mirror of
https://github.com/AuxXxilium/linux_dsm_epyc7002.git
synced 2024-12-27 04:45:23 +07:00
0522236d4f
This patch fixes sleep-in-atomic bugs in AES-CBC and AES-XTS VMX implementations. The problem is that the blkcipher_* functions should not be called in atomic context. The bugs can be reproduced via the AF_ALG interface by trying to encrypt/decrypt sufficiently large buffers (at least 64 KiB) using the VMX implementations of 'cbc(aes)' or 'xts(aes)'. Such operations then trigger BUG in crypto_yield(): [ 891.863680] BUG: sleeping function called from invalid context at include/crypto/algapi.h:424 [ 891.864622] in_atomic(): 1, irqs_disabled(): 0, pid: 12347, name: kcapi-enc [ 891.864739] 1 lock held by kcapi-enc/12347: [ 891.864811] #0: 00000000f5d42c46 (sk_lock-AF_ALG){+.+.}, at: skcipher_recvmsg+0x50/0x530 [ 891.865076] CPU: 5 PID: 12347 Comm: kcapi-enc Not tainted 4.19.0-0.rc0.git3.1.fc30.ppc64le #1 [ 891.865251] Call Trace: [ 891.865340] [c0000003387578c0] [c000000000d67ea4] dump_stack+0xe8/0x164 (unreliable) [ 891.865511] [c000000338757910] [c000000000172a58] ___might_sleep+0x2f8/0x310 [ 891.865679] [c000000338757990] [c0000000006bff74] blkcipher_walk_done+0x374/0x4a0 [ 891.865825] [c0000003387579e0] [d000000007e73e70] p8_aes_cbc_encrypt+0x1c8/0x260 [vmx_crypto] [ 891.865993] [c000000338757ad0] [c0000000006c0ee0] skcipher_encrypt_blkcipher+0x60/0x80 [ 891.866128] [c000000338757b10] [c0000000006ec504] skcipher_recvmsg+0x424/0x530 [ 891.866283] [c000000338757bd0] [c000000000b00654] sock_recvmsg+0x74/0xa0 [ 891.866403] [c000000338757c10] [c000000000b00f64] ___sys_recvmsg+0xf4/0x2f0 [ 891.866515] [c000000338757d90] [c000000000b02bb8] __sys_recvmsg+0x68/0xe0 [ 891.866631] [c000000338757e30] [c00000000000bbe4] system_call+0x5c/0x70 Fixes:8c755ace35
("crypto: vmx - Adding CBC routines for VMX module") Fixes:c07f5d3da6
("crypto: vmx - Adding support for XTS") Cc: stable@vger.kernel.org Signed-off-by: Ondrej Mosnacek <omosnace@redhat.com> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
193 lines
5.2 KiB
C
193 lines
5.2 KiB
C
/**
|
|
* AES CBC routines supporting VMX instructions on the Power 8
|
|
*
|
|
* Copyright (C) 2015 International Business Machines Inc.
|
|
*
|
|
* This program is free software; you can redistribute it and/or modify
|
|
* it under the terms of the GNU General Public License as published by
|
|
* the Free Software Foundation; version 2 only.
|
|
*
|
|
* This program is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
* GNU General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU General Public License
|
|
* along with this program; if not, write to the Free Software
|
|
* Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
|
|
*
|
|
* Author: Marcelo Henrique Cerri <mhcerri@br.ibm.com>
|
|
*/
|
|
|
|
#include <linux/types.h>
|
|
#include <linux/err.h>
|
|
#include <linux/crypto.h>
|
|
#include <linux/delay.h>
|
|
#include <linux/hardirq.h>
|
|
#include <asm/switch_to.h>
|
|
#include <crypto/aes.h>
|
|
#include <crypto/scatterwalk.h>
|
|
#include <crypto/skcipher.h>
|
|
|
|
#include "aesp8-ppc.h"
|
|
|
|
struct p8_aes_cbc_ctx {
|
|
struct crypto_skcipher *fallback;
|
|
struct aes_key enc_key;
|
|
struct aes_key dec_key;
|
|
};
|
|
|
|
static int p8_aes_cbc_init(struct crypto_tfm *tfm)
|
|
{
|
|
const char *alg = crypto_tfm_alg_name(tfm);
|
|
struct crypto_skcipher *fallback;
|
|
struct p8_aes_cbc_ctx *ctx = crypto_tfm_ctx(tfm);
|
|
|
|
fallback = crypto_alloc_skcipher(alg, 0,
|
|
CRYPTO_ALG_ASYNC | CRYPTO_ALG_NEED_FALLBACK);
|
|
|
|
if (IS_ERR(fallback)) {
|
|
printk(KERN_ERR
|
|
"Failed to allocate transformation for '%s': %ld\n",
|
|
alg, PTR_ERR(fallback));
|
|
return PTR_ERR(fallback);
|
|
}
|
|
|
|
crypto_skcipher_set_flags(
|
|
fallback,
|
|
crypto_skcipher_get_flags((struct crypto_skcipher *)tfm));
|
|
ctx->fallback = fallback;
|
|
|
|
return 0;
|
|
}
|
|
|
|
static void p8_aes_cbc_exit(struct crypto_tfm *tfm)
|
|
{
|
|
struct p8_aes_cbc_ctx *ctx = crypto_tfm_ctx(tfm);
|
|
|
|
if (ctx->fallback) {
|
|
crypto_free_skcipher(ctx->fallback);
|
|
ctx->fallback = NULL;
|
|
}
|
|
}
|
|
|
|
static int p8_aes_cbc_setkey(struct crypto_tfm *tfm, const u8 *key,
|
|
unsigned int keylen)
|
|
{
|
|
int ret;
|
|
struct p8_aes_cbc_ctx *ctx = crypto_tfm_ctx(tfm);
|
|
|
|
preempt_disable();
|
|
pagefault_disable();
|
|
enable_kernel_vsx();
|
|
ret = aes_p8_set_encrypt_key(key, keylen * 8, &ctx->enc_key);
|
|
ret += aes_p8_set_decrypt_key(key, keylen * 8, &ctx->dec_key);
|
|
disable_kernel_vsx();
|
|
pagefault_enable();
|
|
preempt_enable();
|
|
|
|
ret += crypto_skcipher_setkey(ctx->fallback, key, keylen);
|
|
return ret;
|
|
}
|
|
|
|
static int p8_aes_cbc_encrypt(struct blkcipher_desc *desc,
|
|
struct scatterlist *dst,
|
|
struct scatterlist *src, unsigned int nbytes)
|
|
{
|
|
int ret;
|
|
struct blkcipher_walk walk;
|
|
struct p8_aes_cbc_ctx *ctx =
|
|
crypto_tfm_ctx(crypto_blkcipher_tfm(desc->tfm));
|
|
|
|
if (in_interrupt()) {
|
|
SKCIPHER_REQUEST_ON_STACK(req, ctx->fallback);
|
|
skcipher_request_set_tfm(req, ctx->fallback);
|
|
skcipher_request_set_callback(req, desc->flags, NULL, NULL);
|
|
skcipher_request_set_crypt(req, src, dst, nbytes, desc->info);
|
|
ret = crypto_skcipher_encrypt(req);
|
|
skcipher_request_zero(req);
|
|
} else {
|
|
blkcipher_walk_init(&walk, dst, src, nbytes);
|
|
ret = blkcipher_walk_virt(desc, &walk);
|
|
while ((nbytes = walk.nbytes)) {
|
|
preempt_disable();
|
|
pagefault_disable();
|
|
enable_kernel_vsx();
|
|
aes_p8_cbc_encrypt(walk.src.virt.addr,
|
|
walk.dst.virt.addr,
|
|
nbytes & AES_BLOCK_MASK,
|
|
&ctx->enc_key, walk.iv, 1);
|
|
disable_kernel_vsx();
|
|
pagefault_enable();
|
|
preempt_enable();
|
|
|
|
nbytes &= AES_BLOCK_SIZE - 1;
|
|
ret = blkcipher_walk_done(desc, &walk, nbytes);
|
|
}
|
|
}
|
|
|
|
return ret;
|
|
}
|
|
|
|
static int p8_aes_cbc_decrypt(struct blkcipher_desc *desc,
|
|
struct scatterlist *dst,
|
|
struct scatterlist *src, unsigned int nbytes)
|
|
{
|
|
int ret;
|
|
struct blkcipher_walk walk;
|
|
struct p8_aes_cbc_ctx *ctx =
|
|
crypto_tfm_ctx(crypto_blkcipher_tfm(desc->tfm));
|
|
|
|
if (in_interrupt()) {
|
|
SKCIPHER_REQUEST_ON_STACK(req, ctx->fallback);
|
|
skcipher_request_set_tfm(req, ctx->fallback);
|
|
skcipher_request_set_callback(req, desc->flags, NULL, NULL);
|
|
skcipher_request_set_crypt(req, src, dst, nbytes, desc->info);
|
|
ret = crypto_skcipher_decrypt(req);
|
|
skcipher_request_zero(req);
|
|
} else {
|
|
blkcipher_walk_init(&walk, dst, src, nbytes);
|
|
ret = blkcipher_walk_virt(desc, &walk);
|
|
while ((nbytes = walk.nbytes)) {
|
|
preempt_disable();
|
|
pagefault_disable();
|
|
enable_kernel_vsx();
|
|
aes_p8_cbc_encrypt(walk.src.virt.addr,
|
|
walk.dst.virt.addr,
|
|
nbytes & AES_BLOCK_MASK,
|
|
&ctx->dec_key, walk.iv, 0);
|
|
disable_kernel_vsx();
|
|
pagefault_enable();
|
|
preempt_enable();
|
|
|
|
nbytes &= AES_BLOCK_SIZE - 1;
|
|
ret = blkcipher_walk_done(desc, &walk, nbytes);
|
|
}
|
|
}
|
|
|
|
return ret;
|
|
}
|
|
|
|
|
|
struct crypto_alg p8_aes_cbc_alg = {
|
|
.cra_name = "cbc(aes)",
|
|
.cra_driver_name = "p8_aes_cbc",
|
|
.cra_module = THIS_MODULE,
|
|
.cra_priority = 2000,
|
|
.cra_type = &crypto_blkcipher_type,
|
|
.cra_flags = CRYPTO_ALG_TYPE_BLKCIPHER | CRYPTO_ALG_NEED_FALLBACK,
|
|
.cra_alignmask = 0,
|
|
.cra_blocksize = AES_BLOCK_SIZE,
|
|
.cra_ctxsize = sizeof(struct p8_aes_cbc_ctx),
|
|
.cra_init = p8_aes_cbc_init,
|
|
.cra_exit = p8_aes_cbc_exit,
|
|
.cra_blkcipher = {
|
|
.ivsize = AES_BLOCK_SIZE,
|
|
.min_keysize = AES_MIN_KEY_SIZE,
|
|
.max_keysize = AES_MAX_KEY_SIZE,
|
|
.setkey = p8_aes_cbc_setkey,
|
|
.encrypt = p8_aes_cbc_encrypt,
|
|
.decrypt = p8_aes_cbc_decrypt,
|
|
},
|
|
};
|