Re: [PATCH 1/8] pmem: Initial version of persistent memory driver

From: Dan Williams
Date: Thu Mar 19 2015 - 20:11:38 EST


On Thu, Mar 19, 2015 at 2:24 AM, Boaz Harrosh <boaz@xxxxxxxxxxxxx> wrote:
> I apologize. It is not my intention to hijack your project. All but the last
> 2 changes I have posted again and again, even those changes I have said that
> I maintain them in a public tree, and made them available publicly ASAP. I
> stopped sending the (last 2) patches because it felt like I was spamming the
> list, since none of my patches got any comments or have been accepted to your
> tree.

That's not true. We talked about your "map=" proposal at length back
in September. You concluded "That the discovery should be elsewhere
in an ARCH/driver LLD and pmem stays generic." [1]. A generic
approach is being specified by the ACPI Working Group and will be
released "Real Soon Now (TM)" (on the order of weeks not months). My
first choice would be to finish waiting for that specification before
we upstream a pmem driver. Outside of that, if we need a pmem driver
"now", Ross's version has the nice property of having an easier to
revert resource discovery mechanism. The kernel command line is
arguably an ABI and the need for "map=" is obviated by a generic
resource discovery mechanism.

[1]: https://lists.01.org/pipermail/linux-nvdimm/2014-September/000043.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/