Hi Nicolas,
I am seeing some compilation issues again on individual patches. Please be
careful. This is causing a lot of duplicate effort.
/home/akhil/up/dpdk-next-crypto/drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c:
In function âdequeue_ldpc_enc_one_op_cbâ:
/home/akhil/up/dpdk-next-crypt
Thanks a lot for working on this.
I can see the support on the kernel side has been pulled in what will be 5.7.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=2fb732b33b0f4d73f391066c0a970b942c600dab
I'd like to get the dpdk support in 20.05, but for this, can I hav
Hi David,
> -Original Message-
> From: David Marchand
> Sent: Saturday, April 18, 2020 17:22
> To: Wang, Haiyue ; Vamsi Attunuru
> ;
> alex.william...@redhat.com
> Cc: dev ; Thomas Monjalon ; Jerin Jacob
> Kollanukkaran
>
> Subject: Re: [PATCH v6 0/2] support for VFIO-PCI VF token inte
> -Original Message-
> From: Lukasz Wojciechowski
> Sent: Saturday, April 18, 2020 3:27 AM
> To: Thomas Monjalon ; Nikhil Rao
> ; Jerin Jacob Kollanukkaran ; Erik
> Gabriel Carrillo
> Cc: dev@dpdk.org
> Subject: [EXT] [PATCH v1 04/17] eventdev: replace library debug flag with
> global o
18/04/2020 07:04, Bill Zhou:
> From: Ferruh Yigit
> > On 4/14/2020 9:32 AM, Dong Zhou wrote:
> > > --- a/lib/librte_ethdev/rte_ethdev.h
> > > +++ b/lib/librte_ethdev/rte_ethdev.h
> > > @@ -3015,6 +3015,7 @@ enum rte_eth_event_type {
> > > RTE_ETH_EVENT_NEW, /**< port is probed */
> > > RT
18/04/2020 11:38, Wang, Haiyue:
> From: David Marchand
> > On the series itself, I did not look in detail, but there is no
> > documentation on how to use the feature.
> > We are missing a release note update.
>
> If needed in 20.05, I can add it in release note. And about how to,
> I just added
18/04/2020 11:41, Jerin Jacob Kollanukkaran:
> From: Lukasz Wojciechowski
> > Use global debug flag RTE_DEBUG instead of RTE_LIBRTE_EVENTDEV_DEBUG.
> > The old define is completely removed from source code and config.
>
>
> May I know the rational for this change?
You know the rationale, it was
> -Original Message-
> From: Thomas Monjalon
> Sent: Saturday, April 18, 2020 17:50
> To: David Marchand ; Vamsi Attunuru
> ;
> alex.william...@redhat.com; Wang, Haiyue
> Cc: dev ; Jerin Jacob Kollanukkaran
> Subject: Re: [PATCH v6 0/2] support for VFIO-PCI VF token interface
>
> 18/04
Hi all,
In my test, entries can't be cleanup in rte_hash table when deleting keys
in rte_hash iterate loop. The test steps:
1. create a hash table table1 with limit 3, ext bucket enabled, and
insert 3 entries into this hash table.
2. create a larger hash table table2 with
On Sat, Apr 18, 2020 at 3:23 PM Thomas Monjalon wrote:
>
> 18/04/2020 11:41, Jerin Jacob Kollanukkaran:
> > From: Lukasz Wojciechowski
> > > Use global debug flag RTE_DEBUG instead of RTE_LIBRTE_EVENTDEV_DEBUG.
> > > The old define is completely removed from source code and config.
> >
> >
> > Ma
The kernel module vfio-pci introduces the VF token to enable SR-IOV
support since 5.7.
The VF token can be set by a vfio-pci based PF driver and must be known
by the vfio-pci based VF driver in order to gain access to the device.
Signed-off-by: Haiyue Wang
Acked-by: Vamsi Attunuru
Tested-by: Va
v7: Add the Fixes tag in uuid, the release note and help
document.
v6: Drop the Fixes tag in uuid, since the file has been
moved to another place, not suitable to apply on stable.
And this is not a bug, just some kind of enhancement.
https://patchwork.dpdk.org/cover/68367/
v5: 1.
Add the dependent header files explicitly, so that the user just needs
to include the 'rte_uuid.h' header file directly to avoid compile error:
(1). rte_uuid.h:97:55: error: unknown type name ‘size_t’
(2). rte_uuid.h:58:2: error: implicit declaration of function ‘memcpy’
Fixes: 6bc67c497a51 ("ea
Static builds can take a lot of space, so reduce the number of examples
built when testing those static builds.
As makefile-based build is close to end of life, completely skip examples
in case of static linkage with make.
Signed-off-by: Thomas Monjalon
---
devtools/test-build.sh| 1 +
18/04/2020 12:05, Jerin Jacob:
> On Sat, Apr 18, 2020 at 3:23 PM Thomas Monjalon wrote:
> >
> > 18/04/2020 11:41, Jerin Jacob Kollanukkaran:
> > > From: Lukasz Wojciechowski
> > > > Use global debug flag RTE_DEBUG instead of RTE_LIBRTE_EVENTDEV_DEBUG.
> > > > The old define is completely removed
18/04/2020 13:16, Haiyue Wang:
> --- a/doc/guides/linux_gsg/linux_drivers.rst
> +++ b/doc/guides/linux_gsg/linux_drivers.rst
> +The Linux kernel since version 5.7 supports the creation of virtual
> functions, and by default this feature is off.
Creation of virtual function has always been support
Introduce stress test for ring enqueue/dequeue operations.
Performs the following pattern on each slave worker:
dequeue/read-write data from the dequeued objects/enqueue.
Serves as both functional and performance test of ring
enqueue/dequeue operations under high contention
(for both over committed
V4 - V5:
1. fix i686 clang build problem
2. fix formal API comments
V3 - V4 changes:
Address comments from Honnappa:
1. for new sync modes make legacy API wrappers around _elem_ calls
2. remove rte_ring_(hts|rts)_generic.h
3. few changes in C11 version
4. peek API - add missing functions for _elem
Introduce head/tail sync mode for MT ring synchronization.
In that mode enqueue/dequeue operation is fully serialized:
only one thread at a time is allowed to perform given op.
Suppose to reduce stall times in case when ring is used on
overcommitted cpus (multiple active threads on the same cpu).
To make these preparations two main things are done:
- Change from *single* to *sync_type* to allow different
synchronisation schemes to be applied.
Mark *single* as deprecated in comments.
Add new functions to allow user to query ring sync types.
Replace direct access to *single* with appr
Introduce new test case to test HTS ring mode under contention.
Signed-off-by: Konstantin Ananyev
---
app/test/Makefile | 1 +
app/test/meson.build| 1 +
app/test/test_ring_hts_stress.c | 32
app/test/test_ring_stress.c | 3 +++
a
Introduce relaxed tail sync (RTS) mode for MT ring synchronization.
Aim to reduce stall times in case when ring is used on
overcommited cpus (multiple active threads on the same cpu).
The main difference from original MP/MC algorithm is that
tail value is increased not by every thread that finished
Introduce new test case to test RTS ring mode under contention.
Signed-off-by: Konstantin Ananyev
---
app/test/Makefile | 1 +
app/test/meson.build| 1 +
app/test/test_ring_rts_stress.c | 32
app/test/test_ring_stress.c | 3 +++
a
Introduce new test case to test MT peek API.
Signed-off-by: Konstantin Ananyev
---
app/test/Makefile| 1 +
app/test/meson.build | 1 +
app/test/test_ring_peek_stress.c | 43
app/test/test_ring_stress.c | 3 +++
app/test/test_ri
Extend test_ring_autotest with new test-cases for RTS/HTS sync modes.
Signed-off-by: Konstantin Ananyev
---
app/test/test_ring.c | 93 ++--
1 file changed, 73 insertions(+), 20 deletions(-)
diff --git a/app/test/test_ring.c b/app/test/test_ring.c
index fb
For rings with producer/consumer in RTE_RING_SYNC_ST, RTE_RING_SYNC_MT_HTS
mode, provide an ability to split enqueue/dequeue operation
into two phases:
- enqueue/dequeue start
- enqueue/dequeue finish
That allows user to inspect objects in the ring without removing
them from it (aka MT
v8: Update the document.
v7: Add the Fixes tag in uuid, the release note and help
document.
https://patchwork.dpdk.org/cover/68845/
v6: Drop the Fixes tag in uuid, since the file has been
moved to another place, not suitable to apply on stable.
And this is not a bug, just some kin
Add the dependent header files explicitly, so that the user just needs
to include the 'rte_uuid.h' header file directly to avoid compile error:
(1). rte_uuid.h:97:55: error: unknown type name ‘size_t’
(2). rte_uuid.h:58:2: error: implicit declaration of function ‘memcpy’
Fixes: 6bc67c497a51 ("ea
The kernel module vfio-pci introduces the VF token to enable SR-IOV
support since 5.7.
The VF token can be set by a vfio-pci based PF driver and must be known
by the vfio-pci based VF driver in order to gain access to the device.
Signed-off-by: Haiyue Wang
Acked-by: Vamsi Attunuru
Tested-by: Va
> -Original Message-
> From: Thomas Monjalon
> Sent: Saturday, April 18, 2020 21:27
> To: alex.william...@redhat.com; david.march...@redhat.com; Wang, Haiyue
>
> Cc: dev@dpdk.org; Burakov, Anatoly ;
> vattun...@marvell.com;
> jer...@marvell.com
> Subject: Re: [PATCH v7 2/2] eal: support
Hi,
It is probably just me but I having issue with this new patch.
I typically rebuild the PMD libraries directly when doing incremental changes
from the Makefile in the PMD directory (ie. not rebuilding full DPDK each time,
which still work okay obviously).
With this new change it doesn't see
Hi Akhil,
> I am seeing some compilation issues again on individual patches. Please be
> careful. This is causing a lot of duplicate effort.
Yes I missed that one unfortunately, something seems to be wrong in our CI for
patch by patch build. I am checking manually for each commit this time.
>
Compilation is broken on ppc:
What commit ID were you testing that failed to build? This must be a
recent change as I'm not seeing it on my system.
Dave
Add stubs for the FPGA 5GNR FEC PMD
Signed-off-by: Nicolas Chautru
---
config/common_base | 5 +
doc/guides/bbdevs/fpga_5gnr_fec.rst| 146
doc/guides/bbdevs/index.rst| 1 +
doc/guides/rel_notes/release_2
Add in the list of registers for the device and related
HW specs definitions.
Signed-off-by: Nicolas Chautru
---
drivers/baseband/fpga_5gnr_fec/fpga_5gnr_fec.h | 189 +
1 file changed, 189 insertions(+)
diff --git a/drivers/baseband/fpga_5gnr_fec/fpga_5gnr_fec.h
b/drive
v5: Fix incremenral patch build and few rebase updates
v4: Fix incremental patch build and expose PMD header file to meson.
v3: Incremental changes from reviews : file name change, moving few inline
function to .h, adding missing inline, doxygen markup fix.
Adding new baseband PMD for FPGA 5GNR
Adding HW specific parsing of error report for
negative scenarios. Not hit through unit test.
Signed-off-by: Nicolas Chautru
---
drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 62 --
1 file changed, 59 insertions(+), 3 deletions(-)
diff --git a/drivers/baseband/fpga_5g
Adding functionality for debug mode to be more
verbose and catch error from unsupported configuration.
Signed-off-by: Nicolas Chautru
---
drivers/baseband/fpga_5gnr_fec/fpga_5gnr_fec.h | 30 ++
drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 356 +
2 files changed,
Adding LDPC processing operations and related documentation.
Signed-off-by: Nicolas Chautru
---
doc/guides/bbdevs/fpga_5gnr_fec.rst| 28 +
drivers/baseband/fpga_5gnr_fec/fpga_5gnr_fec.h | 9 +
drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 660 +
Add in the "info_get" function to the driver, to allow us to query the
device.
No capability are available yet.
Linking bbdev-test to support the PMD with null capability.
Signed-off-by: Nicolas Chautru
---
app/test-bbdev/Makefile| 3 ++
app/test-bbdev/meson.build
Adding function to create and configure queues for
the device. Still no capability.
Signed-off-by: Nicolas Chautru
---
drivers/baseband/fpga_5gnr_fec/fpga_5gnr_fec.h | 110 +++
drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 365 -
2 files changed, 474 insertions
Adding support for interrupt capability in the PMD
and the related operations.
Signed-off-by: Nicolas Chautru
---
drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c | 139 +
1 file changed, 139 insertions(+)
diff --git a/drivers/baseband/fpga_5gnr_fec/rte_fpga_5gnr_fec.c
b/
Adding missing overview page in documentation with
comparison of feature set by each available PMD implementation.
Signed-off-by: Nicolas Chautru
---
.gitignore | 1 +
doc/guides/bbdevs/features/default.ini | 16
doc/guides/bbdevs/feature
Adding optional capability to support loopback preloading
and check of the extern HARQ memory.
This function is required to run the HARQ bit exact test successfully.
Signed-off-by: Nicolas Chautru
---
drivers/baseband/fpga_5gnr_fec/fpga_5gnr_fec.h | 4 +-
drivers/baseband/fpga_5gnr_fec/rte
Add configure function to configure the PF from within
the bbdev-test itself without external application
configuration the device.
Signed-off-by: Nicolas Chautru
---
app/test-bbdev/test_bbdev_perf.c | 57 ++
doc/guides/bbdevs/fpga_5gnr_fec.rst| 123 +++
Sorry for missed this question. The purpose of change function is to skip
device initialization which is not needed in configuration stage.
When features not matched, can just do feature negotiation in configuration
stage and do related actions when virtio device start.
Regards,
Marvin
>
>
> Introduce stress test for ring enqueue/dequeue operations.
> Performs the following pattern on each slave worker:
> dequeue/read-write data from the dequeued objects/enqueue.
> Serves as both functional and performance test of ring enqueue/dequeue
> operations under high contention (for both
> Subject: [PATCH v5 4/9] test/ring: add contention stress test for RTS ring
>
> Introduce new test case to test RTS ring mode under contention.
>
> Signed-off-by: Konstantin Ananyev
Acked-by: Honnappa Nagarahalli
>
> Introduce relaxed tail sync (RTS) mode for MT ring synchronization.
> Aim to reduce stall times in case when ring is used on overcommited cpus
> (multiple active threads on the same cpu).
> The main difference from original MP/MC algorithm is that tail value is
> increased not by every threa
>
> To make these preparations two main things are done:
> - Change from *single* to *sync_type* to allow different
> synchronisation schemes to be applied.
> Mark *single* as deprecated in comments.
> Add new functions to allow user to query ring sync types.
> Replace direct access to *
> Subject: [PATCH v5 5/9] ring: introduce HTS ring mode
>
> Introduce head/tail sync mode for MT ring synchronization.
> In that mode enqueue/dequeue operation is fully serialized:
> only one thread at a time is allowed to perform given op.
> Suppose to reduce stall times in case when ring is us
> Subject: [PATCH v5 8/9] test/ring: add stress test for MT peek API
>
> Introduce new test case to test MT peek API.
>
> Signed-off-by: Konstantin Ananyev
Acked-by: Honnappa Nagarahalli
> Subject: [PATCH v5 7/9] ring: introduce peek style API
>
> For rings with producer/consumer in RTE_RING_SYNC_ST,
> RTE_RING_SYNC_MT_HTS mode, provide an ability to split enqueue/dequeue
> operation into two phases:
> - enqueue/dequeue start
> - enqueue/dequeue finish
> That allows
> Subject: [PATCH v5 6/9] test/ring: add contention stress test for HTS ring
>
> Introduce new test case to test HTS ring mode under contention.
>
> Signed-off-by: Konstantin Ananyev
Acked-by: Honnappa Nagarahalli
> Subject: [PATCH v5 9/9] test/ring: add functional tests for new sync modes
>
> Extend test_ring_autotest with new test-cases for RTS/HTS sync modes.
>
> Signed-off-by: Konstantin Ananyev
Acked-by: Honnappa Nagarahalli
Hi Konstantin,
Changes look good overall, I have integrated RCU defer APIs patch as
well. Please consider adding the following (in another patch?)
1) Release notes
2) Updates to programmer guide for RTS and HTS modes
Thank you,
Honnappa
> -Original Message-
> From: Konstantin An
On 04/19, Liu, Yong wrote:
>Sorry for missed this question. The purpose of change function is to skip
>device initialization which is not needed in configuration stage.
>When features not matched, can just do feature negotiation in configuration
>stage and do related actions when virtio device
Thursday, April 16, 2020 4:20 PM, Maxime Coquelin:
> Subject: Re: [PATCH 1/4] vhost: inroduce operation to get vDPA queue stats
>
> Hi Matan,
>
> On 4/16/20 11:06 AM, Matan Azrad wrote:
> > Hi Maxime
> >
> > Can you point on specific vendor specific counter I suggested?
>
> No, I can't, but I th
> -Original Message-
> From: Jeff Guo
> Sent: Friday, April 17, 2020 9:32 PM
> To: bernard.iremon...@intel.com; Ori Kam ;
> xiaolong...@intel.com; qi.z.zh...@intel.com
> Cc: dev@dpdk.org; jingjing...@intel.com; yahui@intel.com;
> simei...@intel.com; jia@intel.com
> Subject: [dpd
On Sun, Apr 19, 2020 at 12:20 AM David Christensen
wrote:
>
> > Compilation is broken on ppc:
> What commit ID were you testing that failed to build? This must be a
> recent change as I'm not seeing it on my system.
The failure was seen at b11ecebc0.
--
David Marchand
60 matches
Mail list logo
