[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] shim: don't let build modify shim.config



On Thu, Apr 19, 2018 at 05:22:28PM +0100, Wei Liu wrote:
> On Thu, Apr 19, 2018 at 05:14:38PM +0100, Andrew Cooper wrote:
> > On 19/04/18 15:54, Wei Liu wrote:
> > > On Thu, Apr 19, 2018 at 01:01:53PM +0200, Juergen Gross wrote:
> > >> On 19/04/18 12:47, Jan Beulich wrote:
> > >>>>>> On 19.04.18 at 12:30, <jgross@xxxxxxxx> wrote:
> > >>>> --- a/tools/firmware/xen-dir/Makefile
> > >>>> +++ b/tools/firmware/xen-dir/Makefile
> > >>>> @@ -41,21 +41,22 @@ linkfarm.stamp: $(DEP_DIRS) $(DEP_FILES) FORCE
> > >>>>  $(D): linkfarm.stamp
> > >>>>        $(MAKE) -C $(D)/xen distclean
> > >>>>  
> > >>>> -.PHONY: shim-%config
> > >>>> -shim-%config: $(D) FORCE
> > >>>> -      $(MAKE) -C $(D)/xen $*config \
> > >>> I think there was a reason for the % and $*.
> > >> The only one I could think of is the possibility to do
> > >>
> > >> make shim-defaultconfig
> > >>
> > >> or similar calls. Question is who will ever need that. In case it is
> > >> believed to be valuable I can keep it.
> > > Not likely I think.
> > 
> > Using `make shim-menuconfig` was a specific usecase for my original
> > version, but with this difference in representation, the usecase has
> > mostly gone away.
> > 
> > With this change in place, what is the expected workflow for making an
> > update to the configuration?
> > 
> 
> Just commit the must-have's to the tracked file and let Xen's build
> system handle the rest?

+1. As a bare minimum:

CONFIG_XEN_GUEST=y
CONFIG_PVH_GUEST=y
CONFIG_PV_SHIM=y
CONFIG_PV_SHIM_EXCLUSIVE=y

But I expect some more might be needed.

Having a full config will always be a PITA because it will get out of
synch.

Roger.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.