On Thu, 2016-09-08 at 20:57 -0700, Dan Williams wrote: > On Thu, Sep 8, 2016 at 5:28 PM, Verma, Vishal L > <[email protected]> wrote: > > > > On Thu, 2016-09-08 at 16:33 -0700, Dave Jiang wrote: > > > > > > Bad blocks can be injected via /sys/block/pmemN/badblocks. In a > > > situation > > > where legacy pmem is being used or a pmem region created by using > > > memmap > > > kernel parameter, the injected bad blocks are not cleared due to > > > nvdimm_clear_poison() failing from lack of ndctl function pointer. > > > In > > > this case we need to just return as handled and allow the bad > > > blocks > > > to > > > be cleared rather than fail. > > > > > > Signed-off-by: Dave Jiang <[email protected]> > > > --- > > > drivers/nvdimm/bus.c | 6 +++++- > > > 1 file changed, 5 insertions(+), 1 deletion(-) > > > > Looks good to me! > > Reveiewed-by: Vishal Verma <[email protected]> > > Applied, although I just noticed that patchwork did not pick up that > reviewed-by due to the typo.
Oops, that's what happens when I type stuff out instead of using vim macros :) _______________________________________________ Linux-nvdimm mailing list [email protected] https://lists.01.org/mailman/listinfo/linux-nvdimm
