Re: [dpdk-dev] [PATCH v6 4/9] ethdev: support sub function representor

[Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>]

Hi,

On 2/14/21 6:21 AM, Xueming Li wrote:
> SubFunction is a portion of the PCI device, created on demand, a SF
> netdev has its own dedicated queues(txq, rxq). A SF netdev supports
> eswitch representation offload similar to existing PF and VF
> representors.
> 
> To support SF representor, this patch introduces new devargs syntax,
> examples:
>  representor=sf0               - single SubFunction representor
>  representor=sf[1,3,5]         - single list
>  representor=sf[0-3],          - single range
>  representor=sf[0,2-6,8,10-12] - list with singles and ranges
> 
> Signed-off-by: Xueming Li <xuemingl@nvidia.com>
> Acked-by: Viacheslav Ovsiienko <viacheslavo@nvidia.com>
> Acked-by: Thomas Monjalon <thomas@monjalon.net>
> Acked-by: Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>

What happens if I try to mix different types of representors:
A: -a DBDF,representor=sf0,representor=vf1
B: -a DBDF,representor=sf0 -a DBDF,representor=vf1
(DBDF is the same in args B in both cases).

I'm not trying to say that it must work, since most likely
hotplug will be used to add representors. But behaviour
must be consistent (error?).

The question is raised here, since it is the first patch
where the second type of representor appears.

Andrew.