Re: [PATCH] cgroup: remove extra cgroup_migrate_finish() call

From: Daniel Jordan
Date: Thu Apr 04 2019 - 14:30:36 EST


On Wed, Apr 03, 2019 at 04:03:54PM -0700, Shakeel Butt wrote:
> The callers of cgroup_migrate_prepare_dst() correctly call
> cgroup_migrate_finish() for success and failure cases both. No need to
> call it in cgroup_migrate_prepare_dst() in failure case.

Calling cgroup_migrate_finish multiple times is a no-op, so thankfully it
wasn't causing any problems. Looks good to me.

Reviewed-by: Daniel Jordan <daniel.m.jordan@xxxxxxxxxx>