Re: powerpc/pseries: fix error return value in cmm_mem_going_offline

2016-07-05 Thread Michael Ellerman
On Tue, 2016-08-03 at 21:26:17 UTC, Rasmus Villemoes wrote: > 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

Re: powerpc/pseries: fix error return value in cmm_mem_going_offline

2016-07-05 Thread Michael Ellerman
On Tue, 2016-08-03 at 21:26:17 UTC, Rasmus Villemoes wrote: > 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

[PATCH] powerpc/pseries: fix error return value in cmm_mem_going_offline

2016-03-08 Thread Rasmus Villemoes
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.

[PATCH] powerpc/pseries: fix error return value in cmm_mem_going_offline

2016-03-08 Thread Rasmus Villemoes
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.