On Fri, 2016-03-04 at 11:34 +0800, Zhang, Haozhong wrote: > On 03/04/16 11:28, Xiao Guangrong wrote: > > > > > > CCed: Haozhong. > > > > On 03/04/2016 06:39 AM, Vishal Verma wrote: > > > > > > If firmware doesn't implement any of the ARS commands, take that > > > to > > > mean that ARS is unsupported, and continue to initialize regions > > > without > > > bad block lists. We cannot make the assumption that ARS commands > > > will be > > > unconditionally supported on all NVDIMMs. > > > > > > Cc: Dan Williams <dan.j.williams@xxxxxxxxx> > > > Reported-by: Xiao Guangrong <guangrong.xiao@xxxxxxxxxxxxxxx> > > Thank you for the fix, Vishal! it looks good to me. > > > > However, It is haozhong's credit as he is the one reported the bug > > not > > me. :) > > > Thanks! I have tested on QEMU which has not implemented ARS yet and > this patch does work. > > Tested-by: Haozhong Zhang <haozhong.zhang@xxxxxxxxx> Thanks, both Haozhong and Guangrong. Dan, can you please fixup the Reported-by tag when you apply. -Vishal > _______________________________________________ > Linux-nvdimm mailing list > Linux-nvdimm@xxxxxxxxxxxx > https://lists.01.org/mailman/listinfo/linux-nvdimm -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
