Hi Singh and Ferruh, > -----Original Message----- > From: Ferruh Yigit <[email protected]> > Sent: Monday, August 9, 2021 11:31 PM > To: Singh, Aman Deep <[email protected]>; Andrew Rybchenko > <[email protected]>; Xueming(Steven) Li > <[email protected]> > Cc: [email protected]; Slava Ovsiienko <[email protected]>; > NBU-Contact-Thomas Monjalon <[email protected]> > Subject: Re: [dpdk-dev] [RFC] ethdev: change queue release callback > > On 8/9/2021 3:39 PM, Singh, Aman Deep wrote: > > Hi Xueming, > > > > On 7/28/2021 1:10 PM, Andrew Rybchenko wrote: > >> On 7/27/21 6:41 AM, Xueming Li wrote: > >>> To align with other eth device queue configuration callbacks, change > >>> RX and TX queue release callback API parameter from queue object to > >>> device and queue index. > >>> > >>> Signed-off-by: Xueming Li <[email protected]> > >> > >> In fact, there is no strong reasons to do it, but I think it is a > >> nice cleanup to use (dev + queue index) on control path. > >> > >> Hopefully it will not result in any regressions. > > > > Combined there are 100+ API's for Rx/Tx queue_release that need to be > > modified for it. > > > > I believe all regression possibilities here will be caught, in > > compilation phase itself. > > > > Same here, it is a good cleanup but there is no strong reason for it. > > Since it is all internal, there is no ABI restriction on the patch, and > v21.11 will be full ABI break patches, to not cause conflicts with this > change, what would you think to have it on v22.02?
This patch is required by shared-rxq feature which ABI broken, target to 21.11. I'll do it carefully, fortunately, the change is straightforward.
