Re: [PATCH v2] libnvdimm: fix btt vs clear poison locking

From	Dan Williams <dan.j.williams@intel.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH v2] libnvdimm: fix btt vs clear poison locking
Date	2017-04-12 22:50 +0200
Message-ID	<tvxnb-Zv-13@gated-at.bofh.it> (permalink)
References	<tvdeN-58v-3@gated-at.bofh.it> <tvq2l-4yp-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Wed, Apr 12, 2017 at 5:49 AM, Jeff Moyer <jmoyer@redhat.com> wrote:
> Dan Williams <dan.j.williams@intel.com> writes:
>
>> As a minimal fix, disable error clearing when the BTT is enabled for the
>> namespace. For the final fix a larger rework of the poison list locking
>> is needed.
>
> I think "fix" is a strong word for this patch.  ;-)  Looks fine, though.

Heh, true. I'll rename it to "band aid" since the true "fix" is coming in 4.12.

> Reviewed-by: Jeff Moyer <jmoyer@redhat.com>

Thanks!

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH v2] libnvdimm: fix btt vs clear poison locking Dan Williams <dan.j.williams@intel.com> - 2017-04-12 01:20 +0200
  Re: [PATCH v2] libnvdimm: fix btt vs clear poison locking Jeff Moyer <jmoyer@redhat.com> - 2017-04-12 15:00 +0200
    Re: [PATCH v2] libnvdimm: fix btt vs clear poison locking Dan Williams <dan.j.williams@intel.com> - 2017-04-12 22:50 +0200

csiph-web