Messages in this thread | | | Subject | Re: [PATCH] nvme: make controller 'state' sysfs attribute pollable | From | Guan Junxiong <> | Date | Thu, 21 Sep 2017 20:47:10 +0800 |
| |
On 2017/9/20 22:59, Christoph Hellwig wrote: > On Wed, Sep 20, 2017 at 12:40:32PM +0200, Johannes Thumshirn wrote: >> Notify sysfs about changes of a nvme controller so user-space can watch the >> file via poll() or select() in order to react to a state change. > > Userspace has no business polling for the state. >
Please consider this patch. At least upstream multipath-tools is using the sysfs state now: [1] https://git.opensvc.com/gitweb.cgi?p=multipath-tools/.git;a=commitdiff;h=29c3b0446c4d919859f9e87b291563d483aab594 [2] https://git.opensvc.com/gitweb.cgi?p=multipath-tools/.git;a=commitdiff;h=d2561442cc0b444e8a728bac2c1466468816ee9d
> _______________________________________________ > Linux-nvme mailing list > Linux-nvme@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-nvme > >
| |