[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-defa
From: |
Paolo Bonzini |
Subject: |
Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config |
Date: |
Tue, 3 Nov 2020 21:57:39 +0100 |
On Tue, Nov 3, 2020 at 6:01 PM Philippe Mathieu-Daudé <philmd@redhat.com> wrote:
> On 11/3/20 5:52 PM, Daniel P. Berrangé wrote:
> > On Tue, Nov 03, 2020 at 05:46:03PM +0100, Philippe Mathieu-Daudé wrote:
> >> We test './configure --without-default-devices' since commit
> >> 20885b5b169 (".travis.yml: test that no-default-device builds
> >> do not regress") in Travis-CI.
> >>
> >> One minor difference: the GitLab Ubuntu docker image has the
> >> Xen devel packages installed. As it is automatically selected,
> >> we need to disable it with the --disable-xen option, else the
> >> build fails:
> >>
> >> /usr/bin/ld: libcommon.fa.p/hw_xen_xen-legacy-backend.c.o: in function
> >> `xen_be_register_common':
> >> hw/xen/xen-legacy-backend.c:754: undefined reference to `xen_9pfs_ops'
> >> /usr/bin/ld: libcommon.fa.p/fsdev_qemu-fsdev.c.o:(.data.rel+0x8):
> >> undefined reference to `local_ops'
> >> /usr/bin/ld: libcommon.fa.p/fsdev_qemu-fsdev.c.o:(.data.rel+0x20):
> >> undefined reference to `synth_ops'
> >> /usr/bin/ld: libcommon.fa.p/fsdev_qemu-fsdev.c.o:(.data.rel+0x38):
> >> undefined reference to `proxy_ops'
All these symbols are defined in hw/9pfs/meson.build's fs_ss, which is
conditional on CONFIG_9PFS. So:
diff --git a/accel/Kconfig b/accel/Kconfig
index 2ad94a3839..d24664d736 100644
--- a/accel/Kconfig
+++ b/accel/Kconfig
@@ -7,3 +7,4 @@ config KVM
config XEN
bool
select FSDEV_9P if VIRTFS
+ select 9PFS if VIRTFS
or alternatively (and I think better):
diff --git a/hw/9pfs/Kconfig b/hw/9pfs/Kconfig
index d3ebd73730..3ae5749661 100644
--- a/hw/9pfs/Kconfig
+++ b/hw/9pfs/Kconfig
@@ -2,12 +2,8 @@ config FSDEV_9P
bool
depends on VIRTFS
-config 9PFS
- bool
-
config VIRTIO_9P
bool
default y
depends on VIRTFS && VIRTIO
select FSDEV_9P
- select 9PFS
diff --git a/hw/9pfs/meson.build b/hw/9pfs/meson.build
index cc09426212..99be5d9119 100644
--- a/hw/9pfs/meson.build
+++ b/hw/9pfs/meson.build
@@ -15,6 +15,6 @@ fs_ss.add(files(
'coxattr.c',
))
fs_ss.add(when: 'CONFIG_XEN', if_true: files('xen-9p-backend.c'))
-softmmu_ss.add_all(when: 'CONFIG_9PFS', if_true: fs_ss)
+softmmu_ss.add_all(when: 'CONFIG_FSDEV_9P', if_true: fs_ss)
specific_ss.add(when: 'CONFIG_VIRTIO_9P', if_true: files('virtio-9p-device.c'))
Paolo
- Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, (continued)
- Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, Philippe Mathieu-Daudé, 2020/11/03
- Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, Daniel P . Berrangé, 2020/11/03
- Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, Philippe Mathieu-Daudé, 2020/11/03
- Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, Stefano Stabellini, 2020/11/04
- Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, Thomas Huth, 2020/11/04
- Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, Stefano Stabellini, 2020/11/04
- Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, Philippe Mathieu-Daudé, 2020/11/05
- Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, Paolo Bonzini, 2020/11/04
- Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, Stefano Stabellini, 2020/11/04
- Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, Paolo Bonzini, 2020/11/05
- Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config,
Paolo Bonzini <=
- Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, Philippe Mathieu-Daudé, 2020/11/03
- Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, Paolo Bonzini, 2020/11/03
- Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, Philippe Mathieu-Daudé, 2020/11/04
Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, Philippe Mathieu-Daudé, 2020/11/03
Re: [PATCH-for-5.2 2/3] gitlab-ci: Add a job to cover the --without-default-devices config, Thomas Huth, 2020/11/03
[PATCH-for-5.2 3/3] travis-ci: Remove the --without-default-devices job, Philippe Mathieu-Daudé, 2020/11/03