* [PATCH v3 1/3] tests/vhost-user-bridge: move to contrib/
@ 2020-03-02 15:46 Laurent Vivier
2020-03-02 15:46 ` [PATCH v3 2/3] virtiofsd: add it to the tools list Laurent Vivier
2020-03-02 15:46 ` [PATCH v3 3/3] travis: enable tools build on OS X Laurent Vivier
0 siblings, 2 replies; 4+ messages in thread
From: Laurent Vivier @ 2020-03-02 15:46 UTC (permalink / raw)
To: qemu-devel
Cc: Fam Zheng, Laurent Vivier, Juan Quintela, Alex Bennée,
Michael S. Tsirkin, Dr. David Alan Gilbert, Stefan Hajnoczi,
Marc-André Lureau, Philippe Mathieu-Daudé
vhost-user-bridge is not a test. Move it to contrib/.
It will be built with:
make vhost-user-bridge
Suggested-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
---
Notes:
v3: don't add vhost-user-bridge to the tools list
libvhost-user doesn't build on MacOSX and for instance
vhost-user-blk is also built only on demand
v2: update docs
Makefile | 3 +++
Makefile.objs | 1 +
contrib/vhost-user-bridge/Makefile.objs | 1 +
tests/vhost-user-bridge.c => contrib/vhost-user-bridge/main.c | 0
docs/devel/migration.rst | 2 +-
tests/Makefile.include | 1 -
6 files changed, 6 insertions(+), 2 deletions(-)
create mode 100644 contrib/vhost-user-bridge/Makefile.objs
rename tests/vhost-user-bridge.c => contrib/vhost-user-bridge/main.c (100%)
diff --git a/Makefile b/Makefile
index aa9cc0b58475..218b8259a49a 100644
--- a/Makefile
+++ b/Makefile
@@ -445,6 +445,7 @@ dummy := $(call unnest-vars,, \
libvhost-user-obj-y \
vhost-user-scsi-obj-y \
vhost-user-blk-obj-y \
+ vhost-user-bridge-obj-y \
vhost-user-input-obj-y \
vhost-user-gpu-obj-y \
qga-vss-dll-obj-y \
@@ -688,6 +689,8 @@ vhost-user-scsi$(EXESUF): $(vhost-user-scsi-obj-y) libvhost-user.a
$(call LINK, $^)
vhost-user-blk$(EXESUF): $(vhost-user-blk-obj-y) libvhost-user.a
$(call LINK, $^)
+vhost-user-bridge$(EXESUF): $(vhost-user-bridge-obj-y) libvhost-user.a
+ $(call LINK, $^)
rdmacm-mux$(EXESUF): LIBS += "-libumad"
rdmacm-mux$(EXESUF): $(rdmacm-mux-obj-y) $(COMMON_LDADDS)
diff --git a/Makefile.objs b/Makefile.objs
index 8a1cbe8000e6..c282ff77dda5 100644
--- a/Makefile.objs
+++ b/Makefile.objs
@@ -107,6 +107,7 @@ vhost-user-scsi.o-cflags := $(LIBISCSI_CFLAGS)
vhost-user-scsi.o-libs := $(LIBISCSI_LIBS)
vhost-user-scsi-obj-y = contrib/vhost-user-scsi/
vhost-user-blk-obj-y = contrib/vhost-user-blk/
+vhost-user-bridge-obj-y = contrib/vhost-user-bridge/
rdmacm-mux-obj-y = contrib/rdmacm-mux/
vhost-user-input-obj-y = contrib/vhost-user-input/
vhost-user-gpu-obj-y = contrib/vhost-user-gpu/
diff --git a/contrib/vhost-user-bridge/Makefile.objs b/contrib/vhost-user-bridge/Makefile.objs
new file mode 100644
index 000000000000..36a8d9b49a05
--- /dev/null
+++ b/contrib/vhost-user-bridge/Makefile.objs
@@ -0,0 +1 @@
+vhost-user-bridge-obj-y = main.o
diff --git a/tests/vhost-user-bridge.c b/contrib/vhost-user-bridge/main.c
similarity index 100%
rename from tests/vhost-user-bridge.c
rename to contrib/vhost-user-bridge/main.c
diff --git a/docs/devel/migration.rst b/docs/devel/migration.rst
index e88918f7639e..d00424460e23 100644
--- a/docs/devel/migration.rst
+++ b/docs/devel/migration.rst
@@ -807,7 +807,7 @@ The Linux kernel userfault support works on `/dev/shm` memory and on `hugetlbfs`
for hugetlbfs which may be a problem in some configurations).
The vhost-user code in QEMU supports clients that have Postcopy support,
-and the `vhost-user-bridge` (in `tests/`) and the DPDK package have changes
+and the `vhost-user-bridge` (in `contrib/`) and the DPDK package have changes
to support postcopy.
The client needs to open a userfaultfd and register the areas
diff --git a/tests/Makefile.include b/tests/Makefile.include
index edcbd475aa70..2dc95c52c82d 100644
--- a/tests/Makefile.include
+++ b/tests/Makefile.include
@@ -588,7 +588,6 @@ include $(SRC_PATH)/tests/qtest/Makefile.include
tests/test-qga$(EXESUF): qemu-ga$(EXESUF)
tests/test-qga$(EXESUF): tests/test-qga.o $(qtest-obj-y)
-tests/vhost-user-bridge$(EXESUF): tests/vhost-user-bridge.o $(test-util-obj-y) libvhost-user.a
SPEED = quick
--
2.24.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v3 2/3] virtiofsd: add it to the tools list
2020-03-02 15:46 [PATCH v3 1/3] tests/vhost-user-bridge: move to contrib/ Laurent Vivier
@ 2020-03-02 15:46 ` Laurent Vivier
2020-03-02 15:46 ` [PATCH v3 3/3] travis: enable tools build on OS X Laurent Vivier
1 sibling, 0 replies; 4+ messages in thread
From: Laurent Vivier @ 2020-03-02 15:46 UTC (permalink / raw)
To: qemu-devel
Cc: Fam Zheng, Laurent Vivier, Juan Quintela, Alex Bennée,
Michael S. Tsirkin, Dr. David Alan Gilbert, Stefan Hajnoczi,
Marc-André Lureau, Philippe Mathieu-Daudé
it will be built only when tools are enabled (always enabled by default)
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
Reviewed-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
---
Makefile | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/Makefile b/Makefile
index 218b8259a49a..ba8ffc9331c3 100644
--- a/Makefile
+++ b/Makefile
@@ -327,7 +327,7 @@ HELPERS-y += vhost-user-gpu$(EXESUF)
vhost-user-json-y += contrib/vhost-user-gpu/50-qemu-gpu.json
endif
-ifeq ($(CONFIG_LINUX)$(CONFIG_SECCOMP)$(CONFIG_LIBCAP_NG),yyy)
+ifeq ($(CONFIG_LINUX)$(CONFIG_SECCOMP)$(CONFIG_LIBCAP_NG)$(CONFIG_TOOLS),yyyy)
HELPERS-y += virtiofsd$(EXESUF)
vhost-user-json-y += tools/virtiofsd/50-qemu-virtiofsd.json
endif
@@ -697,7 +697,7 @@ rdmacm-mux$(EXESUF): $(rdmacm-mux-obj-y) $(COMMON_LDADDS)
$(call LINK, $^)
# relies on Linux-specific syscalls
-ifeq ($(CONFIG_LINUX)$(CONFIG_SECCOMP)$(CONFIG_LIBCAP_NG),yyy)
+ifeq ($(CONFIG_LINUX)$(CONFIG_SECCOMP)$(CONFIG_LIBCAP_NG)$(CONFIG_TOOLS),yyyy)
virtiofsd$(EXESUF): $(virtiofsd-obj-y) libvhost-user.a $(COMMON_LDADDS)
$(call LINK, $^)
endif
--
2.24.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v3 3/3] travis: enable tools build on OS X
2020-03-02 15:46 [PATCH v3 1/3] tests/vhost-user-bridge: move to contrib/ Laurent Vivier
2020-03-02 15:46 ` [PATCH v3 2/3] virtiofsd: add it to the tools list Laurent Vivier
@ 2020-03-02 15:46 ` Laurent Vivier
2020-03-02 17:15 ` Alex Bennée
1 sibling, 1 reply; 4+ messages in thread
From: Laurent Vivier @ 2020-03-02 15:46 UTC (permalink / raw)
To: qemu-devel
Cc: Fam Zheng, Laurent Vivier, Juan Quintela, Alex Bennée,
Michael S. Tsirkin, Dr. David Alan Gilbert, Stefan Hajnoczi,
Marc-André Lureau, Philippe Mathieu-Daudé
As we can build tools on OS X we should check we don't break build
when we submit new codes.
Signed-off-by: Laurent Vivier <lvivier@redhat.com>
---
.travis.yml | 1 +
1 file changed, 1 insertion(+)
diff --git a/.travis.yml b/.travis.yml
index 9867272177d4..70a24bf2fc8b 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -273,6 +273,7 @@ jobs:
- name: "OSX Xcode 10.3"
env:
+ - BASE_CONFIG="--disable-docs --enable-tools"
- CONFIG="--target-list=i386-softmmu,ppc-softmmu,ppc64-softmmu,m68k-softmmu,x86_64-softmmu"
os: osx
osx_image: xcode10.3
--
2.24.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v3 3/3] travis: enable tools build on OS X
2020-03-02 15:46 ` [PATCH v3 3/3] travis: enable tools build on OS X Laurent Vivier
@ 2020-03-02 17:15 ` Alex Bennée
0 siblings, 0 replies; 4+ messages in thread
From: Alex Bennée @ 2020-03-02 17:15 UTC (permalink / raw)
To: Laurent Vivier
Cc: Fam Zheng, Juan Quintela, Michael S. Tsirkin, qemu-devel,
Dr. David Alan Gilbert, Stefan Hajnoczi, Marc-André Lureau,
Philippe Mathieu-Daudé
Laurent Vivier <lvivier@redhat.com> writes:
> As we can build tools on OS X we should check we don't break build
> when we submit new codes.
I agree, snarfed this patch into testing/next, thanks.
>
> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
> ---
> .travis.yml | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/.travis.yml b/.travis.yml
> index 9867272177d4..70a24bf2fc8b 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -273,6 +273,7 @@ jobs:
>
> - name: "OSX Xcode 10.3"
> env:
> + - BASE_CONFIG="--disable-docs --enable-tools"
> - CONFIG="--target-list=i386-softmmu,ppc-softmmu,ppc64-softmmu,m68k-softmmu,x86_64-softmmu"
> os: osx
> osx_image: xcode10.3
--
Alex Bennée
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2020-03-02 17:20 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-02 15:46 [PATCH v3 1/3] tests/vhost-user-bridge: move to contrib/ Laurent Vivier
2020-03-02 15:46 ` [PATCH v3 2/3] virtiofsd: add it to the tools list Laurent Vivier
2020-03-02 15:46 ` [PATCH v3 3/3] travis: enable tools build on OS X Laurent Vivier
2020-03-02 17:15 ` Alex Bennée
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.