* [PATCH v2] xen: simplify balloon_first_page() with list_first_entry_or_null()
@ 2013-11-13 12:59 Jeff Liu
2013-11-13 14:03 ` David Vrabel
0 siblings, 1 reply; 2+ messages in thread
From: Jeff Liu @ 2013-11-13 12:59 UTC (permalink / raw)
To: xen-devel; +Cc: boris.ostrovsky, David Vrabel
From: Jie Liu <jeff.liu@oracle.com>
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>
---
v2: remove balloon_first_page() with this change.
drivers/xen/balloon.c | 9 +--------
1 file changed, 1 insertion(+), 8 deletions(-)
diff --git a/drivers/xen/balloon.c b/drivers/xen/balloon.c
index 55ea73f..16ffe91 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;
--
1.8.3.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] xen: simplify balloon_first_page() with list_first_entry_or_null()
2013-11-13 12:59 [PATCH v2] xen: simplify balloon_first_page() with list_first_entry_or_null() Jeff Liu
@ 2013-11-13 14:03 ` David Vrabel
0 siblings, 0 replies; 2+ messages in thread
From: David Vrabel @ 2013-11-13 14:03 UTC (permalink / raw)
To: Jeff Liu; +Cc: xen-devel, boris.ostrovsky, David Vrabel
On 13/11/13 12:59, Jeff Liu wrote:
> From: Jie Liu <jeff.liu@oracle.com>
>
> 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>
Reviewed-by: David Vrabel <david.vrabel@citrix.com>
(In case anyone else is wondering this list function was introduced in
3.10.)
David
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-11-13 14:03 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-11-13 12:59 [PATCH v2] xen: simplify balloon_first_page() with list_first_entry_or_null() Jeff Liu
2013-11-13 14:03 ` David Vrabel
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.