On Tue 20-12-16 21:57:43, Geliang Tang wrote: > Use rb_entry_safe() instead of open-coding it. > > Signed-off-by: Geliang Tang <geliangtang@xxxxxxxxx> Acked-by: Michal Hocko <mhocko@xxxxxxxx> > --- > mm/vmalloc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mm/vmalloc.c b/mm/vmalloc.c > index a558438..b9999fc 100644 > --- a/mm/vmalloc.c > +++ b/mm/vmalloc.c > @@ -2309,7 +2309,7 @@ EXPORT_SYMBOL_GPL(free_vm_area); > #ifdef CONFIG_SMP > static struct vmap_area *node_to_va(struct rb_node *n) > { > - return n ? rb_entry(n, struct vmap_area, rb_node) : NULL; > + return rb_entry_safe(n, struct vmap_area, rb_node); > } > > /** > -- > 2.9.3 > -- Michal Hocko SUSE Labs -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>