* [Qemu-devel] [PATCH 0/1] fixup 9pfs dependencies @ 2017-08-08 12:17 Cornelia Huck 2017-08-08 12:17 ` [Qemu-devel] [PATCH 1/1] 9pfs: fix and simplify dependencies Cornelia Huck 0 siblings, 1 reply; 6+ messages in thread From: Cornelia Huck @ 2017-08-08 12:17 UTC (permalink / raw) To: aneesh.kumar, groug; +Cc: borntraeger, agraf, thuth, qemu-devel, Cornelia Huck During my work on cleaning up the pci dependencies on s390x (see my "s390x: zPCI detangling" patch series), I noticed that the 9pfs code is only built if CONFIG_PCI is set. There's really nothing in there that should depend on CONFIG_PCI (it is really transport-agnostic); we do want to make sure that we have a virtio-9p-transport device, however. This patch is on top of master; it does not really make a difference right now, but will be needed after my patch series to avoid an inconsistent config for s390x with pci off. Cornelia Huck (1): 9pfs: fix and simplify dependencies configure | 2 +- default-configs/s390x-softmmu.mak | 1 + fsdev/Makefile.objs | 2 +- hw/Makefile.objs | 2 +- 4 files changed, 4 insertions(+), 3 deletions(-) -- 2.13.4 ^ permalink raw reply [flat|nested] 6+ messages in thread
* [Qemu-devel] [PATCH 1/1] 9pfs: fix and simplify dependencies 2017-08-08 12:17 [Qemu-devel] [PATCH 0/1] fixup 9pfs dependencies Cornelia Huck @ 2017-08-08 12:17 ` Cornelia Huck 2017-08-08 12:50 ` Greg Kurz 0 siblings, 1 reply; 6+ messages in thread From: Cornelia Huck @ 2017-08-08 12:17 UTC (permalink / raw) To: aneesh.kumar, groug; +Cc: borntraeger, agraf, thuth, qemu-devel, Cornelia Huck We want to compile the 9pfs code iff virtfs is enabled and a virtio transport is available (not only pci!). Let's make the generated CONFIG_VIRTFS statement depend on either CONFIG_VIRTIO_PCI or (new) CONFIG_VIRTIO_CCW. The Makefiles for fsdev/ and hw/9pfs/ can then depend on CONFIG_VIRTFS directly. Suggested-by: Thomas Huth <thuth@redhat.com> Signed-off-by: Cornelia Huck <cohuck@redhat.com> --- configure | 2 +- default-configs/s390x-softmmu.mak | 1 + fsdev/Makefile.objs | 2 +- hw/Makefile.objs | 2 +- 4 files changed, 4 insertions(+), 3 deletions(-) diff --git a/configure b/configure index dd73cce62f..709884dff1 100755 --- a/configure +++ b/configure @@ -5771,7 +5771,7 @@ if test "$libattr" = "yes" ; then echo "CONFIG_LIBATTR=y" >> $config_host_mak fi if test "$virtfs" = "yes" ; then - echo "CONFIG_VIRTFS=y" >> $config_host_mak + echo 'CONFIG_VIRTFS=$(call lor, $(CONFIG_VIRTIO_PCI),$(CONFIG_VIRTIO_CCW))' >> $config_host_mak fi if test "$vhost_scsi" = "yes" ; then echo "CONFIG_VHOST_SCSI=y" >> $config_host_mak diff --git a/default-configs/s390x-softmmu.mak b/default-configs/s390x-softmmu.mak index 51191b77df..e4c5236ceb 100644 --- a/default-configs/s390x-softmmu.mak +++ b/default-configs/s390x-softmmu.mak @@ -8,3 +8,4 @@ CONFIG_S390_FLIC=y CONFIG_S390_FLIC_KVM=$(CONFIG_KVM) CONFIG_VFIO_CCW=$(CONFIG_LINUX) CONFIG_WDT_DIAG288=y +CONFIG_VIRTIO_CCW=y diff --git a/fsdev/Makefile.objs b/fsdev/Makefile.objs index 659df6e187..604cc8b6c4 100644 --- a/fsdev/Makefile.objs +++ b/fsdev/Makefile.objs @@ -1,4 +1,4 @@ -ifeq ($(CONFIG_VIRTIO)$(CONFIG_VIRTFS)$(CONFIG_PCI),yyy) +ifeq ($(CONFIG_VIRTFS),y) # Lots of the fsdev/9pcode is pulled in by vl.c via qemu_fsdev_add. # only pull in the actual virtio-9p device if we also enabled virtio. common-obj-y = qemu-fsdev.o 9p-marshal.o 9p-iov-marshal.o diff --git a/hw/Makefile.objs b/hw/Makefile.objs index a2c61f6b09..147d1dc9fb 100644 --- a/hw/Makefile.objs +++ b/hw/Makefile.objs @@ -1,4 +1,4 @@ -devices-dirs-$(call land, $(CONFIG_VIRTIO),$(call land,$(CONFIG_VIRTFS),$(CONFIG_PCI))) += 9pfs/ +devices-dirs-$(CONFIG_VIRTFS) += 9pfs/ devices-dirs-$(CONFIG_SOFTMMU) += acpi/ devices-dirs-$(CONFIG_SOFTMMU) += adc/ devices-dirs-$(CONFIG_SOFTMMU) += audio/ -- 2.13.4 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH 1/1] 9pfs: fix and simplify dependencies 2017-08-08 12:17 ` [Qemu-devel] [PATCH 1/1] 9pfs: fix and simplify dependencies Cornelia Huck @ 2017-08-08 12:50 ` Greg Kurz 2017-08-08 13:00 ` Cornelia Huck 2017-08-08 14:15 ` Greg Kurz 0 siblings, 2 replies; 6+ messages in thread From: Greg Kurz @ 2017-08-08 12:50 UTC (permalink / raw) To: Cornelia Huck; +Cc: aneesh.kumar, borntraeger, agraf, thuth, qemu-devel [-- Attachment #1: Type: text/plain, Size: 2619 bytes --] On Tue, 8 Aug 2017 14:17:01 +0200 Cornelia Huck <cohuck@redhat.com> wrote: > We want to compile the 9pfs code iff virtfs is enabled and a virtio > transport is available (not only pci!). > > Let's make the generated CONFIG_VIRTFS statement depend on either > CONFIG_VIRTIO_PCI or (new) CONFIG_VIRTIO_CCW. The Makefiles for fsdev/ > and hw/9pfs/ can then depend on CONFIG_VIRTFS directly. > > Suggested-by: Thomas Huth <thuth@redhat.com> > Signed-off-by: Cornelia Huck <cohuck@redhat.com> > --- It looks good to me. Acked-by: Greg Kurz <groug@kaod.org> > configure | 2 +- > default-configs/s390x-softmmu.mak | 1 + > fsdev/Makefile.objs | 2 +- > hw/Makefile.objs | 2 +- > 4 files changed, 4 insertions(+), 3 deletions(-) > > diff --git a/configure b/configure > index dd73cce62f..709884dff1 100755 > --- a/configure > +++ b/configure > @@ -5771,7 +5771,7 @@ if test "$libattr" = "yes" ; then > echo "CONFIG_LIBATTR=y" >> $config_host_mak > fi > if test "$virtfs" = "yes" ; then > - echo "CONFIG_VIRTFS=y" >> $config_host_mak > + echo 'CONFIG_VIRTFS=$(call lor, $(CONFIG_VIRTIO_PCI),$(CONFIG_VIRTIO_CCW))' >> $config_host_mak > fi > if test "$vhost_scsi" = "yes" ; then > echo "CONFIG_VHOST_SCSI=y" >> $config_host_mak > diff --git a/default-configs/s390x-softmmu.mak b/default-configs/s390x-softmmu.mak > index 51191b77df..e4c5236ceb 100644 > --- a/default-configs/s390x-softmmu.mak > +++ b/default-configs/s390x-softmmu.mak > @@ -8,3 +8,4 @@ CONFIG_S390_FLIC=y > CONFIG_S390_FLIC_KVM=$(CONFIG_KVM) > CONFIG_VFIO_CCW=$(CONFIG_LINUX) > CONFIG_WDT_DIAG288=y > +CONFIG_VIRTIO_CCW=y > diff --git a/fsdev/Makefile.objs b/fsdev/Makefile.objs > index 659df6e187..604cc8b6c4 100644 > --- a/fsdev/Makefile.objs > +++ b/fsdev/Makefile.objs > @@ -1,4 +1,4 @@ > -ifeq ($(CONFIG_VIRTIO)$(CONFIG_VIRTFS)$(CONFIG_PCI),yyy) > +ifeq ($(CONFIG_VIRTFS),y) > # Lots of the fsdev/9pcode is pulled in by vl.c via qemu_fsdev_add. > # only pull in the actual virtio-9p device if we also enabled virtio. > common-obj-y = qemu-fsdev.o 9p-marshal.o 9p-iov-marshal.o > diff --git a/hw/Makefile.objs b/hw/Makefile.objs > index a2c61f6b09..147d1dc9fb 100644 > --- a/hw/Makefile.objs > +++ b/hw/Makefile.objs > @@ -1,4 +1,4 @@ > -devices-dirs-$(call land, $(CONFIG_VIRTIO),$(call land,$(CONFIG_VIRTFS),$(CONFIG_PCI))) += 9pfs/ > +devices-dirs-$(CONFIG_VIRTFS) += 9pfs/ > devices-dirs-$(CONFIG_SOFTMMU) += acpi/ > devices-dirs-$(CONFIG_SOFTMMU) += adc/ > devices-dirs-$(CONFIG_SOFTMMU) += audio/ [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 181 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH 1/1] 9pfs: fix and simplify dependencies 2017-08-08 12:50 ` Greg Kurz @ 2017-08-08 13:00 ` Cornelia Huck 2017-08-08 14:15 ` Greg Kurz 1 sibling, 0 replies; 6+ messages in thread From: Cornelia Huck @ 2017-08-08 13:00 UTC (permalink / raw) To: Greg Kurz; +Cc: aneesh.kumar, borntraeger, agraf, thuth, qemu-devel On Tue, 8 Aug 2017 14:50:00 +0200 Greg Kurz <groug@kaod.org> wrote: > On Tue, 8 Aug 2017 14:17:01 +0200 > Cornelia Huck <cohuck@redhat.com> wrote: > > > We want to compile the 9pfs code iff virtfs is enabled and a virtio > > transport is available (not only pci!). > > > > Let's make the generated CONFIG_VIRTFS statement depend on either > > CONFIG_VIRTIO_PCI or (new) CONFIG_VIRTIO_CCW. The Makefiles for fsdev/ > > and hw/9pfs/ can then depend on CONFIG_VIRTFS directly. > > > > Suggested-by: Thomas Huth <thuth@redhat.com> > > Signed-off-by: Cornelia Huck <cohuck@redhat.com> > > --- > > It looks good to me. > > Acked-by: Greg Kurz <groug@kaod.org> > > > configure | 2 +- > > default-configs/s390x-softmmu.mak | 1 + > > fsdev/Makefile.objs | 2 +- > > hw/Makefile.objs | 2 +- > > 4 files changed, 4 insertions(+), 3 deletions(-) Thanks! Will you queue this yourself, or should I send it together with the zpci cleanup patches? ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH 1/1] 9pfs: fix and simplify dependencies 2017-08-08 12:50 ` Greg Kurz 2017-08-08 13:00 ` Cornelia Huck @ 2017-08-08 14:15 ` Greg Kurz 2017-08-08 15:21 ` Cornelia Huck 1 sibling, 1 reply; 6+ messages in thread From: Greg Kurz @ 2017-08-08 14:15 UTC (permalink / raw) To: Cornelia Huck; +Cc: qemu-devel, borntraeger, thuth, aneesh.kumar, agraf [-- Attachment #1: Type: text/plain, Size: 3494 bytes --] On Tue, 8 Aug 2017 14:50:00 +0200 Greg Kurz <groug@kaod.org> wrote: > On Tue, 8 Aug 2017 14:17:01 +0200 > Cornelia Huck <cohuck@redhat.com> wrote: > > > We want to compile the 9pfs code iff virtfs is enabled and a virtio > > transport is available (not only pci!). > > > > Let's make the generated CONFIG_VIRTFS statement depend on either > > CONFIG_VIRTIO_PCI or (new) CONFIG_VIRTIO_CCW. The Makefiles for fsdev/ > > and hw/9pfs/ can then depend on CONFIG_VIRTFS directly. > > > > Suggested-by: Thomas Huth <thuth@redhat.com> > > Signed-off-by: Cornelia Huck <cohuck@redhat.com> > > --- > > It looks good to me. > Oops, reading Thomas's other mail, I now realize that with this patch we end up with the following in config-host.mak: CONFIG_VIRTFS=$(call lor, $(CONFIG_VIRTIO_PCI),$(CONFIG_VIRTIO_CCW)) which looks wrong even if it may "work"... but we also get the following in config-host.h: #define CONFIG_VIRTFS $(call lor, $(CONFIG_VIRTIO_PCI),$(CONFIG_VIRTIO_CCW)) which is clearly wrong. Maybe post the changes from your other mail instead ? Message-ID: <20170808111542.16422571@gondolin> The patch isn't that ugly, given the ugliness of the current makefiles :) And we can come up with some cleanup patch later. > Acked-by: Greg Kurz <groug@kaod.org> > NAcked-by: in fact :) > > configure | 2 +- > > default-configs/s390x-softmmu.mak | 1 + > > fsdev/Makefile.objs | 2 +- > > hw/Makefile.objs | 2 +- > > 4 files changed, 4 insertions(+), 3 deletions(-) > > > > diff --git a/configure b/configure > > index dd73cce62f..709884dff1 100755 > > --- a/configure > > +++ b/configure > > @@ -5771,7 +5771,7 @@ if test "$libattr" = "yes" ; then > > echo "CONFIG_LIBATTR=y" >> $config_host_mak > > fi > > if test "$virtfs" = "yes" ; then > > - echo "CONFIG_VIRTFS=y" >> $config_host_mak > > + echo 'CONFIG_VIRTFS=$(call lor, $(CONFIG_VIRTIO_PCI),$(CONFIG_VIRTIO_CCW))' >> $config_host_mak > > fi > > if test "$vhost_scsi" = "yes" ; then > > echo "CONFIG_VHOST_SCSI=y" >> $config_host_mak > > diff --git a/default-configs/s390x-softmmu.mak b/default-configs/s390x-softmmu.mak > > index 51191b77df..e4c5236ceb 100644 > > --- a/default-configs/s390x-softmmu.mak > > +++ b/default-configs/s390x-softmmu.mak > > @@ -8,3 +8,4 @@ CONFIG_S390_FLIC=y > > CONFIG_S390_FLIC_KVM=$(CONFIG_KVM) > > CONFIG_VFIO_CCW=$(CONFIG_LINUX) > > CONFIG_WDT_DIAG288=y > > +CONFIG_VIRTIO_CCW=y > > diff --git a/fsdev/Makefile.objs b/fsdev/Makefile.objs > > index 659df6e187..604cc8b6c4 100644 > > --- a/fsdev/Makefile.objs > > +++ b/fsdev/Makefile.objs > > @@ -1,4 +1,4 @@ > > -ifeq ($(CONFIG_VIRTIO)$(CONFIG_VIRTFS)$(CONFIG_PCI),yyy) > > +ifeq ($(CONFIG_VIRTFS),y) > > # Lots of the fsdev/9pcode is pulled in by vl.c via qemu_fsdev_add. > > # only pull in the actual virtio-9p device if we also enabled virtio. > > common-obj-y = qemu-fsdev.o 9p-marshal.o 9p-iov-marshal.o > > diff --git a/hw/Makefile.objs b/hw/Makefile.objs > > index a2c61f6b09..147d1dc9fb 100644 > > --- a/hw/Makefile.objs > > +++ b/hw/Makefile.objs > > @@ -1,4 +1,4 @@ > > -devices-dirs-$(call land, $(CONFIG_VIRTIO),$(call land,$(CONFIG_VIRTFS),$(CONFIG_PCI))) += 9pfs/ > > +devices-dirs-$(CONFIG_VIRTFS) += 9pfs/ > > devices-dirs-$(CONFIG_SOFTMMU) += acpi/ > > devices-dirs-$(CONFIG_SOFTMMU) += adc/ > > devices-dirs-$(CONFIG_SOFTMMU) += audio/ > [-- Attachment #2: OpenPGP digital signature --] [-- Type: application/pgp-signature, Size: 181 bytes --] ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH 1/1] 9pfs: fix and simplify dependencies 2017-08-08 14:15 ` Greg Kurz @ 2017-08-08 15:21 ` Cornelia Huck 0 siblings, 0 replies; 6+ messages in thread From: Cornelia Huck @ 2017-08-08 15:21 UTC (permalink / raw) To: Greg Kurz; +Cc: qemu-devel, borntraeger, thuth, aneesh.kumar, agraf On Tue, 8 Aug 2017 16:15:59 +0200 Greg Kurz <groug@kaod.org> wrote: > On Tue, 8 Aug 2017 14:50:00 +0200 > Greg Kurz <groug@kaod.org> wrote: > > > On Tue, 8 Aug 2017 14:17:01 +0200 > > Cornelia Huck <cohuck@redhat.com> wrote: > > > > > We want to compile the 9pfs code iff virtfs is enabled and a virtio > > > transport is available (not only pci!). > > > > > > Let's make the generated CONFIG_VIRTFS statement depend on either > > > CONFIG_VIRTIO_PCI or (new) CONFIG_VIRTIO_CCW. The Makefiles for fsdev/ > > > and hw/9pfs/ can then depend on CONFIG_VIRTFS directly. > > > > > > Suggested-by: Thomas Huth <thuth@redhat.com> > > > Signed-off-by: Cornelia Huck <cohuck@redhat.com> > > > --- > > > > It looks good to me. > > > > Oops, reading Thomas's other mail, I now realize that with this patch we > end up with the following in config-host.mak: > > CONFIG_VIRTFS=$(call lor, $(CONFIG_VIRTIO_PCI),$(CONFIG_VIRTIO_CCW)) > > which looks wrong even if it may "work"... but we also get the > following in config-host.h: > > #define CONFIG_VIRTFS $(call lor, $(CONFIG_VIRTIO_PCI),$(CONFIG_VIRTIO_CCW)) > > which is clearly wrong. Eek. Oh well, it would have been nice... > > Maybe post the changes from your other mail instead ? > > Message-ID: <20170808111542.16422571@gondolin> > > The patch isn't that ugly, given the ugliness of the current makefiles :) It is slightly differently ugly :) > > And we can come up with some cleanup patch later. Nod. > > > Acked-by: Greg Kurz <groug@kaod.org> > > > > NAcked-by: in fact :) Heh :) I'll post the other patch properly. ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-08-08 15:22 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-08-08 12:17 [Qemu-devel] [PATCH 0/1] fixup 9pfs dependencies Cornelia Huck 2017-08-08 12:17 ` [Qemu-devel] [PATCH 1/1] 9pfs: fix and simplify dependencies Cornelia Huck 2017-08-08 12:50 ` Greg Kurz 2017-08-08 13:00 ` Cornelia Huck 2017-08-08 14:15 ` Greg Kurz 2017-08-08 15:21 ` Cornelia Huck
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).