From: Ard Biesheuvel Subject: Re: [PATCH v2 0/3] ARM: NEON based fast(er) AES in CBC/CTR/XTS modes Date: Fri, 4 Oct 2013 20:41:35 +0200 Message-ID: References: <1380837566-18242-1-git-send-email-ard.biesheuvel@linaro.org> <20131004174853.GY24303@mudshark.cambridge.arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Cc: Will Deacon , "linux-arm-kernel@lists.infradead.org" , "linux-crypto@vger.kernel.org" , "patches@linaro.org" , "linux@arm.linux.org.uk" To: Nicolas Pitre Return-path: Received: from mail-lb0-f182.google.com ([209.85.217.182]:57281 "EHLO mail-lb0-f182.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750983Ab3JDSlg (ORCPT ); Fri, 4 Oct 2013 14:41:36 -0400 Received: by mail-lb0-f182.google.com with SMTP id c11so3682792lbj.27 for ; Fri, 04 Oct 2013 11:41:35 -0700 (PDT) In-Reply-To: Sender: linux-crypto-owner@vger.kernel.org List-ID: On 4 October 2013 20:34, Nicolas Pitre wrote: > On Fri, 4 Oct 2013, Will Deacon wrote: [...] >> >> Why do you consider it unsuitable to ship the perl script with the kernel? >> Perl 5 is already documented as a build dependency in Documentation/Changes > > Do you have an example of something that does require perl to build the > kernel on ARM? I was under the impression that people try to avoid it > as much as possible in general. > > I'm personally sitting on the fence between effectively adding a new > kernel build dependencies or carrying the output of the perl script. > But if the kernel build does already require perl in practice then this > might tip the balance. > I like Russell's suggestion the most, in fact. In this case, the build time requirement for Perl effectively gets suspended until you start making modifications to the perl script, and the relation between the .S and the .pl files is made explicit by the make rule. Should I put the cmd_perl rule in scripts/Makefile.build ? Or can I just keep it under arch/arm/crypto ? -- Ard.