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.


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]>


<...>



Reply via email to