On Fri, Jan 6, 2017 at 12:32 AM, Samudrala, Sridhar
[off-list ref] wrote:

On 1/5/2017 3:50 AM, Or Gerlitz wrote:

It looks like you are returning the VF port number as phys_port_name() for a
VF rep in en_rep.c. Is this correct?

yeah, correct, what we all call the VF rep actually represents the
eswitch vport the VF is connected to. The most flexible input for user
space would be just to get this port number along with the switchdev
ID which tells them to which switch this port belongs.

By default i am creating VFPR netdev with name as <pf_name>_VF<vf_num>
For ex; if enp5s0f0 is the pf name, VFPR netdev for VF0 will be enp5s0f0_vf0

If we want udev to follow this syntax should i return '_vf0'  as
get_phys_port_name() for VF rep 0?

just return a number. Typically the PF is vport #0, so for vf0 you
would return 1, this is aligned with the kernel switchdev
documentation [1]

Or.

[1] http://lxr.free-electrons.com/source/Documentation/networking/switchdev.txt#L104