On Fri, Jan 27, 2023 at 11:33:57PM +0000, Matthew Wilcox wrote: > On Fri, Jan 27, 2023 at 02:42:02PM -0800, Eric Biggers wrote: > > +++ b/fs/buffer.c > > @@ -307,8 +307,8 @@ static void decrypt_bh(struct work_struct *work) > > struct buffer_head *bh = ctx->bh; > > int err; > > > > - err = fscrypt_decrypt_pagecache_blocks(bh->b_page, bh->b_size, > > - bh_offset(bh)); > > + err = fscrypt_decrypt_pagecache_blocks(page_folio(bh->b_page), > > + bh->b_size, bh_offset(bh)); > > FYI, Andrew's been carrying "buffer: add b_folio as an alias of b_page" > in his tree since mid-December. buffer_heads always point to the > head page, aka folio. > > This all looks good to me. > > Reviewed-by: Matthew Wilcox (Oracle) <willy@xxxxxxxxxxxxx> Indeed, but it's not upstream yet, so I decided to use page_folio() for now to avoid a cross-tree dependency. - Eric
