On Fri, Jan 6, 2017 at 12:32 AM, Samudrala, Sridhar <sridhar.samudr...@intel.com> 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
