[dpdk-dev] [PATCH v3 2/2] net/virtio: add link speed devarg
i.dyukov at samsung.com
Fri Feb 14 08:59:23 CET 2020
13.02.2020 16:54, Maxime Coquelin пишет:
> 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?
I have a lot of work which is not related to opensource contribution. I need few days to prepare the patch.
More information about the dev