Re: [RFC PATCH 5/7] libnvdimm, e820: make CONFIG_X86_PMEM_LEGACY a tristate option

From: Dan Williams
Date: Sat Aug 15 2015 - 12:04:09 EST

On Sat, Aug 15, 2015 at 8:58 AM, Christoph Hellwig <hch@xxxxxx> wrote:
> On Sat, Aug 15, 2015 at 08:28:35AM -0700, Dan Williams wrote:
>> 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
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at