Re: [RFC PATCH v3 2/5] ndctl: monitor: add ndctl create-monitor command

2018-02-18 Thread Yasunori Goto
> On Tue, Feb 13, 2018 at 2:12 AM, Yasunori Goto wrote: > >> Hi, > [..] > > I would like to ask one more thing. > > > > What should nvdimm daemon do on detected events of > > bus/region/namespace/dimm ? > > IIRC, udev handles such hotplug events. > > What is relationship/roles between nvdimm daem

Re: [RFC PATCH v3 2/5] ndctl: monitor: add ndctl create-monitor command

2018-02-16 Thread Dan Williams
On Wed, Feb 14, 2018 at 9:51 PM, Qi, Fuli wrote: > >>> Hi Qi, >>> >>> This is getting closer to where I want to see this go, but still some >>> architecture details to incorporate. I mentioned on the cover letter >>> that systemd can handle starting, stopping, and show the status of >>> the >>> mo

Re: [RFC PATCH v3 2/5] ndctl: monitor: add ndctl create-monitor command

2018-02-16 Thread Dan Williams
On Tue, Feb 13, 2018 at 2:12 AM, Yasunori Goto wrote: >> Hi, [..] > I would like to ask one more thing. > > What should nvdimm daemon do on detected events of bus/region/namespace/dimm ? > IIRC, udev handles such hotplug events. > What is relationship/roles between nvdimm daemon and udev? UDEV ha

Re: [RFC PATCH v3 2/5] ndctl: monitor: add ndctl create-monitor command

2018-02-16 Thread Dan Williams
On Tue, Feb 13, 2018 at 1:58 AM, Yasunori Goto wrote: > Hi, > >> On Fri, Feb 9, 2018 at 12:02 AM, QI Fuli wrote: >> > This patch is used to add $ndctl create-monitor command, by which users can >> > create a new monitor. Users can select the DIMMS to be monitored by using >> > [--dimm] [--bus] [-

Re: [RFC PATCH v3 2/5] ndctl: monitor: add ndctl create-monitor command

2018-02-14 Thread Qi, Fuli
Hi Qi, This is getting closer to where I want to see this go, but still some architecture details to incorporate. I mentioned on the cover letter that systemd can handle starting, stopping, and show the status of the monitor. The other detail to incorporate is that monitor events can come DIMMs

Re: [RFC PATCH v3 2/5] ndctl: monitor: add ndctl create-monitor command

2018-02-13 Thread Yasunori Goto
> Hi, > > > On Fri, Feb 9, 2018 at 12:02 AM, QI Fuli wrote: > > > This patch is used to add $ndctl create-monitor command, by which users > > > can > > > create a new monitor. Users can select the DIMMS to be monitored by using > > > [--dimm] [--bus] [--region] [--namespace] options. The notific

Re: [RFC PATCH v3 2/5] ndctl: monitor: add ndctl create-monitor command

2018-02-13 Thread Yasunori Goto
Hi, > On Fri, Feb 9, 2018 at 12:02 AM, QI Fuli wrote: > > This patch is used to add $ndctl create-monitor command, by which users can > > create a new monitor. Users can select the DIMMS to be monitored by using > > [--dimm] [--bus] [--region] [--namespace] options. The notifications can > > be o

Re: [RFC PATCH v3 2/5] ndctl: monitor: add ndctl create-monitor command

2018-02-12 Thread Verma, Vishal L
On Sun, 2018-02-11 at 12:48 -0800, Dan Williams wrote: > On Fri, Feb 9, 2018 at 12:02 AM, QI Fuli > wrote: > > This patch is used to add $ndctl create-monitor command, by which > > users can > > create a new monitor. Users can select the DIMMS to be monitored by > > using > > [--dimm] [--bus] [--r

Re: [RFC PATCH v3 2/5] ndctl: monitor: add ndctl create-monitor command

2018-02-11 Thread Dan Williams
On Fri, Feb 9, 2018 at 12:02 AM, QI Fuli wrote: > This patch is used to add $ndctl create-monitor command, by which users can > create a new monitor. Users can select the DIMMS to be monitored by using > [--dimm] [--bus] [--region] [--namespace] options. The notifications can > be outputed to a sp