Re: [PATCH RFC 3/4] barriers: convert a control to a data dependency

2019-01-06 Thread Jason Wang
On 2019/1/3 上午4:57, Michael S. Tsirkin wrote: It's not uncommon to have two access two unrelated memory locations in a specific order. At the moment one has to use a memory barrier for this. However, if the first access was a read and the second used an address depending on the first one we w

Re: [PATCH RFC 3/4] barriers: convert a control to a data dependency

2019-01-06 Thread Jason Wang
On 2019/1/7 下午12:23, Michael S. Tsirkin wrote: On Mon, Jan 07, 2019 at 11:58:23AM +0800, Jason Wang wrote: On 2019/1/3 上午4:57, Michael S. Tsirkin wrote: It's not uncommon to have two access two unrelated memory locations in a specific order. At the moment one has to use a memory barrie

Re: [PATCH RFC v5 07/10] tun: Introduce virtio-net RSS

2024-10-09 Thread Jason Wang
On Tue, Oct 8, 2024 at 2:55 PM Akihiko Odaki wrote: > > RSS is a receive steering algorithm that can be negotiated to use with > virtio_net. Conventionally the hash calculation was done by the VMM. > However, computing the hash after the queue was chosen defeats the > purpose of RSS. > > Another a

Re: [PATCH RFC v5 05/10] tun: Pad virtio header with zero

2024-10-09 Thread Jason Wang
On Tue, Oct 8, 2024 at 2:55 PM Akihiko Odaki wrote: > > tun used to simply advance iov_iter when it needs to pad virtio header, > which leaves the garbage in the buffer as is. This is especially > problematic when tun starts to allow enabling the hash reporting > feature; even if the feature is en

Re: [PATCH RFC v5 06/10] tun: Introduce virtio-net hash reporting feature

2024-10-09 Thread Jason Wang
On Tue, Oct 8, 2024 at 2:55 PM Akihiko Odaki wrote: > > Allow the guest to reuse the hash value to make receive steering > consistent between the host and guest, and to save hash computation. > > Signed-off-by: Akihiko Odaki I wonder if this would cause overhead when hash reporting is not enable

Re: [PATCH RFC v5 07/10] tun: Introduce virtio-net RSS

2024-10-18 Thread Jason Wang
On Sat, Oct 12, 2024 at 6:29 PM Akihiko Odaki wrote: > > On 2024/10/09 17:14, Jason Wang wrote: > > On Tue, Oct 8, 2024 at 2:55 PM Akihiko Odaki > > wrote: > >> > >> RSS is a receive steering algorithm that can be negotiated to use with > >> virtio

Re: [PATCH RFC v4 7/9] tun: Introduce virtio-net RSS

2024-09-24 Thread Jason Wang
On Tue, Sep 24, 2024 at 5:01 PM Akihiko Odaki wrote: > > RSS is a receive steering algorithm that can be negotiated to use with > virtio_net. Conventionally the hash calculation was done by the VMM. > However, computing the hash after the queue was chosen defeats the > purpose of RSS. > > Another

Re: [PATCH RFC v4 0/9] tun: Introduce virtio-net hashing feature

2024-09-24 Thread Jason Wang
On Tue, Sep 24, 2024 at 5:01 PM Akihiko Odaki wrote: > > virtio-net have two usage of hashes: one is RSS and another is hash > reporting. Conventionally the hash calculation was done by the VMM. > However, computing the hash after the queue was chosen defeats the > purpose of RSS. > > Another appr

Re: [PATCH RFC v4 0/9] tun: Introduce virtio-net hashing feature

2024-09-26 Thread Jason Wang
On Fri, Sep 27, 2024 at 10:11 AM Akihiko Odaki wrote: > > On 2024/09/25 12:30, Jason Wang wrote: > > On Tue, Sep 24, 2024 at 5:01 PM Akihiko Odaki > > wrote: > >> > >> virtio-net have two usage of hashes: one is RSS and another is hash > >> repor

Re: [PATCH RFC v4 0/9] tun: Introduce virtio-net hashing feature

