From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754199AbcHVSuW (ORCPT ); Mon, 22 Aug 2016 14:50:22 -0400 Received: from aserp1040.oracle.com ([141.146.126.69]:51508 "EHLO aserp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751573AbcHVSuU (ORCPT ); Mon, 22 Aug 2016 14:50:20 -0400 Date: Mon, 22 Aug 2016 21:50:06 +0300 From: Dan Carpenter To: "Michael S. Tsirkin" Cc: Jonathan Corbet , linux-kernel@vger.kernel.org, Julia Lawall , Jason Wang , linux-doc@vger.kernel.org, virtualization@lists.linux-foundation.org Subject: Re: [PATCH] CodingStyle: add some more error handling guidelines Message-ID: <20160822185006.GF4129@mwanda> References: <1471874251-7721-1-git-send-email-mst@redhat.com> <20160822081617.386db8cd@lwn.net> <20160822174355-mutt-send-email-mst@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20160822174355-mutt-send-email-mst@kernel.org> User-Agent: Mutt/1.5.21 (2010-09-15) X-Source-IP: aserv0022.oracle.com [141.146.126.234] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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