virtualization.lists.linux-foundation.org archive mirror
 help / color / mirror / Atom feed
* [PATCH net-next] vhost: remove unnecessary forward declarations in vhost.h
@ 2014-11-27  6:41 Jason Wang
  2014-11-30  5:04 ` David Miller
  2014-12-01  9:18 ` Michael S. Tsirkin
  0 siblings, 2 replies; 4+ messages in thread
From: Jason Wang @ 2014-11-27  6:41 UTC (permalink / raw)
  To: mst, virtualization, netdev, linux-kernel; +Cc: kvm

Signed-off-by: Jason Wang <jasowang@redhat.com>
---
 drivers/vhost/vhost.h | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/vhost/vhost.h b/drivers/vhost/vhost.h
index 3eda654..7d039ef 100644
--- a/drivers/vhost/vhost.h
+++ b/drivers/vhost/vhost.h
@@ -12,8 +12,6 @@
 #include <linux/virtio_ring.h>
 #include <linux/atomic.h>
 
-struct vhost_device;
-
 struct vhost_work;
 typedef void (*vhost_work_fn_t)(struct vhost_work *work);
 
@@ -54,8 +52,6 @@ struct vhost_log {
 	u64 len;
 };
 
-struct vhost_virtqueue;
-
 /* The virtqueue structure describes a queue attached to a device. */
 struct vhost_virtqueue {
 	struct vhost_dev *dev;
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH net-next] vhost: remove unnecessary forward declarations in vhost.h
  2014-11-27  6:41 [PATCH net-next] vhost: remove unnecessary forward declarations in vhost.h Jason Wang
@ 2014-11-30  5:04 ` David Miller
  2014-12-01  5:34   ` Jason Wang
  2014-12-01  9:18 ` Michael S. Tsirkin
  1 sibling, 1 reply; 4+ messages in thread
From: David Miller @ 2014-11-30  5:04 UTC (permalink / raw)
  To: jasowang; +Cc: netdev, virtualization, linux-kernel, kvm, mst

From: Jason Wang <jasowang@redhat.com>
Date: Thu, 27 Nov 2014 14:41:21 +0800

> Signed-off-by: Jason Wang <jasowang@redhat.com>

I don't think generic vhost patches should go via my tree.

If you disagree, let me know why, thanks :)

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH net-next] vhost: remove unnecessary forward declarations in vhost.h
  2014-11-30  5:04 ` David Miller
@ 2014-12-01  5:34   ` Jason Wang
  0 siblings, 0 replies; 4+ messages in thread
From: Jason Wang @ 2014-12-01  5:34 UTC (permalink / raw)
  To: David Miller, Michael S. Tsirkin
  Cc: netdev, linux-kernel, kvm, virtualization



On Sun, Nov 30, 2014 at 1:04 PM, David Miller <davem@davemloft.net> 
wrote:
> From: Jason Wang <jasowang@redhat.com>
> Date: Thu, 27 Nov 2014 14:41:21 +0800
> 
>>  Signed-off-by: Jason Wang <jasowang@redhat.com>
> 
> I don't think generic vhost patches should go via my tree.
> 
> If you disagree, let me know why, thanks :)

Agree. Michael, could you pls pick this into vhost tree?

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH net-next] vhost: remove unnecessary forward declarations in vhost.h
  2014-11-27  6:41 [PATCH net-next] vhost: remove unnecessary forward declarations in vhost.h Jason Wang
  2014-11-30  5:04 ` David Miller
@ 2014-12-01  9:18 ` Michael S. Tsirkin
  1 sibling, 0 replies; 4+ messages in thread
From: Michael S. Tsirkin @ 2014-12-01  9:18 UTC (permalink / raw)
  To: Jason Wang; +Cc: netdev, linux-kernel, kvm, virtualization

On Thu, Nov 27, 2014 at 02:41:21PM +0800, Jason Wang wrote:
> Signed-off-by: Jason Wang <jasowang@redhat.com>

Applied, thanks.

> ---
>  drivers/vhost/vhost.h | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/drivers/vhost/vhost.h b/drivers/vhost/vhost.h
> index 3eda654..7d039ef 100644
> --- a/drivers/vhost/vhost.h
> +++ b/drivers/vhost/vhost.h
> @@ -12,8 +12,6 @@
>  #include <linux/virtio_ring.h>
>  #include <linux/atomic.h>
>  
> -struct vhost_device;
> -
>  struct vhost_work;
>  typedef void (*vhost_work_fn_t)(struct vhost_work *work);
>  
> @@ -54,8 +52,6 @@ struct vhost_log {
>  	u64 len;
>  };
>  
> -struct vhost_virtqueue;
> -
>  /* The virtqueue structure describes a queue attached to a device. */
>  struct vhost_virtqueue {
>  	struct vhost_dev *dev;
> -- 
> 1.9.1

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-12-01  9:18 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-27  6:41 [PATCH net-next] vhost: remove unnecessary forward declarations in vhost.h Jason Wang
2014-11-30  5:04 ` David Miller
2014-12-01  5:34   ` Jason Wang
2014-12-01  9:18 ` Michael S. Tsirkin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).