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

Roman Gushchin
2019-05-28T22:46:33Z

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@gmail.com>
> Acked-by: Hillf Danton <hdanton@sina.com>

Reviewed-by: Roman Gushchin <guro@fb.com>

Thanks!