>-----Original Message----- >From: Ferruh Yigit <[email protected]> >Sent: Tuesday, March 9, 2021 12:22 AM >To: Xueming(Steven) Li <[email protected]>; Andrew Rybchenko ><[email protected]> >Cc: [email protected]; Slava Ovsiienko <[email protected]>; Asaf Penso ><[email protected]>; NBU-Contact-Thomas Monjalon ><[email protected]>; [email protected] >Subject: Re: [PATCH v8 2/9] ethdev: support representor port list > >On 3/8/2021 3:58 PM, Xueming(Steven) Li wrote: >> >>> -----Original Message----- >>> From: Ferruh Yigit <[email protected]> >>> Sent: Monday, March 8, 2021 10:38 PM >>> To: Xueming(Steven) Li <[email protected]>; Andrew Rybchenko >>> <[email protected]> >>> Cc: [email protected]; Slava Ovsiienko <[email protected]>; Asaf >>> Penso <[email protected]>; NBU-Contact-Thomas Monjalon >>> <[email protected]>; [email protected] >>> Subject: Re: [PATCH v8 2/9] ethdev: support representor port list >>> >>> On 3/4/2021 2:30 PM, Xueming Li wrote: >>>> To support extended representor syntax, this patch extends the >>>> representor list parsing to support for representor port range in >>>> devargs, examples: >>>> representor=[1,2,3] - single list >>>> representor=[1,3-5,7,9-11] - list with singles and ranges >>>> >>> >>> Isn't above syntax already supported, can you please document what is added? >>> >>> From documentation, what already supported is: >>> -w pci:dbdf,representor=0 >>> -w pci:dbdf,representor=[0-3] >>> -w pci:dbdf,representor=[0,5-11] >>> >> >> Just a refactor to prepare for the extended representor syntax which need to >> reuse the value parsing function for controller and pf >section. >> > >OK, can you please highlight this is a refactoring, from patch title and >commit log it feels like new syntax support is added.
Sure. > >>> >>>> Signed-off-by: Xueming Li <[email protected]> >>>> Acked-by: Viacheslav Ovsiienko <[email protected]> >>>> Acked-by: Thomas Monjalon <[email protected]> >>>> Acked-by: Andrew Rybchenko <[email protected]> >>> >>> >>> <...> >>> >>
