mirror of
https://mirrors.bfsu.edu.cn/git/linux.git
synced 2024-11-11 12:28:41 +08:00
gfs2: Clean up use of fault_in_iov_iter_{read,write}able
No need to store the return value of the fault_in functions in separate variables. Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com>
This commit is contained in:
parent
42e4c3bdca
commit
6d22ff4710
@ -846,12 +846,10 @@ retry_under_glock:
|
||||
read = ret;
|
||||
|
||||
if (should_fault_in_pages(ret, to, &prev_count, &window_size)) {
|
||||
size_t leftover;
|
||||
|
||||
gfs2_holder_allow_demote(gh);
|
||||
leftover = fault_in_iov_iter_writeable(to, window_size);
|
||||
window_size -= fault_in_iov_iter_writeable(to, window_size);
|
||||
gfs2_holder_disallow_demote(gh);
|
||||
if (leftover != window_size) {
|
||||
if (window_size) {
|
||||
if (gfs2_holder_queued(gh))
|
||||
goto retry_under_glock;
|
||||
goto retry;
|
||||
@ -915,12 +913,10 @@ retry_under_glock:
|
||||
written = ret;
|
||||
|
||||
if (should_fault_in_pages(ret, from, &prev_count, &window_size)) {
|
||||
size_t leftover;
|
||||
|
||||
gfs2_holder_allow_demote(gh);
|
||||
leftover = fault_in_iov_iter_readable(from, window_size);
|
||||
window_size -= fault_in_iov_iter_readable(from, window_size);
|
||||
gfs2_holder_disallow_demote(gh);
|
||||
if (leftover != window_size) {
|
||||
if (window_size) {
|
||||
if (gfs2_holder_queued(gh))
|
||||
goto retry_under_glock;
|
||||
goto retry;
|
||||
@ -983,12 +979,10 @@ retry_under_glock:
|
||||
read += ret;
|
||||
|
||||
if (should_fault_in_pages(ret, to, &prev_count, &window_size)) {
|
||||
size_t leftover;
|
||||
|
||||
gfs2_holder_allow_demote(&gh);
|
||||
leftover = fault_in_iov_iter_writeable(to, window_size);
|
||||
window_size -= fault_in_iov_iter_writeable(to, window_size);
|
||||
gfs2_holder_disallow_demote(&gh);
|
||||
if (leftover != window_size) {
|
||||
if (window_size) {
|
||||
if (gfs2_holder_queued(&gh))
|
||||
goto retry_under_glock;
|
||||
goto retry;
|
||||
@ -1058,13 +1052,11 @@ retry_under_glock:
|
||||
|
||||
from->count = orig_count - written;
|
||||
if (should_fault_in_pages(ret, from, &prev_count, &window_size)) {
|
||||
size_t leftover;
|
||||
|
||||
gfs2_holder_allow_demote(gh);
|
||||
leftover = fault_in_iov_iter_readable(from, window_size);
|
||||
window_size -= fault_in_iov_iter_readable(from, window_size);
|
||||
gfs2_holder_disallow_demote(gh);
|
||||
if (leftover != window_size) {
|
||||
from->count = min(from->count, window_size - leftover);
|
||||
if (window_size) {
|
||||
from->count = min(from->count, window_size);
|
||||
if (gfs2_holder_queued(gh))
|
||||
goto retry_under_glock;
|
||||
goto retry;
|
||||
|
Loading…
Reference in New Issue
Block a user