Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754305AbbETWUn (ORCPT ); Wed, 20 May 2015 18:20:43 -0400 Received: from mail-qg0-f53.google.com ([209.85.192.53]:34472 "EHLO mail-qg0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753212AbbETWUj (ORCPT ); Wed, 20 May 2015 18:20:39 -0400 MIME-Version: 1.0 In-Reply-To: <40F65EF2B5E2254199711F58E3ACB84D7825D377@MX104CL02.corp.emc.com> References: <40F65EF2B5E2254199711F58E3ACB84D7825D377@MX104CL02.corp.emc.com> From: Bjorn Helgaas Date: Wed, 20 May 2015 17:20:18 -0500 Message-ID: Subject: Re: [PATCH 03/16] ntb: Enable link training for RP mode in the driver probe To: "Hubbe, Allen" Cc: "linux-ntb@googlegroups.com" , "linux-kernel@vger.kernel.org" , "linux-pci@vger.kernel.org" , Jon Mason , Dave Jiang Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4347 Lines: 123 On Wed, May 20, 2015 at 4:46 PM, Hubbe, Allen wrote: > From: Bjorn Helgaas [mailto:bhelgaas@google.com] >> Please run "git log --oneline drivers/ntb" and make your subject lines >> consistent. The convention I use in PCI (and what was used in NTB >> until recently) is >> >> - Acronyms and initialisms are capitalized >> - Subject line is a subsystem prefix ("PCI:", "NTB:", etc.) followed >> by a complete sentence starting with a capitalized verb, with no >> period at the end > > These are the new subject lines that will be in the v2 of this set. That helps a lot, thanks. > NTB: Fix small code format issues in transport > NTB: Default to CPU memcpy for performance > NTB: Improve performance with write combining > NTB: Use NUMA aware memory allocation > NTB: Use NUMA memory and DMA chan in transport > NTB: Reset transport QP link stats on down > NTB: Do not advance transport RX on link down > NTB: Differentiate transport link down messages > NTB: Rate limit ntb_qp_link_work > NTB: Add tool client > NTB: Add ping pong client I think these are test cases, not part of the kernel itself. It'd be nice to know that from the summary. I don't know what the precedent is for that (if there is one). > NTB: Intel NTB params for SNB B2B addresses This sentence no verb :) Not sure what you're doing with those params. > NTB: Check the DID for workaround error flags I think there's room to spell out "device ID". > NTB: Enable link training for RP mode in probe Is that Root Port? > NTB: Add NTB hardware abstraction layer > NTB: Move files in preparation for NTB abstraction Of course, none of this has anything to do with the patches themselves, which I don't intend to review because they're out of my area :) >> >> On Wed, May 20, 2015 at 10:41 AM, Allen Hubbe >> wrote: >> > From: Dave Jiang >> > >> > Link training for RP should be enabled in the driver probe. We should >> > not have to wait for transport loaded for this to hapen. Otherwise the >> >> s/hapen/happen/ >> >> > device will not show up on the transparent bridge side. >> > >> > Signed-off-by: Dave Jiang >> > --- >> > drivers/ntb/hw/intel/ntb_hw_intel.c | 10 ++++++++++ >> > 1 file changed, 10 insertions(+) >> > >> > diff --git a/drivers/ntb/hw/intel/ntb_hw_intel.c >> b/drivers/ntb/hw/intel/ntb_hw_intel.c >> > index 05c4b77..d162f22 100644 >> > --- a/drivers/ntb/hw/intel/ntb_hw_intel.c >> > +++ b/drivers/ntb/hw/intel/ntb_hw_intel.c >> > @@ -1333,6 +1333,9 @@ static int snb_poll_link(struct intel_ntb_dev >> *ndev) >> > >> > static int snb_link_is_up(struct intel_ntb_dev *ndev) >> > { >> > + if (ndev->ntb.topo == NTB_TOPO_SEC) >> > + return 1; >> > + >> > return NTB_LNK_STA_ACTIVE(ndev->lnk_sta); >> > } >> > >> > @@ -1642,6 +1645,7 @@ static int snb_setup_b2b_mw(struct intel_ntb_dev >> *ndev, >> > static int snb_init_ntb(struct intel_ntb_dev *ndev) >> > { >> > int rc; >> > + u32 ntb_ctl; >> > >> > if (ndev->bar4_split) >> > ndev->mw_count = HSX_SPLIT_BAR_MW_COUNT; >> > @@ -1658,6 +1662,12 @@ static int snb_init_ntb(struct intel_ntb_dev >> *ndev) >> > dev_err(ndev_dev(ndev), "NTB Primary config >> disabled\n"); >> > return -EINVAL; >> > } >> > + >> > + /* enable link to allow secondary side device to >> appear */ >> > + ntb_ctl = ioread32(ndev->self_mmio + ndev->reg- >> >ntb_ctl); >> > + ntb_ctl &= ~NTB_CTL_DISABLE; >> > + iowrite32(ntb_ctl, ndev->self_mmio + ndev->reg- >> >ntb_ctl); >> > + >> > /* use half the spads for the peer */ >> > ndev->spad_count >>= 1; >> > ndev->self_reg = &snb_pri_reg; >> > -- >> > 2.4.0.rc0.43.gcf8a8c6 >> > >> > -- >> > To unsubscribe from this list: send the line "unsubscribe linux-pci" >> in >> > the body of a message to majordomo@vger.kernel.org >> > More majordomo info at http://vger.kernel.org/majordomo-info.html -- 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/