On Mon 26-01-26 05:50:49, Christoph Hellwig wrote: > Use IS_ENABLED to disable this code, leading to a slight size reduction: > > text data bss dec hex filename > 4121 376 16 4513 11a1 fs/ext4/readpage.o.old > 4030 328 16 4374 1116 fs/ext4/readpage.o > > Signed-off-by: Christoph Hellwig <[email protected]>
Looks good. Feel free to add: Reviewed-by: Jan Kara <[email protected]> Honza > --- > fs/ext4/readpage.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/fs/ext4/readpage.c b/fs/ext4/readpage.c > index e7f2350c725b..267594ef0b2c 100644 > --- a/fs/ext4/readpage.c > +++ b/fs/ext4/readpage.c > @@ -130,7 +130,8 @@ static void bio_post_read_processing(struct > bio_post_read_ctx *ctx) > ctx->cur_step++; > fallthrough; > case STEP_VERITY: > - if (ctx->enabled_steps & (1 << STEP_VERITY)) { > + if (IS_ENABLED(CONFIG_FS_VERITY) && > + ctx->enabled_steps & (1 << STEP_VERITY)) { > INIT_WORK(&ctx->work, verity_work); > fsverity_enqueue_verify_work(&ctx->work); > return; > -- > 2.47.3 > -- Jan Kara <[email protected]> SUSE Labs, CR _______________________________________________ Linux-f2fs-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
