18/01/2021 19:00, Ajit Khaparde: > On Mon, Jan 18, 2021 at 9:57 AM Thomas Monjalon <tho...@monjalon.net> wrote: > > 18/01/2021 18:42, Ajit Khaparde: > > > On Mon, Jan 18, 2021 at 3:17 AM Xueming Li <xuemi...@nvidia.com> wrote: > > > > +enum rte_eth_representor_type { > > > > + RTE_ETH_REPRESENTOR_NONE, /**< not a representor. */ > > > > + RTE_ETH_REPRESENTOR_VF, /**< representor of VF. */ > > > > + RTE_ETH_REPRESENTOR_SF, /**< representor of SF. */ > > > Till we get used to the terminology... > > > Can we also have SF = "Sub Function" mentioned in the docs or comments? > > > > Are we sure about the definition? > > I remember seeing SF = Scalable Function somewhere else (maybe from Intel) > That complicates it. But if they mean the same thing, let's pick one.
I think "Sub Function" and "Virtual Function" are easy to understand for everybody. I suggest picking these two for comments above.