RE: [ndctl PATCH v3] ndctl, test: add a new unit test for monitor

2018-07-10 Thread Qi, Fuli
> -Original Message- > From: Dan Williams [mailto:dan.j.willi...@intel.com] > Sent: Tuesday, July 10, 2018 12:46 AM > To: Qi, Fuli/斉 福利 > Cc: linux-nvdimm > Subject: Re: [ndctl PATCH v3] ndctl, test: add a new unit test for monitor > > On Mon, Jul 9, 2018 at

Re: [ndctl PATCH v3] ndctl, test: add a new unit test for monitor

2018-07-09 Thread Dan Williams
>> Subject: Re: [ndctl PATCH v3] ndctl, test: add a new unit test for monitor >> >> On Fri, Jul 6, 2018 at 10:35 PM, QI Fuli wrote: >> > Add a new unit test to test the following options of the monitor command. >> >--dimm >> >--bus >> >

RE: [ndctl PATCH v3] ndctl, test: add a new unit test for monitor

2018-07-09 Thread Qi, Fuli
Hi Dan, Thanks for your comments. > -Original Message- > From: Dan Williams [mailto:dan.j.willi...@intel.com] > Sent: Sunday, July 8, 2018 3:56 AM > To: Qi, Fuli/斉 福利 > Cc: linux-nvdimm > Subject: Re: [ndctl PATCH v3] ndctl, test: add a new unit test for monitor >

Re: [ndctl PATCH v3] ndctl, test: add a new unit test for monitor

2018-07-07 Thread Dan Williams
On Fri, Jul 6, 2018 at 10:35 PM, QI Fuli wrote: > Add a new unit test to test the following options of the monitor command. >--dimm >--bus >--region >--namespace >--logfile >--config-file > > Based-on-patch-by: Yasunori Goto > Signed-off-by: QI Fuli > --- > v2 -> v3: >

[ndctl PATCH v3] ndctl, test: add a new unit test for monitor

2018-07-06 Thread QI Fuli
Add a new unit test to test the following options of the monitor command. --dimm --bus --region --namespace --logfile --config-file Based-on-patch-by: Yasunori Goto Signed-off-by: QI Fuli --- v2 -> v3: - Add filter_obj instead of hard-coded values v1 -> v2: - Add init() -