From: "Hubbe, Allen" <Allen.Hubbe@emc.com>
To: Bjorn Helgaas <bhelgaas@google.com>
CC: "linux-ntb@googlegroups.com" <linux-ntb@googlegroups.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
        Jon Mason <jdmason@kudzu.us>, Dave Jiang <dave.jiang@intel.com>
Subject: RE: [PATCH 03/16] ntb: Enable link training for RP mode in the
 driver probe
Thread-Topic: [PATCH 03/16] ntb: Enable link training for RP mode in the
 driver probe
Thread-Index: AQHQkz1v7cL4yn0pgk6+buW6rqOeBp2FonSA///CWRA=
Date: Wed, 20 May 2015 21:46:58 +0000
Message-ID: <40F65EF2B5E2254199711F58E3ACB84D7825D377@MX104CL02.corp.emc.com>
References: <cover.1432135631.git.Allen.Hubbe@emc.com>
 <ae66adf68441cdaf893d2a562924615c229489c0.1432135631.git.Allen.Hubbe@emc.com>
 <CAErSpo5gf9VhegEGY5S+dak0UuMNEMDD29JT4TcTz_UPey3aNQ@mail.gmail.com>
In-Reply-To: <CAErSpo5gf9VhegEGY5S+dak0UuMNEMDD29JT4TcTz_UPey3aNQ@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Content-Transfer-Encoding: 8bit
Content-Length: 3597
Lines: 98

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.

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
NTB: Intel NTB params for SNB B2B addresses
NTB: Check the DID for workaround error flags
NTB: Enable link training for RP mode in probe
NTB: Add NTB hardware abstraction layer
NTB: Move files in preparation for NTB abstraction

> 
> On Wed, May 20, 2015 at 10:41 AM, Allen Hubbe <Allen.Hubbe@emc.com>
> wrote:
> > From: Dave Jiang <dave.jiang@intel.com>
> >
> > 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 <dave.jiang@intel.com>
> > ---
> >  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
????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m????????????I?