diff options
| author | Tejun Heo <tj@kernel.org> | 2010-10-19 11:08:41 +0200 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2010-10-19 11:08:41 +0200 |
| commit | 10ccd84695c2a03075bad2f4fc728575fe9051f8 (patch) | |
| tree | 262fbf9d7114c6f4790cf31db35a12e06bf6dcb6 /mm | |
| parent | shpchp: update workqueue usage (diff) | |
| download | linux-10ccd84695c2a03075bad2f4fc728575fe9051f8.tar.xz linux-10ccd84695c2a03075bad2f4fc728575fe9051f8.zip | |
memory_hotplug: drop spurious calls to flush_scheduled_work()
lru_add_drain_all() uses schedule_on_each_cpu() which is synchronous.
There is no reason to call flush_scheduled_work() after
lru_add_drain_all(). Drop the spurious calls.
This is to prepare for the deprecation and removal of
flush_scheduled_work().
Signed-off-by: Tejun Heo <tj@kernel.org>
Acked-by: KAMEZAWA Hiroyuki <kamezawa.hiroyu@jp.fujitsu.com>
Reviewed-by: Minchan Kim <minchan.kim@gmail.com>
Acked-by: Mel Gorman <mel@csn.ul.ie>
Diffstat (limited to 'mm')
| -rw-r--r-- | mm/memory_hotplug.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/mm/memory_hotplug.c b/mm/memory_hotplug.c index dd186c1a5d53..d4e940a26945 100644 --- a/mm/memory_hotplug.c +++ b/mm/memory_hotplug.c @@ -840,7 +840,6 @@ repeat: ret = 0; if (drain) { lru_add_drain_all(); - flush_scheduled_work(); cond_resched(); drain_all_pages(); } @@ -862,7 +861,6 @@ repeat: } /* drain all zone's lru pagevec, this is asyncronous... */ lru_add_drain_all(); - flush_scheduled_work(); yield(); /* drain pcp pages , this is synchrouns. */ drain_all_pages(); |