[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID:
<AS2P194MB21709F0B79D6FB686D373B199AA52@AS2P194MB2170.EURP194.PROD.OUTLOOK.COM>
Date: Fri, 12 Jul 2024 01:02:00 +0200
From: Luigi Leonardi <luigi.leonardi@...look.com>
To: ameryhung@...il.com,
Bobby Eshleman <bobby.eshleman@...edance.com>
Cc: amery.hung@...edance.com,
bpf@...r.kernel.org,
bryantan@...are.com,
dan.carpenter@...aro.org,
davem@...emloft.net,
decui@...rosoft.com,
edumazet@...gle.com,
haiyangz@...rosoft.com,
jasowang@...hat.com,
jiang.wang@...edance.com,
kuba@...nel.org,
kvm@...r.kernel.org,
kys@...rosoft.com,
linux-hyperv@...r.kernel.org,
linux-kernel@...r.kernel.org,
mst@...hat.com,
netdev@...r.kernel.org,
oxffffaa@...il.com,
pabeni@...hat.com,
pv-drivers@...are.com,
sgarzare@...hat.com,
simon.horman@...igine.com,
stefanha@...hat.com,
vdasa@...are.com,
virtualization@...ts.linux-foundation.org,
wei.liu@...nel.org,
xiyou.wangcong@...il.com,
xuanzhuo@...ux.alibaba.com,
Luigi Leonardi <luigi.leonardi@...look.com>
Subject: Re: [RFC PATCH net-next v6 13/14] virtio/vsock: implement datagram support
Hi Bobby, Amery
Thank you for working on this!
> This commit implements datagram support with a new version of
> ->dgram_allow().
Commit messages should be imperative "This commit implements X" -> "Implements X".
https://www.kernel.org/doc/html/latest/process/submitting-patches.html#describe-your-changes
This suggestion applies to many of the commits in this series.
> +static bool virtio_transport_dgram_allow(u32 cid, u32 port)
> +{
> + struct virtio_vsock *vsock;
> + bool dgram_allow;
> +
> + dgram_allow = false;
I think you can initialize the variable in the declaration.
> + rcu_read_lock();
> + vsock = rcu_dereference(the_virtio_vsock);
> + if (vsock)
> + dgram_allow = vsock->dgram_allow;
> + rcu_read_unlock();
> +
> + return dgram_allow;
> +}
> +
The rest LGTM.
Thanks,
Luigi
Powered by blists - more mailing lists