On 3/8/2021 3:58 PM, Xueming(Steven) Li wrote:
-----Original Message-----
From: Ferruh Yigit <ferruh.yi...@intel.com>
Sent: Monday, March 8, 2021 10:38 PM
To: Xueming(Steven) Li <xuemi...@nvidia.com>; Andrew Rybchenko
<andrew.rybche...@oktetlabs.ru>
Cc: dev@dpdk.org; Slava Ovsiienko <viachesl...@nvidia.com>; Asaf Penso
<as...@nvidia.com>; NBU-Contact-Thomas Monjalon
<tho...@monjalon.net>; declan.dohe...@intel.com
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 <xuemi...@nvidia.com>
Acked-by: Viacheslav Ovsiienko <viachesl...@nvidia.com>
Acked-by: Thomas Monjalon <tho...@monjalon.net>
Acked-by: Andrew Rybchenko <andrew.rybche...@oktetlabs.ru>
<...>
