Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965630AbbDVP21 (ORCPT ); Wed, 22 Apr 2015 11:28:27 -0400 Received: from cantor2.suse.de ([195.135.220.15]:49986 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757190AbbDVP2Y (ORCPT ); Wed, 22 Apr 2015 11:28:24 -0400 Date: Wed, 22 Apr 2015 17:28:21 +0200 From: "Luis R. Rodriguez" To: Doug Ledford Cc: "Luis R. Rodriguez" , infinipath@intel.com, roland@kernel.org, sean.hefty@intel.com, hal.rosenstock@gmail.com, linux-rdma@vger.kernel.org, luto@amacapital.net, mst@redhat.com, linux-kernel@vger.kernel.org, cocci@systeme.lip6.fr, Toshi Kani , Suresh Siddha , Ingo Molnar , Thomas Gleixner , Juergen Gross , Daniel Vetter , Dave Airlie , Antonino Daplas , Jean-Christophe Plagniol-Villard , Tomi Valkeinen , linux-fbdev@vger.kernel.org Subject: Re: [PATCH v4 1/2] IB/qib: add acounting for MTRR Message-ID: <20150422152821.GC5622@wotan.suse.de> References: <1429653035-19424-1-git-send-email-mcgrof@do-not-panic.com> <1429653035-19424-2-git-send-email-mcgrof@do-not-panic.com> <1429710292.45956.90.camel@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1429710292.45956.90.camel@redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2654 Lines: 64 On Wed, Apr 22, 2015 at 09:44:52AM -0400, Doug Ledford wrote: > On Tue, 2015-04-21 at 14:50 -0700, Luis R. Rodriguez wrote: > > From: "Luis R. Rodriguez" > > > > There is no good reason not to, we eventually delete it as well. > > > > Cc: Toshi Kani > > Cc: Suresh Siddha > > Cc: Ingo Molnar > > Cc: Thomas Gleixner > > Cc: Juergen Gross > > Cc: Daniel Vetter > > Cc: Andy Lutomirski > > Cc: Dave Airlie > > Cc: Antonino Daplas > > Cc: Jean-Christophe Plagniol-Villard > > Cc: Tomi Valkeinen > > Cc: Mike Marciniszyn > > Cc: Roland Dreier > > Cc: Sean Hefty > > Cc: Hal Rosenstock > > Cc: linux-rdma@vger.kernel.org > > Cc: linux-fbdev@vger.kernel.org > > Cc: linux-kernel@vger.kernel.org > > Signed-off-by: Luis R. Rodriguez > > --- > > drivers/infiniband/hw/qib/qib_wc_x86_64.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/infiniband/hw/qib/qib_wc_x86_64.c b/drivers/infiniband/hw/qib/qib_wc_x86_64.c > > index 81b225f..fe0850a 100644 > > --- a/drivers/infiniband/hw/qib/qib_wc_x86_64.c > > +++ b/drivers/infiniband/hw/qib/qib_wc_x86_64.c > > @@ -118,7 +118,7 @@ int qib_enable_wc(struct qib_devdata *dd) > > if (!ret) { > > int cookie; > > > > - cookie = mtrr_add(pioaddr, piolen, MTRR_TYPE_WRCOMB, 0); > > + cookie = mtrr_add(pioaddr, piolen, MTRR_TYPE_WRCOMB, 1); > > if (cookie < 0) { > > { > > qib_devinfo(dd->pcidev, > > Skip this patch please. You remove this line entirely in your next > patch, so this becomes a single kernel out of all possible bisectable > kernels with this accounting enabled, and then the very next kernel does > away with it. No, the next patch uses accounting enabled as well, it also makes somse other changes. This change is done in order to add accounting to match the grammar used by arch_phys_wc_add() so it is in fact an atomic comittiable and highly recommmended bisectable commit to be present. > It makes no sense to have a single outlying bisectable > kernel like that. This is an atomic difference worth keeping record of. Luis -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/