> Il giorno 23 mar 2017, alle ore 12:20, Wei Liu <wei.l...@citrix.com> ha 
> scritto:
> 
> On Thu, Mar 23, 2017 at 11:07:04AM +0000, Wei Liu wrote:
>> On Thu, Mar 23, 2017 at 02:31:37AM +0100, Luca Miccio wrote:
>>> Execute the clean target for both arm and x86 architecture.
>>> 
>>> When trying to build Xen for a different architecture in the same
>>> tree, the command make clean will only remove temporary files for
>>> the host architecture.
>>> This will lead a compilation error when trying to build ARM64 and
>>> ARM32 Xen in the same tree.
>>> (See also: 
>>> https://lists.xenproject.org/archives/html/xen-devel/2016-11/msg02176.html)
>>> 
>>> Signed-off-by: Luca Miccio <lucmic...@gmail.com>
>>> ---
>>> xen/Makefile | 3 ++-
>>> 1 file changed, 2 insertions(+), 1 deletion(-)
>>> 
>>> diff --git a/xen/Makefile b/xen/Makefile
>>> index dc6862e04d..fcd5c7e9d6 100644
>>> --- a/xen/Makefile
>>> +++ b/xen/Makefile
>>> @@ -115,7 +115,8 @@ _clean: delete-unfresh-files
>>>     $(MAKE) -f $(BASEDIR)/Rules.mk -C drivers clean
>>>     $(MAKE) -f $(BASEDIR)/Rules.mk -C xsm clean
>>>     $(MAKE) -f $(BASEDIR)/Rules.mk -C crypto clean
>>> -   $(MAKE) -f $(BASEDIR)/Rules.mk -C arch/$(TARGET_ARCH) clean
>>> +   $(MAKE) -f $(BASEDIR)/Rules.mk -C arch/arm clean
>>> +   $(MAKE) -f $(BASEDIR)/Rules.mk -C arch/x86 clean
>> 
>> A more future-proof way of doing this is to use find to collect a list
>> of directory under arch/ and iterate.
>> 
>> But I don't see a new architecture support coming any time soon, so
>> this is just a suggestion.
>> 

I had the same doubt, but, as you said there is no new architecture support 
coming soon.
However i could add this option.
> 
> Also, you seem to have missed overriding TARGET_ARCH and TARGET_SUBARCH.
> I believe there are references to them in arch-specific Makefiles.
> 
> Wei.
I don’t get your point/suggestion. When we use “make clean”, with my patch 
applied, we 
simply execute the clean target for every architecture supported for now. 
I took a look at the clean target used both from the arm and the x86 Makefile 
and i didn’t find
any references to TARGET_ARCH and TARGET_SUBARCH.

Also, with the current command:
$(MAKE) -f $(BASEDIR)/Rules.mk -C arch/$(TARGET_ARCH) clean
The target clean is called only for the $TARGET_ARCH architecture, which, if 
it’s not set ( "make clean | distclean" at the xen root directory),
is the host one.

Maybe i’m missing something.

Luca.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Reply via email to