On Fri, 2018-07-06 at 20:43 -0700, Dan Williams wrote:
> On Fri, Jul 6, 2018 at 6:48 PM, QI Fuli wrote:
> > Hi Masa,
> >
> > Thank you for your comments.
> >
> > On 7/7/2018 12:08 AM, Masayoshi Mizuma wrote:
> > >
> > > Hi Qi,
> > >
> > > On 07/06/2018 01:22 AM, QI Fuli wrote:
> > > >
> > >
> -Original Message-
> From: Dan Williams [mailto:dan.j.willi...@intel.com]
> Sent: Saturday, July 7, 2018 12:58 PM
> To: Qi, Fuli/斉 福利
> Cc: linux-nvdimm
> Subject: Re: [ndctl PATCH v2] ndctl, test: add a new unit test for monitor
>
> On Thu, Jul 5, 2018 at
On Thu, Jul 5, 2018 at 10:22 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
> Acked-by: Masayoshi Mizuma
> Signed-off-by:
On Fri, Jul 6, 2018 at 6:48 PM, QI Fuli wrote:
> Hi Masa,
>
> Thank you for your comments.
>
> On 7/7/2018 12:08 AM, Masayoshi Mizuma wrote:
>>
>> Hi Qi,
>>
>> On 07/06/2018 01:22 AM, QI Fuli wrote:
>>>
>>> Add a new unit test to test the following options of the monitor command.
>>> --dimm
Hi Masa,
Thank you for your comments.
On 7/7/2018 12:08 AM, Masayoshi Mizuma wrote:
Hi Qi,
On 07/06/2018 01:22 AM, QI Fuli wrote:
Add a new unit test to test the following options of the monitor command.
--dimm
--bus
--region
--namespace
--logfile
--config-file
Hi Qi,
On 07/06/2018 01:22 AM, 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
> Acked-by: Masayoshi Mizuma
I think this
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
Acked-by: Masayoshi Mizuma
Signed-off-by: QI Fuli
---
v1 -> v2:
- Add init()
- Add get_filter_dimm() to get