s390/pai: move enum definition to header file
authorThomas Richter <tmricht@linux.ibm.com>
Thu, 20 Oct 2022 08:52:24 +0000 (10:52 +0200)
committerVasily Gorbik <gor@linux.ibm.com>
Wed, 26 Oct 2022 12:46:51 +0000 (14:46 +0200)
Move enum definition to header file. This is done in preparation
for a follow on patch where this enum will be used in another source
file.
Also change the enum name from paiext_mode to paievt_mode
to indicate this enum is now used for several events.
Make naming consistent and rename PAI_MODE_COUNTER to PAI_MODE_COUNTING.

Signed-off-by: Thomas Richter <tmricht@linux.ibm.com>
Acked-by: Sumanth Korikkar <sumanthk@linux.ibm.com>
Signed-off-by: Vasily Gorbik <gor@linux.ibm.com>
arch/s390/include/asm/pai.h
arch/s390/kernel/perf_pai_ext.c

index 1a8a6b1..7d1888e 100644 (file)
@@ -75,4 +75,10 @@ static __always_inline void pai_kernel_exit(struct pt_regs *regs)
        WRITE_ONCE(S390_lowcore.ccd, S390_lowcore.ccd & ~PAI_CRYPTO_KERNEL_OFFSET);
 }
 
+enum paievt_mode {
+       PAI_MODE_NONE,
+       PAI_MODE_SAMPLING,
+       PAI_MODE_COUNTING,
+};
+
 #endif
index d5c7c1e..2da5869 100644 (file)
 static debug_info_t *paiext_dbg;
 static unsigned int paiext_cnt;        /* Extracted with QPACI instruction */
 
-enum paiext_mode {
-       PAI_MODE_NONE,
-       PAI_MODE_SAMPLING,
-       PAI_MODE_COUNTER,
-};
-
 struct pai_userdata {
        u16 num;
        u64 value;
@@ -54,7 +48,7 @@ struct paiext_cb {            /* PAI extension 1 control block */
 struct paiext_map {
        unsigned long *area;            /* Area for CPU to store counters */
        struct pai_userdata *save;      /* Area to store non-zero counters */
-       enum paiext_mode mode;          /* Type of event */
+       enum paievt_mode mode;          /* Type of event */
        unsigned int active_events;     /* # of PAI Extension users */
        unsigned int refcnt;
        struct perf_event *event;       /* Perf event for sampling */
@@ -192,14 +186,14 @@ static int paiext_alloc(struct perf_event_attr *a, struct perf_event *event)
                        goto unlock;
                }
                cpump->mode = a->sample_period ? PAI_MODE_SAMPLING
-                                              : PAI_MODE_COUNTER;
+                                              : PAI_MODE_COUNTING;
        } else {
                /* Multiple invocation, check whats active.
                 * Supported are multiple counter events or only one sampling
                 * event concurrently at any one time.
                 */
                if (cpump->mode == PAI_MODE_SAMPLING ||
-                   (cpump->mode == PAI_MODE_COUNTER && a->sample_period)) {
+                   (cpump->mode == PAI_MODE_COUNTING && a->sample_period)) {
                        rc = -EBUSY;
                        goto unlock;
                }