Re: [PATCH 13/13] libnvdimm, namespace: Publish page structure init state / control

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
Subject: Re: [PATCH 13/13] libnvdimm, namespace: Publish page structure init state / control
From: Dan Williams <dan.j.williams@xxxxxxxxx>
Date: Thu, 5 Jul 2018 13:34:01 -0700
Cc: Johannes Thumshirn <jthumshirn@xxxxxxx>, Ross Zwisler <ross.zwisler@xxxxxxxxxxxxxxx>, Vishal Verma <vishal.l.verma@xxxxxxxxx>, Dave Jiang <dave.jiang@xxxxxxxxx>, Jeff Moyer <jmoyer@xxxxxxxxxx>, Christoph Hellwig <hch@xxxxxx>, linux-nvdimm <linux-nvdimm@xxxxxxxxxxxx>, Linux MM <linux-mm@xxxxxxxxx>, Linux Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>
In-reply-to: <20180705132455.2a40de08dbe3a9bb384fb870@linux-foundation.org>
References: <153077334130.40830.2714147692560185329.stgit@dwillia2-desk3.amr.corp.intel.com> <153077341292.40830.11333232703318633087.stgit@dwillia2-desk3.amr.corp.intel.com> <20180705082931.echvdqipgvwhghf2@linux-x5ow.site> <CAPcyv4h1L6ZMCqWXhWD_ZJ=sH7SVzuUGMG2Ln=6Cy6sR4S=VUw@mail.gmail.com> <20180705144941.drfiwhqcnqqorqu3@linux-x5ow.site> <20180705132455.2a40de08dbe3a9bb384fb870@linux-foundation.org>

On Thu, Jul 5, 2018 at 1:24 PM, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote:
> On Thu, 5 Jul 2018 16:49:41 +0200 Johannes Thumshirn <jthumshirn@xxxxxxx> wrote:
>
>> On Thu, Jul 05, 2018 at 07:46:05AM -0700, Dan Williams wrote:
>> > ...but that also allows 'echo "syncAndThenSomeGarbage" >
>> > /sys/.../memmap_state' to succeed.
>>
>> Yep it does :-(.
>>
>> Damn
>
> sysfs_streq()

Nice... /me stares down a long list of needed cleanups in the
libnvdimm sysfs implementation with that gem.

Follow-Ups:
- Re: [PATCH 13/13] libnvdimm, namespace: Publish page structure init state / control
  - From: Johannes Thumshirn

References:
- [PATCH 00/13] mm: Asynchronous + multithreaded memmap init for ZONE_DEVICE
  - From: Dan Williams
- [PATCH 13/13] libnvdimm, namespace: Publish page structure init state / control
  - From: Dan Williams
- Re: [PATCH 13/13] libnvdimm, namespace: Publish page structure init state / control
  - From: Johannes Thumshirn
- Re: [PATCH 13/13] libnvdimm, namespace: Publish page structure init state / control
  - From: Dan Williams
- Re: [PATCH 13/13] libnvdimm, namespace: Publish page structure init state / control
  - From: Johannes Thumshirn
- Re: [PATCH 13/13] libnvdimm, namespace: Publish page structure init state / control
  - From: Andrew Morton

Prev by Date: Re: [PATCH 13/13] libnvdimm, namespace: Publish page structure init state / control
Next by Date: Re: [PATCH 13/13] libnvdimm, namespace: Publish page structure init state / control
Previous by thread: Re: [PATCH 13/13] libnvdimm, namespace: Publish page structure init state / control
Next by thread: Re: [PATCH 13/13] libnvdimm, namespace: Publish page structure init state / control
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]