[dpdk-dev] [PATCH v6 00/38] new bnxt poll mode driver library

2016-06-21 Thread Bruce Richardson
On Tue, Jun 21, 2016 at 09:25:18AM -0500, Ajit Khaparde wrote:
> Bruce,
> We have a set of patches which add support to a couple of new Broadcom PCI
> devices.
> Should I use the dpdk-next-net/rel_16_07 to push them out?
> Let me know if there is any other way to do this.
> 
> Thanks
> 

They should be based on dpdk-next-net/rel_16_07.

However, be aware that it is very late in the DPDK 16.07 release cycle for us
to take new content in, so please minimise the patchset to only the changes that
are absolutely necessary for you. At this stage we can only take contributions
on a best-effort basis, so having a smaller patchset will increase your chances
that the patch may be merged.

Regards,
/Bruce

> On Thu, Jun 16, 2016 at 1:51 PM, Ajit Khaparde 
> wrote:
> 
> >
> > On Thu, Jun 16, 2016 at 9:24 AM, Bruce Richardson <
> > bruce.richardson at intel.com> wrote:
> >
> >> On Wed, Jun 15, 2016 at 02:23:00PM -0700, Stephen Hurd wrote:
> >> > The bnxt poll mode library (librte_pmd_bnxt) implements support for
> >> > Broadcom NetXtreme C-Series.  These adapters support Standards-
> >> > compliant 10/25/50Gbps 30MPPS full-duplex throughput.
> >> >
> >> > Information about this family of adapters can be found in the
> >> > NetXtreme Brand section https://goo.gl/4H7q63 of the Broadcom web
> >> > site http://www.broadcom.com/
> >> >
> >> > With the current driver, allocated mbufs must be large enough to hold
> >> > the entire received frame.  If the mbufs are not large enough, the
> >> > packets will be dropped.  This is most limiting when jumbo frames are
> >> > used.
> >> >
> >>
> >> Applied to dpdk-next-net/rel_16_07
> >>
> >> On apply I got conflicts with the nic overview document, so please check
> >> the
> >> resulting information in that document is correct in the next-net tree.
> >> I also added a very short entry to the release notes for this new driver
> >> as
> >> part of patch 1, since that was missing. Please also check that for
> >> correctness
> >> and send on any additional comments/corrections you want on that.
> >>
> > Thanks Bruce. I had a cursory glance ?and it looked good.
> > We will update the
> > ?m? further
> > if necessary.
> > ?
> >
> >
> >> Thanks for all the work on this driver.
> >>
> >> Regards,
> >> /Bruce
> >>
> >
> >


[dpdk-dev] [PATCH v6 00/38] new bnxt poll mode driver library

2016-06-21 Thread Ajit Khaparde
Bruce,
We have a set of patches which add support to a couple of new Broadcom PCI
devices.
Should I use the dpdk-next-net/rel_16_07 to push them out?
Let me know if there is any other way to do this.

Thanks

On Thu, Jun 16, 2016 at 1:51 PM, Ajit Khaparde 
wrote:

>
> On Thu, Jun 16, 2016 at 9:24 AM, Bruce Richardson <
> bruce.richardson at intel.com> wrote:
>
>> On Wed, Jun 15, 2016 at 02:23:00PM -0700, Stephen Hurd wrote:
>> > The bnxt poll mode library (librte_pmd_bnxt) implements support for
>> > Broadcom NetXtreme C-Series.  These adapters support Standards-
>> > compliant 10/25/50Gbps 30MPPS full-duplex throughput.
>> >
>> > Information about this family of adapters can be found in the
>> > NetXtreme Brand section https://goo.gl/4H7q63 of the Broadcom web
>> > site http://www.broadcom.com/
>> >
>> > With the current driver, allocated mbufs must be large enough to hold
>> > the entire received frame.  If the mbufs are not large enough, the
>> > packets will be dropped.  This is most limiting when jumbo frames are
>> > used.
>> >
>>
>> Applied to dpdk-next-net/rel_16_07
>>
>> On apply I got conflicts with the nic overview document, so please check
>> the
>> resulting information in that document is correct in the next-net tree.
>> I also added a very short entry to the release notes for this new driver
>> as
>> part of patch 1, since that was missing. Please also check that for
>> correctness
>> and send on any additional comments/corrections you want on that.
>>
> Thanks Bruce. I had a cursory glance ?and it looked good.
> We will update the
> ?m? further
> if necessary.
> ?
>
>
>> Thanks for all the work on this driver.
>>
>> Regards,
>> /Bruce
>>
>
>


