diff options
-rw-r--r-- | fs/fs-writeback.c | 18 | ||||
-rw-r--r-- | include/linux/writeback.h | 1 | ||||
-rw-r--r-- | mm/page-writeback.c | 9 |
3 files changed, 23 insertions, 5 deletions
diff --git a/fs/fs-writeback.c b/fs/fs-writeback.c index 3fe782d70a71..0b3064079fa5 100644 --- a/fs/fs-writeback.c +++ b/fs/fs-writeback.c | |||
@@ -284,7 +284,17 @@ __sync_single_inode(struct inode *inode, struct writeback_control *wbc) | |||
284 | * soon as the queue becomes uncongested. | 284 | * soon as the queue becomes uncongested. |
285 | */ | 285 | */ |
286 | inode->i_state |= I_DIRTY_PAGES; | 286 | inode->i_state |= I_DIRTY_PAGES; |
287 | requeue_io(inode); | 287 | if (wbc->nr_to_write <= 0) { |
288 | /* | ||
289 | * slice used up: queue for next turn | ||
290 | */ | ||
291 | requeue_io(inode); | ||
292 | } else { | ||
293 | /* | ||
294 | * somehow blocked: retry later | ||
295 | */ | ||
296 | redirty_tail(inode); | ||
297 | } | ||
288 | } else { | 298 | } else { |
289 | /* | 299 | /* |
290 | * Otherwise fully redirty the inode so that | 300 | * Otherwise fully redirty the inode so that |
@@ -468,8 +478,12 @@ sync_sb_inodes(struct super_block *sb, struct writeback_control *wbc) | |||
468 | iput(inode); | 478 | iput(inode); |
469 | cond_resched(); | 479 | cond_resched(); |
470 | spin_lock(&inode_lock); | 480 | spin_lock(&inode_lock); |
471 | if (wbc->nr_to_write <= 0) | 481 | if (wbc->nr_to_write <= 0) { |
482 | wbc->more_io = 1; | ||
472 | break; | 483 | break; |
484 | } | ||
485 | if (!list_empty(&sb->s_more_io)) | ||
486 | wbc->more_io = 1; | ||
473 | } | 487 | } |
474 | return; /* Leave any unwritten inodes on s_io */ | 488 | return; /* Leave any unwritten inodes on s_io */ |
475 | } | 489 | } |
diff --git a/include/linux/writeback.h b/include/linux/writeback.h index b2cd826a8c90..b7b3362f7717 100644 --- a/include/linux/writeback.h +++ b/include/linux/writeback.h | |||
@@ -62,6 +62,7 @@ struct writeback_control { | |||
62 | unsigned for_reclaim:1; /* Invoked from the page allocator */ | 62 | unsigned for_reclaim:1; /* Invoked from the page allocator */ |
63 | unsigned for_writepages:1; /* This is a writepages() call */ | 63 | unsigned for_writepages:1; /* This is a writepages() call */ |
64 | unsigned range_cyclic:1; /* range_start is cyclic */ | 64 | unsigned range_cyclic:1; /* range_start is cyclic */ |
65 | unsigned more_io:1; /* more io to be dispatched */ | ||
65 | }; | 66 | }; |
66 | 67 | ||
67 | /* | 68 | /* |
diff --git a/mm/page-writeback.c b/mm/page-writeback.c index a4ca162666c5..5e00f1772c20 100644 --- a/mm/page-writeback.c +++ b/mm/page-writeback.c | |||
@@ -567,6 +567,7 @@ static void background_writeout(unsigned long _min_pages) | |||
567 | global_page_state(NR_UNSTABLE_NFS) < background_thresh | 567 | global_page_state(NR_UNSTABLE_NFS) < background_thresh |
568 | && min_pages <= 0) | 568 | && min_pages <= 0) |
569 | break; | 569 | break; |
570 | wbc.more_io = 0; | ||
570 | wbc.encountered_congestion = 0; | 571 | wbc.encountered_congestion = 0; |
571 | wbc.nr_to_write = MAX_WRITEBACK_PAGES; | 572 | wbc.nr_to_write = MAX_WRITEBACK_PAGES; |
572 | wbc.pages_skipped = 0; | 573 | wbc.pages_skipped = 0; |
@@ -574,8 +575,9 @@ static void background_writeout(unsigned long _min_pages) | |||
574 | min_pages -= MAX_WRITEBACK_PAGES - wbc.nr_to_write; | 575 | min_pages -= MAX_WRITEBACK_PAGES - wbc.nr_to_write; |
575 | if (wbc.nr_to_write > 0 || wbc.pages_skipped > 0) { | 576 | if (wbc.nr_to_write > 0 || wbc.pages_skipped > 0) { |
576 | /* Wrote less than expected */ | 577 | /* Wrote less than expected */ |
577 | congestion_wait(WRITE, HZ/10); | 578 | if (wbc.encountered_congestion || wbc.more_io) |
578 | if (!wbc.encountered_congestion) | 579 | congestion_wait(WRITE, HZ/10); |
580 | else | ||
579 | break; | 581 | break; |
580 | } | 582 | } |
581 | } | 583 | } |
@@ -640,11 +642,12 @@ static void wb_kupdate(unsigned long arg) | |||
640 | global_page_state(NR_UNSTABLE_NFS) + | 642 | global_page_state(NR_UNSTABLE_NFS) + |
641 | (inodes_stat.nr_inodes - inodes_stat.nr_unused); | 643 | (inodes_stat.nr_inodes - inodes_stat.nr_unused); |
642 | while (nr_to_write > 0) { | 644 | while (nr_to_write > 0) { |
645 | wbc.more_io = 0; | ||
643 | wbc.encountered_congestion = 0; | 646 | wbc.encountered_congestion = 0; |
644 | wbc.nr_to_write = MAX_WRITEBACK_PAGES; | 647 | wbc.nr_to_write = MAX_WRITEBACK_PAGES; |
645 | writeback_inodes(&wbc); | 648 | writeback_inodes(&wbc); |
646 | if (wbc.nr_to_write > 0) { | 649 | if (wbc.nr_to_write > 0) { |
647 | if (wbc.encountered_congestion) | 650 | if (wbc.encountered_congestion || wbc.more_io) |
648 | congestion_wait(WRITE, HZ/10); | 651 | congestion_wait(WRITE, HZ/10); |
649 | else | 652 | else |
650 | break; /* All the old data is written */ | 653 | break; /* All the old data is written */ |