Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1545137

[PATCH] xen/blkback: use rb_entry()

From Geliang Tang <geliangtang@gmail.com>
Newsgroups linux.kernel
Subject [PATCH] xen/blkback: use rb_entry()
Date 2016-12-20 15:10 +0100
Message-ID <sQth7-6Ck-3@gated-at.bofh.it> (permalink)
References <sQth7-6Ck-5@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


To make the code clearer, use rb_entry() instead of container_of() to
deal with rbtree.

Signed-off-by: Geliang Tang <geliangtang@gmail.com>
---
 drivers/block/xen-blkback/blkback.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/block/xen-blkback/blkback.c b/drivers/block/xen-blkback/blkback.c
index 726c32e..7e59fae 100644
--- a/drivers/block/xen-blkback/blkback.c
+++ b/drivers/block/xen-blkback/blkback.c
@@ -191,10 +191,10 @@ static void make_response(struct xen_blkif_ring *ring, u64 id,
 			  unsigned short op, int st);
 
 #define foreach_grant_safe(pos, n, rbtree, node) \
-	for ((pos) = container_of(rb_first((rbtree)), typeof(*(pos)), node), \
+	for ((pos) = rb_entry(rb_first((rbtree)), typeof(*(pos)), node), \
 	     (n) = (&(pos)->node != NULL) ? rb_next(&(pos)->node) : NULL; \
 	     &(pos)->node != NULL; \
-	     (pos) = container_of(n, typeof(*(pos)), node), \
+	     (pos) = rb_entry(n, typeof(*(pos)), node), \
 	     (n) = (&(pos)->node != NULL) ? rb_next(&(pos)->node) : NULL)
 
 
@@ -223,7 +223,7 @@ static int add_persistent_gnt(struct xen_blkif_ring *ring,
 	/* Figure out where to put new node */
 	new = &ring->persistent_gnts.rb_node;
 	while (*new) {
-		this = container_of(*new, struct persistent_gnt, node);
+		this = rb_entry(*new, struct persistent_gnt, node);
 
 		parent = *new;
 		if (persistent_gnt->gnt < this->gnt)
@@ -254,7 +254,7 @@ static struct persistent_gnt *get_persistent_gnt(struct xen_blkif_ring *ring,
 
 	node = ring->persistent_gnts.rb_node;
 	while (node) {
-		data = container_of(node, struct persistent_gnt, node);
+		data = rb_entry(node, struct persistent_gnt, node);
 
 		if (gref < data->gnt)
 			node = node->rb_left;
-- 
2.9.3

Back to linux.kernel | Previous | NextNext in thread | Find similar | Unroll thread


Thread

[PATCH] xen/blkback: use rb_entry() Geliang Tang <geliangtang@gmail.com> - 2016-12-20 15:10 +0100
  Re: [PATCH] xen/blkback: use rb_entry() Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> - 2016-12-20 17:50 +0100
    Re: [PATCH] xen/blkback: use rb_entry() Roger Pau Monné <roger.pau@citrix.com> - 2016-12-20 18:50 +0100
      Re: [PATCH] xen/blkback: use rb_entry() Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> - 2016-12-20 19:00 +0100
        Re: [PATCH] xen/blkback: use rb_entry() Eric Dumazet <eric.dumazet@gmail.com> - 2016-12-20 23:00 +0100
          Re: [PATCH] xen/blkback: use rb_entry() Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> - 2016-12-20 23:10 +0100

csiph-web