On 6 May 2016 at 21:47, Jan Viktorin wrote:
> Generalize the PCI-specific pci_unbind_kernel_driver. It is now divided into
> two parts. First, determination of the path and string identification of the
> device to be unbound. Second, the actual unbind operation which is generic.
>
> Signed-off-by:
On 12 May 2016 at 18:31, Santosh Shukla
wrote:
> On Thu, May 12, 2016 at 05:52:54PM +0800, Jianbo Liu wrote:
>> On 12 May 2016 at 16:57, Santosh Shukla
>> wrote:
>> > On Thu, May 12, 2016 at 01:54:13PM +0800, Jianbo Liu wrote:
>> >> On 12 May 2016 at 13:06, Santosh Shukla
>> >> wrote:
>> >> > On
On 5/13/2016 12:40 AM, Yuanhan Liu wrote:
> On Thu, May 12, 2016 at 03:08:05PM +0800, Tan, Jianfeng wrote:
+static int
+vdev_setup_queue(struct virtio_hw *hw __rte_unused, struct virtqueue *vq)
+{
+ /* Changed to use virtual addr */
+ vq->vq_ring_mem = (phys_addr_t)vq->
Hi Michael,
On 5/13/2016 1:02 AM, Michael S. Tsirkin wrote:
> On Thu, May 12, 2016 at 03:08:05PM +0800, Tan, Jianfeng wrote:
>> (2) It's more aligned to previous logic to hide the detail to differentiate
>> modern/legacy device.
> Why is there a need to support legacy interfaces at all? It's a c
Hi, Nikita
dev_conf.rxmode.max_rx_pkt_len is different with MTU concept.
The max_rx_pkt_len indicates the maximum packet length it can receive, it
should be larger than MTU.
There is another patch which is enabling set_mtu ops.
http://dpdk.org/dev/patchwork/patch/12218/
You can definitely help t
On 6 May 2016 at 21:47, Jan Viktorin wrote:
> This option has the same meaning for the SoC infra as the --no-pci
> for the PCI infra.
>
> Signed-off-by: Jan Viktorin
> ---
> lib/librte_eal/common/eal_common_options.c | 5 +
> lib/librte_eal/common/eal_internal_cfg.h | 1 +
> lib/librte_eal
> -Original Message-
> From: Jianbo Liu [mailto:jianbo.liu at linaro.org]
> Sent: Friday, May 13, 2016 7:13 AM
> To: Santosh Shukla
> Cc: Stephen Hemminger ; Jerin Jacob
> ; Hemant Agrawal
> ; dev at dpdk.org; Thomas Monjalon
>
> Subject: Re: [dpdk-dev] [PATCHv3 1/2] config/armv8a: disa
This patch docs the issue on EAL argument that the last EAL
argument is replaced by program name in argv[].
Reported-by: Ziye Yang
Signed-off-by: Jingjing Wu
---
doc/guides/rel_notes/known_issues.rst | 19 +++
1 file changed, 19 insertions(+)
diff --git a/doc/guides/rel_notes/k
Hi Team,
I started working on implementing the QoS Shaping in OVS+DPDK by making use of
rte_sched library provided in DPDK. Meanwhile to compare the performance,
started performance test with DPDK sample scheduling application. Below are the
configuration details of system which I am using,
S
On Fri, May 13, 2016 at 03:37:01AM +, Hemant Agrawal wrote:
>
>
> > -Original Message-
> > From: Jianbo Liu [mailto:jianbo.liu at linaro.org]
> > Sent: Friday, May 13, 2016 7:13 AM
> > To: Santosh Shukla
> > Cc: Stephen Hemminger ; Jerin Jacob
> > ; Hemant Agrawal
> > ; dev at dpdk.o
On Thu, May 12, 2016 at 03:39:56PM +, Pattan, Reshma wrote:
>
>
> > -Original Message-
> > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jerin Jacob
> > Sent: Saturday, May 7, 2016 4:16 PM
> > To: dev at dpdk.org
> > Cc: thomas.monjalon at 6wind.com; Richardson, Bruce
> > ;
This patch enables configuring MTU for i40e.
Since changing MTU needs to reconfigure queue, stop port first
before configuring MTU.
Signed-off-by: Beilei Xing
---
v3 changes:
Add frame size with extra I40E_VLAN_TAG_SIZE.
Delete i40e_dev_rx_init(pf) cause it will be called when port starts.
v2
2016-05-10 10:24, Arek Kusztal:
> From: Arkadiusz Kusztal
>
> Changed to -O3 optimization flag in Intel QuickAssist Technology Makefile
There is another change below. Should it be a separate patch?
> --- a/drivers/crypto/qat/qat_adf/qat_algs_build_desc.c
> +++ b/drivers/crypto/qat/qat_adf/qat_a
Hi Thomas,
This change was made to fix the compilation issues arising when optimization
flag was changed. Hence its related to same optimization flag patch.
Do you want us to separate the Makefile and .c file change?
Regards,
Deepak
-Original Message-
From: Thomas Monjalon [mailto:thoma
Fix issue reported by Coverity.
Coverity ID 30693: Unchecked return value
check_return: Calling rte_meter_srtcm_config without checking return value.
Fixes: e6541fdec8b2 ("meter: initial import")
Signed-off-by: Slawomir Mrozowicz
---
examples/qos_meter/main.c | 16
examples/qo
On Thu, May 12, 2016 at 4:41 PM, Jan Viktorin
wrote:
> Hi,
>
> Just a note, please, when replying inline, do not prepend ">" before your
> new text. I could not find your replies.
>
>
My gmail interface does not show that prepend character... It seems I have
to leave a white line before my replie
> -Original Message-
> From: Mrozowicz, SlawomirX
> Sent: Friday, May 13, 2016 9:35 AM
> To: Dumitrescu, Cristian
> Cc: dev at dpdk.org; Singh, Jasvinder ;
> Mrozowicz, SlawomirX
> Subject: [PATCH v4] examples/qos_meter: fix unchecked return value
>
> Fix issue reported by Coverity.
>
On Tue, May 10, 2016 at 02:01:04PM +0530, Jianbo Liu wrote:
> Other APP may call rte_memcpy by function pointer,
Instead of "Other APP" may be better to use DPDK application
> so change it to an inline function.
>
> Signed-off-by: Jianbo Liu
Acked-by: Jerin Jacob
> ---
> lib/librte_eal/comm
> -Original Message-
> From: Mrzyglod, DanielX T
> Sent: Friday, May 6, 2016 6:55 PM
> To: Kerlin, MarcinX ; Dumitrescu, Cristian
> ; Singh, Jasvinder
>
> Cc: dev at dpdk.org; Mrzyglod, DanielX T
> Subject: [PATCH] examples/ip_pipline: fix memory initialization in firewall
> bulk functi
On 13 May 2016 at 15:47, Jerin Jacob wrote:
> On Fri, May 13, 2016 at 03:37:01AM +, Hemant Agrawal wrote:
>>
>>
>> > -Original Message-
>> > From: Jianbo Liu [mailto:jianbo.liu at linaro.org]
>> > Sent: Friday, May 13, 2016 7:13 AM
>> > To: Santosh Shukla
>> > Cc: Stephen Hemminger ;
2016-05-11 10:46, Ferruh Yigit:
> On 5/10/2016 6:18 PM, Dumitrescu, Cristian wrote:
> > As previously discussed on this email list, the rte_bitmap_free() is an API
> > function that works as a placeholder for any resource freeing that needs to
> > be done for the bitmap. The API function should n
2016-05-12 08:55, Sergio Gonzalez Monroy:
> On 11/05/2016 17:01, Daniel Mrzyglod wrote:
> > Fix issue reported by Coverity.
> > Coverity ID 97920
> >
> > munmap structure of hugepage
> >
> > leaked_storage: Variable hugepage going out of scope leaks the storage
> > it points to.
> >
> > The system
> -Original Message-
> From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com]
> Sent: Friday, May 13, 2016 11:12 AM
> To: Dumitrescu, Cristian
> Cc: dev at dpdk.org; Yigit, Ferruh ; Mrzyglod,
> DanielX
> T
> Subject: Re: [dpdk-dev] [PATCH] sched: fix useless call
>
> 2016-05-11 1
2016-05-11 19:17, Hemant Agrawal:
> IGB_UIO not supported for arm64 arch in kernel so disable.
If I understand well, a patch is needed in the kernel to make
igb_uio works? Please confirm.
In that case, yes, the default configuration should be to disable
igb_uio.
Please note it's just a default to
This patchset allows virtio-net pmd to run on ppc64 processors.
The main thing that as missing was the support of ioports in EAL.
It also fixes some endianess issues in PCI config accesses in
legacy mode.
This is validated with test-pmd:
=== HOST
mkdir -p /mnt/huge
mount -t hugetlbfs nodev /mnt/
Fix some typos and add missing comments related to ioports API in
rte_pci.h.
Fixes: 756ce64b1 ("eal: introduce PCI ioport API")
Signed-off-by: Olivier Matz
---
lib/librte_eal/common/include/rte_pci.h | 9 ++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/lib/librte_eal/comm
>From iopl(2) man page: "This call is mostly for the x86 architecture. On
many other architectures it does not exist or will always return an
error".
This patch removes the call to iopl() in rte_eal_iopl_init() for
architectures other than x86, and always return 0 (success). This was
already done
In a previous commit, the file used to map the PCI resources changed
from "/dev/uio" to "/sys/bus/pci/devices//resource", making
the comment wrong. Remove it.
Fixes: 9e67561acd1a ("eal/linux: mmap uio resources using resourceX files")
Signed-off-by: Olivier Matz
---
lib/librte_eal/linuxapp/eal/e
Split pci_parse_sysfs_resource() and introduce
pci_parse_one_sysfs_resource() that parses one line of sysfs resource
file.
This new function will be exported and used in next commits when
mapping the ioports resources.
No functional change.
Signed-off-by: Olivier Matz
---
lib/librte_eal/linuxa
On PPC64, the ioports are mapped in memory. Implement the missing part
of ioport API for PPC64 when using uio. This may also work on other
architectures but it has not been tested.
Signed-off-by: David Marchand
Signed-off-by: Olivier Matz
---
lib/librte_eal/common/include/rte_pci.h| 4 +-
From: David Marchand
Although ppc supports both endianesses, qemu supposes that the cpu is
big endian and enforces this for the virtio-net stuff.
Fix PCI accesses in legacy mode. Only ppc64le is supported at the moment.
Signed-off-by: David Marchand
Signed-off-by: Olivier Matz
---
drivers/ne
Now that virtio pmd is supported on ppc, enable it.
Signed-off-by: Olivier Matz
---
config/defconfig_ppc_64-power8-linuxapp-gcc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/config/defconfig_ppc_64-power8-linuxapp-gcc
b/config/defconfig_ppc_64-power8-linuxapp-gcc
index 9eb
On Fri, May 13, 2016 at 02:50:48PM +0200, Thomas Monjalon wrote:
> 2016-05-11 19:17, Hemant Agrawal:
> > IGB_UIO not supported for arm64 arch in kernel so disable.
>
> If I understand well, a patch is needed in the kernel to make
> igb_uio works? Please confirm.
>
Yes. User need this [1] out-of-t
We don't want to have this instructions in the generated docs, so use
comments. It's also less confusing for people adding entries in the
documentation.
Signed-off-by: Olivier Matz
---
doc/guides/rel_notes/release_16_07.rst | 86 +-
1 file changed, 43 insertions(+
2016-05-13 09:38, Alejandro Lucero:
> On Thu, May 12, 2016 at 4:41 PM, Jan Viktorin
> > Alejandro Lucero wrote:
> > > On Thu, May 12, 2016 at 3:52 PM, Jan Viktorin
> > > > "Alejandro.Lucero" wrote:
> > > > > - New dma_mask field in rte_eth_dev_data.
> > > > > - If PMD sets device dma_mask, cal
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Jerin Jacob
> Sent: Saturday, May 7, 2016 4:16 PM
> To: dev at dpdk.org
> Cc: thomas.monjalon at 6wind.com; Richardson, Bruce
> ; Jerin Jacob
> ; Maciej Czekaj
> ; Kamil Rytarowski
> ; Zyta Szpak
> ; Slawomir R
Hello Olivier,
On Fri, May 13, 2016 at 2:50 PM, Olivier Matz wrote:
> From: David Marchand
>
> Although ppc supports both endianesses, qemu supposes that the cpu is
> big endian and enforces this for the virtio-net stuff.
>
> Fix PCI accesses in legacy mode. Only ppc64le is supported at the mome
On Fri, May 13, 2016 at 2:50 PM, Olivier Matz wrote:
> On PPC64, the ioports are mapped in memory. Implement the missing part
> of ioport API for PPC64 when using uio. This may also work on other
> architectures but it has not been tested.
>
> Signed-off-by: David Marchand
> Signed-off-by: Olivie
2016-05-09 23:34, Jan Viktorin:
> The qede PMD driver is failing when building for ARMv7:
>
> drivers/net/qede/base/ecore_dev.c: In function ?ecore_hw_init_pf?:
> drivers/net/qede/base/ecore_dev.c:1150:6:
> error: variable ?prs_reg? set but not used
> [-Werror=unused-but-set-variable]
> u
On Thu, 12 May 2016 17:34:13 +0200
Thomas Monjalon wrote:
> 2016-05-10 20:13, Jan Viktorin:
> > The current test_pci is just a single test case that tests the blacklisting
> > of devices. Rename it to test_pci_blacklist and call it from the test_pci.
>
> The functions are also moved. It is con
> >fix errors:
> >icc: command line warning #10006: ignoring unknown option
> >'-Wno-unused-value'
> >icc: command line warning #10006: ignoring unknown option
> >'-Wno-format-nonliteral'
> >icc: command line warning #10006: ignoring unknown option
> >'-Wno-shift-negative-value'
> >qede/base/ecore_
On Thu, 12 May 2016 18:10:07 +0200
Thomas Monjalon wrote:
> 2016-05-12 17:46, Jan Viktorin:
> > On Thu, 12 May 2016 17:41:22 +0200
> > Thomas Monjalon wrote:
> > > 2016-05-10 20:13, Jan Viktorin:
> > > > + orig = pci_get_sysfs_path();
> > > > + ret = setenv("SYSFS_PCI_DEVICES", "
2016-05-13 17:19, Jan Viktorin:
> On Thu, 12 May 2016 17:34:13 +0200
> Thomas Monjalon wrote:
>
> > 2016-05-10 20:13, Jan Viktorin:
> > > The current test_pci is just a single test case that tests the
> > > blacklisting
> > > of devices. Rename it to test_pci_blacklist and call it from the
> >
Self review:
* missing #ifdef VFIO_PRESENT ... #endif in eal_vfio.c leads to fail
when VFIO build is disabled
Jan
On Fri, 29 Apr 2016 15:44:12 +0200
Jan Viktorin wrote:
> The vfio_cfg is a module-global variable and so together with this
> variable, it is necessary to move functions:
>
> *
2016-05-10 13:01, Panu Matilainen:
> With gcc >= 6.0, qede base driver fails to build with:
> drivers/net/qede/base/ecore_cxt.c: In function 'ecore_cdu_init_common':
> cc1: error: left shift of negative value [-Werror=shift-negative-value]
>
> Since the base drivers are untouchable, work around by
The new tool uses /sys/devices instead of /proc directory, which
does not exist on all systems. If the procfs is not available
then memory and huge page information is skipped.
The tool also can emit a json format in short or long form to
allow for machine readable information.
Here is the usage
Sorry this one should have been an RFC instead of a patch.
I create this new tool to combine some information and use /sys/devices
instead. What I was hoping was some of you could try this script and see if it
works correctly. Also I was hope to find out if this script is useful and what
other
2016-05-10 14:01, Jianbo Liu:
> Other APP may call rte_memcpy by function pointer,
> so change it to an inline function.
Any example in mind?
> --- a/lib/librte_eal/common/include/arch/arm/rte_memcpy_64.h
> +++ b/lib/librte_eal/common/include/arch/arm/rte_memcpy_64.h
> -#define rte_memcpy(d, s, n
2016-04-29 17:23, Mauricio Vasquez B:
> The RTE_ETH_VALID_PORTID_OR_ERR_RET macro is used in some places
> to check if a port id is valid or not. This commit makes use of it in
> some new parts of the code.
There are other occurences:
rte_eth_dev_socket_id
rte_eth_add_rx_callback
Hi David,
On 05/13/2016 04:28 PM, David Marchand wrote:
>> +#ifdef RTE_ARCH_PPC_64
>> + switch (length) {
>> + case 4:
>> + *(uint32_t *)dst = rte_be_to_cpu_32(*(uint32_t *)dst);
>> + break;
>> + case 2:
>> + *(uint16_t *)dst = rte_be_to_
On 05/13/2016 04:33 PM, David Marchand wrote:
>> --- a/lib/librte_eal/common/include/rte_pci.h
>> +++ b/lib/librte_eal/common/include/rte_pci.h
>> @@ -105,7 +105,8 @@ extern struct pci_device_list pci_device_list; /**<
>> Global list of PCI devices.
>> /** Nb. of values in PCI resource format.
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Stephen Hemminger
> Sent: Friday, May 06, 2016 4:56 PM
> To: Doherty, Declan
> Cc: Iremonger, Bernard; dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH 1/5] bonding: replace spinlock with read/write
> lock
>
sorry for the noise, it turned out I need to add the virtio pmd in the
mTCP app Makefile as below
LIBS += -m64 -g -O3 -pthread -lrt -march=native -Wl,-export-dynamic
${MTCP_FLD}/lib/libmtcp.a -L../../dpdk/lib -Wl,-lnuma -Wl,-lmtcp
-Wl,-lpthread -Wl,-lrt -Wl,-ldl -Wl,--whole-archive
-Wl,-lrte_distr
> -Original Message-
> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Ananyev, Konstantin
> Sent: Friday, May 13, 2016 6:11 PM
> To: Stephen Hemminger; Doherty, Declan
> Cc: Iremonger, Bernard; dev at dpdk.org
> Subject: Re: [dpdk-dev] [PATCH 1/5] bonding: replace spinlock with r
Prefer !ptr to ptr == NULL
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_ethdev.c | 10 +-
drivers/net/bnxt/bnxt_filter.c | 2 +-
drivers/net/bnxt/bnxt_hwrm.c | 6 +++---
drivers/net/bnxt/bnxt_ring.c | 2 +-
drivers/net/bnxt/bnxt_vnic.c
Add flow_ctrl_get and flow_ctrl_set device operations.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_ethdev.c | 83 ++
1 file changed, 83 insertions(+)
diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnx
Add rss_hash_update and rss_hash_conf_get
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_ethdev.c | 117 +
1 file changed, 117 insertions(+)
diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethde
Update/query reta operations
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_ethdev.c | 56 ++
1 file changed, 56 insertions(+)
diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
index 3396
Add/remove MAC addresses
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_ethdev.c | 69 ++
1 file changed, 69 insertions(+)
diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
index edb7427.
Enables/disables all multicast traffic.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_ethdev.c | 30 ++
1 file changed, 30 insertions(+)
diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
index f5de4
BNXT driver will now minimally pass traffic with testpmd.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_ethdev.c | 267 +
1 file changed, 267 insertions(+)
diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net
Add a function to initialize ring groups, and a function to
allocate the rings via HWRM.
This should be the last functionality needed to add start/stop
device operations.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_ring.c | 119 +++
Allows setting and clearing L2 context RX masks per vnic
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_hwrm.c | 45 +++
drivers/net/bnxt/bnxt_hwrm.h | 3 +
drivers/net/bnxt/hsi_struct_def_dpdk.h | 135 ++
Add function and associated structures and definitions as well as
some convenienct functions for manipulating the state of the entire
function.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_hwrm.c | 219 +
driver
Add HWRM calls to allocate and free TX/RX/CMPL rings along with
the associated structs and definitions.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_hwrm.c | 108
drivers/net/bnxt/bnxt_hwrm.h | 7 +
drivers/net/bnxt/hs
Perform allocation and free()ing of ring information structures for
TX, RX, and completion rings.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_cpr.c | 28 +++-
drivers/net/bnxt/bnxt_cpr.h | 2 +-
drivers/net/bnxt/bnxt_rxq.c | 17
This requires a group info array in struct bnxt, so add that, save
the max size from the func_qcap response, and alloc/free in init/uninit
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt.h| 2 +
drivers/net/bnxt/bnxt_hwrm.c | 33
Add code to create/destroy TX queues. This still requires TX ring support
to be completed in a future commit.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/Makefile | 1 +
drivers/net/bnxt/bnxt_ethdev.c | 3 +
drivers/net/bnxt/bnxt_txq.c| 125 ++
Initial implementation of rx_pkt_burst
Add code to allocate rings to bnxt_ring.c
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/Makefile | 1 +
drivers/net/bnxt/bnxt_ethdev.c | 3 +-
drivers/net/bnxt/bnxt_ring.c | 20 +-
dri
Initial implementation of rx_pkt_burst
Add code to allocate rings to bnxt_ring.c
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/Makefile | 1 +
drivers/net/bnxt/bnxt_cpr.h| 4 +-
drivers/net/bnxt/bnxt_ethdev.c | 5 +-
dri
Add the L2 filter structure and the alloc/init/free functions for
dealing with them.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/Makefile | 1 +
drivers/net/bnxt/bnxt.h| 3 +
drivers/net/bnxt/bnxt_filter.c | 175
Add functions to allocate, initialize, and free vnics.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/Makefile | 1 +
drivers/net/bnxt/bnxt.h| 14 ++
drivers/net/bnxt/bnxt_vnic.c | 277 +
Initial skeleton simply fails init.
Add nic guide and tie into build system.
Signed-off-by: Stephen Hurd
---
MAINTAINERS | 5 ++
config/common_base | 5 ++
doc/guides/nics/bnxt.rst| 49 +++
driv
Gets device info from the bp structure filled in the init() function.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt.h| 3 ++
drivers/net/bnxt/bnxt_ethdev.c | 96 +-
2 files changed, 98 insertions(+), 1 del
Move init() cleanup into uninit() function
Fix .dev_private_size
Add require hwrm calls:
bnxt_hwrm_func_driver_register()
bnxt_hwrm_func_driver_unregister()
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt.h| 1 +
drivers/n
Start adding HWRM support.
Initial commit just performs necessary HWRM queries for init, then
fails as before.
Thee used HWRM calls so far:
bnxt_hwrm_func_qcaps:
Queries device capabilities.
bnxt_hwrm_ver_get:
Gets the firmware version and interface specifications.
Returns
This adds the bnxt_hwrm_port_phy_cfg() HWRM call, and copies required
information into the new struct bnxt_link_info.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt.h| 32 +++
drivers/net/bnxt/bnxt_ethdev.c | 24 ++
drivers/net/b
Declare ring structures and a ring free() function.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/Makefile| 1 +
drivers/net/bnxt/bnxt_ring.c | 51
drivers/net/bnxt/bnxt_ring.h | 92
3
Structures, macros, and functions for working with completion rings
in the driver.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/Makefile | 1 +
drivers/net/bnxt/bnxt.h| 6 +
drivers/net/bnxt/bnxt_cpr.c| 139 ++
Initial create/destroy queue code. Requires RX ring support to be
functional.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/Makefile | 1 +
drivers/net/bnxt/bnxt.h| 2 +
drivers/net/bnxt/bnxt_ethdev.c | 3 +
drivers
Frees a vnic allocated by vnic_alloc.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_hwrm.c | 21 +
drivers/net/bnxt/bnxt_hwrm.h | 1 +
drivers/net/bnxt/hsi_struct_def_dpdk.h | 82 ++
3 files ch
Add get and clear staitstics operations and the asociated HWRM calls.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/Makefile | 1 +
drivers/net/bnxt/bnxt.h| 5 +-
drivers/net/bnxt/bnxt_cpr.c| 5 +-
drivers/net/bn
More HWRM calls.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_hwrm.c | 38
drivers/net/bnxt/bnxt_hwrm.h | 2 +
drivers/net/bnxt/hsi_struct_def_dpdk.h | 153 +
3 files changed, 193 insertion
Add HWRM code to allocate a statistics context and a helper function
to allocate one for evert completion ring.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_hwrm.c | 52
drivers/net/bnxt/bnxt_hwrm.h | 3 ++
driv
Add bnxt_hwrm_func_reset() function and supporting structs and macros.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_hwrm.c | 17 +
drivers/net/bnxt/bnxt_hwrm.h | 1 +
drivers/net/bnxt/hsi_struct_def_dpdk.h | 129 ++
Used to enable RSS configuration
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_hwrm.c | 24
drivers/net/bnxt/bnxt_hwrm.h | 2 ++
drivers/net/bnxt/hsi_struct_def_dpdk.h | 1 +
3 files changed, 27 insertions(+
Configurs a vnic allocaed by vnic_alloc function.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_hwrm.c | 34
drivers/net/bnxt/bnxt_hwrm.h | 3 +-
drivers/net/bnxt/hsi_struct_def_dpdk.h | 155 +++
Some HWRM versions will stop responding if we request poll mode interrupt.
As a workaround, request an MSI interrupt even though we never enable it.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_hwrm.c | 6 +-
1 file changed, 5 insertions(+), 1 delet
Add missing forward declaration of struct bnxt_ring_struct to avoid
requiring additional headers for the function declaation.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_hwrm.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/bnxt/bnxt_h
Add HWRM ring group add/free functions and associated structs and
definitions.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_hwrm.c | 84 +++
drivers/net/bnxt/bnxt_hwrm.h | 4 +
drivers/net/bnxt/hsi_struct_def_dpdk.h |
Enables and diables promiscuous mode.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_ethdev.c | 30 ++
1 file changed, 30 insertions(+)
diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
index a91e1c2
Frees all resources except the hwrm ones, which are required to notify
the HWRM that the driver is unloaded (these are freed in uninit()).
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_ethdev.c | 11 +++
1 file changed, 11 insertions(+)
diff --g
Add HWRM port pgy qcfg HWRM command and bnxt_get_hwrm_link_config()
wrapper which parses the link state.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_hwrm.c | 120 +
drivers/net/bnxt/bnxt_hwrm.h | 1 +
drivers/net/bnxt/hsi_stru
Sets link to up or down as appropriate.
Signed-off-by: Stephen Hurd
Reviewed-by: Ajit Kumar Khaparde
---
drivers/net/bnxt/bnxt_ethdev.c | 20
1 file changed, 20 insertions(+)
diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
index 7056f98..339607
94 matches
Mail list logo
