Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1470930

Re: [PATCH 2/3] acpi, nfit: add dimm device notification support

From "Kani, Toshimitsu" <toshi.kani@hpe.com>
Newsgroups linux.kernel
Subject Re: [PATCH 2/3] acpi, nfit: add dimm device notification support
Date 2016-08-26 20:30 +0200
Message-ID <sau37-2a3-5@gated-at.bofh.it> (permalink)
References <s9rTI-1jl-11@gated-at.bofh.it> <s9rTI-1jl-9@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On Tue, 2016-08-23 at 14:54 -0700, Dan Williams wrote:
> Per "ACPI 6.1 Section 9.20.3" NVDIMM devices, children of the
> ACPI0012 NVDIMM Root device, can receive health event notifications.
> 
> Given that these devices are precluded from registering a
> notification handler via acpi_driver.acpi_device_ops (due to no
> _HID), we use acpi_install_notify_handler() directly.  

I've confirmed that this ACPI notify handler is called properly.

> The registered handler, acpi_nvdimm_notify(), triggers a poll(2)
> event on the nmemX/nfit/flags sysfs attribute when a health event
> notification is received.

This sounds good idea, but should we document that the value of sysfs
'flags' itself does not get updated?  User space program will then need
to call its _DSM to get health status.

Thanks,
-Toshi

Back to linux.kernel | Previous | NextNext in thread | Find similar | Unroll thread


Thread

Re: [PATCH 2/3] acpi, nfit: add dimm device notification support "Kani, Toshimitsu" <toshi.kani@hpe.com> - 2016-08-26 20:30 +0200
  Re: [PATCH 2/3] acpi, nfit: add dimm device notification support Dan Williams <dan.j.williams@intel.com> - 2016-08-26 20:50 +0200
    Re: [PATCH 2/3] acpi, nfit: add dimm device notification support "Kani, Toshimitsu" <toshi.kani@hpe.com> - 2016-08-30 00:20 +0200

csiph-web