> On 04/13/2016 03:15 PM, Greg Kurz wrote: >> The <sys/uio.h> system header doesn't exist on all host platforms. Code >> should >> include "qemu/osdep.h" instead to avoid build breaks on plafforms that >> don't >> define CONFIG_IOVEC (like win32, if it is to support 9p one day). >> >> Signed-off-by: Greg Kurz <gk...@linux.vnet.ibm.com> > > Acked-by: Cédric Le Goater <c...@kaod.org> Acked-by: Michael Fritscher <mich...@fritscher.net>
Yes, I still plan to upstream the win32 9p support in the next weeks, when my stress level goes down a bit ;-) Best regards, Michael Fritscher >> --- >> fsdev/9p-iov-marshal.c | 1 - >> fsdev/9p-marshal.c | 1 - >> fsdev/file-op-9p.h | 2 +- >> 3 files changed, 1 insertion(+), 3 deletions(-) >> >> diff --git a/fsdev/9p-iov-marshal.c b/fsdev/9p-iov-marshal.c >> index fb40bdf0d5f6..a564637b73c1 100644 >> --- a/fsdev/9p-iov-marshal.c >> +++ b/fsdev/9p-iov-marshal.c >> @@ -15,7 +15,6 @@ >> #include <glib.h> >> #include <glib/gprintf.h> >> #include <utime.h> >> -#include <sys/uio.h> >> >> #include "9p-iov-marshal.h" >> #include "qemu/bswap.h" >> diff --git a/fsdev/9p-marshal.c b/fsdev/9p-marshal.c >> index 183d3667c681..df9d4f8e23c9 100644 >> --- a/fsdev/9p-marshal.c >> +++ b/fsdev/9p-marshal.c >> @@ -16,7 +16,6 @@ >> #include <glib/gprintf.h> >> #include <dirent.h> >> #include <utime.h> >> -#include <sys/uio.h> >> >> #include "9p-marshal.h" >> >> diff --git a/fsdev/file-op-9p.h b/fsdev/file-op-9p.h >> index b8c26024a955..8736d1fa6d52 100644 >> --- a/fsdev/file-op-9p.h >> +++ b/fsdev/file-op-9p.h >> @@ -12,9 +12,9 @@ >> */ >> #ifndef _FILEOP_H >> #define _FILEOP_H >> +#include "qemu/osdep.h" >> #include <dirent.h> >> #include <utime.h> >> -#include <sys/uio.h> >> #include <sys/vfs.h> >> >> #define SM_LOCAL_MODE_BITS 0600 >> >> > >
