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>