Thread (59 messages) 59 messages, 10 authors, 2022-07-11

Re: [dpdk-dev] [PATCH 6/8] examples/ip_pipeline: remove setting of PCI ID and address

From: Xia, Chenbo <hidden>
Date: 2021-09-10 08:21:51

Hi David,
-----Original Message-----
From: David Marchand <redacted>
Sent: Friday, September 10, 2021 3:18 PM
To: Xia, Chenbo <redacted>
Cc: dev <redacted>; Dumitrescu, Cristian <redacted>;
Yigit, Ferruh [off-list ref]
Subject: Re: [dpdk-dev] [PATCH 6/8] examples/ip_pipeline: remove setting of
PCI ID and address

On Fri, Sep 10, 2021 at 4:38 AM Chenbo Xia [off-list ref] wrote:
quoted
PCI ID and address in structure rte_kni_conf are never used and
will be removed in kni library. So remove the setting of them
first in the example.

Signed-off-by: Chenbo Xia <redacted>
(forgot to Cc: Ferruh in previous comment)
Ditto patch 5.

I would tag with a Fixes: on ethtool removal.
Plus, this is the same kind of cleanup wrt kni, you can squash those
two patches together.
OK. Will do.

Too bad we do not have a deprecation notice.
lib/kni/rte_kni.h:      struct rte_pci_addr addr; /* depreciated */
lib/kni/rte_kni.h:      struct rte_pci_id id; /* depreciated */

Can we plan for this cleanup?
My bad. I did't notice this when I send the deprecation notice for PCI bus
last release.

If we all agree, we can make this happen 22.11 (Or this release if tech-board
agrees).

Thanks,
Chenbo

--
David Marchand
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help