Re: [PATCH v3] bus/cdx: provide driver flag for optional resource mapping

2023-10-13 Thread David Marchand
On Fri, Oct 13, 2023 at 1:52 PM Gangurde, Abhijit wrote: > > > > > > +/** > > > > > + * Map the CDX device resources in user space virtual memory address. > > > > > + * > > > > > + * Note that driver should not call this function when flag > > > > > + * RTE_CDX_DRV_NEED_MAPPING is set, as EAL will

RE: [PATCH v3] bus/cdx: provide driver flag for optional resource mapping

2023-10-13 Thread Gangurde, Abhijit
> > > > +/** > > > > + * Map the CDX device resources in user space virtual memory address. > > > > + * > > > > + * Note that driver should not call this function when flag > > > > + * RTE_CDX_DRV_NEED_MAPPING is set, as EAL will do that for > > > > + * you when it's on. > > > > > > Why should we e

Re: [PATCH v3] bus/cdx: provide driver flag for optional resource mapping

2023-10-04 Thread David Marchand
On Wed, Oct 4, 2023 at 12:06 PM Gangurde, Abhijit wrote: > > > +/** > > > + * Map the CDX device resources in user space virtual memory address. > > > + * > > > + * Note that driver should not call this function when flag > > > + * RTE_CDX_DRV_NEED_MAPPING is set, as EAL will do that for > > > + *

RE: [PATCH v3] bus/cdx: provide driver flag for optional resource mapping

2023-10-04 Thread Gangurde, Abhijit
[AMD Official Use Only - General] > wrote: > > @@ -383,10 +384,12 @@ cdx_probe_one_driver(struct rte_cdx_driver *dr, > > CDX_BUS_DEBUG(" probe device %s using driver: %s", dev_name, > > dr->driver.name); > > > > - ret = cdx_vfio_map_resource(dev); > > - if (re

Re: [PATCH v3] bus/cdx: provide driver flag for optional resource mapping

2023-09-29 Thread David Marchand
On Tue, Jul 11, 2023 at 7:52 AM Abhijit Gangurde wrote: > @@ -383,10 +384,12 @@ cdx_probe_one_driver(struct rte_cdx_driver *dr, > CDX_BUS_DEBUG(" probe device %s using driver: %s", dev_name, > dr->driver.name); > > - ret = cdx_vfio_map_resource(dev); > - if (re

Re: [PATCH v3] bus/cdx: provide driver flag for optional resource mapping

2023-07-11 Thread Gupta, Nipun
Please use 'in-reply-to' of the first submission when sending any subsequent patches. On 7/11/2023 11:21 AM, Abhijit Gangurde wrote: Provide driver flag which gives an option to map the cdx device resource before probing the device driver. Also, make rte_cdx_map_device() API as public to map de