Re: [PATCH] ext4: Remove references to bh->b_page

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu 13-02-25 18:23:01, Matthew Wilcox (Oracle) wrote:
> Buffer heads are attached to folios, not to pages.  Also
> flush_dcache_page() is now deprecated in favour of flush_dcache_folio().
> 
> Signed-off-by: Matthew Wilcox (Oracle) <willy@xxxxxxxxxxxxx>

Looks good. Feel free to add:

Reviewed-by: Jan Kara <jack@xxxxxxx>

								Honza

> ---
>  fs/ext4/inode.c   | 2 +-
>  fs/ext4/super.c   | 2 +-
>  fs/jbd2/journal.c | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c
> index 7c54ae5fcbd4..bd579f46c7f3 100644
> --- a/fs/ext4/inode.c
> +++ b/fs/ext4/inode.c
> @@ -751,7 +751,7 @@ static void ext4_update_bh_state(struct buffer_head *bh, unsigned long flags)
>  	flags &= EXT4_MAP_FLAGS;
>  
>  	/* Dummy buffer_head? Set non-atomically. */
> -	if (!bh->b_page) {
> +	if (!bh->b_folio) {
>  		bh->b_state = (bh->b_state & ~EXT4_MAP_FLAGS) | flags;
>  		return;
>  	}
> diff --git a/fs/ext4/super.c b/fs/ext4/super.c
> index a50e5c31b937..366ce891bcc3 100644
> --- a/fs/ext4/super.c
> +++ b/fs/ext4/super.c
> @@ -7288,7 +7288,7 @@ static ssize_t ext4_quota_write(struct super_block *sb, int type,
>  	}
>  	lock_buffer(bh);
>  	memcpy(bh->b_data+offset, data, len);
> -	flush_dcache_page(bh->b_page);
> +	flush_dcache_folio(bh->b_folio);
>  	unlock_buffer(bh);
>  	err = ext4_handle_dirty_metadata(handle, NULL, bh);
>  	brelse(bh);
> diff --git a/fs/jbd2/journal.c b/fs/jbd2/journal.c
> index d8084b31b361..e5a4e4ba7837 100644
> --- a/fs/jbd2/journal.c
> +++ b/fs/jbd2/journal.c
> @@ -947,7 +947,7 @@ int jbd2_journal_bmap(journal_t *journal, unsigned long blocknr,
>   * descriptor blocks we do need to generate bona fide buffers.
>   *
>   * After the caller of jbd2_journal_get_descriptor_buffer() has finished modifying
> - * the buffer's contents they really should run flush_dcache_page(bh->b_page).
> + * the buffer's contents they really should run flush_dcache_folio(bh->b_folio).
>   * But we don't bother doing that, so there will be coherency problems with
>   * mmaps of blockdevs which hold live JBD-controlled filesystems.
>   */
> -- 
> 2.47.2
> 
-- 
Jan Kara <jack@xxxxxxxx>
SUSE Labs, CR




[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [NTFS 3]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [NTFS 3]     [Samba]     [Device Mapper]     [CEPH Development]

  Powered by Linux