Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934362AbcCIXFH (ORCPT ); Wed, 9 Mar 2016 18:05:07 -0500 Received: from mga04.intel.com ([192.55.52.120]:6475 "EHLO mga04.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752693AbcCIXFF (ORCPT ); Wed, 9 Mar 2016 18:05:05 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.24,313,1455004800"; d="scan'208";a="920518772" From: "Verma, Vishal L" To: "Williams, Dan J" , "linux-nvdimm@lists.01.org" CC: "linux-kernel@vger.kernel.org" , "ross.zwisler@linux.intel.com" , "stable@vger.kernel.org" , "x86@kernel.org" , "linux-acpi@vger.kernel.org" Subject: Re: [PATCH 0/3] nfit, libnvdimm, pmem: clear poison support Thread-Topic: [PATCH 0/3] nfit, libnvdimm, pmem: clear poison support Thread-Index: AQHReYyHzyFVC+FrAkijyw7pgQVXH59SQzYA Date: Wed, 9 Mar 2016 23:05:03 +0000 Message-ID: <1457564698.4525.33.camel@intel.com> References: <20160308224713.16298.33547.stgit@dwillia2-desk3.jf.intel.com> In-Reply-To: <20160308224713.16298.33547.stgit@dwillia2-desk3.jf.intel.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.232.112.171] Content-Type: text/plain; charset="utf-8" Content-ID: <61A8C159E16CCA4189856919446AA353@intel.com> MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id u29N5CtW003853 Content-Length: 1281 Lines: 30 On Tue, 2016-03-08 at 14:47 -0800, Dan Williams wrote: > ACPI 6.1 introduces the ability to send "clear error" commands to the > ACPI0012:00 device representing the root of an "nvdimm bus". > > Similar to relocating a bad block on a disk, this support clears > media errors in response to a write. > > --- > > Dan Williams (3): >       nfit, libnvdimm: clear poison command support >       libnvdimm, pmem: fix kmap_atomic() leak in error path >       libnvdimm, pmem: clear poison on write > > >  arch/x86/include/asm/pmem.h      |    5 +++ >  drivers/acpi/nfit.c              |   12 ++++++- >  drivers/nvdimm/bus.c             |   65 > ++++++++++++++++++++++++++++++++++++++ >  drivers/nvdimm/nd.h              |    2 + >  drivers/nvdimm/pmem.c            |   40 ++++++++++++++++++++--- >  include/linux/pmem.h             |   19 +++++++++++ >  include/uapi/linux/ndctl.h       |   13 ++++++++ >  tools/testing/nvdimm/test/nfit.c |   29 +++++++++++++++++ >  8 files changed, 179 insertions(+), 6 deletions(-) Except for the one comment in patch 1, this looks good to me! For the series, Reviewed-by: Vishal Verma