From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <xiaojuntime@gmail.com>
Received: from mail-ig0-f173.google.com (mail-ig0-f173.google.com
 [209.85.213.173]) by dpdk.org (Postfix) with ESMTP id A56EB106A
 for <dev@dpdk.org>; Sat,  7 Feb 2015 10:15:38 +0100 (CET)
Received: by mail-ig0-f173.google.com with SMTP id a13so7721971igq.0
 for <dev@dpdk.org>; Sat, 07 Feb 2015 01:15:38 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113;
 h=mime-version:in-reply-to:references:from:date:message-id:subject:to
 :cc:content-type;
 bh=BE9/5eTBzJ4icZQhR1g6rvufhpBq3ESZyy1+/PR1G0g=;
 b=GYdeVoKdExWpdB/+Wtcec4wFUp4uEWX2hN91ywOTzFV2sAwY/9MnWWrtWMLNnHiepV
 vovu0opt/bWul2rcsTwGz45f8h5XikEdHK0mkdq25qvN9e09b8bdp2cOyVrD4N8lwq7A
 cu2n80NyjL8PYXAdIYXn2aR7NUDwe8k9S323y477XWO3Ph4kd24eJs1SinsNiqgRW7eo
 7Qbm2sAFTLsQh2QGXAGTNW3ZO9jxNg2pKDu4icjqE+J9znIvE8TqsTVuQ8I83QRpEgIY
 EE/dpr0S1Xmp6Zy2ijAgjRHY0BdrVXllfpFiZVaY6t6ah5UW1tvV42y+yrEDErxgyUO3
 Y0NQ==
X-Received: by 10.50.117.41 with SMTP id kb9mr6537028igb.37.1423300538059;
 Sat, 07 Feb 2015 01:15:38 -0800 (PST)
MIME-Version: 1.0
Received: by 10.36.96.132 with HTTP; Sat, 7 Feb 2015 01:15:17 -0800 (PST)
In-Reply-To: <1423247795-22399-1-git-send-email-stephen@networkplumber.org>
References: <1423247795-22399-1-git-send-email-stephen@networkplumber.org>
From: Jun Xiao <xiaojuntime@gmail.com>
Date: Sat, 7 Feb 2015 17:15:17 +0800
Message-ID: <CACyt=TG8EgADhVpiWN4cu56z904hBFH4LyUJAPvgrWhddqemBQ@mail.gmail.com>
To: Stephen Hemminger <stephen@networkplumber.org>
Content-Type: text/plain; charset=UTF-8
Cc: dev <dev@dpdk.org>, Stephen Hemminger <shemming@brocade.com>
Subject: Re: [dpdk-dev] [PATCH 0/4] Broadcom 10G NIC Poll Mode Driver
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: patches and discussions about DPDK <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Sat, 07 Feb 2015 09:15:39 -0000

Hi Stephen,

Thanks for your great work on this!

Could you elaborate on a few things:
- What's the methodology used in the driver porting?
  e.g. what's the base source you ported from?
  what's the major change against the base source?
- What kind of tests have been done on Linux? any known limitations?

Thanks,
Jun

