Hi Matthew,
I had to apply this in this patch, since fio can be reassigned after
checking fio->in_list check.
--- a/fs/f2fs/data.c
+++ b/fs/f2fs/data.c
@@ -946,7 +946,6 @@ static bool is_end_zone_blkaddr(struct f2fs_sb_info *sbi,
block_t blkaddr)
void f2fs_submit_page_write(struct f2fs_io_info *fio)
{
- struct inode *inode = fio_inode(fio);
struct f2fs_sb_info *sbi = fio->sbi;
enum page_type btype = PAGE_TYPE_OF_BIO(fio->type);
struct f2fs_bio_info *io = sbi->write_io[btype] + fio->temp;
@@ -996,13 +995,13 @@ void f2fs_submit_page_write(struct f2fs_io_info *fio)
if (io->bio &&
(!io_is_mergeable(sbi, io->bio, io, fio, io->last_block_in_bio,
fio->new_blkaddr) ||
- !f2fs_crypt_mergeable_bio(io->bio, inode,
+ !f2fs_crypt_mergeable_bio(io->bio, fio_inode(fio),
page_folio(bio_page)->index, fio)))
__submit_merged_bio(io);
alloc_new:
if (io->bio == NULL) {
io->bio = __bio_alloc(fio, BIO_MAX_VECS);
- f2fs_set_bio_crypt_ctx(io->bio, inode,
+ f2fs_set_bio_crypt_ctx(io->bio, fio_inode(fio),
page_folio(bio_page)->index, fio, GFP_NOIO);
io->fio = *fio;
}
On 03/31, Matthew Wilcox (Oracle) wrote:
> This helper returns the inode associated with the f2fs_io_info. That's a
> relatively common thing to want, mildly awkward to get and provides one
> place to change if we decide to record it directly, or change fio->page
> to fio->folio.
>
> Signed-off-by: Matthew Wilcox (Oracle) <[email protected]>
> ---
> fs/f2fs/data.c | 9 +++++----
> fs/f2fs/f2fs.h | 5 +++++
> fs/f2fs/segment.c | 6 +++---
> 3 files changed, 13 insertions(+), 7 deletions(-)
>
> diff --git a/fs/f2fs/data.c b/fs/f2fs/data.c
> index 54f89f0ee69b..597d1e82cb55 100644
> --- a/fs/f2fs/data.c
> +++ b/fs/f2fs/data.c
> @@ -946,6 +946,7 @@ static bool is_end_zone_blkaddr(struct f2fs_sb_info *sbi,
> block_t blkaddr)
>
> void f2fs_submit_page_write(struct f2fs_io_info *fio)
> {
> + struct inode *inode = fio_inode(fio);
> struct f2fs_sb_info *sbi = fio->sbi;
> enum page_type btype = PAGE_TYPE_OF_BIO(fio->type);
> struct f2fs_bio_info *io = sbi->write_io[btype] + fio->temp;
> @@ -995,13 +996,13 @@ void f2fs_submit_page_write(struct f2fs_io_info *fio)
> if (io->bio &&
> (!io_is_mergeable(sbi, io->bio, io, fio, io->last_block_in_bio,
> fio->new_blkaddr) ||
> - !f2fs_crypt_mergeable_bio(io->bio, fio->page->mapping->host,
> + !f2fs_crypt_mergeable_bio(io->bio, inode,
> page_folio(bio_page)->index, fio)))
> __submit_merged_bio(io);
> alloc_new:
> if (io->bio == NULL) {
> io->bio = __bio_alloc(fio, BIO_MAX_VECS);
> - f2fs_set_bio_crypt_ctx(io->bio, fio->page->mapping->host,
> + f2fs_set_bio_crypt_ctx(io->bio, inode,
> page_folio(bio_page)->index, fio, GFP_NOIO);
> io->fio = *fio;
> }
> @@ -2501,7 +2502,7 @@ static void f2fs_readahead(struct readahead_control
> *rac)
>
> int f2fs_encrypt_one_page(struct f2fs_io_info *fio)
> {
> - struct inode *inode = fio->page->mapping->host;
> + struct inode *inode = fio_inode(fio);
> struct page *mpage, *page;
> gfp_t gfp_flags = GFP_NOFS;
>
> @@ -2631,7 +2632,7 @@ bool f2fs_should_update_outplace(struct inode *inode,
> struct f2fs_io_info *fio)
>
> static inline bool need_inplace_update(struct f2fs_io_info *fio)
> {
> - struct inode *inode = fio->page->mapping->host;
> + struct inode *inode = fio_inode(fio);
>
> if (f2fs_should_update_outplace(inode, fio))
> return false;
> diff --git a/fs/f2fs/f2fs.h b/fs/f2fs/f2fs.h
> index f1576dc6ec67..c37092e82354 100644
> --- a/fs/f2fs/f2fs.h
> +++ b/fs/f2fs/f2fs.h
> @@ -3858,6 +3858,11 @@ unsigned int f2fs_usable_blks_in_seg(struct
> f2fs_sb_info *sbi,
> unsigned long long f2fs_get_section_mtime(struct f2fs_sb_info *sbi,
> unsigned int segno);
>
> +static inline struct inode *fio_inode(struct f2fs_io_info *fio)
> +{
> + return page_folio(fio->page)->mapping->host;
> +}
> +
> #define DEF_FRAGMENT_SIZE 4
> #define MIN_FRAGMENT_SIZE 1
> #define MAX_FRAGMENT_SIZE 512
> diff --git a/fs/f2fs/segment.c b/fs/f2fs/segment.c
> index 396ef71f41e3..d2f2f68da098 100644
> --- a/fs/f2fs/segment.c
> +++ b/fs/f2fs/segment.c
> @@ -3584,7 +3584,7 @@ static int __get_segment_type_2(struct f2fs_io_info
> *fio)
> static int __get_segment_type_4(struct f2fs_io_info *fio)
> {
> if (fio->type == DATA) {
> - struct inode *inode = fio->page->mapping->host;
> + struct inode *inode = fio_inode(fio);
>
> if (S_ISDIR(inode->i_mode))
> return CURSEG_HOT_DATA;
> @@ -3618,7 +3618,7 @@ static int __get_age_segment_type(struct inode *inode,
> pgoff_t pgofs)
> static int __get_segment_type_6(struct f2fs_io_info *fio)
> {
> if (fio->type == DATA) {
> - struct inode *inode = fio->page->mapping->host;
> + struct inode *inode = fio_inode(fio);
> int type;
>
> if (is_inode_flag_set(inode, FI_ALIGNED_WRITE))
> @@ -4023,7 +4023,7 @@ int f2fs_inplace_write_data(struct f2fs_io_info *fio)
> if (!err) {
> f2fs_update_device_state(fio->sbi, fio->ino,
> fio->new_blkaddr, 1);
> - f2fs_update_iostat(fio->sbi, fio->page->mapping->host,
> + f2fs_update_iostat(fio->sbi, fio_inode(fio),
> fio->io_type, F2FS_BLKSIZE);
> }
>
> --
> 2.47.2
_______________________________________________
Linux-f2fs-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
