[dpdk-dev] [PATCH v3 2/2] net/virtio: add link speed devarg
Maxime Coquelin
maxime.coquelin at redhat.com
Thu Feb 13 14:54:55 CET 2020
Hi Ivan,
On 2/12/20 11:40 AM, Maxime Coquelin wrote:
>
>
> On 2/12/20 11:35 AM, Tiwei Bie wrote:
>> On Fri, Feb 07, 2020 at 02:25:26PM +0300, Ivan Dyukov wrote:
>>> Some applications like pktgen use link_speed to calculate
>>> transmit rate. It limits outcome traffic to hardcoded 10G.
>>>
>>> This patch adds link_speed devarg which allows to configure
>>> link_speed of virtio device.
>>>
>>> Signed-off-by: Ivan Dyukov <i.dyukov at samsung.com>
>>> ---
>>> doc/guides/nics/virtio.rst | 7 ++
>>> drivers/net/virtio/virtio_ethdev.c | 101 ++++++++++++++++++++++++-----
>>> drivers/net/virtio/virtio_pci.h | 1 +
>>> 3 files changed, 92 insertions(+), 17 deletions(-)
>>
>> Maybe I missed something.. Why not enable the VIRTIO_NET_F_SPEED_DUPLEX
>> support directly? When that feature is supported and negotiated,
>> we will ignore this devarg?
>
> IMHO, yes. When VIRTIO_NET_F_SPEED_DUPLEX will be implemented and
> negotiated, this devarg will be ignored.
>
>> If we want this devarg, it looks better to support it in virtio-user
>> as well (most code is shared between virtio-PMD and virtio-user).
>
> That's a valid point, Virtio-user should also support it.
> Ivan, can you send a new revision with Virtio-user support as well?
Do you plan to submit the new revision today?
Thanks,
Maxime
> Thanks,
> Maxime
>
>> Thanks!
>> Tiwei
>>
More information about the dev
mailing list