s390/pai_crypto: remove per-cpu variable assignement in event initialization
authorThomas Richter <tmricht@linux.ibm.com>
Wed, 18 Oct 2023 09:53:52 +0000 (11:53 +0200)
committerVasily Gorbik <gor@linux.ibm.com>
Wed, 25 Oct 2023 13:08:29 +0000 (15:08 +0200)
Function paicrypt_event_init() initializes the PMU device driver
specific details for an event. It is called once per event creation.
The function paicrypt_event_init() is not necessarily executed on
that CPU the event will be used for.
When an event is activated, function paicrypt_start() is used to
start the event on that CPU.
The per CPU data structure struct paicrypt_map has a pointer to
the event which is active for a particular CPU. This pointer is
set in function paicrypt_start() to point to the currently installed
event. There is no need to also set this pointer in function
paicrypt_event_init() where is might be assigned to the wrong CPU.
Therefore remove this assignment in paicrypt_event_init().

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

index 35940234da750199a86fff77f807785c75291fbe..77fd24e6cbb643bccb79777da90481c748f4feaf 100644 (file)
@@ -285,7 +285,6 @@ static int paicrypt_event_init(struct perf_event *event)
         * are active at the same time.
         */
        event->hw.last_tag = 0;
-       cpump->event = event;
        event->destroy = paicrypt_event_destroy;
 
        if (a->sample_period) {