Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1418532 > unrolled thread

[PATCH -next] virtio_net: Update the feature bit to comply with spec

Started byAaron Conole <aconole@redhat.com>
First post2016-06-09 19:50 +0200
Last post2016-06-10 08:40 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH -next] virtio_net: Update the feature bit to comply with spec Aaron Conole <aconole@redhat.com> - 2016-06-09 19:50 +0200
    Re: [PATCH -next] virtio_net: Update the feature bit to comply  with spec David Miller <davem@davemloft.net> - 2016-06-10 08:40 +0200

#1418532 — [PATCH -next] virtio_net: Update the feature bit to comply with spec

FromAaron Conole <aconole@redhat.com>
Date2016-06-09 19:50 +0200
Subject[PATCH -next] virtio_net: Update the feature bit to comply with spec
Message-ID<rIcfD-18x-11@gated-at.bofh.it>
A draft version of the MTU Advice feature bit was specified as 25.  This
bit is not within the allowed range for network device feature bits, and
should be changed to be feature bit 3 to fully comply with the spec.

Fixes 14de9d114a82 ('virtio-net: Add initial MTU advice feature')
Signed-off-by: Aaron Conole <aconole@redhat.com>
Suggested-by: "Michael S. Tsirkin" <mst@redhat.com>
---
 include/uapi/linux/virtio_net.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/uapi/linux/virtio_net.h b/include/uapi/linux/virtio_net.h
index 1ab4ea6..0da0e3a 100644
--- a/include/uapi/linux/virtio_net.h
+++ b/include/uapi/linux/virtio_net.h
@@ -35,6 +35,7 @@
 #define VIRTIO_NET_F_CSUM	0	/* Host handles pkts w/ partial csum */
 #define VIRTIO_NET_F_GUEST_CSUM	1	/* Guest handles pkts w/ partial csum */
 #define VIRTIO_NET_F_CTRL_GUEST_OFFLOADS 2 /* Dynamic offload configuration. */
+#define VIRTIO_NET_F_MTU	3	/* Initial MTU advice */
 #define VIRTIO_NET_F_MAC	5	/* Host has given MAC address. */
 #define VIRTIO_NET_F_GUEST_TSO4	7	/* Guest can handle TSOv4 in. */
 #define VIRTIO_NET_F_GUEST_TSO6	8	/* Guest can handle TSOv6 in. */
@@ -55,7 +56,6 @@
 #define VIRTIO_NET_F_MQ	22	/* Device supports Receive Flow
 					 * Steering */
 #define VIRTIO_NET_F_CTRL_MAC_ADDR 23	/* Set MAC address */
-#define VIRTIO_NET_F_MTU 25	/* Initial MTU advice */
 
 #ifndef VIRTIO_NET_NO_LEGACY
 #define VIRTIO_NET_F_GSO	6	/* Host handles pkts w/ any GSO type */
-- 
2.5.5

[toc] | [next] | [standalone]


#1419061 — Re: [PATCH -next] virtio_net: Update the feature bit to comply with spec

FromDavid Miller <davem@davemloft.net>
Date2016-06-10 08:40 +0200
SubjectRe: [PATCH -next] virtio_net: Update the feature bit to comply with spec
Message-ID<rIogN-16D-1@gated-at.bofh.it>
In reply to#1418532
From: Aaron Conole <aconole@redhat.com>
Date: Thu,  9 Jun 2016 13:41:15 -0400

> A draft version of the MTU Advice feature bit was specified as 25.  This
> bit is not within the allowed range for network device feature bits, and
> should be changed to be feature bit 3 to fully comply with the spec.
> 
> Fixes 14de9d114a82 ('virtio-net: Add initial MTU advice feature')
> Signed-off-by: Aaron Conole <aconole@redhat.com>
> Suggested-by: "Michael S. Tsirkin" <mst@redhat.com>

Applied.

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web