On Sat, Feb 7, 2015 at 2:36 AM, Stephen Hemminger
<stephen@networkplumber.org> wrote:
> From: Stephen Hemminger <shemming@brocade.com>
>
> These are the patches to enable supporting the Broadcom
> NetExtreme II 10G devices (show up as bnx2x on Linux).
>
> The driver has only been tested on Linux, there maybe issues
> with firmware loading and PCI config access on BSD.
>
> Stephen Hemminger (4):
>   pci: allow access to PCI config space
>   bcm: add BCM pci device ids
>   bcm: new poll mode driver
>   bcm: enable BCM poll mode driver in config
>
>  config/common_linuxapp                          |    10 +
>  lib/Makefile                                    |     1 +
>  lib/librte_eal/common/include/rte_pci.h         |    29 +
>  lib/librte_eal/common/include/rte_pci_dev_ids.h |    30 +
>  lib/librte_eal/linuxapp/eal/eal_pci.c           |    15 +
>  lib/librte_eal/linuxapp/eal/eal_pci_uio.c       |    10 +
>  lib/librte_eal/linuxapp/eal/rte_eal_version.map |     2 +
>  lib/librte_pmd_bcm/Makefile                     |    28 +
>  lib/librte_pmd_bcm/bcm.c                        | 11817 +++++++++++++++++++
>  lib/librte_pmd_bcm/bcm.h                        |  1998 ++++
>  lib/librte_pmd_bcm/bcm_ethdev.c                 |   544 +
>  lib/librte_pmd_bcm/bcm_ethdev.h                 |    79 +
>  lib/librte_pmd_bcm/bcm_logs.h                   |    51 +
>  lib/librte_pmd_bcm/bcm_rxtx.c                   |   487 +
>  lib/librte_pmd_bcm/bcm_rxtx.h                   |    85 +
>  lib/librte_pmd_bcm/bcm_stats.c                  |  1619 +++
>  lib/librte_pmd_bcm/bcm_stats.h                  |   633 +
>  lib/librte_pmd_bcm/bcm_vfpf.c                   |   597 +
>  lib/librte_pmd_bcm/bcm_vfpf.h                   |   315 +
>  lib/librte_pmd_bcm/debug.c                      |   113 +
>  lib/librte_pmd_bcm/ecore_fw_defs.h              |   423 +
>  lib/librte_pmd_bcm/ecore_hsi.h                  |  6349 ++++++++++
>  lib/librte_pmd_bcm/ecore_init.h                 |   842 ++
>  lib/librte_pmd_bcm/ecore_init_ops.h             |   886 ++
>  lib/librte_pmd_bcm/ecore_mfw_req.h              |   207 +
>  lib/librte_pmd_bcm/ecore_reg.h                  |  3664 ++++++
>  lib/librte_pmd_bcm/ecore_sp.c                   |  5455 +++++++++
>  lib/librte_pmd_bcm/ecore_sp.h                   |  1796 +++
>  lib/librte_pmd_bcm/elink.c                      | 13378 ++++++++++++++++++++++
>  lib/librte_pmd_bcm/elink.h                      |   610 +
>  30 files changed, 52073 insertions(+)
>  create mode 100644 lib/librte_pmd_bcm/Makefile
>  create mode 100644 lib/librte_pmd_bcm/bcm.c
>  create mode 100644 lib/librte_pmd_bcm/bcm.h
>  create mode 100644 lib/librte_pmd_bcm/bcm_ethdev.c
>  create mode 100644 lib/librte_pmd_bcm/bcm_ethdev.h
>  create mode 100644 lib/librte_pmd_bcm/bcm_logs.h
>  create mode 100644 lib/librte_pmd_bcm/bcm_rxtx.c
>  create mode 100644 lib/librte_pmd_bcm/bcm_rxtx.h
>  create mode 100644 lib/librte_pmd_bcm/bcm_stats.c
>  create mode 100644 lib/librte_pmd_bcm/bcm_stats.h
>  create mode 100644 lib/librte_pmd_bcm/bcm_vfpf.c
>  create mode 100644 lib/librte_pmd_bcm/bcm_vfpf.h
>  create mode 100644 lib/librte_pmd_bcm/debug.c
>  create mode 100644 lib/librte_pmd_bcm/ecore_fw_defs.h
>  create mode 100644 lib/librte_pmd_bcm/ecore_hsi.h
>  create mode 100644 lib/librte_pmd_bcm/ecore_init.h
>  create mode 100644 lib/librte_pmd_bcm/ecore_init_ops.h
>  create mode 100644 lib/librte_pmd_bcm/ecore_mfw_req.h
>  create mode 100644 lib/librte_pmd_bcm/ecore_reg.h
>  create mode 100644 lib/librte_pmd_bcm/ecore_sp.c
>  create mode 100644 lib/librte_pmd_bcm/ecore_sp.h
>  create mode 100644 lib/librte_pmd_bcm/elink.c
>  create mode 100644 lib/librte_pmd_bcm/elink.h
>
> --
> 2.1.4
>