Re: [Xen-devel] [Resend][PATCH 06/17] rbtree: break out of rb_insert_color loop after tree rotation

[Dario Faggioli <dario.faggioli@xxxxxxxxxx>]

On Thu, 2017-06-01 at 02:50 +0530, Praveen Kumar wrote:
> --- a/xen/common/rbtree.c
> +++ b/xen/common/rbtree.c
> @@ -110,11 +110,8 @@ void rb_insert_color(struct rb_node *node,
> struct rb_root *root)
>  
>              if (parent->rb_right == node)
>              {
> -                register struct rb_node *tmp;
>                  __rb_rotate_left(parent, root);
> -                tmp = parent;
>                  parent = node;
> -                node = tmp;
>              }
>  
>              rb_set_black(parent);
> @@ -135,11 +132,8 @@ void rb_insert_color(struct rb_node *node,
> struct rb_root *root)
>  
>              if (parent->rb_left == node)
>              {
> -                register struct rb_node *tmp;
>                  __rb_rotate_right(parent, root);
> -                tmp = parent;
>                  parent = node;
> -                node = tmp;
>              }
>  
>              rb_set_black(parent);
>
It looks to me like something went wrong while you applied/prepared the
patch.

In fact, I can see the removed lines are the same, between the patch
itself and the original Linux comment, but you are not adding any code
(while the original Linux commit does!)

Regards,
Dario
-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel