From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx2.suse.de (mx2.suse.de [195.135.220.15]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 7C2EE1A1E63 for ; Thu, 24 Mar 2016 03:11:04 -0700 (PDT) From: Johannes Thumshirn Subject: Re: [PATCH 01/13] libnvdimm, pfn: fix nvdimm_namespace_add_poison() vs section alignment Date: Thu, 24 Mar 2016 11:10:37 +0100 Message-ID: <2305610.tLOzfXxl0W@c203> In-Reply-To: <20160324012526.21436.70359.stgit@dwillia2-desk3.jf.intel.com> References: <20160324012520.21436.22505.stgit@dwillia2-desk3.jf.intel.com> <20160324012526.21436.70359.stgit@dwillia2-desk3.jf.intel.com> MIME-Version: 1.0 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Errors-To: linux-nvdimm-bounces@lists.01.org Sender: "Linux-nvdimm" To: linux-nvdimm@lists.01.org Cc: linux-kernel@vger.kernel.org, stable@vger.kernel.org List-ID: On Mittwoch, 23. M=E4rz 2016 18:25:26 CET Dan Williams wrote: > When section alignment padding is in effect we need to shift / truncate > the range that is queried for poison by the 'start_pad' or 'end_trunc' > reservations. > = > It's easiest if we just pass in an adjusted resource range rather than > deriving it from the passed in namespace. With the resource range > resolution pushed out to the caller we can also push the > namespace-to-region lookup to the caller and drop the implicit pmem-type > assumption about the passed in namespace object. > = > Cc: Vishal Verma > Cc: > Signed-off-by: Dan Williams Does this really qualify for inclusion into stable? -- = Johannes Thumshirn Storage jthumshirn@suse.de +49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 N=FCrnberg GF: Felix Imend=F6rffer, Jane Smithard, Graham Norton HRB 21284 (AG N=FCrnberg) Key fingerprint =3D EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850 _______________________________________________ Linux-nvdimm mailing list Linux-nvdimm@lists.01.org https://lists.01.org/mailman/listinfo/linux-nvdimm From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757185AbcCXKKp (ORCPT ); Thu, 24 Mar 2016 06:10:45 -0400 Received: from mx2.suse.de ([195.135.220.15]:48935 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755189AbcCXKKj convert rfc822-to-8bit (ORCPT ); Thu, 24 Mar 2016 06:10:39 -0400 From: Johannes Thumshirn To: linux-nvdimm@ml01.01.org Cc: Dan Williams , linux-kernel@vger.kernel.org, stable@vger.kernel.org Subject: Re: [PATCH 01/13] libnvdimm, pfn: fix nvdimm_namespace_add_poison() vs section alignment Date: Thu, 24 Mar 2016 11:10:37 +0100 Message-ID: <2305610.tLOzfXxl0W@c203> User-Agent: KMail/5.1.2 (Linux/4.5.0-1-default; KDE/5.20.0; x86_64; ; ) In-Reply-To: <20160324012526.21436.70359.stgit@dwillia2-desk3.jf.intel.com> References: <20160324012520.21436.22505.stgit@dwillia2-desk3.jf.intel.com> <20160324012526.21436.70359.stgit@dwillia2-desk3.jf.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="iso-8859-1" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mittwoch, 23. März 2016 18:25:26 CET Dan Williams wrote: > When section alignment padding is in effect we need to shift / truncate > the range that is queried for poison by the 'start_pad' or 'end_trunc' > reservations. > > It's easiest if we just pass in an adjusted resource range rather than > deriving it from the passed in namespace. With the resource range > resolution pushed out to the caller we can also push the > namespace-to-region lookup to the caller and drop the implicit pmem-type > assumption about the passed in namespace object. > > Cc: Vishal Verma > Cc: > Signed-off-by: Dan Williams Does this really qualify for inclusion into stable? -- Johannes Thumshirn Storage jthumshirn@suse.de +49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Johannes Thumshirn To: linux-nvdimm@lists.01.org Cc: Dan Williams , linux-kernel@vger.kernel.org, stable@vger.kernel.org Subject: Re: [PATCH 01/13] libnvdimm, pfn: fix nvdimm_namespace_add_poison() vs section alignment Date: Thu, 24 Mar 2016 11:10:37 +0100 Message-ID: <2305610.tLOzfXxl0W@c203> In-Reply-To: <20160324012526.21436.70359.stgit@dwillia2-desk3.jf.intel.com> References: <20160324012520.21436.22505.stgit@dwillia2-desk3.jf.intel.com> <20160324012526.21436.70359.stgit@dwillia2-desk3.jf.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="iso-8859-1" Sender: linux-kernel-owner@vger.kernel.org List-ID: On Mittwoch, 23. M�rz 2016 18:25:26 CET Dan Williams wrote: > When section alignment padding is in effect we need to shift / truncate > the range that is queried for poison by the 'start_pad' or 'end_trunc' > reservations. > > It's easiest if we just pass in an adjusted resource range rather than > deriving it from the passed in namespace. With the resource range > resolution pushed out to the caller we can also push the > namespace-to-region lookup to the caller and drop the implicit pmem-type > assumption about the passed in namespace object. > > Cc: Vishal Verma > Cc: > Signed-off-by: Dan Williams Does this really qualify for inclusion into stable? -- Johannes Thumshirn Storage jthumshirn@suse.de +49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 N�rnberg GF: Felix Imend�rffer, Jane Smithard, Graham Norton HRB 21284 (AG N�rnberg) Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850