From mboxrd@z Thu Jan 1 00:00:00 1970 From: Alex =?utf-8?Q?Benn=C3=A9e?= Subject: Re: [PATCH 03/27] arm64: efi: Add missing Kconfig dependency on KERNEL_MODE_NEON Date: Fri, 18 Aug 2017 13:02:36 +0100 Message-ID: <87o9rdhyn7.fsf@linaro.org> References: <1502280338-23002-1-git-send-email-Dave.Martin@arm.com> <1502280338-23002-4-git-send-email-Dave.Martin@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-reply-to: <1502280338-23002-4-git-send-email-Dave.Martin@arm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: kvmarm-bounces@lists.cs.columbia.edu Sender: kvmarm-bounces@lists.cs.columbia.edu To: Dave Martin Cc: linux-arch@vger.kernel.org, libc-alpha@sourceware.org, Ard Biesheuvel , Szabolcs Nagy , Catalin Marinas , Will Deacon , Richard Sandiford , kvmarm@lists.cs.columbia.edu, linux-arm-kernel@lists.infradead.org List-Id: linux-arch.vger.kernel.org CkRhdmUgTWFydGluIDxEYXZlLk1hcnRpbkBhcm0uY29tPiB3cml0ZXM6Cgo+IFRoZSBFRkkgcnVu dGltZSBzZXJ2aWNlcyBBQkkgcGVybWl0cyBjYWxscyB0byBFRkkgdG8gY2xvYmJlcgo+IGNlcnRh aW4gRlBTSU1EL05FT04gcmVnaXN0ZXJzLCBhcyBwZXIgdGhlIEFBcmNoNjQgcHJvY2VkdXJlIGNh bGwKPiBzdGFuZGFyZC4KPgo+IFNhdmluZy9yZXN0b3JpbmcgdGhlIGNsb2JiZXJlZCByZWdpc3Rl cnMgYXJvdW5kIHN1Y2ggY2FsbHMgbmVlZHMKPiBLRVJORUxfTU9ERV9ORU9OLCBidXQgdGhlIGRl cGVuZGVuY3kgaXMgbWlzc2luZyBmcm9tIEtjb25maWcuCj4KPiBUaGlzIHBhdGNoIGFkZHMgdGhl IG1pc3NpbmcgZGVwZW5kZW5jeS4KPgo+IFRoaXMgd2lsbCBhaWQgYmlzZWN0aW9uIG9mIHRoZSBw YXRjaGVzIGltcGxlbWVudGluZyBzdXBwb3J0IGZvciB0aGUKPiBBUk0gU2NhbGFibGUgVmVjdG9y IEV4dGVuc2lvbiAoU1ZFKS4KPgo+IFNpZ25lZC1vZmYtYnk6IERhdmUgTWFydGluIDxEYXZlLk1h cnRpbkBhcm0uY29tPgoKUmV2aWV3ZWQtYnk6IEFsZXggQmVubsOpZSA8YWxleC5iZW5uZWVAbGlu YXJvLm9yZz4KCj4gLS0tCj4gIGFyY2gvYXJtNjQvS2NvbmZpZyB8IDEgKwo+ICAxIGZpbGUgY2hh bmdlZCwgMSBpbnNlcnRpb24oKykKPgo+IGRpZmYgLS1naXQgYS9hcmNoL2FybTY0L0tjb25maWcg Yi9hcmNoL2FybTY0L0tjb25maWcKPiBpbmRleCBkZmQ5MDg2Li5mZGMzMDJlIDEwMDY0NAo+IC0t LSBhL2FyY2gvYXJtNjQvS2NvbmZpZwo+ICsrKyBiL2FyY2gvYXJtNjQvS2NvbmZpZwo+IEBAIC0x MDUwLDYgKzEwNTAsNyBAQCBjb25maWcgRUZJX1NUVUIKPiAgY29uZmlnIEVGSQo+ICAJYm9vbCAi VUVGSSBydW50aW1lIHN1cHBvcnQiCj4gIAlkZXBlbmRzIG9uIE9GICYmICFDUFVfQklHX0VORElB Tgo+ICsJZGVwZW5kcyBvbiBLRVJORUxfTU9ERV9ORU9OCj4gIAlzZWxlY3QgTElCRkRUCj4gIAlz ZWxlY3QgVUNTMl9TVFJJTkcKPiAgCXNlbGVjdCBFRklfUEFSQU1TX0ZST01fRkRUCgoKLS0KQWxl eCBCZW5uw6llCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f Cmt2bWFybSBtYWlsaW5nIGxpc3QKa3ZtYXJtQGxpc3RzLmNzLmNvbHVtYmlhLmVkdQpodHRwczov L2xpc3RzLmNzLmNvbHVtYmlhLmVkdS9tYWlsbWFuL2xpc3RpbmZvL2t2bWFybQo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f178.google.com ([209.85.128.178]:36748 "EHLO mail-wr0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751489AbdHRMCj (ORCPT ); Fri, 18 Aug 2017 08:02:39 -0400 Received: by mail-wr0-f178.google.com with SMTP id f8so25011991wrf.3 for ; Fri, 18 Aug 2017 05:02:38 -0700 (PDT) References: <1502280338-23002-1-git-send-email-Dave.Martin@arm.com> <1502280338-23002-4-git-send-email-Dave.Martin@arm.com> From: Alex =?utf-8?Q?Benn=C3=A9e?= Subject: Re: [PATCH 03/27] arm64: efi: Add missing Kconfig dependency on KERNEL_MODE_NEON In-reply-to: <1502280338-23002-4-git-send-email-Dave.Martin@arm.com> Date: Fri, 18 Aug 2017 13:02:36 +0100 Message-ID: <87o9rdhyn7.fsf@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Sender: linux-arch-owner@vger.kernel.org List-ID: To: Dave Martin Cc: linux-arm-kernel@lists.infradead.org, linux-arch@vger.kernel.org, libc-alpha@sourceware.org, Ard Biesheuvel , Szabolcs Nagy , Catalin Marinas , Will Deacon , Richard Sandiford , kvmarm@lists.cs.columbia.edu Message-ID: <20170818120236.KN-tJLj03dP_bEhF489mut2sMU_LHqRhWs_D8E4jJAU@z> Dave Martin writes: > The EFI runtime services ABI permits calls to EFI to clobber > certain FPSIMD/NEON registers, as per the AArch64 procedure call > standard. > > Saving/restoring the clobbered registers around such calls needs > KERNEL_MODE_NEON, but the dependency is missing from Kconfig. > > This patch adds the missing dependency. > > This will aid bisection of the patches implementing support for the > ARM Scalable Vector Extension (SVE). > > Signed-off-by: Dave Martin Reviewed-by: Alex Bennée > --- > arch/arm64/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig > index dfd9086..fdc302e 100644 > --- a/arch/arm64/Kconfig > +++ b/arch/arm64/Kconfig > @@ -1050,6 +1050,7 @@ config EFI_STUB > config EFI > bool "UEFI runtime support" > depends on OF && !CPU_BIG_ENDIAN > + depends on KERNEL_MODE_NEON > select LIBFDT > select UCS2_STRING > select EFI_PARAMS_FROM_FDT -- Alex Bennée