2013-05-06 10:37:45

by Michael S. Tsirkin

[permalink] [raw]
Subject: [PATCH] vhost: drop virtio_net.h dependency

There's no net specific code in vhost.c anymore,
don't include the virtio_net.h header.

Signed-off-by: Michael S. Tsirkin <[email protected]>
---

This is on top of Asias' patches, already queued so
just FYI.

drivers/vhost/vhost.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/vhost/vhost.c b/drivers/vhost/vhost.c
index de9441a..dcde269 100644
--- a/drivers/vhost/vhost.c
+++ b/drivers/vhost/vhost.c
@@ -13,7 +13,6 @@

#include <linux/eventfd.h>
#include <linux/vhost.h>
-#include <linux/virtio_net.h>
#include <linux/mm.h>
#include <linux/mmu_context.h>
#include <linux/miscdevice.h>
--
MST


2013-05-06 12:07:21

by Asias He

[permalink] [raw]
Subject: Re: [PATCH] vhost: drop virtio_net.h dependency

On Mon, May 06, 2013 at 01:37:34PM +0300, Michael S. Tsirkin wrote:
> There's no net specific code in vhost.c anymore,
> don't include the virtio_net.h header.
>
> Signed-off-by: Michael S. Tsirkin <[email protected]>

Reviewed-by: Asias He <[email protected]>

> ---
>
> This is on top of Asias' patches, already queued so
> just FYI.
>
> drivers/vhost/vhost.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/vhost/vhost.c b/drivers/vhost/vhost.c
> index de9441a..dcde269 100644
> --- a/drivers/vhost/vhost.c
> +++ b/drivers/vhost/vhost.c
> @@ -13,7 +13,6 @@
>
> #include <linux/eventfd.h>
> #include <linux/vhost.h>
> -#include <linux/virtio_net.h>
> #include <linux/mm.h>
> #include <linux/mmu_context.h>
> #include <linux/miscdevice.h>
> --
> MST

--
Asias

2013-05-06 13:23:36

by Asias He

[permalink] [raw]
Subject: Re: [PATCH] vhost: drop virtio_net.h dependency

On Mon, May 06, 2013 at 01:37:34PM +0300, Michael S. Tsirkin wrote:
> There's no net specific code in vhost.c anymore,
> don't include the virtio_net.h header.

Did you push the it to your tree. I am not seeing it.

> Signed-off-by: Michael S. Tsirkin <[email protected]>
> ---
>
> This is on top of Asias' patches, already queued so
> just FYI.
>
> drivers/vhost/vhost.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/vhost/vhost.c b/drivers/vhost/vhost.c
> index de9441a..dcde269 100644
> --- a/drivers/vhost/vhost.c
> +++ b/drivers/vhost/vhost.c
> @@ -13,7 +13,6 @@
>
> #include <linux/eventfd.h>
> #include <linux/vhost.h>
> -#include <linux/virtio_net.h>
> #include <linux/mm.h>
> #include <linux/mmu_context.h>
> #include <linux/miscdevice.h>
> --
> MST

--
Asias