From: Herbert Xu <herbert@gondor.apana.org.au>
To: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: linux-crypto@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v3 00/10] crypto - AES for ARM/arm64 updates for v4.11 (round #2)
Date: Fri, 3 Feb 2017 18:22:12 +0800 [thread overview]
Message-ID: <20170203102212.GG2632@gondor.apana.org.au> (raw)
In-Reply-To: <1485645939-17126-1-git-send-email-ard.biesheuvel@linaro.org>
On Sat, Jan 28, 2017 at 11:25:29PM +0000, Ard Biesheuvel wrote:
> Patch #1 is a fix for the CBC chaining issue that was discussed on the
> mailing list. The driver itself is queued for v4.11, so this fix can go
> right on top.
>
> Patches #2 - #6 clear the cra_alignmasks of various drivers: all NEON
> capable CPUs can perform unaligned accesses, and the advantage of using
> the slightly faster aligned accessors (which only exist on ARM not arm64)
> is certainly outweighed by the cost of copying data to suitably aligned
> buffers.
>
> NOTE: patch #5 won't apply unless 'crypto: arm64/aes-blk - honour iv_out
> requirement in CBC and CTR modes' is applied first, which was sent out
> separately as a bugfix for v3.16 - v4.9. If this is a problem, this patch
> can wait.
>
> Patch #7 and #8 are minor tweaks to the new scalar AES code.
>
> Patch #9 improves the performance of the plain NEON AES code, to make it
> more suitable as a fallback for the new bitsliced NEON code, which can
> only operate on 8 blocks in parallel, and needs another driver to perform
> CBC encryption or XTS tweak generation.
>
> Patch #10 updates the new bitsliced AES NEON code to switch to the plain
> NEON driver as a fallback.
>
> Patches #9 and #10 improve the performance of CBC encryption by ~35% on
> low end cores such as the Cortex-A53 that can be found in the Raspberry Pi3
>
> Changes since v2:
> - use polynomial multiply NEON instruction for multiplication by x^2, this
> eliminates 4 instructions from the decrypt path (#9)
>
> Changes since v1:
> - shave off another few cycles from the sequential AES NEON code (patch #9)
Patches 2-10 applied.
Thanks,
--
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
prev parent reply other threads:[~2017-02-03 10:22 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-01-28 23:25 [PATCH v3 00/10] crypto - AES for ARM/arm64 updates for v4.11 (round #2) Ard Biesheuvel
2017-01-28 23:25 ` [PATCH v3 01/10] crypto: arm64/aes-neon-bs - honour iv_out requirement in CTR mode Ard Biesheuvel
2017-01-28 23:25 ` [PATCH v3 02/10] crypto: arm/aes-ce - remove cra_alignmask Ard Biesheuvel
2017-01-28 23:25 ` [PATCH v3 03/10] crypto: arm/chacha20 " Ard Biesheuvel
2017-01-28 23:25 ` [PATCH v3 04/10] crypto: arm64/aes-ce-ccm " Ard Biesheuvel
2017-01-28 23:25 ` [PATCH v3 05/10] crypto: arm64/aes-blk " Ard Biesheuvel
2017-01-28 23:25 ` [PATCH v3 06/10] crypto: arm64/chacha20 " Ard Biesheuvel
2017-01-28 23:25 ` [PATCH v3 07/10] crypto: arm64/aes - avoid literals for cross-module symbol references Ard Biesheuvel
2017-01-28 23:25 ` [PATCH v3 08/10] crypto: arm64/aes - performance tweak Ard Biesheuvel
2017-01-28 23:25 ` [PATCH v3 09/10] crypto: arm64/aes-neon-blk - tweak performance for low end cores Ard Biesheuvel
2017-01-28 23:25 ` [PATCH v3 10/10] crypto: arm64/aes - replace scalar fallback with plain NEON fallback Ard Biesheuvel
2017-02-03 10:22 ` Herbert Xu [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170203102212.GG2632@gondor.apana.org.au \
--to=herbert@gondor.apana.org.au \
--cc=ard.biesheuvel@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-crypto@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).