On Wed, Aug 31, 2016 at 05:28:26PM -0700, David Rientjes wrote:
2. store_mem_state() still needs a tweak, right? It was only
returning -EINVAL by accident, due to the convoluted sequence I
listed in the patch.
Yes, absolutely. It returning -EINVAL for "nline" is what is accidently
preserving it's backwards compatibility :) Note that device_online()
returns 1 if already online and memory_subsys_online() returns 0 if online
in this case. So we want store_mem_state() to return -EINVAL if
device_online() returns non-zero (this was in my first email).
I'll spin a v3 patch to do this.
Thank you for your review!
--
Reza Arbab
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>