[dpdk-dev] [PATCH v6 00/38] new bnxt poll mode driver library

2016-06-16 Thread Bruce Richardson
On Wed, Jun 15, 2016 at 02:23:00PM -0700, Stephen Hurd wrote:
> The bnxt poll mode library (librte_pmd_bnxt) implements support for
> Broadcom NetXtreme C-Series.  These adapters support Standards-
> compliant 10/25/50Gbps 30MPPS full-duplex throughput.
> 
> Information about this family of adapters can be found in the
> NetXtreme Brand section https://goo.gl/4H7q63 of the Broadcom web
> site http://www.broadcom.com/
> 
> With the current driver, allocated mbufs must be large enough to hold
> the entire received frame.  If the mbufs are not large enough, the
> packets will be dropped.  This is most limiting when jumbo frames are
> used.
> 

Applied to dpdk-next-net/rel_16_07

On apply I got conflicts with the nic overview document, so please check the
resulting information in that document is correct in the next-net tree.
I also added a very short entry to the release notes for this new driver as
part of patch 1, since that was missing. Please also check that for correctness
and send on any additional comments/corrections you want on that.

Thanks for all the work on this driver.

Regards,
/Bruce


[dpdk-dev] [PATCH v6 00/38] new bnxt poll mode driver library

2016-06-16 Thread Ajit Khaparde
On Thu, Jun 16, 2016 at 9:24 AM, Bruce Richardson <
bruce.richardson at intel.com> wrote:

> On Wed, Jun 15, 2016 at 02:23:00PM -0700, Stephen Hurd wrote:
> > The bnxt poll mode library (librte_pmd_bnxt) implements support for
> > Broadcom NetXtreme C-Series.  These adapters support Standards-
> > compliant 10/25/50Gbps 30MPPS full-duplex throughput.
> >
> > Information about this family of adapters can be found in the
> > NetXtreme Brand section https://goo.gl/4H7q63 of the Broadcom web
> > site http://www.broadcom.com/
> >
> > With the current driver, allocated mbufs must be large enough to hold
> > the entire received frame.  If the mbufs are not large enough, the
> > packets will be dropped.  This is most limiting when jumbo frames are
> > used.
> >
>
> Applied to dpdk-next-net/rel_16_07
>
> On apply I got conflicts with the nic overview document, so please check
> the
> resulting information in that document is correct in the next-net tree.
> I also added a very short entry to the release notes for this new driver as
> part of patch 1, since that was missing. Please also check that for
> correctness
> and send on any additional comments/corrections you want on that.
>
Thanks Bruce. I had a cursory glance ?and it looked good.
We will update the
?m? further
if necessary.
?


> Thanks for all the work on this driver.
>
> Regards,
> /Bruce
>


[dpdk-dev] [PATCH v6 00/38] new bnxt poll mode driver library

2016-06-15 Thread Stephen Hurd
The bnxt poll mode library (librte_pmd_bnxt) implements support for
Broadcom NetXtreme C-Series.  These adapters support Standards-
compliant 10/25/50Gbps 30MPPS full-duplex throughput.

Information about this family of adapters can be found in the
NetXtreme Brand section https://goo.gl/4H7q63 of the Broadcom web
site http://www.broadcom.com/

With the current driver, allocated mbufs must be large enough to hold
the entire received frame.  If the mbufs are not large enough, the
packets will be dropped.  This is most limiting when jumbo frames are
used.

