On Sat, Aug 15, 2015 at 8:58 AM, Christoph Hellwig [off-list ref] wrote:

On Sat, Aug 15, 2015 at 08:28:35AM -0700, Dan Williams wrote:

quoted

I'm not grokking the argument against allowing this functionality to
be modular.

You're adding a another layer of platform_devices just to make a tivially
small piece of code modular so that you can hook into it.  I don't think
that's a good reason, and neither is the after thought of preventing
potentially future buggy firmware.

What other layer? /sys/devices/platform/e820_pmem is that exact same
device we had before this patch.  We just have a proper driver for it
now.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>