Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <e77232b9-a290-545b-cb02-ae1fbc2cd5cb@linux.alibaba.com>
Date:   Wed, 12 Jul 2023 16:34:00 +0800
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0)
 Gecko/20100101 Thunderbird/102.12.0
Subject: Re: [PATCH net-next V1 0/4] virtio_net: add per queue interrupt
 coalescing support
To:     Gavin Li <gavinl@nvidia.com>
Cc:     virtualization@lists.linux-foundation.org, netdev@vger.kernel.org,
        linux-kernel@vger.kernel.org, bpf@vger.kernel.org, mst@redhat.com,
        jasowang@redhat.com, xuanzhuo@linux.alibaba.com,
        davem@davemloft.net, edumazet@google.com, kuba@kernel.org,
        pabeni@redhat.com, ast@kernel.org, daniel@iogearbox.net,
        hawk@kernel.org, john.fastabend@gmail.com, jiri@nvidia.com,
        dtatulea@nvidia.com
References: <20230710092005.5062-1-gavinl@nvidia.com>
From:   Heng Qi <hengqi@linux.alibaba.com>
In-Reply-To: <20230710092005.5062-1-gavinl@nvidia.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: bulk


在 2023/7/10 下午5:20, Gavin Li 写道:
> Currently, coalescing parameters are grouped for all transmit and receive
> virtqueues. This patch series add support to set or get the parameters for
> a specified virtqueue.
>
> When the traffic between virtqueues is unbalanced, for example, one virtqueue
> is busy and another virtqueue is idle, then it will be very useful to
> control coalescing parameters at the virtqueue granularity.


We definitely did the same thing, and I'm waiting for our hardware 
implementation to be ready to
push the ethtool + netdim implementation.

Since this commit log is completely copied from the implementation of 
the virtio spec, consulting
each other's scheduling plan in advance can be friendly to avoid us 
doing the same thing and consuming an extra effort.

Thanks.

>
> Example command:
> $ ethtool -Q eth5 queue_mask 0x1 --coalesce tx-packets 10
> Would set max_packets=10 to VQ 1.
> $ ethtool -Q eth5 queue_mask 0x1 --coalesce rx-packets 10
> Would set max_packets=10 to VQ 0.
> $ ethtool -Q eth5 queue_mask 0x1 --show-coalesce
>   Queue: 0
>   Adaptive RX: off  TX: off
>   stats-block-usecs: 0
>   sample-interval: 0
>   pkt-rate-low: 0
>   pkt-rate-high: 0
>
>   rx-usecs: 222
>   rx-frames: 0
>   rx-usecs-irq: 0
>   rx-frames-irq: 256
>
>   tx-usecs: 222
>   tx-frames: 0
>   tx-usecs-irq: 0
>   tx-frames-irq: 256
>
>   rx-usecs-low: 0
>   rx-frame-low: 0
>   tx-usecs-low: 0
>   tx-frame-low: 0
>
>   rx-usecs-high: 0
>   rx-frame-high: 0
>   tx-usecs-high: 0
>   tx-frame-high: 0
>
> In this patch series:
> Patch-1: Extract interrupt coalescing settings to a structure.
> Patch-2: Extract get/set interrupt coalesce to a function.
> Patch-3: Support per queue interrupt coalesce command.
> Patch-4: Enable per queue interrupt coalesce feature.
>
> Gavin Li (4):
>    virtio_net: extract interrupt coalescing settings to a structure
>    virtio_net: extract get/set interrupt coalesce to a function
>    virtio_net: support per queue interrupt coalesce command
>    virtio_net: enable per queue interrupt coalesce feature
>
>   drivers/net/virtio_net.c        | 169 ++++++++++++++++++++++++++------
>   include/uapi/linux/virtio_net.h |  14 +++
>   2 files changed, 154 insertions(+), 29 deletions(-)
>