Re: [PATCH v3 3/4] mm/vmap: get rid of one single unlink_va() when merge

From: Roman Gushchin
Date: Tue May 28 2019 - 18:49:55 EST


On Mon, May 27, 2019 at 11:38:41AM +0200, Uladzislau Rezki (Sony) wrote:
> It does not make sense to try to "unlink" the node that is
> definitely not linked with a list nor tree. On the first
> merge step VA just points to the previously disconnected
> busy area.
>
> On the second step, check if the node has been merged and do
> "unlink" if so, because now it points to an object that must
> be linked.
>
> Signed-off-by: Uladzislau Rezki (Sony) <urezki@xxxxxxxxx>
> Acked-by: Hillf Danton <hdanton@xxxxxxxx>

Reviewed-by: Roman Gushchin <guro@xxxxxx>

Thanks!