diff mbox

arm64: add boot image dependencies to not generate invalid images

Message ID 1466473440-18887-1-git-send-email-yamada.masahiro@socionext.com (mailing list archive)
State New, archived
Headers show

Commit Message

Masahiro Yamada June 21, 2016, 1:44 a.m. UTC
I fixed boot image dependencies for arch/arm in commit 3939f3345050
("ARM: 8418/1: add boot image dependencies to not generate invalid
images").

I see a similar problem for arch/arm64; "make -jN Image Image.gz"
would sometimes end up generating bad images where N > 1.

Fix the dependency in arch/arm64/Makefile to avoid the race
between "make Image" and "make Image.*".

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
---

 arch/arm64/Makefile | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Catalin Marinas June 21, 2016, 10:32 a.m. UTC | #1
On Tue, Jun 21, 2016 at 10:44:00AM +0900, Masahiro Yamada wrote:
> I fixed boot image dependencies for arch/arm in commit 3939f3345050
> ("ARM: 8418/1: add boot image dependencies to not generate invalid
> images").
> 
> I see a similar problem for arch/arm64; "make -jN Image Image.gz"
> would sometimes end up generating bad images where N > 1.
> 
> Fix the dependency in arch/arm64/Makefile to avoid the race
> between "make Image" and "make Image.*".
> 
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>

Acked-by: Catalin Marinas <catalin.marinas@arm.com>

Will, could you please pick this up for 4.7? Maybe with a cc stable.
Thanks.
Will Deacon June 21, 2016, 10:34 a.m. UTC | #2
On Tue, Jun 21, 2016 at 11:32:17AM +0100, Catalin Marinas wrote:
> On Tue, Jun 21, 2016 at 10:44:00AM +0900, Masahiro Yamada wrote:
> > I fixed boot image dependencies for arch/arm in commit 3939f3345050
> > ("ARM: 8418/1: add boot image dependencies to not generate invalid
> > images").
> > 
> > I see a similar problem for arch/arm64; "make -jN Image Image.gz"
> > would sometimes end up generating bad images where N > 1.
> > 
> > Fix the dependency in arch/arm64/Makefile to avoid the race
> > between "make Image" and "make Image.*".
> > 
> > Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> 
> Acked-by: Catalin Marinas <catalin.marinas@arm.com>
> 
> Will, could you please pick this up for 4.7? Maybe with a cc stable.

Yup, will do.

Will
Masahiro Yamada June 21, 2016, 10:36 a.m. UTC | #3
2016-06-21 19:32 GMT+09:00 Catalin Marinas <catalin.marinas@arm.com>:
> On Tue, Jun 21, 2016 at 10:44:00AM +0900, Masahiro Yamada wrote:
>> I fixed boot image dependencies for arch/arm in commit 3939f3345050
>> ("ARM: 8418/1: add boot image dependencies to not generate invalid
>> images").
>>
>> I see a similar problem for arch/arm64; "make -jN Image Image.gz"
>> would sometimes end up generating bad images where N > 1.
>>
>> Fix the dependency in arch/arm64/Makefile to avoid the race
>> between "make Image" and "make Image.*".
>>
>> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
>
> Acked-by: Catalin Marinas <catalin.marinas@arm.com>
>
> Will, could you please pick this up for 4.7? Maybe with a cc stable.
> Thanks.


If possible, could you change the subject?


arm64: add boot image dependencies to not generate invalid images

 --->

arm64: fix boot image dependencies to not generate invalid images
diff mbox

Patch

diff --git a/arch/arm64/Makefile b/arch/arm64/Makefile
index 7085e32..648a32c 100644
--- a/arch/arm64/Makefile
+++ b/arch/arm64/Makefile
@@ -95,7 +95,7 @@  boot := arch/arm64/boot
 Image: vmlinux
 	$(Q)$(MAKE) $(build)=$(boot) $(boot)/$@
 
-Image.%: vmlinux
+Image.%: Image
 	$(Q)$(MAKE) $(build)=$(boot) $(boot)/$@
 
 zinstall install: