Skip to contentSkip to conversation

i40e PMD contains copy-pasted bug from mempool library

Bug 923 [email protected] 2 participants 3 messages Jan 20, 2022 – Jul 14, 2022

lore
Share & tools
lore
#0 lore raw

https://bugs.dpdk.org/show_bug.cgi?id=923

Bug ID: 923
Summary: i40e PMD contains copy-pasted bug from mempool library
Product: DPDK
Version: unspecified
Hardware: All
OS: All
Status: UNCONFIRMED
Severity: normal
Priority: Normal
Component: ethdev
Assignee: [email protected]
Reporter: [email protected]
Target Milestone: ---

The mempool library does not flush its cache correctly, but leaves it full after flushing.
This bug is fixed in [1], but remains in the i40e PMD, which contains copy-pasted code [2] from the mempool library.

Other PMDs with copy-pasted code may have the same bug.

[1]
http://inbox.dpdk.org/dev/[email protected]/T/#u

[2]
http://code.dpdk.org/dpdk/latest/source/drivers/net/i40e/i40e_rxtx_vec_avx512.c#L919

Signature · 2 lines
You are receiving this mail because:
You are the assignee for the bug.
#1 lore raw

To: i40e, iavf, ice PMD maintainers
CC: mempool library maintainers

Quoted 20 lines

From: [email protected] [mailto:[email protected]]
Sent: Thursday, 20 January 2022 09.43

https://bugs.dpdk.org/show_bug.cgi?id=923

Bug ID: 923
Summary: i40e PMD contains copy-pasted bug from mempool
library
Product: DPDK
Version: unspecified
Hardware: All
OS: All
Status: UNCONFIRMED
Severity: normal
Priority: Normal
Component: ethdev
Assignee: [email protected]
Reporter: [email protected]
Target Milestone: ---

Quoted 7 lines

The mempool library does not flush its cache correctly, but leaves it full
after flushing.
This bug is fixed in [1], but remains in the i40e PMD, which contains copy-pasted code [2] from the mempool library.

Other PMDs with copy-pasted code may have the same bug.

Grep rte_mempool_default_cache could indicate that the AVX512 code in the iavf and ice PMDs also have this bug.

Quoted 12 lines

[1]
http://inbox.dpdk.org/dev/20220119150301.42583-1-
[email protected]/T/#u

[2]
http://code.dpdk.org/dpdk/latest/source/drivers/net/i40e/i40e_rxtx_vec_
avx512.c#L919

--
You are receiving this mail because:
You are the assignee for the bug.

#2 lore raw
In reply to [email protected]

https://bugs.dpdk.org/show_bug.cgi?id=923

Morten Brørup ([email protected]) changed:

What |Removed |Added
----------------------------------------------------------------------------

             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |DUPLICATE

--- Comment #1 from Morten Brørup ([email protected]) --- Copy-pasting internal code and not using the mempool API causes more problems than just the risk of spreading bugs from the library it is copied from to the destinations where it is copied.

*** This bug has been marked as a duplicate of bug 1052 ***

Signature · 2 lines
You are receiving this mail because:
You are the assignee for the bug.