On Mon, Mar 14, 2016 at 03:48:23PM +0100, Ladi Prosek wrote:
> Virtio 1.0 introduces new constants and data structures, common to
> all devices as well as specific to virtio-net. This commit adds a
> subset of these to be able to drive the virtio-net 1.0 network
> device.
> 
> Signed-off-by: Ladi Prosek <lpro...@redhat.com>
> ---
>  src/drivers/net/virtio-net.h   | 18 +++++++++++++
>  src/include/ipxe/virtio-pci.h  | 60 
> ++++++++++++++++++++++++++++++++++++++++++
>  src/include/ipxe/virtio-ring.h |  8 ++++++
>  3 files changed, 86 insertions(+)
> 
> diff --git a/src/drivers/net/virtio-net.h b/src/drivers/net/virtio-net.h
> index 3abef28..5bde7d7 100644
> --- a/src/drivers/net/virtio-net.h
> +++ b/src/drivers/net/virtio-net.h
> @@ -15,6 +15,14 @@
>  #define VIRTIO_NET_F_HOST_ECN   13      /* Host can handle TSO[6] w/ ECN in. 
> */
>  #define VIRTIO_NET_F_HOST_UFO   14      /* Host can handle UFO in. */
>  
> +/* Virtio 1.0 feature bits for virtio net */
> +#define VIRTIO_NET_F_MRG_RXBUF  15      /* Driver can merge receive buffers. 
> */
> +#define VIRTIO_NET_F_STATUS     16      /* Configuration status field is 
> available. */
> +#define VIRTIO_NET_F_CTRL_VQ    17      /* Control channel is available. */
> +#define VIRTIO_NET_F_CTRL_RX    18      /* Control channel RX mode support. 
> */
> +#define VIRTIO_NET_F_CTRL_VLAN  19      /* Control channel VLAN filtering. */
> +#define VIRTIO_NET_F_GUEST_ANNOUNCE 21  /* Driver can send gratuitous 
> packets. */

Please drop the comment about Virtio 1.0, these were all in virtio
0.9.5:
https://ozlabs.org/~rusty/virtio-spec/virtio-0.9.5.pdf

None of them seem necessary for the VIRTIO 1.0 implementation but there
is little harm in adding these #defines.

Attachment: signature.asc
Description: PGP signature

_______________________________________________
ipxe-devel mailing list
ipxe-devel@lists.ipxe.org
https://lists.ipxe.org/mailman/listinfo.cgi/ipxe-devel

Reply via email to