On 3/8/2021 3:58 PM, Xueming(Steven) Li wrote:

quoted

-----Original Message-----
From: Ferruh Yigit <redacted>
Sent: Monday, March 8, 2021 10:38 PM
To: Xueming(Steven) Li <redacted>; Andrew Rybchenko <redacted>
Cc: dev@dpdk.org; Slava Ovsiienko <redacted>; Asaf Penso <redacted>; NBU-Contact-Thomas Monjalon
[off-list ref]; declan.doherty@intel.com
Subject: Re: [PATCH v8 2/9] ethdev: support representor port list

On 3/4/2021 2:30 PM, Xueming Li wrote:

quoted

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.

quoted

quoted

Signed-off-by: Xueming Li <redacted>
Acked-by: Viacheslav Ovsiienko <redacted>
Acked-by: Thomas Monjalon <redacted>
Acked-by: Andrew Rybchenko <redacted>

<...>