2024-09-28 Thread Jason Wang
On Fri, Sep 27, 2024 at 3:51 PM Akihiko Odaki wrote: > > On 2024/09/27 13:31, Jason Wang wrote: > > On Fri, Sep 27, 2024 at 10:11 AM Akihiko Odaki > > wrote: > >> > >> On 2024/09/25 12:30, Jason Wang wrote: > >>> On Tue, Sep 24, 2024 at 5:01 PM A

Re: [PATCH v2 3/3] tun: Set num_buffers for virtio 1.0

2025-01-09 Thread Jason Wang
On Thu, Jan 9, 2025 at 2:59 PM Akihiko Odaki wrote: > > The specification says the device MUST set num_buffers to 1 if > VIRTIO_NET_F_MRG_RXBUF has not been negotiated. Have we agreed on how to fix the spec or not? As I replied in the spec patch, if we just remove this "MUST", it looks like we a

Re: [PATCH v2 2/3] tun: Pad virtio header with zero

2025-01-09 Thread Jason Wang
On Thu, Jan 9, 2025 at 2:59 PM Akihiko Odaki wrote: > > tun used to simply advance iov_iter when it needs to pad virtio header, > which leaves the garbage in the buffer as is. This is especially > problematic when tun starts to allow enabling the hash reporting > feature; even if the feature is en

Re: [PATCH v2 1/3] tun: Unify vnet implementation

2025-01-09 Thread Jason Wang
On Thu, Jan 9, 2025 at 2:59 PM Akihiko Odaki wrote: > > Both tun and tap exposes the same set of virtio-net-related features. > Unify their implementations to ease future changes. > > Signed-off-by: Akihiko Odaki > --- > MAINTAINERS| 1 + > drivers/net/Kconfig| 5 ++ > driver

Re: [PATCH net v3 9/9] tap: Use tun's vnet-related code

2025-01-19 Thread Jason Wang
On Fri, Jan 17, 2025 at 6:35 PM Akihiko Odaki wrote: > > On 2025/01/17 18:23, Willem de Bruijn wrote: > > Akihiko Odaki wrote: > >> tun and tap implements the same vnet-related features so reuse the code. > >> > >> Signed-off-by: Akihiko Odaki > >> --- > >> drivers/net/Kconfig| 1 + > >>

Re: [PATCH v2 3/3] tun: Set num_buffers for virtio 1.0

2025-01-19 Thread Jason Wang
On Thu, Jan 16, 2025 at 1:30 PM Akihiko Odaki wrote: > > On 2025/01/16 10:06, Jason Wang wrote: > > On Wed, Jan 15, 2025 at 1:07 PM Akihiko Odaki > > wrote: > >> > >> On 2025/01/13 12:04, Jason Wang wrote: > >>> On Fri, Jan 10, 2025 at 7:12 PM Ak

Re: [PATCH v2 3/3] tun: Set num_buffers for virtio 1.0

2025-01-12 Thread Jason Wang
On Fri, Jan 10, 2025 at 7:12 PM Akihiko Odaki wrote: > > On 2025/01/10 19:23, Michael S. Tsirkin wrote: > > On Fri, Jan 10, 2025 at 11:27:13AM +0800, Jason Wang wrote: > >> On Thu, Jan 9, 2025 at 2:59 PM Akihiko Odaki > >> wrote: > >>> > &

Re: [PATCH v2 3/3] tun: Set num_buffers for virtio 1.0

2025-01-15 Thread Jason Wang
On Wed, Jan 15, 2025 at 1:07 PM Akihiko Odaki wrote: > > On 2025/01/13 12:04, Jason Wang wrote: > > On Fri, Jan 10, 2025 at 7:12 PM Akihiko Odaki > > wrote: > >> > >> On 2025/01/10 19:23, Michael S. Tsirkin wrote: > >>> On Fri, Jan 10, 2025 at 1

Re: [PATCH net-next] tun: Pad virtio headers

2025-02-16 Thread Jason Wang
On Sat, Feb 15, 2025 at 1:25 PM Akihiko Odaki wrote: > > On 2025/02/14 0:43, Michael S. Tsirkin wrote: > > On Thu, Feb 13, 2025 at 06:23:55PM +0900, Akihiko Odaki wrote: > >> On 2025/02/13 16:18, Michael S. Tsirkin wrote: > >>> > >>> Commit log needs some work. > >>> > >>> So my understanding is,

