<html xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body lang="en-IT" link="blue" vlink="purple" style="word-wrap:break-word"> <div class="WordSection1"> <div> > On Wed, May 31, 2023 at 10:44 AM Elena Agostini <a href="mailto:eagostini@nvidia.com">eagostini@nvidia.com</a> wrote:<o:p></o:p> > ><o:p></o:p> > > > On Tue, May 30, 2023 at 1:48 PM <a href="mailto:eagostini@nvidia.com">eagostini@nvidia.com</a> wrote:<o:p></o:p> > ><o:p></o:p> > > > ><o:p></o:p> > ><o:p></o:p> > > > > From: Elena Agostini <a href="mailto:eagostini@nvidia.com"> eagostini@nvidia.com</a><o:p></o:p> > ><o:p></o:p> > > > ><o:p></o:p> > ><o:p></o:p> > > > > In DPDK 22.11 pci bus related structure have been hidden internally<o:p></o:p> > ><o:p></o:p> > > > > so the application doesn't have a direct access to those info anymore.<o:p></o:p> > ><o:p></o:p> > > > ><o:p></o:p> > ><o:p></o:p> > > > > This patch introduces a get function to retrieve a PCI address<o:p></o:p> > ><o:p></o:p> > > > > from an rte_device handler.<o:p></o:p> > ><o:p></o:p> > > > ><o:p></o:p> > ><o:p></o:p> > > > > Signed-off-by: Elena Agostini <a href="mailto:eagostini@nvidia.com">eagostini@nvidia.com</a><o:p></o:p> > ><o:p></o:p> > > ><o:p></o:p> > ><o:p></o:p> > > > (no need to Cc: stable, I removed it)<o:p></o:p> > ><o:p></o:p> > > ><o:p></o:p> > ><o:p></o:p> > > > I would prefer we don't add specific bus API when there is an alternative.<o:p></o:p> > ><o:p></o:p> > > ><o:p></o:p> > ><o:p></o:p> > > > The PCI address is already reported as a string in the generic device<o:p></o:p> > ><o:p></o:p> > > > object name.<o:p></o:p> > ><o:p></o:p> > > > I checked the different ways this name is set and afaics, it is consistent:<o:p></o:p> > ><o:p></o:p> > > > - devarg case <a href="https://git.dpdk.org/dpdk/tree/drivers/bus/pci/pci_common.c#n112"> https://git.dpdk.org/dpdk/tree/drivers/bus/pci/pci_common.c#n112</a><o:p></o:p> > ><o:p></o:p> > > > + <a href="https://git.dpdk.org/dpdk/tree/drivers/bus/pci/pci_params.c#n117"> https://git.dpdk.org/dpdk/tree/drivers/bus/pci/pci_params.c#n117</a><o:p></o:p> > ><o:p></o:p> > > > - no devarg case<o:p></o:p> > ><o:p></o:p> > > > <a href="https://git.dpdk.org/dpdk/tree/drivers/bus/pci/pci_common.c#n115"> https://git.dpdk.org/dpdk/tree/drivers/bus/pci/pci_common.c#n115</a> +<o:p></o:p> > ><o:p></o:p> > > > <a href="https://git.dpdk.org/dpdk/tree/drivers/bus/pci/pci_common.c#n100"> https://git.dpdk.org/dpdk/tree/drivers/bus/pci/pci_common.c#n100</a><o:p></o:p> > ><o:p></o:p> > > ><o:p></o:p> > ><o:p></o:p> > > > Would that be enough for your usecase?<o:p></o:p> > ><o:p></o:p> > ><o:p></o:p> > ><o:p></o:p> > > No as I need to parse anyway the PCI address string in the form of domain/bus/devid/function.> <o:p></o:p> <o:p> </o:p> > I am curious. Can you explain why you would need such information?> <o:p></o:p> <o:p> </o:p> Use-case is the Aerial 5G where two processes have to exchange info<o:p></o:p> about PCI devices sending messages according to some specific format.<o:p></o:p> <o:p> </o:p> > ><o:p></o:p> > > Also, the device name can be changed as it’s exposed to application level.> <o:p></o:p> <o:p> </o:p> > ?<o:p></o:p> > If you mean the application can bust the device name, well, it's the<o:p></o:p> > application problem.> <o:p> </o:p> > <o:p></o:p> <o:p> </o:p> > --<o:p></o:p> > David Marchand<o:p></o:p> </div> </div> </body> </html>