All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-devel] Re: vhost.c: include <linux/vhost.h> last
  2010-04-08 20:49 [Qemu-devel] vhost.c: include <linux/vhost.h> last Marcelo Tosatti
@ 2010-04-08 20:48 ` Michael S. Tsirkin
  2010-04-13 22:12   ` Aurelien Jarno
  0 siblings, 1 reply; 3+ messages in thread
From: Michael S. Tsirkin @ 2010-04-08 20:48 UTC (permalink / raw)
  To: Marcelo Tosatti; +Cc: qemu-devel

On Thu, Apr 08, 2010 at 05:49:50PM -0300, Marcelo Tosatti wrote:
> 
> So the userspace headers define KERNEL_STRICT_NAMES and there's no
> conflict on type definition for older kernels.
> 
> igned-off-by: Marcelo Tosatti <mtosatti@redhat.com>

Acked-by: Michael S. Tsirkin <mst@redhat.com>

> diff --git a/hw/vhost.c b/hw/vhost.c
> index ad2f98a..d37a66e 100644
> --- a/hw/vhost.c
> +++ b/hw/vhost.c
> @@ -10,13 +10,13 @@
>   * the COPYING file in the top-level directory.
>   */
>  
> -#include <linux/vhost.h>
>  #include <sys/ioctl.h>
>  #include <sys/eventfd.h>
>  #include "vhost.h"
>  #include "hw/hw.h"
>  /* For range_get_last */
>  #include "pci.h"
> +#include <linux/vhost.h>
>  
>  static void vhost_dev_sync_region(struct vhost_dev *dev,
>                                    uint64_t mfirst, uint64_t mlast,

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

* [Qemu-devel] vhost.c: include <linux/vhost.h> last
@ 2010-04-08 20:49 Marcelo Tosatti
  2010-04-08 20:48 ` [Qemu-devel] " Michael S. Tsirkin
  0 siblings, 1 reply; 3+ messages in thread
From: Marcelo Tosatti @ 2010-04-08 20:49 UTC (permalink / raw)
  To: qemu-devel; +Cc: Michael S. Tsirkin


So the userspace headers define KERNEL_STRICT_NAMES and there's no
conflict on type definition for older kernels.

igned-off-by: Marcelo Tosatti <mtosatti@redhat.com>

diff --git a/hw/vhost.c b/hw/vhost.c
index ad2f98a..d37a66e 100644
--- a/hw/vhost.c
+++ b/hw/vhost.c
@@ -10,13 +10,13 @@
  * the COPYING file in the top-level directory.
  */
 
-#include <linux/vhost.h>
 #include <sys/ioctl.h>
 #include <sys/eventfd.h>
 #include "vhost.h"
 #include "hw/hw.h"
 /* For range_get_last */
 #include "pci.h"
+#include <linux/vhost.h>
 
 static void vhost_dev_sync_region(struct vhost_dev *dev,
                                   uint64_t mfirst, uint64_t mlast,

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

* Re: [Qemu-devel] Re: vhost.c: include <linux/vhost.h> last
  2010-04-08 20:48 ` [Qemu-devel] " Michael S. Tsirkin
@ 2010-04-13 22:12   ` Aurelien Jarno
  0 siblings, 0 replies; 3+ messages in thread
From: Aurelien Jarno @ 2010-04-13 22:12 UTC (permalink / raw)
  To: Michael S. Tsirkin; +Cc: Marcelo Tosatti, qemu-devel

On Thu, Apr 08, 2010 at 11:48:56PM +0300, Michael S. Tsirkin wrote:
> On Thu, Apr 08, 2010 at 05:49:50PM -0300, Marcelo Tosatti wrote:
> > 
> > So the userspace headers define KERNEL_STRICT_NAMES and there's no
> > conflict on type definition for older kernels.
> > 
> > igned-off-by: Marcelo Tosatti <mtosatti@redhat.com>
> 
> Acked-by: Michael S. Tsirkin <mst@redhat.com>

Thanks, applied.

> > diff --git a/hw/vhost.c b/hw/vhost.c
> > index ad2f98a..d37a66e 100644
> > --- a/hw/vhost.c
> > +++ b/hw/vhost.c
> > @@ -10,13 +10,13 @@
> >   * the COPYING file in the top-level directory.
> >   */
> >  
> > -#include <linux/vhost.h>
> >  #include <sys/ioctl.h>
> >  #include <sys/eventfd.h>
> >  #include "vhost.h"
> >  #include "hw/hw.h"
> >  /* For range_get_last */
> >  #include "pci.h"
> > +#include <linux/vhost.h>
> >  
> >  static void vhost_dev_sync_region(struct vhost_dev *dev,
> >                                    uint64_t mfirst, uint64_t mlast,
> 
> 
> 

-- 
Aurelien Jarno                          GPG: 1024D/F1BCDB73
aurelien@aurel32.net                 http://www.aurel32.net

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

end of thread, other threads:[~2010-04-13 22:14 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-04-08 20:49 [Qemu-devel] vhost.c: include <linux/vhost.h> last Marcelo Tosatti
2010-04-08 20:48 ` [Qemu-devel] " Michael S. Tsirkin
2010-04-13 22:12   ` Aurelien Jarno

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.