Re: [PATCH net-next v2] tun: Pad virtio headers

2025-02-20 Thread Jason Wang
On Thu, Feb 20, 2025 at 4:45 PM Michael S. Tsirkin wrote: > > On Thu, Feb 20, 2025 at 08:58:38AM +0100, Paolo Abeni wrote: > > Hi, > > > > On 2/15/25 7:04 AM, Akihiko Odaki wrote: > > > tun simply advances iov_iter when it needs to pad virtio header, > > > which leaves the garbage in the buffer as

Re: [PATCH net-next v7 3/6] tun: Introduce virtio-net hash feature

2025-03-03 Thread Jason Wang
On Fri, Feb 28, 2025 at 3:59 PM Akihiko Odaki wrote: > > Hash reporting > -- > > Allow the guest to reuse the hash value to make receive steering > consistent between the host and guest, and to save hash computation. > > RSS > --- > > RSS is a receive steering algorithm that can be neg

Re: [PATCH net-next v9 6/6] vhost/net: Support VIRTIO_NET_F_HASH_REPORT

2025-03-10 Thread Jason Wang
On Mon, Mar 10, 2025 at 3:04 PM Akihiko Odaki wrote: > > On 2025/03/10 13:43, Jason Wang wrote: > > On Fri, Mar 7, 2025 at 7:02 PM Akihiko Odaki > > wrote: > >> > >> VIRTIO_NET_F_HASH_REPORT allows to report hash values calculated on the > >> host

Re: [PATCH net-next v9 3/6] tun: Introduce virtio-net hash feature

2025-03-10 Thread Jason Wang
On Mon, Mar 10, 2025 at 3:45 PM Akihiko Odaki wrote: > > On 2025/03/10 12:55, Jason Wang wrote: > > On Fri, Mar 7, 2025 at 7:01 PM Akihiko Odaki > > wrote: > >> > >> Hash reporting > >> == > >> > >> Allow the guest t

Re: [PATCH net-next v9 1/6] virtio_net: Add functions for hashing

2025-03-10 Thread Jason Wang
On Mon, Mar 10, 2025 at 2:53 PM Akihiko Odaki wrote: > > On 2025/03/10 12:55, Jason Wang wrote: > > On Fri, Mar 7, 2025 at 7:01 PM Akihiko Odaki > > wrote: > >> > >> They are useful to implement VIRTIO_NET_F_RSS and > >> VIRTIO_NET_F_HASH_R

Re: [PATCH net-next v9 3/6] tun: Introduce virtio-net hash feature

2025-03-10 Thread Jason Wang
On Mon, Mar 10, 2025 at 3:59 PM Akihiko Odaki wrote: > > On 2025/03/10 12:55, Jason Wang wrote: > > On Fri, Mar 7, 2025 at 7:01 PM Akihiko Odaki > > wrote: > >> > >> Hash reporting > >> == > >> > >> Allow the guest t

Re: [PATCH net-next v9 6/6] vhost/net: Support VIRTIO_NET_F_HASH_REPORT

2025-03-09 Thread Jason Wang
On Fri, Mar 7, 2025 at 7:02 PM Akihiko Odaki wrote: > > VIRTIO_NET_F_HASH_REPORT allows to report hash values calculated on the > host. When VHOST_NET_F_VIRTIO_NET_HDR is employed, it will report no > hash values (i.e., the hash_report member is always set to > VIRTIO_NET_HASH_REPORT_NONE). Otherw

Re: [PATCH net-next v9 1/6] virtio_net: Add functions for hashing

2025-03-09 Thread Jason Wang
On Fri, Mar 7, 2025 at 7:01 PM Akihiko Odaki wrote: > > They are useful to implement VIRTIO_NET_F_RSS and > VIRTIO_NET_F_HASH_REPORT. > > Signed-off-by: Akihiko Odaki > Tested-by: Lei Yang > --- > include/linux/virtio_net.h | 188 > + > 1 file change

Re: [PATCH net-next v9 3/6] tun: Introduce virtio-net hash feature

