From: Dan Carpenter <dan.carpenter@oracle.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: kbuild@lists.01.org, lkp@intel.com, kbuild-all@lists.01.org,
Linux Memory Management List <linux-mm@kvack.org>
Subject: Re: [linux-next:master 1517/2389] sound/pci/als4000.c:930 snd_card_als4000_probe() warn: 'pci' not released on lines: 852,872.
Date: Mon, 26 Jul 2021 15:45:15 +0300 [thread overview]
Message-ID: <20210726124515.GJ25548@kadam> (raw)
In-Reply-To: <s5hr1fqjp88.wl-tiwai@suse.de>
On Thu, Jul 22, 2021 at 06:19:19PM +0200, Takashi Iwai wrote:
> > ^1da177e4c3f41 Linus Torvalds 2005-04-16 827
> > ^1da177e4c3f41 Linus Torvalds 2005-04-16 828 /* enable PCI device */
> > 0e175f665960c7 Takashi Iwai 2021-07-15 829 err = pcim_enable_device(pci);
> > 61ea06e8a88163 Takashi Iwai 2021-06-08 830 if (err < 0)
> > ^1da177e4c3f41 Linus Torvalds 2005-04-16 831 return err;
> > 61ea06e8a88163 Takashi Iwai 2021-06-08 832
> > ^1da177e4c3f41 Linus Torvalds 2005-04-16 833 /* check, if we can restrict PCI DMA transfers to 24 bits */
> > 669f65eaeb969e Takashi Iwai 2021-01-14 834 if (dma_set_mask_and_coherent(&pci->dev, DMA_BIT_MASK(24))) {
> > d85d878efb3047 Takashi Iwai 2014-02-25 835 dev_err(&pci->dev, "architecture does not support 24bit PCI busmaster DMA\n");
> > ^1da177e4c3f41 Linus Torvalds 2005-04-16 836 return -ENXIO;
> > ^1da177e4c3f41 Linus Torvalds 2005-04-16 837 }
> > ^1da177e4c3f41 Linus Torvalds 2005-04-16 838
> > 61ea06e8a88163 Takashi Iwai 2021-06-08 839 err = pci_request_regions(pci, "ALS4000");
> > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> > 0e175f665960c7 Takashi Iwai 2021-07-15 840 if (err < 0)
> > ^1da177e4c3f41 Linus Torvalds 2005-04-16 841 return err;
> > c0874449849175 Andreas Mohr 2008-08-20 842 iobase = pci_resource_start(pci, 0);
> > ^1da177e4c3f41 Linus Torvalds 2005-04-16 843
> > ^1da177e4c3f41 Linus Torvalds 2005-04-16 844 pci_read_config_word(pci, PCI_COMMAND, &word);
> > ^1da177e4c3f41 Linus Torvalds 2005-04-16 845 pci_write_config_word(pci, PCI_COMMAND, word | PCI_COMMAND_IO);
> > ^1da177e4c3f41 Linus Torvalds 2005-04-16 846 pci_set_master(pci);
> > ^1da177e4c3f41 Linus Torvalds 2005-04-16 847
> > 0e175f665960c7 Takashi Iwai 2021-07-15 848 err = snd_devm_card_new(&pci->dev, index[dev], id[dev], THIS_MODULE,
> > e58de7baf7de11 Takashi Iwai 2008-12-28 849 sizeof(*acard) /* private_data: acard */,
> > e58de7baf7de11 Takashi Iwai 2008-12-28 850 &card);
> > 0e175f665960c7 Takashi Iwai 2021-07-15 851 if (err < 0)
> > e58de7baf7de11 Takashi Iwai 2008-12-28 852 return err;
> > ^^^^^^^^^^
> > Smatch complains because there is no error handling. Maybe the other
> > error paths use devm_ magic? Or more likely Smatch just isn't smart
> > enough...
>
> I believe those are false-positive.
>
> When the PCI device is already managed via pcim_enable_device(), all
> resources (even the ones without pcim_ or devm_) are freed
> automagically.
Thanks! I've marked everything after pcim_enable_device or
pci_enable_device() as managed (will push later).
regards,
dan carpenter
prev parent reply other threads:[~2021-07-26 12:45 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-22 9:28 [linux-next:master 1517/2389] sound/pci/als4000.c:930 snd_card_als4000_probe() warn: 'pci' not released on lines: 852,872 Dan Carpenter
2021-07-22 16:19 ` Takashi Iwai
2021-07-26 12:45 ` Dan Carpenter [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20210726124515.GJ25548@kadam \
--to=dan.carpenter@oracle.com \
--cc=kbuild-all@lists.01.org \
--cc=kbuild@lists.01.org \
--cc=linux-mm@kvack.org \
--cc=lkp@intel.com \
--cc=tiwai@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).