Re: [PATCH] CodingStyle: add some more error handling guidelines

From: Dan Carpenter
Date: Mon Aug 22 2016 - 14:50:25 EST


On Mon, Aug 22, 2016 at 05:53:02PM +0300, Michael S. Tsirkin wrote:
> The point is really naming label for the part of init that failed
> (and so needs to be skipped), rather than the part that will run.

Naming labels after what "needs to be skipped" doesn't work. How does
that meaning make sense for err_cgroup in vhost_dev_set_owner()? What
needs to be skipped here?

regards,
dan carpenter