]> git.proxmox.com Git - mirror_ubuntu-hirsute-kernel.git/commitdiff
powerpc/pseries/hibernation: pass stream id via function arguments
authorNathan Lynch <nathanl@linux.ibm.com>
Mon, 7 Dec 2020 21:51:50 +0000 (15:51 -0600)
committerMichael Ellerman <mpe@ellerman.id.au>
Tue, 8 Dec 2020 10:41:00 +0000 (21:41 +1100)
There is no need for the stream id to be a file-global variable; pass
it from hibernate_store() to pseries_suspend_begin() for the
H_VASI_STATE call.

Signed-off-by: Nathan Lynch <nathanl@linux.ibm.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Link: https://lore.kernel.org/r/20201207215200.1785968-19-nathanl@linux.ibm.com
arch/powerpc/platforms/pseries/suspend.c

index 3eaa9d59dc7ab4a3faa0694b59c76bb20d9ca3c1..232621f33510fe846bc2eccc16efbc3798a9ebba 100644 (file)
@@ -15,7 +15,6 @@
 #include <asm/topology.h>
 #include "../../kernel/cacheinfo.h"
 
-static u64 stream_id;
 static struct device suspend_dev;
 static DECLARE_COMPLETION(suspend_work);
 static struct rtas_suspend_me_data suspend_data;
@@ -29,7 +28,7 @@ static atomic_t suspending;
  * Return value:
  *     0 on success / other on failure
  **/
-static int pseries_suspend_begin(suspend_state_t state)
+static int pseries_suspend_begin(u64 stream_id)
 {
        long vasi_state, rc;
        unsigned long retbuf[PLPAR_HCALL_BUFSIZE];
@@ -132,6 +131,7 @@ static ssize_t store_hibernate(struct device *dev,
                               struct device_attribute *attr,
                               const char *buf, size_t count)
 {
+       u64 stream_id;
        int rc;
 
        if (!capable(CAP_SYS_ADMIN))
@@ -140,7 +140,7 @@ static ssize_t store_hibernate(struct device *dev,
        stream_id = simple_strtoul(buf, NULL, 16);
 
        do {
-               rc = pseries_suspend_begin(PM_SUSPEND_MEM);
+               rc = pseries_suspend_begin(stream_id);
                if (rc == -EAGAIN)
                        ssleep(1);
        } while (rc == -EAGAIN);
@@ -148,8 +148,6 @@ static ssize_t store_hibernate(struct device *dev,
        if (!rc)
                rc = pm_suspend(PM_SUSPEND_MEM);
 
-       stream_id = 0;
-
        if (!rc)
                rc = count;