dm-verity: make verity_hash() take dm_verity_io instead of ahash_request

In preparation for adding shash support to dm-verity, change
verity_hash() to take a pointer to a struct dm_verity_io instead of a
pointer to the ahash_request embedded inside it.

Reviewed-by: Sami Tolvanen <samitolvanen@google.com>
Acked-by: Ard Biesheuvel <ardb@kernel.org>
Signed-off-by: Eric Biggers <ebiggers@google.com>
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
This commit is contained in:
Eric Biggers 2024-07-02 16:40:41 +02:00 committed by Mikulas Patocka
parent cf715f4b7e
commit e8f5e93301
3 changed files with 13 additions and 16 deletions

View File

@ -186,8 +186,7 @@ error:
static int fec_is_erasure(struct dm_verity *v, struct dm_verity_io *io, static int fec_is_erasure(struct dm_verity *v, struct dm_verity_io *io,
u8 *want_digest, u8 *data) u8 *want_digest, u8 *data)
{ {
if (unlikely(verity_hash(v, verity_io_hash_req(v, io), if (unlikely(verity_hash(v, io, data, 1 << v->data_dev_block_bits,
data, 1 << v->data_dev_block_bits,
verity_io_real_digest(v, io), true))) verity_io_real_digest(v, io), true)))
return 0; return 0;
@ -388,8 +387,7 @@ static int fec_decode_rsb(struct dm_verity *v, struct dm_verity_io *io,
} }
/* Always re-validate the corrected block against the expected hash */ /* Always re-validate the corrected block against the expected hash */
r = verity_hash(v, verity_io_hash_req(v, io), fio->output, r = verity_hash(v, io, fio->output, 1 << v->data_dev_block_bits,
1 << v->data_dev_block_bits,
verity_io_real_digest(v, io), true); verity_io_real_digest(v, io), true);
if (unlikely(r < 0)) if (unlikely(r < 0))
return r; return r;

View File

@ -180,9 +180,10 @@ out:
return r; return r;
} }
int verity_hash(struct dm_verity *v, struct ahash_request *req, int verity_hash(struct dm_verity *v, struct dm_verity_io *io,
const u8 *data, size_t len, u8 *digest, bool may_sleep) const u8 *data, size_t len, u8 *digest, bool may_sleep)
{ {
struct ahash_request *req = verity_io_hash_req(v, io);
int r; int r;
struct crypto_wait wait; struct crypto_wait wait;
@ -325,8 +326,7 @@ static int verity_verify_level(struct dm_verity *v, struct dm_verity_io *io,
goto release_ret_r; goto release_ret_r;
} }
r = verity_hash(v, verity_io_hash_req(v, io), r = verity_hash(v, io, data, 1 << v->hash_dev_block_bits,
data, 1 << v->hash_dev_block_bits,
verity_io_real_digest(v, io), !io->in_bh); verity_io_real_digest(v, io), !io->in_bh);
if (unlikely(r < 0)) if (unlikely(r < 0))
goto release_ret_r; goto release_ret_r;
@ -428,8 +428,7 @@ static noinline int verity_recheck(struct dm_verity *v, struct dm_verity_io *io,
if (unlikely(r)) if (unlikely(r))
goto free_ret; goto free_ret;
r = verity_hash(v, verity_io_hash_req(v, io), buffer, r = verity_hash(v, io, buffer, 1 << v->data_dev_block_bits,
1 << v->data_dev_block_bits,
verity_io_real_digest(v, io), true); verity_io_real_digest(v, io), true);
if (unlikely(r)) if (unlikely(r))
goto free_ret; goto free_ret;
@ -544,7 +543,7 @@ static int verity_verify_io(struct dm_verity_io *io)
continue; continue;
} }
r = verity_hash(v, verity_io_hash_req(v, io), data, block_size, r = verity_hash(v, io, data, block_size,
verity_io_real_digest(v, io), !io->in_bh); verity_io_real_digest(v, io), !io->in_bh);
if (unlikely(r < 0)) { if (unlikely(r < 0)) {
kunmap_local(data); kunmap_local(data);
@ -991,7 +990,7 @@ static int verity_alloc_most_once(struct dm_verity *v)
static int verity_alloc_zero_digest(struct dm_verity *v) static int verity_alloc_zero_digest(struct dm_verity *v)
{ {
int r = -ENOMEM; int r = -ENOMEM;
struct ahash_request *req; struct dm_verity_io *io;
u8 *zero_data; u8 *zero_data;
v->zero_digest = kmalloc(v->digest_size, GFP_KERNEL); v->zero_digest = kmalloc(v->digest_size, GFP_KERNEL);
@ -999,9 +998,9 @@ static int verity_alloc_zero_digest(struct dm_verity *v)
if (!v->zero_digest) if (!v->zero_digest)
return r; return r;
req = kmalloc(v->ahash_reqsize, GFP_KERNEL); io = kmalloc(sizeof(*io) + v->ahash_reqsize, GFP_KERNEL);
if (!req) if (!io)
return r; /* verity_dtr will free zero_digest */ return r; /* verity_dtr will free zero_digest */
zero_data = kzalloc(1 << v->data_dev_block_bits, GFP_KERNEL); zero_data = kzalloc(1 << v->data_dev_block_bits, GFP_KERNEL);
@ -1009,11 +1008,11 @@ static int verity_alloc_zero_digest(struct dm_verity *v)
if (!zero_data) if (!zero_data)
goto out; goto out;
r = verity_hash(v, req, zero_data, 1 << v->data_dev_block_bits, r = verity_hash(v, io, zero_data, 1 << v->data_dev_block_bits,
v->zero_digest, true); v->zero_digest, true);
out: out:
kfree(req); kfree(io);
kfree(zero_data); kfree(zero_data);
return r; return r;

View File

@ -116,7 +116,7 @@ static inline u8 *verity_io_want_digest(struct dm_verity *v,
return io->want_digest; return io->want_digest;
} }
extern int verity_hash(struct dm_verity *v, struct ahash_request *req, extern int verity_hash(struct dm_verity *v, struct dm_verity_io *io,
const u8 *data, size_t len, u8 *digest, bool may_sleep); const u8 *data, size_t len, u8 *digest, bool may_sleep);
extern int verity_hash_for_block(struct dm_verity *v, struct dm_verity_io *io, extern int verity_hash_for_block(struct dm_verity *v, struct dm_verity_io *io,