On Thu, Jul 21, 2022 at 08:50:24PM +0200, David Sterba wrote: > The use of kmap() is being deprecated in favor of kmap_local_page() > where it is feasible. For kmap around a memcpy there's a convenience > helper memcpy_to_page, use it. > > CC: Ira Weiny <ira.weiny@xxxxxxxxx> Typo in the subject: s/memcpy_to_page/memcpy_to_zero Other than that. Reviewed-by: Ira Weiny <ira.weiny@xxxxxxxxx> > CC: Fabio M. De Francesco <fmdefrancesco@xxxxxxxxx> > Signed-off-by: David Sterba <dsterba@xxxxxxxx> > --- > fs/affs/file.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/fs/affs/file.c b/fs/affs/file.c > index cd00a4c68a12..45a21729f358 100644 > --- a/fs/affs/file.c > +++ b/fs/affs/file.c > @@ -526,7 +526,6 @@ affs_do_readpage_ofs(struct page *page, unsigned to, int create) > struct inode *inode = page->mapping->host; > struct super_block *sb = inode->i_sb; > struct buffer_head *bh; > - char *data; > unsigned pos = 0; > u32 bidx, boff, bsize; > u32 tmp; > @@ -545,9 +544,7 @@ affs_do_readpage_ofs(struct page *page, unsigned to, int create) > return PTR_ERR(bh); > tmp = min(bsize - boff, to - pos); > BUG_ON(pos + tmp > to || tmp > bsize); > - data = kmap_atomic(page); > - memcpy(data + pos, AFFS_DATA(bh) + boff, tmp); > - kunmap_atomic(data); > + memcpy_to_page(page, pos, AFFS_DATA(bh) + boff, tmp); > affs_brelse(bh); > bidx++; > pos += tmp; > -- > 2.36.1 >
