udf: Convert udf_expand_file_adinicb() to use a folio

[ Upstream commit db6754090a ]

Use the folio APIs throughout this function.

Signed-off-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Fixes: 1eeceaec79 ("udf: Convert udf_expand_file_adinicb() to avoid kmap_atomic()")
Signed-off-by: Jan Kara <jack@suse.cz>
Message-Id: <20240417150416.752929-4-willy@infradead.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Matthew Wilcox (Oracle) 2024-04-17 16:04:09 +01:00 committed by Greg Kroah-Hartman
parent 7745b1b785
commit a758e99cd6

View File

@ -341,7 +341,7 @@ const struct address_space_operations udf_aops = {
*/ */
int udf_expand_file_adinicb(struct inode *inode) int udf_expand_file_adinicb(struct inode *inode)
{ {
struct page *page; struct folio *folio;
struct udf_inode_info *iinfo = UDF_I(inode); struct udf_inode_info *iinfo = UDF_I(inode);
int err; int err;
@ -357,12 +357,13 @@ int udf_expand_file_adinicb(struct inode *inode)
return 0; return 0;
} }
page = find_or_create_page(inode->i_mapping, 0, GFP_KERNEL); folio = __filemap_get_folio(inode->i_mapping, 0,
if (!page) FGP_LOCK | FGP_ACCESSED | FGP_CREAT, GFP_KERNEL);
return -ENOMEM; if (IS_ERR(folio))
return PTR_ERR(folio);
if (!PageUptodate(page)) if (!folio_test_uptodate(folio))
udf_adinicb_readpage(page); udf_adinicb_readpage(&folio->page);
down_write(&iinfo->i_data_sem); down_write(&iinfo->i_data_sem);
memset(iinfo->i_data + iinfo->i_lenEAttr, 0x00, memset(iinfo->i_data + iinfo->i_lenEAttr, 0x00,
iinfo->i_lenAlloc); iinfo->i_lenAlloc);
@ -371,22 +372,22 @@ int udf_expand_file_adinicb(struct inode *inode)
iinfo->i_alloc_type = ICBTAG_FLAG_AD_SHORT; iinfo->i_alloc_type = ICBTAG_FLAG_AD_SHORT;
else else
iinfo->i_alloc_type = ICBTAG_FLAG_AD_LONG; iinfo->i_alloc_type = ICBTAG_FLAG_AD_LONG;
set_page_dirty(page); folio_mark_dirty(folio);
unlock_page(page); folio_unlock(folio);
up_write(&iinfo->i_data_sem); up_write(&iinfo->i_data_sem);
err = filemap_fdatawrite(inode->i_mapping); err = filemap_fdatawrite(inode->i_mapping);
if (err) { if (err) {
/* Restore everything back so that we don't lose data... */ /* Restore everything back so that we don't lose data... */
lock_page(page); folio_lock(folio);
down_write(&iinfo->i_data_sem); down_write(&iinfo->i_data_sem);
memcpy_to_page(page, 0, iinfo->i_data + iinfo->i_lenEAttr, memcpy_from_folio(iinfo->i_data + iinfo->i_lenEAttr,
inode->i_size); folio, 0, inode->i_size);
unlock_page(page); folio_unlock(folio);
iinfo->i_alloc_type = ICBTAG_FLAG_AD_IN_ICB; iinfo->i_alloc_type = ICBTAG_FLAG_AD_IN_ICB;
iinfo->i_lenAlloc = inode->i_size; iinfo->i_lenAlloc = inode->i_size;
up_write(&iinfo->i_data_sem); up_write(&iinfo->i_data_sem);
} }
put_page(page); folio_put(folio);
mark_inode_dirty(inode); mark_inode_dirty(inode);
return err; return err;