Jason Wang
2022-May-10 02:05 UTC
[PATCH] virtio_net: Remove unused case in virtio_skb_set_hash()
On Mon, May 9, 2022 at 9:17 PM Tang Bin <tangbin at cmss.chinamobile.com> wrote:> > In this function, "VIRTIO_NET_HASH_REPORT_NONE" is included > in "default", so it canbe removed. > > Signed-off-by: Tang Bin <tangbin at cmss.chinamobile.com> > --- > drivers/net/virtio_net.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c > index 87838cbe3..b3e5d8637 100644 > --- a/drivers/net/virtio_net.c > +++ b/drivers/net/virtio_net.c > @@ -1172,7 +1172,6 @@ static void virtio_skb_set_hash(const struct virtio_net_hdr_v1_hash *hdr_hash, > case VIRTIO_NET_HASH_REPORT_IPv6_EX: > rss_hash_type = PKT_HASH_TYPE_L3; > break; > - case VIRTIO_NET_HASH_REPORT_NONE: > default: > rss_hash_type = PKT_HASH_TYPE_NONE;I wonder if we need to do things in the reverse. Warn for default and only set NONE when it's NONE? Thanks> } > -- > 2.20.1.windows.1 > > >