RE: [PATCH v2] staging: mkspec: added aarch64 ifarch case.

From	James Tau <jtau@mellanox.com>
Newsgroups	linux.kernel
Subject	RE: [PATCH v2] staging: mkspec: added aarch64 ifarch case.
Date	2017-03-03 07:30 +0100
Message-ID	<tgOT0-5tc-3@gated-at.bofh.it> (permalink)
References	<tgqnD-5cO-3@gated-at.bofh.it> <tgqnD-5cO-1@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi Will,

This patch (http://lkml.kernel.org/r/20161122213434.14788-1-mmarek@suse.com) looks better.  It has what Linus calls "good taste". ;-)  I didn't see it in mmarek's kbuild branches (for-next,rc-fixes), however.  Still making its way there?

But it doesn't quite fix the native 'make rpm' build completely.  While it gets beyond the point at which 'make rpm' fails without my patch, it exposes another issue for which I am debugging right now:

  ld -EL -r  -T ./scripts/module-common.lds --build-id  -o net/unix/unix.ko net/unix/unix.o net/unix/unix.mod.o ;  true
make -f ./scripts/Makefile.fwinst obj=firmware __fw_modbuild
error: Bad exit status from /var/tmp/rpm-tmp.YcfiLf (%build)
    Bad exit status from /var/tmp/rpm-tmp.YcfiLf (%build)

RPM build errors:
make[1]: *** [rpm] Error 1
make: *** [rpm] Error 2

If I succeed in root-causing the problem, I'll submit a patch for that (if another doesn't beat me to it).  And assuming that patch is accepted for having Linusian "good taste", then it, and http://lkml.kernel.org/r/20161122213434.14788-1-mmarek@suse.com, will make my current submitted patch extraneous.

Thanks,
James

-----Original Message-----
From: Will Deacon [mailto:will.deacon@arm.com] 
Sent: Wednesday, March 1, 2017 11:06 PM
To: James Tau <jtau@mellanox.com>
Cc: linux-kernel@vger.kernel.org; linux-kbuild@vger.kernel.org; mmarek@suse.com; catalin.marinas@arm.com; Chris Metcalf <cmetcalf@mellanox.com>
Subject: Re: [PATCH v2] staging: mkspec: added aarch64 ifarch case.

On Wed, Mar 01, 2017 at 09:24:14AM -0800, James Tau wrote:
> Patch attempting to fix native 'make rpm' build on ARM64 machines by 
> adding an "ifarch aarch64" case.  Without it, build fails because the 
> 'cp ...' in the default case can't find the built image.
> 
> Signed-off-by: James Tau <jtau@mellanox.com>
> ---
>  scripts/package/mkspec | 4 ++++
>  1 file changed, 4 insertions(+)

Is this the same issue that was fixed by:

http://lkml.kernel.org/r/20161122213434.14788-1-mmarek@suse.com

?

I was assuming that Michael was going to queue those, but I could be wrong.

Will

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH v2] staging: mkspec: added aarch64 ifarch case. Will Deacon <will.deacon@arm.com> - 2017-03-02 05:20 +0100
  RE: [PATCH v2] staging: mkspec: added aarch64 ifarch case. James Tau <jtau@mellanox.com> - 2017-03-03 07:30 +0100

csiph-web