Skip to content

Commit 60b4fcb

Browse files
Christoph HellwigEric Biggers
authored andcommitted
fscrypt: pass a byte offset to fscrypt_generate_dun
Logical offsets into an inode are usually expressed as bytes in the VFS. Switch fscrypt_generate_dun to that convention and remove the ci_data_units_per_block_bits member in struct fscrypt_inode_info that was only used to cache the DUN shift based on the logical block size granularity. Signed-off-by: Christoph Hellwig <hch@lst.de> Link: https://lore.kernel.org/r/20260302141922.370070-7-hch@lst.de Signed-off-by: Eric Biggers <ebiggers@kernel.org>
1 parent a18b1ab commit 60b4fcb

3 files changed

Lines changed: 4 additions & 11 deletions

File tree

fs/crypto/fscrypt_private.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -278,9 +278,6 @@ struct fscrypt_inode_info {
278278
*/
279279
u8 ci_data_unit_bits;
280280

281-
/* Cached value: log2 of number of data units per FS block */
282-
u8 ci_data_units_per_block_bits;
283-
284281
/* Hashed inode number. Only set for IV_INO_LBLK_32 */
285282
u32 ci_hashed_ino;
286283

fs/crypto/inline_crypt.c

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -268,14 +268,12 @@ bool __fscrypt_inode_uses_inline_crypto(const struct inode *inode)
268268
EXPORT_SYMBOL_GPL(__fscrypt_inode_uses_inline_crypto);
269269

270270
static void fscrypt_generate_dun(const struct fscrypt_inode_info *ci,
271-
u64 lblk_num,
272-
u64 dun[BLK_CRYPTO_DUN_ARRAY_SIZE])
271+
loff_t pos, u64 dun[BLK_CRYPTO_DUN_ARRAY_SIZE])
273272
{
274-
u64 index = lblk_num << ci->ci_data_units_per_block_bits;
275273
union fscrypt_iv iv;
276274
int i;
277275

278-
fscrypt_generate_iv(&iv, index, ci);
276+
fscrypt_generate_iv(&iv, pos >> ci->ci_data_unit_bits, ci);
279277

280278
BUILD_BUG_ON(FSCRYPT_MAX_IV_SIZE > BLK_CRYPTO_MAX_IV_SIZE);
281279
memset(dun, 0, BLK_CRYPTO_MAX_IV_SIZE);
@@ -309,7 +307,7 @@ void fscrypt_set_bio_crypt_ctx(struct bio *bio, const struct inode *inode,
309307
return;
310308
ci = fscrypt_get_inode_info_raw(inode);
311309

312-
fscrypt_generate_dun(ci, first_lblk, dun);
310+
fscrypt_generate_dun(ci, first_lblk << inode->i_blkbits, dun);
313311
bio_crypt_set_ctx(bio, ci->ci_enc_key.blk_key, dun, gfp_mask);
314312
}
315313
EXPORT_SYMBOL_GPL(fscrypt_set_bio_crypt_ctx);
@@ -356,7 +354,7 @@ bool fscrypt_mergeable_bio(struct bio *bio, const struct inode *inode,
356354
if (bc->bc_key != ci->ci_enc_key.blk_key)
357355
return false;
358356

359-
fscrypt_generate_dun(ci, next_lblk, next_dun);
357+
fscrypt_generate_dun(ci, next_lblk << inode->i_blkbits, next_dun);
360358
return bio_crypt_dun_is_contiguous(bc, bio->bi_iter.bi_size, next_dun);
361359
}
362360
EXPORT_SYMBOL_GPL(fscrypt_mergeable_bio);

fs/crypto/keysetup.c

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -609,8 +609,6 @@ fscrypt_setup_encryption_info(struct inode *inode,
609609

610610
crypt_info->ci_data_unit_bits =
611611
fscrypt_policy_du_bits(&crypt_info->ci_policy, inode);
612-
crypt_info->ci_data_units_per_block_bits =
613-
inode->i_blkbits - crypt_info->ci_data_unit_bits;
614612

615613
res = setup_file_encryption_key(crypt_info, need_dirhash_key, &mk);
616614
if (res)

0 commit comments

Comments
 (0)