diff options
| author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2012-09-05 02:34:32 -0400 |
|---|---|---|
| committer | Alex Elder <elder@inktank.com> | 2012-10-01 15:30:49 -0400 |
| commit | cc4829e5967de577794b25dfcd1a65e509d171ed (patch) | |
| tree | 99e14d39ac747a678df303caee99c41ce48ebd9f /net/ceph | |
| parent | 523f32582f30768ab4e56b71b276fc1ea71f48cc (diff) | |
ceph: use list_move_tail instead of list_del/list_add_tail
Using list_move_tail() instead of list_del() + list_add_tail().
Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
Signed-off-by: Sage Weil <sage@inktank.com>
Diffstat (limited to 'net/ceph')
| -rw-r--r-- | net/ceph/pagelist.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/net/ceph/pagelist.c b/net/ceph/pagelist.c index 665cd23020ff..92866bebb65f 100644 --- a/net/ceph/pagelist.c +++ b/net/ceph/pagelist.c | |||
| @@ -1,4 +1,3 @@ | |||
| 1 | |||
| 2 | #include <linux/module.h> | 1 | #include <linux/module.h> |
| 3 | #include <linux/gfp.h> | 2 | #include <linux/gfp.h> |
| 4 | #include <linux/pagemap.h> | 3 | #include <linux/pagemap.h> |
| @@ -134,8 +133,8 @@ int ceph_pagelist_truncate(struct ceph_pagelist *pl, | |||
| 134 | ceph_pagelist_unmap_tail(pl); | 133 | ceph_pagelist_unmap_tail(pl); |
| 135 | while (pl->head.prev != c->page_lru) { | 134 | while (pl->head.prev != c->page_lru) { |
| 136 | page = list_entry(pl->head.prev, struct page, lru); | 135 | page = list_entry(pl->head.prev, struct page, lru); |
| 137 | list_del(&page->lru); /* remove from pagelist */ | 136 | /* move from pagelist to reserve */ |
| 138 | list_add_tail(&page->lru, &pl->free_list); /* add to reserve */ | 137 | list_move_tail(&page->lru, &pl->free_list); |
| 139 | ++pl->num_pages_free; | 138 | ++pl->num_pages_free; |
| 140 | } | 139 | } |
| 141 | pl->room = c->room; | 140 | pl->room = c->room; |