summaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorJie Liu <jeff.liu@oracle.com>2013-11-13 13:59:58 +0100
committerKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>2014-01-06 16:07:29 +0100
commit9346c2a8defab777d1fba6bcc284f6ada181fe96 (patch)
tree4597a6c868c0e106dc19058df0efaaea1a6b2eb0 /drivers
parentasm/xen/page.h: remove redundant semicolon (diff)
downloadlinux-9346c2a8defab777d1fba6bcc284f6ada181fe96.tar.xz
linux-9346c2a8defab777d1fba6bcc284f6ada181fe96.zip
xen: simplify balloon_first_page() with list_first_entry_or_null()
Replace the code logic at balloon_first_page() by calling list_first_entry_or_null() directly. since here is only one user of that routine, therefore we can just remove it. Signed-off-by: Jie Liu <jeff.liu@oracle.com> Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> Reviewed-by: David Vrabel <david.vrabel@citrix.com>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/xen/balloon.c9
1 files changed, 1 insertions, 8 deletions
diff --git a/drivers/xen/balloon.c b/drivers/xen/balloon.c
index 4c02e2b94103..37d06ea624aa 100644
--- a/drivers/xen/balloon.c
+++ b/drivers/xen/balloon.c
@@ -157,13 +157,6 @@ static struct page *balloon_retrieve(bool prefer_highmem)
return page;
}
-static struct page *balloon_first_page(void)
-{
- if (list_empty(&ballooned_pages))
- return NULL;
- return list_entry(ballooned_pages.next, struct page, lru);
-}
-
static struct page *balloon_next_page(struct page *page)
{
struct list_head *next = page->lru.next;
@@ -328,7 +321,7 @@ static enum bp_state increase_reservation(unsigned long nr_pages)
if (nr_pages > ARRAY_SIZE(frame_list))
nr_pages = ARRAY_SIZE(frame_list);
- page = balloon_first_page();
+ page = list_first_entry_or_null(&ballooned_pages, struct page, lru);
for (i = 0; i < nr_pages; i++) {
if (!page) {
nr_pages = i;