> -----Original Message----- > From: Dan Williams [mailto:[email protected]] > Sent: Friday, July 6, 2018 6:26 AM > To: Masayoshi Mizuma <[email protected]> > Cc: Qi, Fuli/斉 福利 <[email protected]>; linux-nvdimm > <[email protected]> > Subject: Re: [ndctl PATCH] ndctl, test: add a new unit test for monitor > > On Thu, Jul 5, 2018 at 2:06 PM, Masayoshi Mizuma <[email protected]> > wrote: > > Hi Qi, > > > > I tried this test after I applied your latest monitor patches [1]. > > I found some points. Could you check the following? > > > > [1] https://lists.01.org/pipermail/linux-nvdimm/2018-June/016562.html > > > > On 07/05/2018 07:23 AM, QI Fuli wrote: > [..] > > Is this "nmem0:nmem1:nmem2:nmem3"? > > Or why don't you getting the dimms by ndctl list command instead of > > hard-cording? > > Yes, this is needed. The test fails for me because I have an e820 defined bus > and > a typical ACPI NFIT bus along with the nfit_test buses on my test system.
Masa, Dan, Thank you very much for your comments. I will fix it in next version. QI _______________________________________________ Linux-nvdimm mailing list [email protected] https://lists.01.org/mailman/listinfo/linux-nvdimm
