From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752018AbdHVR5t (ORCPT ); Tue, 22 Aug 2017 13:57:49 -0400 Received: from mx2.suse.de ([195.135.220.15]:34530 "EHLO mx1.suse.de" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751716AbdHVR5s (ORCPT ); Tue, 22 Aug 2017 13:57:48 -0400 Date: Tue, 22 Aug 2017 19:57:46 +0200 Message-ID: From: Takashi Iwai To: "SF Markus Elfring" Cc: , "Julia Lawall" , "Jaroslav Kysela" , , "LKML" Subject: Re: [PATCH] ALSA: via82xx: Use common error handling code in snd_via82xx_create() In-Reply-To: <2332472c-61d2-8c21-bf46-5c80745d6380@users.sourceforge.net> References: <2332472c-61d2-8c21-bf46-5c80745d6380@users.sourceforge.net> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/25.2 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 22 Aug 2017 19:43:08 +0200, SF Markus Elfring wrote: > > From: Markus Elfring > Date: Tue, 22 Aug 2017 19:37:26 +0200 > > Add jump targets so that a bit of exception handling can be better reused > at the end of this function. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring > --- > sound/pci/via82xx_modem.c | 33 +++++++++++++++++++-------------- > 1 file changed, 19 insertions(+), 14 deletions(-) > > diff --git a/sound/pci/via82xx_modem.c b/sound/pci/via82xx_modem.c > index 55f79b2599e7..8659776887b2 100644 > --- a/sound/pci/via82xx_modem.c > +++ b/sound/pci/via82xx_modem.c > @@ -1110,8 +1110,8 @@ static int snd_via82xx_create(struct snd_card *card, > return err; > > if ((chip = kzalloc(sizeof(*chip), GFP_KERNEL)) == NULL) { > - pci_disable_device(pci); > - return -ENOMEM; > + err = -ENOMEM; > + goto disable_device; > } > > spin_lock_init(&chip->reg_lock); > @@ -1121,30 +1121,27 @@ static int snd_via82xx_create(struct snd_card *card, > > if ((err = pci_request_regions(pci, card->driver)) < 0) { > kfree(chip); > - pci_disable_device(pci); > - return err; > + goto disable_device; > } > chip->port = pci_resource_start(pci, 0); > if (request_irq(pci->irq, snd_via82xx_interrupt, IRQF_SHARED, > KBUILD_MODNAME, chip)) { > dev_err(card->dev, "unable to grab IRQ %d\n", pci->irq); > - snd_via82xx_free(chip); > - return -EBUSY; > + err = -EBUSY; > + goto free_chip; > } > chip->irq = pci->irq; > if (ac97_clock >= 8000 && ac97_clock <= 48000) > chip->ac97_clock = ac97_clock; > synchronize_irq(chip->irq); > > - if ((err = snd_via82xx_chip_init(chip)) < 0) { > - snd_via82xx_free(chip); > - return err; > - } > + err = snd_via82xx_chip_init(chip); > + if (err < 0) > + goto free_chip; > > - if ((err = snd_device_new(card, SNDRV_DEV_LOWLEVEL, chip, &ops)) < 0) { > - snd_via82xx_free(chip); > - return err; > - } > + err = snd_device_new(card, SNDRV_DEV_LOWLEVEL, chip, &ops); > + if (err < 0) > + goto free_chip; > > /* The 8233 ac97 controller does not implement the master bit > * in the pci command register. IMHO this is a violation of the PCI spec. > @@ -1153,6 +1150,14 @@ static int snd_via82xx_create(struct snd_card *card, > > *r_via = chip; > return 0; > + > +disable_device: > + pci_disable_device(pci); > + goto exit; > +free_chip: > + snd_via82xx_free(chip); > +exit: > + return err; Doubly goto doesn't look like an improvement. Takashi