lkml.org 
[lkml]   [2019]   [Jan]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH] nfit: Collect shutdown status for NVDIMM_FAMILY_HYPERV
Date
> From: Linux-nvdimm <linux-nvdimm-bounces@lists.01.org> On Behalf Of
> Dexuan Cui
> Sent: Wednesday, January 30, 2019 12:03 PM
> To: Greg KH <gregkh@linuxfoundation.org>
> Cc: Josh Poulson <jopoulso@microsoft.com>; linux-nvdimm@lists.01.org;
> Haiyang Zhang <haiyangz@microsoft.com>;
> driverdev-devel@linuxdriverproject.org; Rafael J. Wysocki
> <rjw@rjwysocki.net>; linux-kernel@vger.kernel.org; Michael Kelley
> <mikelley@microsoft.com>; Sasha Levin <Alexander.Levin@microsoft.com>;
> linux-acpi@vger.kernel.org; Ross Zwisler <zwisler@kernel.org>; Stephen
> Hemminger <sthemmin@microsoft.com>; Len Brown <lenb@kernel.org>
> Subject: RE: [PATCH] nfit: Collect shutdown status for
> NVDIMM_FAMILY_HYPERV
>
> > From: Greg KH <gregkh@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

\
 
 \ /
  Last update: 2019-01-30 21:11    [W:0.029 / U:3.240 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site