If we don't set the rc, we will return "0", making it look like we
succeeded.

Cc: Benjamin Herrenschmidt <b...@kernel.crashing.org>
Cc: Paul Mackerras <pau...@samba.org>
Cc: Michael Ellerman <m...@ellerman.id.au>
Cc: Andrew Morton <a...@linux-foundation.org>
Cc: Pavel Tatashin <pasha.tatas...@soleen.com>
Cc: Richard Fontana <rfont...@redhat.com>
Cc: Allison Randal <alli...@lohutok.net>
Cc: Thomas Gleixner <t...@linutronix.de>
Cc: Arun KS <aru...@codeaurora.org>
Signed-off-by: David Hildenbrand <da...@redhat.com>
---
 arch/powerpc/platforms/pseries/cmm.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/arch/powerpc/platforms/pseries/cmm.c 
b/arch/powerpc/platforms/pseries/cmm.c
index 572651a5c87b..fab049d3ea1e 100644
--- a/arch/powerpc/platforms/pseries/cmm.c
+++ b/arch/powerpc/platforms/pseries/cmm.c
@@ -683,8 +683,12 @@ static int cmm_init(void)
        if ((rc = cmm_sysfs_register(&cmm_dev)))
                goto out_reboot_notifier;
 
-       if (register_memory_notifier(&cmm_mem_nb) ||
-           register_memory_isolate_notifier(&cmm_mem_isolate_nb))
+       rc = register_memory_notifier(&cmm_mem_nb);
+       if (rc)
+               goto out_unregister_notifier;
+
+       rc = register_memory_isolate_notifier(&cmm_mem_isolate_nb);
+       if (rc)
                goto out_unregister_notifier;
 
        if (cmm_disabled)
-- 
2.21.0

Reply via email to