From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751571AbdCCGYO (ORCPT ); Fri, 3 Mar 2017 01:24:14 -0500 Received: from mail-ve1eur01on0086.outbound.protection.outlook.com ([104.47.1.86]:53242 "EHLO EUR01-VE1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751333AbdCCGYJ (ORCPT ); Fri, 3 Mar 2017 01:24:09 -0500 From: James Tau To: Will Deacon CC: "linux-kernel@vger.kernel.org" , "linux-kbuild@vger.kernel.org" , "mmarek@suse.com" , "catalin.marinas@arm.com" , Chris Metcalf Subject: RE: [PATCH v2] staging: mkspec: added aarch64 ifarch case. Thread-Topic: [PATCH v2] staging: mkspec: added aarch64 ifarch case. Thread-Index: AQHSkrC4QJ5QvFH/SEGFRpbz4LzE3KGA7yAAgADNHfA= Date: Fri, 3 Mar 2017 04:44:01 +0000 Message-ID: References: <20170301172414.GA29242@mtbu-fae-softiron1.internal.tilera.com> <20170302040539.GB15590@arm.com> In-Reply-To: <20170302040539.GB15590@arm.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: arm.com; dkim=none (message not signed) header.d=none;arm.com; dmarc=none action=none header.from=mellanox.com; x-originating-ip: [12.216.194.146] x-ms-office365-filtering-correlation-id: 83bf90e0-af52-4464-448b-08d461efea01 x-ms-office365-filtering-ht: Tenant x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001)(48565401081);SRVR:HE1PR0501MB2763; x-microsoft-exchange-diagnostics: 1;HE1PR0501MB2763;7:r96b6VaatskCjxBEPJGJ5MvIBnVYRNIWijrYb1+EgO26eVemI0nrzaezMWlcM5wR/BuomE4vhcYPAtv7edsWfy8xfCrdJHDoMV48hkmg32422ADfR5YJZhN1yf4LZf+qXh8/hhDQK+BbOh6I+o1DkmmzH1IvRCE7jx03OB71YHlGmvIepBvq3UjMoOvs5k9SylnGBOd6WaBM74cowcvi7VZoy9j6h1AyjP450XMhiAqgvM4Tgm638t8ozwb5pAGlPSNW7CXEp8y0I7uX0Bmci5mU1KeGmKQonesdfQrek/fdzR27KU+LyJzuVmh1obuGriivK+h8+G/xO2JGX5im6w== x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(180628864354917)(9452136761055)(42068640409301); x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(6040375)(601004)(2401047)(8121501046)(5005006)(3002001)(10201501046)(6055026)(6041248)(20161123562025)(20161123564025)(20161123560025)(20161123555025)(20161123558025)(6072148);SRVR:HE1PR0501MB2763;BCL:0;PCL:0;RULEID:;SRVR:HE1PR0501MB2763; x-forefront-prvs: 0235CBE7D0 x-forefront-antispam-report: SFV:NSPM;SFS:(10009020)(979002)(6009001)(7916002)(39850400002)(39860400002)(39450400003)(39840400002)(39410400002)(13464003)(377454003)(24454002)(3280700002)(6116002)(102836003)(38730400002)(53936002)(7696004)(2906002)(966004)(110136004)(107886003)(7736002)(6436002)(50986999)(3846002)(6246003)(305945005)(229853002)(53546006)(5660300001)(3660700001)(54356999)(6916009)(76176999)(2950100002)(8936002)(106116001)(6306002)(66066001)(8676002)(54906002)(2900100001)(4326008)(9686003)(92566002)(81166006)(99286003)(55016002)(86362001)(6506006)(189998001)(33656002)(5250100002)(74316002)(969003)(989001)(999001)(1009001)(1019001);DIR:OUT;SFP:1101;SCL:1;SRVR:HE1PR0501MB2763;H:AM3PR05MB371.eurprd05.prod.outlook.com;FPR:;SPF:None;MLV:ovrnspm;PTR:InfoNoRecords;LANG:en; spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 X-OriginatorOrg: Mellanox.com X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Mar 2017 04:44:01.5080 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: a652971c-7d2e-4d9b-a6a4-d149256f461b X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0501MB2763 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v236OKUl016099 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 Cc: linux-kernel@vger.kernel.org; linux-kbuild@vger.kernel.org; mmarek@suse.com; catalin.marinas@arm.com; Chris Metcalf 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 > --- > 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