2025-03-09 Thread Jason Wang
On Fri, Mar 7, 2025 at 7:01 PM Akihiko Odaki wrote: > > Hash reporting > == > > Allow the guest to reuse the hash value to make receive steering > consistent between the host and guest, and to save hash computation. > > RSS > === > > RSS is a receive steering algorithm that can be nego

Re: [PATCH net-next v9 3/6] tun: Introduce virtio-net hash feature

2025-03-09 Thread Jason Wang
On Mon, Mar 10, 2025 at 11:55 AM Jason Wang wrote: > > On Fri, Mar 7, 2025 at 7:01 PM Akihiko Odaki wrote: > > > > Hash reporting > > == > > > > Allow the guest to reuse the hash value to make receive steering > > consistent between the ho

Re: [PATCH net-next v9 5/6] selftest: tun: Add tests for virtio-net hashing

2025-03-09 Thread Jason Wang
On Fri, Mar 7, 2025 at 7:02 PM Akihiko Odaki wrote: > > The added tests confirm tun can perform RSS and hash reporting, and > reject invalid configurations for them. Let's be more verbose here. E.g what's the network topology used here. > > Signed-off-by: Akihiko Odaki > Tested-by: Lei Yang >

Re: [PATCH net-next v9 3/6] tun: Introduce virtio-net hash feature

2025-03-11 Thread Jason Wang
On Tue, Mar 11, 2025 at 2:11 PM Akihiko Odaki wrote: > > On 2025/03/11 9:38, Jason Wang wrote: > > On Mon, Mar 10, 2025 at 3:45 PM Akihiko Odaki > > wrote: > >> > >> On 2025/03/10 12:55, Jason Wang wrote: > >>> On Fri, Mar 7, 2025 at 7:01 PM

Re: [PATCH net-next v9 3/6] tun: Introduce virtio-net hash feature

2025-03-11 Thread Jason Wang
On Tue, Mar 11, 2025 at 2:17 PM Akihiko Odaki wrote: > > On 2025/03/11 9:38, Jason Wang wrote: > > On Mon, Mar 10, 2025 at 3:45 PM Akihiko Odaki > > wrote: > >> > >> On 2025/03/10 12:55, Jason Wang wrote: > >>> On Fri, Mar 7, 2025 at 7:01 PM

Re: [PATCH net-next v9 6/6] vhost/net: Support VIRTIO_NET_F_HASH_REPORT

2025-03-11 Thread Jason Wang
On Tue, Mar 11, 2025 at 2:24 PM Akihiko Odaki wrote: > > On 2025/03/11 9:42, Jason Wang wrote: > > On Mon, Mar 10, 2025 at 3:04 PM Akihiko Odaki > > wrote: > >> > >> On 2025/03/10 13:43, Jason Wang wrote: > >>> On Fri, Ma

Re: [PATCH net-next v9 3/6] tun: Introduce virtio-net hash feature

2025-03-16 Thread Jason Wang
On Wed, Mar 12, 2025 at 1:03 PM Akihiko Odaki wrote: > > On 2025/03/12 11:35, Jason Wang wrote: > > On Tue, Mar 11, 2025 at 2:11 PM Akihiko Odaki > > wrote: > >> > >> On 2025/03/11 9:38, Jason Wang wrote: > >>> On Mon, Mar 10, 2025 at 3:45 PM Ak

Re: [PATCH net-next v9 3/6] tun: Introduce virtio-net hash feature

2025-03-16 Thread Jason Wang
On Wed, Mar 12, 2025 at 1:55 PM Akihiko Odaki wrote: > > On 2025/03/12 11:59, Jason Wang wrote: > > On Tue, Mar 11, 2025 at 2:17 PM Akihiko Odaki > > wrote: > >> > >> On 2025/03/11 9:38, Jason Wang wrote: > >>> On Mon, Mar 10, 2025 at 3:45 PM Ak

Re: [PATCH net-next v9 6/6] vhost/net: Support VIRTIO_NET_F_HASH_REPORT

2025-03-16 Thread Jason Wang
On Wed, Mar 12, 2025 at 1:59 PM Akihiko Odaki wrote: > > On 2025/03/12 12:36, Jason Wang wrote: > > On Tue, Mar 11, 2025 at 2:24 PM Akihiko Odaki > > wrote: > >> > >> On 2025/03/11 9:42, Jason Wang wrote: > >>> On Mon, Mar 10, 2025 at 3:04 PM Ak

