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

[Juergen Gross <jgross@xxxxxxxx>]

On 20/04/18 11:33, Andrew Cooper wrote:
> On 20/04/18 10:28, Roger Pau Monné wrote:
>> 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.
> 
> CONFIG_SHADOW_PAGING=n
> CONFIG_NR_CPUS=32
> CONFIG_VIDEO=n
> CONFIG_HVM_FEP=n
> CONFIG_TBOOT=n
> CONFIG_KEXEC=n
> CONFIG_TMEM=n
> CONFIG_XENOPROF=n
> 
> Remember that shim also wants to be as much of a slimline build as possible

Attached is the minimal shim.config to be fed into "make olddefconfig"
resulting in the current config used for the shim.


Juergen

Attachment: shim.config
Description: application/config

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