On 10/6/21 12:56 AM, Thomas Monjalon wrote:
> 13/09/2021 13:26, Andrew Rybchenko:
>> From: Viacheslav Galaktionov <viacheslav.galaktio...@oktetlabs.ru>
>>
>> Getting a list of representors from a representor does not make sense.
>> Instead, a parent device should be used.
> 
> I don't understand which issue it is fixing.
> This function was not working before if not using the backer port?

The function, rte_eth_representor_id_get(), is used in
eth_representor_cmp() which is required in ethdev class
iterator to search ethdev port ID by name (representor case).
Before the patch the function is called on the representor
itself it tries to get representors info to match.

It was found by OvS+DPDK testing:
 1. OvS hotplugs representor
 2. OvS tries to find DPDK port ID for just hotplugged
    representor and fails (if PMD does not provider
    representors info on the representor itself)

> Is it fixing a specific PMD?

It is a generic fix for PMD which do *not* provide representors
info on the representor itself.


Reply via email to