Messages in this thread | | | From | Dan Williams <> | Date | Fri, 2 Dec 2016 12:51:55 -0800 | Subject | Re: [PATCH 3/8 linux-next] libnvdimm, namespace: move persistence warning |
| |
On Fri, Dec 2, 2016 at 11:20 AM, Fabian Frederick <fabf@skynet.be> wrote: > There's no need to warn on persistence yet as some validity tests > are on the way. > > Signed-off-by: Fabian Frederick <fabf@skynet.be> > --- > drivers/nvdimm/pmem.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/nvdimm/pmem.c b/drivers/nvdimm/pmem.c > index 822a712..204912c 100644 > --- a/drivers/nvdimm/pmem.c > +++ b/drivers/nvdimm/pmem.c > @@ -266,9 +266,6 @@ static int pmem_attach_disk(struct device *dev, > dev_set_drvdata(dev, pmem); > pmem->phys_addr = res->start; > pmem->size = resource_size(res); > - if (nvdimm_has_flush(nd_region) < 0) > - dev_warn(dev, "unable to guarantee persistence of writes due to unknown flushing capability of the memory region\n"); > - > if (!devm_request_mem_region(dev, res->start, resource_size(res), > dev_name(dev))) { > dev_warn(dev, "could not reserve region %pR\n", res); > @@ -279,6 +276,9 @@ static int pmem_attach_disk(struct device *dev, > if (!q) > return -ENOMEM; > > + if (nvdimm_has_flush(nd_region) < 0) > + dev_warn(dev, "unable to guarantee persistence of writes due to unknown flushing capability of the memory region\n"); > + > pmem->pfn_flags = PFN_DEV; > if (is_nd_pfn(dev)) { > addr = devm_memremap_pages(dev, &pfn_res, &q->q_usage_counter,
With the new sub-section hotplug support I don't think we need this.
| |