Re: [PATCH v7 09/16] libnvdimm, pmem: move pmem to drivers/nvdimm/

From: Geert Uytterhoeven
Date: Wed Jul 01 2015 - 12:49:46 EST


On Wed, Jul 1, 2015 at 6:41 PM, Dan Williams <dan.j.williams@xxxxxxxxx> wrote:
> On Wed, Jul 1, 2015 at 6:13 AM, Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote:
>> On Thu, Jun 18, 2015 at 1:14 AM, Dan Williams <dan.j.williams@xxxxxxxxx> wrote:
>>> Prepare the pmem driver to consume PMEM namespaces emitted by regions of
>>> an nvdimm_bus instance. No functional change.
>>
>> As LIBNVDIMM depends on PHYS_ADDR_T_64BIT, the driver can no
>> longer be enabled on pure 32-bit platforms. Is that intended?
>
> Yes it was intentional. It still allows 32-bit platforms with 64-bit
> resource_size_t to compile. Do you otherwise have a 32-bit use case
> for pmem? I'm of course open to working through the changes to add
> wider architecture support.

Nope, just wondering, as the original didn't depend on PHYS_ADDR_T_64BIT.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
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/