Re: [XEN PATCH v7 00/51] xen: Build system improvements, now with out-of-tree build!

[Anthony PERARD <anthony.perard@citrix.com>]

I hope this is useful:

On Tue, Aug 24, 2021 at 11:49:47AM +0100, Anthony PERARD wrote:
> Anthony PERARD (51):
>   build: introduce cpp_flags macro
>   build: use if_changed on built_in.o
>   build: use if_changed_rules with %.o:%.c targets

all 3 ready to commit

>   build: factorise generation of the linker scripts

may need a ack from arm maintainers, does it? Otherwise, ready.

>   x86/mm: avoid building multiple .o from a single .c file
>   build,include: rework compat-build-source.py
>   build,include: rework compat-build-header.py
>   build: fix clean targets when subdir-y is used
>   build: use subdir-y in test/Makefile

all 5 ready to commit

>   build,arm: move LDFLAGS change to arch.mk
need edit commit description, but otherwise ready
    not needed before [PATCH 21/51] build: set ALL_OBJS to main Makefile; move prelink.o to main Makefile

>   build: move make option changes check earlier
>   build: avoid building arm/arm/*/head.o twice
both ready

So up to here (12/51) all patch seems to be ready to commit.

>   build: convert binfile use to if_changed
>   xen: move include/asm-* to arch/*/include/asm
both need a ack

>   build,riscv: tell the build system about riscv64/head.S
patch ready
but depends on "[PATCH 14/51] xen: move include/asm-* to arch/*/include/asm"
    for the rune written in the commit message to work.


>   build: generate "include/xen/compile.h" with if_changed
>   build: set XEN_BUILD_EFI earlier
>   build: fix $(TARGET).efi creation in arch/arm
patches needs review

>   build: fix arch/x86/note.o rule
ready, standalone


The rest of the series needs review.

Cheers,

-- 
Anthony PERARD