On Tue, Dec 04, 2018 at 03:45:07PM -0800, Eric Biggers wrote: > On Thu, Sep 06, 2018 at 12:43:41PM +0200, Ard Biesheuvel wrote: > > On 5 September 2018 at 21:24, Eric Biggers <ebiggers@xxxxxxxxxx> wrote: > > > From: Eric Biggers <ebiggers@xxxxxxxxxx> > > > > > > fscrypt doesn't use the CTR mode of operation for anything, so there's > > > no need to select CRYPTO_CTR. It was added by commit 71dea01ea2ed > > > ("ext4 crypto: require CONFIG_CRYPTO_CTR if ext4 encryption is > > > enabled"). But, I've been unable to identify the arm64 crypto bug it > > > was supposedly working around. > > > > > > I suspect the issue was seen only on some old Android device kernel > > > (circa 3.10?). So if the fix wasn't mistaken, the real bug is probably > > > already fixed. Or maybe it was actually a bug in a non-upstream crypto > > > driver. > > > > > > So, remove the dependency. If it turns out there's actually still a > > > bug, we'll fix it properly. > > > > > > Signed-off-by: Eric Biggers <ebiggers@xxxxxxxxxx> > > > > Acked-by: Ard Biesheuvel <ard.biesheuvel@xxxxxxxxxx> Thanks, applied. - Ted