Ajit Khaparde (38):
  bnxt: new driver for Broadcom NetXtreme-C devices
  bnxt: add HWRM init code
  bnxt: add driver register/unregister support
  bnxt: add dev infos get operation
  bnxt: add dev configure operation
  bnxt: add vnic functions and structs
  bnxt: declare generic ring structs and free() func
  bnxt: add completion ring support
  bnxt: add L2 filter alloc/init/free
  bnxt: add Tx queue operations (nonfunctional)
  bnxt: add Rx queue create/destroy operations
  bnxt: add statistics operations
  bnxt: add initial Tx code implementation
  bnxt: add initial Rx code implementation
  bnxt: add code to alloc/free Tx Rx and cmpl rings
  bnxt: add HWRM function reset command
  bnxt: add HWRM vnic alloc function
  bnxt: add HWRM vnic free function
  bnxt: add HWRM vnic configure function
  bnxt: add API to allow configuration of vnic
  bnxt: add HWRM API to configure RSS
  bnxt: add API for L2 Rx mask set/clear functions
  bnxt: add HWRM API for stats context allocation
  bnxt: add HWRM ring alloc/free functions
  bnxt: add ring group alloc/free functions
  bnxt: add HWRM stat context free function
  bnxt: add HWRM API to set and clear filters
  bnxt: allocate and free all HWRM rings and groups
  bnxt: add HWRM port PHY config call and helpers
  bnxt: add start/stop/link update operations
  bnxt: add promiscuous enable/disable operations
  bnxt: add all multicast enable/disable operations
  bnxt: free memory in close operation
  bnxt: add MAC address add/remove dev ops
  bnxt: add set link up/down operations
  bnxt: add reta update/query operations
  bnxt: add RSS device operations
  bnxt: add flow control operations

 MAINTAINERS |5 +
 config/common_base  |5 +
 doc/guides/nics/bnxt.rst|   49 +
 doc/guides/nics/index.rst   |1 +
 doc/guides/nics/overview.rst|  138 +-
 drivers/net/Makefile|1 +
 drivers/net/bnxt/Makefile   |   73 +
 drivers/net/bnxt/bnxt.h |  176 +
 drivers/net/bnxt/bnxt_cpr.c |  159 +
 drivers/net/bnxt/bnxt_cpr.h |   86 +
 drivers/net/bnxt/bnxt_ethdev.c  | 1049 
 drivers/net/bnxt/bnxt_filter.c  |  175 +
 drivers/net/bnxt/bnxt_filter.h  |   74 +
 drivers/net/bnxt/bnxt_hwrm.c| 1491 ++
 drivers/net/bnxt/bnxt_hwrm.h|  104 +
 drivers/net/bnxt/bnxt_ring.c|  306 ++
 drivers/net/bnxt/bnxt_ring.h|  103 +
 drivers/net/bnxt/bnxt_rxq.c |  319 ++
 drivers/net/bnxt/bnxt_rxq.h |   74 +
 drivers/net/bnxt/bnxt_rxr.c |  367 ++
 drivers/net/bnxt/bnxt_rxr.h |   62 +
 drivers/net/bnxt/bnxt_stats.c   |  142 +
 drivers/net/bnxt/bnxt_stats.h   |   44 +
 drivers/net/bnxt/bnxt_txq.c |  162 +
 drivers/net/bnxt/bnxt_txq.h |   75 +
 drivers/net/bnxt/bnxt_txr.c |  339 ++
 drivers/net/bnxt/bnxt_txr.h |   71 +
 drivers/net/bnxt/bnxt_vnic.c|  277 ++
 drivers/net/bnxt/bnxt_vnic.h|   80 +
 drivers/net/bnxt/hsi_struct_def_dpdk.h  | 5799 +++
 drivers/net/bnxt/rte_pmd_bnxt_version.map   |4 +
 lib/librte_eal/common/include/rte_pci_dev_ids.h |   38 +-
 mk/rte.app.mk   |1 +
 33 files changed, 11775 insertions(+), 74 deletions(-)
 create mode 100644 doc/guides/nics/bnxt.rst
 create mode 100644 drivers/net/bnxt/Makefile
 create mode 100644 drivers/net/bnxt/bnxt.h
 create mode 100644 drivers/net/bnxt/bnxt_cpr.c
 create mode 100644 drivers/net/bnxt/bnxt_cpr.h
 create mode 100644 drivers/net/bnxt/bnxt_ethdev.c
 create mode 100644 drivers/net/bnxt/bnxt_filter.c
 create mode 100644 drivers/net/bnxt/bnxt_filter.h
 create mode 100644 drivers/net/bnxt/bnxt_hwrm.c
 create mode 100644 drivers/net/bnxt/bnxt_hwrm.h
 create mode 100644 drivers/net/bnxt/bnxt_ring.c
 create mode 100644 drivers/net/bnxt/bnxt_ring.h
 create mode 100644 drivers/net/bnxt/bnxt_rxq.c
 create mode 100644 drivers/net/bnxt/bnxt_rxq.h
 create mode 100644 drivers/net/bnxt/bnxt_rxr.c
 create mode 100644 drivers/net/bnxt/bnxt_rxr.h
 create