diff options
author | Rakib Mullick <rakib.mullick@gmail.com> | 2011-11-02 16:40:04 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2011-11-02 19:07:03 -0400 |
commit | 6d994a7e42ab219ba3c10d5ffccf20990252881e (patch) | |
tree | dc10f63454fa7baa3ac379e54550d44832ae7c0c /drivers/misc | |
parent | 3fd306c85adcde7209281cb663dd8ea247e97cc3 (diff) |
drivers/misc/vmw_balloon.c: determine page allocation flag can_sleep outside loop
In vmballoon_reserve_page(), flags has been passed from the callee
function (vmballoon_inflate here). So, we can determine can_sleep outside
the loop.
Signed-off-by: Rakib Mullick <rakib.mullick@gmail.com>
Acked-by: Dmitry Torokhov <dtor@vmware.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/misc')
-rw-r--r-- | drivers/misc/vmw_balloon.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/misc/vmw_balloon.c b/drivers/misc/vmw_balloon.c index 053d36caf955..6983d80563cb 100644 --- a/drivers/misc/vmw_balloon.c +++ b/drivers/misc/vmw_balloon.c | |||
@@ -412,6 +412,7 @@ static int vmballoon_reserve_page(struct vmballoon *b, bool can_sleep) | |||
412 | gfp_t flags; | 412 | gfp_t flags; |
413 | unsigned int hv_status; | 413 | unsigned int hv_status; |
414 | bool locked = false; | 414 | bool locked = false; |
415 | flags = can_sleep ? VMW_PAGE_ALLOC_CANSLEEP : VMW_PAGE_ALLOC_NOSLEEP; | ||
415 | 416 | ||
416 | do { | 417 | do { |
417 | if (!can_sleep) | 418 | if (!can_sleep) |
@@ -419,7 +420,6 @@ static int vmballoon_reserve_page(struct vmballoon *b, bool can_sleep) | |||
419 | else | 420 | else |
420 | STATS_INC(b->stats.sleep_alloc); | 421 | STATS_INC(b->stats.sleep_alloc); |
421 | 422 | ||
422 | flags = can_sleep ? VMW_PAGE_ALLOC_CANSLEEP : VMW_PAGE_ALLOC_NOSLEEP; | ||
423 | page = alloc_page(flags); | 423 | page = alloc_page(flags); |
424 | if (!page) { | 424 | if (!page) { |
425 | if (!can_sleep) | 425 | if (!can_sleep) |