linux-kernel - Re: [PATCH 03/16] ntb: Enable link training for RP mode in the driver probe

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAErSpo5gf9VhegEGY5S+dak0UuMNEMDD29JT4TcTz_UPey3aNQ@mail.gmail.com>
Date:	Wed, 20 May 2015 16:21:31 -0500
From:	Bjorn Helgaas <bhelgaas@...gle.com>
To:	Allen Hubbe <Allen.Hubbe@....com>
Cc:	linux-ntb@...glegroups.com,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linux-pci@...r.kernel.org" <linux-pci@...r.kernel.org>,
	Jon Mason <jdmason@...zu.us>, Dave Jiang <dave.jiang@...el.com>
Subject: Re: [PATCH 03/16] ntb: Enable link training for RP mode in the driver probe

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

On Wed, May 20, 2015 at 10:41 AM, Allen Hubbe <Allen.Hubbe@....com> wrote:
> From: Dave Jiang <dave.jiang@...el.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@...el.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@...r.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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/