[PATCH] nfit: Collect shutdown status for NVDIMM_FAMILY_HYPERV

Dexuan Cui decui at microsoft.com
Wed Jan 30 20:10:49 UTC 2019


> From: Linux-nvdimm <linux-nvdimm-bounces at lists.01.org> On Behalf Of
> Dexuan Cui
> Sent: Wednesday, January 30, 2019 12:03 PM
> To: Greg KH <gregkh at linuxfoundation.org>
> Cc: Josh Poulson <jopoulso at microsoft.com>; linux-nvdimm at lists.01.org;
> Haiyang Zhang <haiyangz at microsoft.com>;
> driverdev-devel at linuxdriverproject.org; Rafael J. Wysocki
> <rjw at rjwysocki.net>; linux-kernel at vger.kernel.org; Michael Kelley
> <mikelley at microsoft.com>; Sasha Levin <Alexander.Levin at microsoft.com>;
> linux-acpi at vger.kernel.org; Ross Zwisler <zwisler at kernel.org>; Stephen
> Hemminger <sthemmin at microsoft.com>; Len Brown <lenb at kernel.org>
> Subject: RE: [PATCH] nfit: Collect shutdown status for
> NVDIMM_FAMILY_HYPERV
> 
> > From: Greg KH <gregkh at linuxfoundation.org>
> > Sent: Wednesday, January 30, 2019 11:38 AM
> >
> > On Wed, Jan 30, 2019 at 06:48:40PM +0000, Dexuan Cui wrote:
> > >
> > > Let's expose the info to the userspace (e.g. ntctl) via sysfs.
> >
> > If you add a new sysfs file, you need to add a new Documentation/ABI/
> > update as well :(
> 
> It's an existing sysfs node, which was originally added by Dan in Sep 2018:
> /sys/bus/nd/devices/nmem0/nfit/dirty_shutdown.
> 
> Before this patch, the node doesn't appear when Linux runs on Hyper-V,
> and with this patch, the node can appear now.
> 
> However, indeed, the node and the other related nodes have not been
> documented in Documentation/ABI/testing/sysfs-bus-nfit yet.
> I suppose Dan would add that?
Actually the other nodes have been documented, and only the
"dirty_shutdown" is missed. 

It should be straightfoward. Let me make a patch for this.

Thanks for the reminder, Greg!
 
Thanks,
-- Dexuan


More information about the devel mailing list