Re: [Xen-devel] [XEN PATCH v3 07/23] xen/build: Use obj-y += subdir/ instead of subdir-y

From: Jan Beulich <jbeulich@suse.com>
To: Julien Grall <julien@xen.org>,
	Stefano Stabellini <sstabellini@kernel.org>,
	Kevin Tian <kevin.tian@intel.com>,
	Daniel De Graaf <dgdegra@tycho.nsa.gov>
Cc: "Wei Liu" <wl@xen.org>,
	"Konrad Rzeszutek Wilk" <konrad.wilk@oracle.com>,
	"George Dunlap" <George.Dunlap@eu.citrix.com>,
	"Andrew Cooper" <andrew.cooper3@citrix.com>,
	"Ian Jackson" <ian.jackson@eu.citrix.com>,
	"Anthony PERARD" <anthony.perard@citrix.com>,
	xen-devel@lists.xenproject.org,
	"Volodymyr Babchuk" <Volodymyr_Babchuk@epam.com>,
	"Roger Pau Monné" <roger.pau@citrix.com>
Subject: Re: [Xen-devel] [XEN PATCH v3 07/23] xen/build: Use obj-y += subdir/ instead of subdir-y
Date: Thu, 5 Mar 2020 10:24:28 +0100	[thread overview]
Message-ID: <155c8c7d-df12-5177-45ab-1dd2a6bc9b45@suse.com> (raw)
In-Reply-To: <20200226113355.2532224-8-anthony.perard@citrix.com>

On 26.02.2020 12:33, Anthony PERARD wrote:
> This is part of upgrading our build system and import more of Linux's
> one.
> 
> In Linux, subdir-y in Makefiles is only used to descend into
> subdirectory when there are no object to build, Xen doesn't have that
> and all subdir have object to be included in the final binary.
> 
> To allow the new syntax, the "obj-y" and "subdir-*" calculation in
> Rules.mk is changed and partially imported from Linux's Kbuild.
> 
> The command used to modify the Makefile was:
>     sed -i -r 's#^subdir-(.*)#obj-\1/#;' **/Makefile
> 
> Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
> ---
> 
> Notes:
>     v3:
>     - no more tabs
>     - reshuffle variable, and remove __subdir-y
> 
>  xen/Rules.mk                         | 19 ++++++++-----------
>  xen/arch/arm/Makefile                | 14 +++++++-------
>  xen/arch/arm/arm32/Makefile          |  2 +-
>  xen/arch/arm/arm64/Makefile          |  2 +-

Julien, Stefano - any chance of getting an ack here?

>  xen/arch/x86/Makefile                | 18 +++++++++---------
>  xen/arch/x86/acpi/Makefile           |  2 +-
>  xen/arch/x86/cpu/Makefile            |  4 ++--
>  xen/arch/x86/guest/Makefile          |  4 ++--
>  xen/arch/x86/hvm/Makefile            |  6 +++---
>  xen/arch/x86/mm/Makefile             |  4 ++--
>  xen/arch/x86/x86_64/Makefile         |  2 +-
>  xen/common/Makefile                  | 10 +++++-----
>  xen/drivers/Makefile                 | 14 +++++++-------
>  xen/drivers/acpi/Makefile            |  6 +++---
>  xen/drivers/passthrough/Makefile     |  8 ++++----
>  xen/drivers/passthrough/vtd/Makefile |  2 +-

Kevin, how about this one?

>  xen/lib/Makefile                     |  2 +-
>  xen/xsm/Makefile                     |  2 +-
>  xen/xsm/flask/Makefile               |  2 +-

Daniel, how about these?

I guess the latter two are small enough to skip further waiting for
acks once the Arm one would be in place. Getting this patch in
would unblock a fair part of the remainder of this series.

Jan

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