> -----Original Message-----
> From: David Marchand <david.march...@redhat.com>
> Sent: Friday, July 1, 2022 16:15
> To: Huang, Wei <wei.hu...@intel.com>
> Cc: Xu, Rosen <rosen...@intel.com>; Zhang, Tianfei <tianfei.zh...@intel.com>;
> dev@dpdk.org; Ray Kinsella <m...@ashroe.eu>
> Subject: Re: [PATCH] doc: announce some raw/ifpga API removal
>
> On Fri, Jul 1, 2022 at 10:09 AM David Marchand <david.march...@redhat.com>
> wrote:
> > > > > 2. Get pci_dev by specified PCI address, and remove it
> > > > > TAILQ_FOREACH(pci_dev, &pci_bus->device_list, next) {
> > > > > if (!rte_pci_addr_cmp(&pci_dev->addr, &addr))
> > > > > return pci_dev;
> > > > > }
> > > > > ....
> > > > > pci_drv = pci_dev->driver;
> > > > > pci_drv->remove(pci_dev);
> > > >
> > > > Can't this application use rte_dev_remove and rte_dev_probe?
> > > > If not, we should add the missing parts in the API.
> > > >
> > > Both rte_dev_remove and rte_dev_probe need rte_device pointer. In
> > > this application, it only know the device's PCI address, is there an
> > > existing API
> to get the rte_pci_device pointer by its PCI address ?
> >
> > rte_dev_probe takes a devargs string as input.
> > int rte_dev_probe(const char *devargs);
> >
> > You need the rte_device for removing which can be retrieved from
> > rte_rawdev_info_get.
>
> Additionnaly, rte_eal_hotplug_{add,remove} do the same jobs, but with an
> easier(?) interface.
>
>
I checked rte_eal_hotplug_{add,remove}, they should meet my requirements,
thanks a lot.
> --
> David Marchand