Re: [PATCH net-next v9 1/6] virtio_net: Add functions for hashing

2025-03-16 Thread Jason Wang
On Tue, Mar 11, 2025 at 1:49 PM Akihiko Odaki wrote: > > On 2025/03/11 9:47, Jason Wang wrote: > > On Mon, Mar 10, 2025 at 2:53 PM Akihiko Odaki > > wrote: > >> > >> On 2025/03/10 12:55, Jason Wang wrote: > >>> On Fri, Mar 7, 2025 at 7:01 PM Aki

Re: [PATCH net-next v10 03/10] tun: Allow steering eBPF program to fall back

2025-03-16 Thread Jason Wang
On Thu, Mar 13, 2025 at 3:01 PM Akihiko Odaki wrote: > > This clarifies a steering eBPF program takes precedence over the other > steering algorithms. > > Signed-off-by: Akihiko Odaki > --- I think we should elaborate more on the advantages of *not* making this implicit. Or what's the advantage

Re: [PATCH net-next v9 3/6] tun: Introduce virtio-net hash feature

2025-03-17 Thread Jason Wang
On Mon, Mar 17, 2025 at 3:07 PM Akihiko Odaki wrote: > > On 2025/03/17 10:12, Jason Wang wrote: > > On Wed, Mar 12, 2025 at 1:03 PM Akihiko Odaki > > wrote: > >> > >> On 2025/03/12 11:35, Jason Wang wrote: > >>> On Tue, Mar 11, 2025 at 2:11 PM Ak

Re: [PATCH net-next v11 00/10] tun: Introduce virtio-net hashing feature

2025-03-17 Thread Jason Wang
On Mon, Mar 17, 2025 at 6:58 PM Akihiko Odaki wrote: > > virtio-net have two usage of hashes: one is RSS and another is hash > reporting. Conventionally the hash calculation was done by the VMM. > However, computing the hash after the queue was chosen defeats the > purpose of RSS. > > Another appr

Re: [PATCH net-next v9 3/6] tun: Introduce virtio-net hash feature

2025-03-18 Thread Jason Wang
On Tue, Mar 18, 2025 at 6:10 PM Akihiko Odaki wrote: > > On 2025/03/18 9:15, Jason Wang wrote: > > On Mon, Mar 17, 2025 at 3:07 PM Akihiko Odaki > > wrote: > >> > >> On 2025/03/17 10:12, Jason Wang wrote: > >>> On Wed, Mar 12, 2025 at 1:03 PM Ak

Re: [PATCH net-next v9 3/6] tun: Introduce virtio-net hash feature

2025-03-19 Thread Jason Wang
On Wed, Mar 19, 2025 at 1:29 PM Akihiko Odaki wrote: > > On 2025/03/19 9:58, Jason Wang wrote: > > On Tue, Mar 18, 2025 at 6:10 PM Akihiko Odaki > > wrote: > >> > >> On 2025/03/18 9:15, Jason Wang wrote: > >>> On Mon, Mar 17, 2025 at 3:07 PM Akihi

Re: [PATCH net-next v9 3/6] tun: Introduce virtio-net hash feature

2025-03-20 Thread Jason Wang
On Thu, Mar 20, 2025 at 1:33 PM Akihiko Odaki wrote: > > On 2025/03/20 10:31, Jason Wang wrote: > > On Wed, Mar 19, 2025 at 1:29 PM Akihiko Odaki > > wrote: > >> > >> On 2025/03/19 9:58, Jason Wang wrote: > >>> On Tue, Mar 18, 2025 at 6:10 PM Ak

Re: [PATCH net-next v9 3/6] tun: Introduce virtio-net hash feature

2025-03-23 Thread Jason Wang
On Fri, Mar 21, 2025 at 1:57 PM Akihiko Odaki wrote: > > On 2025/03/21 10:13, Jason Wang wrote: > > On Thu, Mar 20, 2025 at 1:33 PM Akihiko Odaki > > wrote: > >> > >> On 2025/03/20 10:31, Jason Wang wrote: > >>> On Wed, Mar 19, 2025 at 1:29 PM Ak