mirror of
https://github.com/tbsdtv/linux_media.git
synced 2025-07-23 12:43:29 +02:00
gfs2: Replace deprecated kmap_atomic with kmap_local_page
kmap_atomic() is deprecated in favor of kmap_local_{folio,page}(). Therefore, replace kmap_atomic() with kmap_local_page() in gfs2_internal_read() and stuffed_readpage(). kmap_atomic() disables page-faults and preemption (the latter only for !PREEMPT_RT kernels), However, the code within the mapping/un-mapping in gfs2_internal_read() and stuffed_readpage() does not depend on the above-mentioned side effects. Therefore, a mere replacement of the old API with the new one is all that is required (i.e., there is no need to explicitly add any calls to pagefault_disable() and/or preempt_disable()). Signed-off-by: Deepak R Varma <drv@mailo.com> Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
This commit is contained in:
committed by
Andreas Gruenbacher
parent
f246dd4b78
commit
58721bd46c
@@ -431,10 +431,10 @@ static int stuffed_readpage(struct gfs2_inode *ip, struct page *page)
|
|||||||
if (error)
|
if (error)
|
||||||
return error;
|
return error;
|
||||||
|
|
||||||
kaddr = kmap_atomic(page);
|
kaddr = kmap_local_page(page);
|
||||||
memcpy(kaddr, dibh->b_data + sizeof(struct gfs2_dinode), dsize);
|
memcpy(kaddr, dibh->b_data + sizeof(struct gfs2_dinode), dsize);
|
||||||
memset(kaddr + dsize, 0, PAGE_SIZE - dsize);
|
memset(kaddr + dsize, 0, PAGE_SIZE - dsize);
|
||||||
kunmap_atomic(kaddr);
|
kunmap_local(kaddr);
|
||||||
flush_dcache_page(page);
|
flush_dcache_page(page);
|
||||||
brelse(dibh);
|
brelse(dibh);
|
||||||
SetPageUptodate(page);
|
SetPageUptodate(page);
|
||||||
@@ -497,12 +497,12 @@ int gfs2_internal_read(struct gfs2_inode *ip, char *buf, loff_t *pos,
|
|||||||
continue;
|
continue;
|
||||||
return PTR_ERR(page);
|
return PTR_ERR(page);
|
||||||
}
|
}
|
||||||
p = kmap_atomic(page);
|
p = kmap_local_page(page);
|
||||||
amt = size - copied;
|
amt = size - copied;
|
||||||
if (offset + size > PAGE_SIZE)
|
if (offset + size > PAGE_SIZE)
|
||||||
amt = PAGE_SIZE - offset;
|
amt = PAGE_SIZE - offset;
|
||||||
memcpy(buf + copied, p + offset, amt);
|
memcpy(buf + copied, p + offset, amt);
|
||||||
kunmap_atomic(p);
|
kunmap_local(p);
|
||||||
put_page(page);
|
put_page(page);
|
||||||
copied += amt;
|
copied += amt;
|
||||||
index++;
|
index++;
|
||||||
|
Reference in New Issue
Block a user