Re: [PATCH v10 08/12] xen: add /buildinfo/config entry to hypervisor filesystem

[Jürgen Groß <jgross@xxxxxxxx>]

On 02.06.20 11:03, Andrew Cooper wrote:
> On 19/05/2020 08:21, Juergen Gross wrote:
> > diff --git a/xen/common/Makefile b/xen/common/Makefile
> > index bf7d0e25a3..3d61239fbf 100644
> > --- a/xen/common/Makefile
> > +++ b/xen/common/Makefile
> > @@ -1,6 +1,7 @@
> >   obj-$(CONFIG_ARGO) += argo.o
> >   obj-y += bitmap.o
> >   obj-y += bsearch.o
> > +obj-$(CONFIG_HYPFS_CONFIG) += config_data.o
> >   obj-$(CONFIG_CORE_PARKING) += core_parking.o
> >   obj-y += cpu.o
> >   obj-$(CONFIG_DEBUG_TRACE) += debugtrace.o
> > @@ -73,3 +74,14 @@ obj-$(CONFIG_UBSAN) += ubsan/
> >   
> >   obj-$(CONFIG_NEEDS_LIBELF) += libelf/
> >   obj-$(CONFIG_HAS_DEVICE_TREE) += libfdt/
> > +
> > +config.gz: ../.config
> > +       gzip -c $< >$@
> I had to disable HYPFS in the XenServer build.  Inside RPM, this fails with
Disabling CONFIG_HYPFS_CONFIG should work, too.

> make[3]: *** No rule to make target `../.config', needed by
> `config.gz'.  Stop.
> make[3]: *** Waiting for unfinished jobs....
>
> This needs to be an expression involving $(KCONFIG_CONFIG) because
> .config is only the default, and it surely needs a $(XEN_ROOT) in there?
Will send a patch.

Oh, in fact I'll send two patches, as I guess pv-shim doesn't want to
have CONFIG_HYPFS set.


Juergen