diff options
| author | Rasmus Villemoes <linux@rasmusvillemoes.dk> | 2016-03-08 22:26:17 +0100 |
|---|---|---|
| committer | Michael Ellerman <mpe@ellerman.id.au> | 2016-07-05 23:58:52 +1000 |
| commit | e2be23712a9c8e3710c9e2684a3204bd20c685d6 (patch) | |
| tree | 56db0a9c7f91870ffa5498c132c755072ed32b6e | |
| parent | a9862c7440f191439a51f77233f89f7e40efe02e (diff) | |
| download | talos-obmc-linux-e2be23712a9c8e3710c9e2684a3204bd20c685d6.tar.gz talos-obmc-linux-e2be23712a9c8e3710c9e2684a3204bd20c685d6.zip | |
powerpc/pseries: Fix error return value in cmm_mem_going_offline()
cmm_mem_going_offline() is (only) called from cmm_memory_cb(), which
sends the return value through notifier_from_errno(). The latter
expects 0 or -errno (notifier_to_errno(notifier_from_errno(x)) is 0
for any x >= 0, so passing a positive value cannot make sense). Hence
negate ENOMEM.
Signed-off-by: Rasmus Villemoes <linux@rasmusvillemoes.dk>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
| -rw-r--r-- | arch/powerpc/platforms/pseries/cmm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/powerpc/platforms/pseries/cmm.c b/arch/powerpc/platforms/pseries/cmm.c index fc44ad0475f8..66e7227469b8 100644 --- a/arch/powerpc/platforms/pseries/cmm.c +++ b/arch/powerpc/platforms/pseries/cmm.c @@ -574,7 +574,7 @@ static int cmm_mem_going_offline(void *arg) cmm_dbg("Failed to allocate memory for list " "management. Memory hotplug " "failed.\n"); - return ENOMEM; + return -ENOMEM; } memcpy(npa, pa_curr, PAGE_SIZE); if (pa_curr == cmm_page_list) |
