From: Dan Carpenter <dan.carpenter@oracle.com>
To: kbuild@lists.01.org, zuoqilin1@163.com, tiwai@suse.com,
lars@metafoo.de, joe@perches.com, broonie@kernel.org,
perex@perex.cz
Cc: zuoqilin <zuoqilin@yulong.com>,
alsa-devel@alsa-project.org, kbuild-all@lists.01.org,
lkp@intel.com
Subject: [kbuild] Re: [PATCH] sound: Remove redundant steps
Date: Wed, 10 Mar 2021 21:33:13 +0300 [thread overview]
Message-ID: <20210310183312.GK21246@kadam> (raw)
In-Reply-To: <20210310063426.524-1-zuoqilin1@163.com>
[-- Attachment #1: Type: text/plain, Size: 5070 bytes --]
Hi,
url: https://github.com/0day-ci/linux/commits/zuoqilin1-163-com/sound-Remove-redundant-steps/20210310-183451
base: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next
config: x86_64-randconfig-s022-20210309 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce:
# apt-get install sparse
# sparse version: v0.6.3-262-g5e674421-dirty
# https://github.com/0day-ci/linux/commit/7477aca62fd7115a8e7034e78fd7b5d427deb74b
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review zuoqilin1-163-com/sound-Remove-redundant-steps/20210310-183451
git checkout 7477aca62fd7115a8e7034e78fd7b5d427deb74b
# save the attached .config to linux build tree
make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
"sparse warnings: (new ones prefixed by >>)"
>> sound/core/oss/mixer_oss.c:832:7: sparse: sparse: unused label '__free_only'
sound/core/oss/mixer_oss.c:886:7: sparse: sparse: unused label '__free_only'
vim +/__free_only +832 sound/core/oss/mixer_oss.c
f956b4a3ae790e1 Takashi Iwai 2005-11-17 789 static int snd_mixer_oss_get_recsrc2(struct snd_mixer_oss_file *fmixer, unsigned int *active_index)
^1da177e4c3f415 Linus Torvalds 2005-04-16 790 {
f956b4a3ae790e1 Takashi Iwai 2005-11-17 791 struct snd_card *card = fmixer->card;
f956b4a3ae790e1 Takashi Iwai 2005-11-17 792 struct snd_mixer_oss *mixer = fmixer->mixer;
f956b4a3ae790e1 Takashi Iwai 2005-11-17 793 struct snd_kcontrol *kctl;
f956b4a3ae790e1 Takashi Iwai 2005-11-17 794 struct snd_mixer_oss_slot *pslot;
^1da177e4c3f415 Linus Torvalds 2005-04-16 795 struct slot *slot;
f956b4a3ae790e1 Takashi Iwai 2005-11-17 796 struct snd_ctl_elem_info *uinfo;
f956b4a3ae790e1 Takashi Iwai 2005-11-17 797 struct snd_ctl_elem_value *uctl;
^1da177e4c3f415 Linus Torvalds 2005-04-16 798 int err, idx;
^1da177e4c3f415 Linus Torvalds 2005-04-16 799
ca2c0966562cfbf Takashi Iwai 2005-09-09 800 uinfo = kzalloc(sizeof(*uinfo), GFP_KERNEL);
ca2c0966562cfbf Takashi Iwai 2005-09-09 801 uctl = kzalloc(sizeof(*uctl), GFP_KERNEL);
^1da177e4c3f415 Linus Torvalds 2005-04-16 802 if (uinfo == NULL || uctl == NULL) {
7477aca62fd7115 zuoqilin 2021-03-10 803 return -ENOMEM;
^1da177e4c3f415 Linus Torvalds 2005-04-16 804 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 805 down_read(&card->controls_rwsem);
^1da177e4c3f415 Linus Torvalds 2005-04-16 806 kctl = snd_mixer_oss_test_id(mixer, "Capture Source", 0);
7c22f1aaa23370b Takashi Iwai 2005-10-10 807 if (! kctl) {
7c22f1aaa23370b Takashi Iwai 2005-10-10 808 err = -ENOENT;
7c22f1aaa23370b Takashi Iwai 2005-10-10 809 goto __unlock;
7c22f1aaa23370b Takashi Iwai 2005-10-10 810 }
7c22f1aaa23370b Takashi Iwai 2005-10-10 811 if ((err = kctl->info(kctl, uinfo)) < 0)
7c22f1aaa23370b Takashi Iwai 2005-10-10 812 goto __unlock;
7c22f1aaa23370b Takashi Iwai 2005-10-10 813 if ((err = kctl->get(kctl, uctl)) < 0)
7c22f1aaa23370b Takashi Iwai 2005-10-10 814 goto __unlock;
^1da177e4c3f415 Linus Torvalds 2005-04-16 815 for (idx = 0; idx < 32; idx++) {
^1da177e4c3f415 Linus Torvalds 2005-04-16 816 if (!(mixer->mask_recsrc & (1 << idx)))
^1da177e4c3f415 Linus Torvalds 2005-04-16 817 continue;
^1da177e4c3f415 Linus Torvalds 2005-04-16 818 pslot = &mixer->slots[idx];
9fe856e47e17512 Joe Perches 2010-09-04 819 slot = pslot->private_data;
^1da177e4c3f415 Linus Torvalds 2005-04-16 820 if (slot->signature != SNDRV_MIXER_OSS_SIGNATURE)
^1da177e4c3f415 Linus Torvalds 2005-04-16 821 continue;
^1da177e4c3f415 Linus Torvalds 2005-04-16 822 if (!(slot->present & SNDRV_MIXER_OSS_PRESENT_CAPTURE))
^1da177e4c3f415 Linus Torvalds 2005-04-16 823 continue;
^1da177e4c3f415 Linus Torvalds 2005-04-16 824 if (slot->capture_item == uctl->value.enumerated.item[0]) {
^1da177e4c3f415 Linus Torvalds 2005-04-16 825 *active_index = idx;
^1da177e4c3f415 Linus Torvalds 2005-04-16 826 break;
^1da177e4c3f415 Linus Torvalds 2005-04-16 827 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 828 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 829 err = 0;
^1da177e4c3f415 Linus Torvalds 2005-04-16 830 __unlock:
^1da177e4c3f415 Linus Torvalds 2005-04-16 831 up_read(&card->controls_rwsem);
838c364ff05c143 Jaroslav Kysela 2010-10-08 @832 __free_only:
^^^^^^^^^^^
Unused
^1da177e4c3f415 Linus Torvalds 2005-04-16 833 kfree(uctl);
^1da177e4c3f415 Linus Torvalds 2005-04-16 834 kfree(uinfo);
^1da177e4c3f415 Linus Torvalds 2005-04-16 835 return err;
^1da177e4c3f415 Linus Torvalds 2005-04-16 836 }
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 37046 bytes --]
[-- Attachment #3: Type: text/plain, Size: 149 bytes --]
_______________________________________________
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-leave@lists.01.org
WARNING: multiple messages have this Message-ID (diff)
From: Dan Carpenter <dan.carpenter@oracle.com>
To: kbuild@lists.01.org
Subject: Re: [PATCH] sound: Remove redundant steps
Date: Wed, 10 Mar 2021 21:33:13 +0300 [thread overview]
Message-ID: <20210310183312.GK21246@kadam> (raw)
In-Reply-To: <20210310063426.524-1-zuoqilin1@163.com>
[-- Attachment #1: Type: text/plain, Size: 5310 bytes --]
Hi,
url: https://github.com/0day-ci/linux/commits/zuoqilin1-163-com/sound-Remove-redundant-steps/20210310-183451
base: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next
config: x86_64-randconfig-s022-20210309 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce:
# apt-get install sparse
# sparse version: v0.6.3-262-g5e674421-dirty
# https://github.com/0day-ci/linux/commit/7477aca62fd7115a8e7034e78fd7b5d427deb74b
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review zuoqilin1-163-com/sound-Remove-redundant-steps/20210310-183451
git checkout 7477aca62fd7115a8e7034e78fd7b5d427deb74b
# save the attached .config to linux build tree
make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
"sparse warnings: (new ones prefixed by >>)"
>> sound/core/oss/mixer_oss.c:832:7: sparse: sparse: unused label '__free_only'
sound/core/oss/mixer_oss.c:886:7: sparse: sparse: unused label '__free_only'
vim +/__free_only +832 sound/core/oss/mixer_oss.c
f956b4a3ae790e1 Takashi Iwai 2005-11-17 789 static int snd_mixer_oss_get_recsrc2(struct snd_mixer_oss_file *fmixer, unsigned int *active_index)
^1da177e4c3f415 Linus Torvalds 2005-04-16 790 {
f956b4a3ae790e1 Takashi Iwai 2005-11-17 791 struct snd_card *card = fmixer->card;
f956b4a3ae790e1 Takashi Iwai 2005-11-17 792 struct snd_mixer_oss *mixer = fmixer->mixer;
f956b4a3ae790e1 Takashi Iwai 2005-11-17 793 struct snd_kcontrol *kctl;
f956b4a3ae790e1 Takashi Iwai 2005-11-17 794 struct snd_mixer_oss_slot *pslot;
^1da177e4c3f415 Linus Torvalds 2005-04-16 795 struct slot *slot;
f956b4a3ae790e1 Takashi Iwai 2005-11-17 796 struct snd_ctl_elem_info *uinfo;
f956b4a3ae790e1 Takashi Iwai 2005-11-17 797 struct snd_ctl_elem_value *uctl;
^1da177e4c3f415 Linus Torvalds 2005-04-16 798 int err, idx;
^1da177e4c3f415 Linus Torvalds 2005-04-16 799
ca2c0966562cfbf Takashi Iwai 2005-09-09 800 uinfo = kzalloc(sizeof(*uinfo), GFP_KERNEL);
ca2c0966562cfbf Takashi Iwai 2005-09-09 801 uctl = kzalloc(sizeof(*uctl), GFP_KERNEL);
^1da177e4c3f415 Linus Torvalds 2005-04-16 802 if (uinfo == NULL || uctl == NULL) {
7477aca62fd7115 zuoqilin 2021-03-10 803 return -ENOMEM;
^1da177e4c3f415 Linus Torvalds 2005-04-16 804 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 805 down_read(&card->controls_rwsem);
^1da177e4c3f415 Linus Torvalds 2005-04-16 806 kctl = snd_mixer_oss_test_id(mixer, "Capture Source", 0);
7c22f1aaa23370b Takashi Iwai 2005-10-10 807 if (! kctl) {
7c22f1aaa23370b Takashi Iwai 2005-10-10 808 err = -ENOENT;
7c22f1aaa23370b Takashi Iwai 2005-10-10 809 goto __unlock;
7c22f1aaa23370b Takashi Iwai 2005-10-10 810 }
7c22f1aaa23370b Takashi Iwai 2005-10-10 811 if ((err = kctl->info(kctl, uinfo)) < 0)
7c22f1aaa23370b Takashi Iwai 2005-10-10 812 goto __unlock;
7c22f1aaa23370b Takashi Iwai 2005-10-10 813 if ((err = kctl->get(kctl, uctl)) < 0)
7c22f1aaa23370b Takashi Iwai 2005-10-10 814 goto __unlock;
^1da177e4c3f415 Linus Torvalds 2005-04-16 815 for (idx = 0; idx < 32; idx++) {
^1da177e4c3f415 Linus Torvalds 2005-04-16 816 if (!(mixer->mask_recsrc & (1 << idx)))
^1da177e4c3f415 Linus Torvalds 2005-04-16 817 continue;
^1da177e4c3f415 Linus Torvalds 2005-04-16 818 pslot = &mixer->slots[idx];
9fe856e47e17512 Joe Perches 2010-09-04 819 slot = pslot->private_data;
^1da177e4c3f415 Linus Torvalds 2005-04-16 820 if (slot->signature != SNDRV_MIXER_OSS_SIGNATURE)
^1da177e4c3f415 Linus Torvalds 2005-04-16 821 continue;
^1da177e4c3f415 Linus Torvalds 2005-04-16 822 if (!(slot->present & SNDRV_MIXER_OSS_PRESENT_CAPTURE))
^1da177e4c3f415 Linus Torvalds 2005-04-16 823 continue;
^1da177e4c3f415 Linus Torvalds 2005-04-16 824 if (slot->capture_item == uctl->value.enumerated.item[0]) {
^1da177e4c3f415 Linus Torvalds 2005-04-16 825 *active_index = idx;
^1da177e4c3f415 Linus Torvalds 2005-04-16 826 break;
^1da177e4c3f415 Linus Torvalds 2005-04-16 827 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 828 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 829 err = 0;
^1da177e4c3f415 Linus Torvalds 2005-04-16 830 __unlock:
^1da177e4c3f415 Linus Torvalds 2005-04-16 831 up_read(&card->controls_rwsem);
838c364ff05c143 Jaroslav Kysela 2010-10-08 @832 __free_only:
^^^^^^^^^^^
Unused
^1da177e4c3f415 Linus Torvalds 2005-04-16 833 kfree(uctl);
^1da177e4c3f415 Linus Torvalds 2005-04-16 834 kfree(uinfo);
^1da177e4c3f415 Linus Torvalds 2005-04-16 835 return err;
^1da177e4c3f415 Linus Torvalds 2005-04-16 836 }
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
_______________________________________________
kbuild mailing list -- kbuild(a)lists.01.org
To unsubscribe send an email to kbuild-leave(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 37046 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Dan Carpenter <dan.carpenter@oracle.com>
To: kbuild-all@lists.01.org
Subject: [kbuild] Re: [PATCH] sound: Remove redundant steps
Date: Wed, 10 Mar 2021 21:33:13 +0300 [thread overview]
Message-ID: <20210310183312.GK21246@kadam> (raw)
In-Reply-To: <20210310063426.524-1-zuoqilin1@163.com>
[-- Attachment #1: Type: text/plain, Size: 5310 bytes --]
Hi,
url: https://github.com/0day-ci/linux/commits/zuoqilin1-163-com/sound-Remove-redundant-steps/20210310-183451
base: https://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git for-next
config: x86_64-randconfig-s022-20210309 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce:
# apt-get install sparse
# sparse version: v0.6.3-262-g5e674421-dirty
# https://github.com/0day-ci/linux/commit/7477aca62fd7115a8e7034e78fd7b5d427deb74b
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review zuoqilin1-163-com/sound-Remove-redundant-steps/20210310-183451
git checkout 7477aca62fd7115a8e7034e78fd7b5d427deb74b
# save the attached .config to linux build tree
make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
"sparse warnings: (new ones prefixed by >>)"
>> sound/core/oss/mixer_oss.c:832:7: sparse: sparse: unused label '__free_only'
sound/core/oss/mixer_oss.c:886:7: sparse: sparse: unused label '__free_only'
vim +/__free_only +832 sound/core/oss/mixer_oss.c
f956b4a3ae790e1 Takashi Iwai 2005-11-17 789 static int snd_mixer_oss_get_recsrc2(struct snd_mixer_oss_file *fmixer, unsigned int *active_index)
^1da177e4c3f415 Linus Torvalds 2005-04-16 790 {
f956b4a3ae790e1 Takashi Iwai 2005-11-17 791 struct snd_card *card = fmixer->card;
f956b4a3ae790e1 Takashi Iwai 2005-11-17 792 struct snd_mixer_oss *mixer = fmixer->mixer;
f956b4a3ae790e1 Takashi Iwai 2005-11-17 793 struct snd_kcontrol *kctl;
f956b4a3ae790e1 Takashi Iwai 2005-11-17 794 struct snd_mixer_oss_slot *pslot;
^1da177e4c3f415 Linus Torvalds 2005-04-16 795 struct slot *slot;
f956b4a3ae790e1 Takashi Iwai 2005-11-17 796 struct snd_ctl_elem_info *uinfo;
f956b4a3ae790e1 Takashi Iwai 2005-11-17 797 struct snd_ctl_elem_value *uctl;
^1da177e4c3f415 Linus Torvalds 2005-04-16 798 int err, idx;
^1da177e4c3f415 Linus Torvalds 2005-04-16 799
ca2c0966562cfbf Takashi Iwai 2005-09-09 800 uinfo = kzalloc(sizeof(*uinfo), GFP_KERNEL);
ca2c0966562cfbf Takashi Iwai 2005-09-09 801 uctl = kzalloc(sizeof(*uctl), GFP_KERNEL);
^1da177e4c3f415 Linus Torvalds 2005-04-16 802 if (uinfo == NULL || uctl == NULL) {
7477aca62fd7115 zuoqilin 2021-03-10 803 return -ENOMEM;
^1da177e4c3f415 Linus Torvalds 2005-04-16 804 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 805 down_read(&card->controls_rwsem);
^1da177e4c3f415 Linus Torvalds 2005-04-16 806 kctl = snd_mixer_oss_test_id(mixer, "Capture Source", 0);
7c22f1aaa23370b Takashi Iwai 2005-10-10 807 if (! kctl) {
7c22f1aaa23370b Takashi Iwai 2005-10-10 808 err = -ENOENT;
7c22f1aaa23370b Takashi Iwai 2005-10-10 809 goto __unlock;
7c22f1aaa23370b Takashi Iwai 2005-10-10 810 }
7c22f1aaa23370b Takashi Iwai 2005-10-10 811 if ((err = kctl->info(kctl, uinfo)) < 0)
7c22f1aaa23370b Takashi Iwai 2005-10-10 812 goto __unlock;
7c22f1aaa23370b Takashi Iwai 2005-10-10 813 if ((err = kctl->get(kctl, uctl)) < 0)
7c22f1aaa23370b Takashi Iwai 2005-10-10 814 goto __unlock;
^1da177e4c3f415 Linus Torvalds 2005-04-16 815 for (idx = 0; idx < 32; idx++) {
^1da177e4c3f415 Linus Torvalds 2005-04-16 816 if (!(mixer->mask_recsrc & (1 << idx)))
^1da177e4c3f415 Linus Torvalds 2005-04-16 817 continue;
^1da177e4c3f415 Linus Torvalds 2005-04-16 818 pslot = &mixer->slots[idx];
9fe856e47e17512 Joe Perches 2010-09-04 819 slot = pslot->private_data;
^1da177e4c3f415 Linus Torvalds 2005-04-16 820 if (slot->signature != SNDRV_MIXER_OSS_SIGNATURE)
^1da177e4c3f415 Linus Torvalds 2005-04-16 821 continue;
^1da177e4c3f415 Linus Torvalds 2005-04-16 822 if (!(slot->present & SNDRV_MIXER_OSS_PRESENT_CAPTURE))
^1da177e4c3f415 Linus Torvalds 2005-04-16 823 continue;
^1da177e4c3f415 Linus Torvalds 2005-04-16 824 if (slot->capture_item == uctl->value.enumerated.item[0]) {
^1da177e4c3f415 Linus Torvalds 2005-04-16 825 *active_index = idx;
^1da177e4c3f415 Linus Torvalds 2005-04-16 826 break;
^1da177e4c3f415 Linus Torvalds 2005-04-16 827 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 828 }
^1da177e4c3f415 Linus Torvalds 2005-04-16 829 err = 0;
^1da177e4c3f415 Linus Torvalds 2005-04-16 830 __unlock:
^1da177e4c3f415 Linus Torvalds 2005-04-16 831 up_read(&card->controls_rwsem);
838c364ff05c143 Jaroslav Kysela 2010-10-08 @832 __free_only:
^^^^^^^^^^^
Unused
^1da177e4c3f415 Linus Torvalds 2005-04-16 833 kfree(uctl);
^1da177e4c3f415 Linus Torvalds 2005-04-16 834 kfree(uinfo);
^1da177e4c3f415 Linus Torvalds 2005-04-16 835 return err;
^1da177e4c3f415 Linus Torvalds 2005-04-16 836 }
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
_______________________________________________
kbuild mailing list -- kbuild(a)lists.01.org
To unsubscribe send an email to kbuild-leave(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 37046 bytes --]
next prev parent reply other threads:[~2021-03-10 18:34 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-03-10 6:34 [PATCH] sound: Remove redundant steps zuoqilin1
2021-03-10 9:09 ` Takashi Iwai
2021-03-10 13:19 ` kernel test robot
2021-03-10 13:19 ` kernel test robot
2021-03-10 18:33 ` Dan Carpenter [this message]
2021-03-10 18:33 ` [kbuild] " Dan Carpenter
2021-03-10 18:33 ` Dan Carpenter
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=20210310183312.GK21246@kadam \
--to=dan.carpenter@oracle.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=joe@perches.com \
--cc=kbuild-all@lists.01.org \
--cc=kbuild@lists.01.org \
--cc=lars@metafoo.de \
--cc=lkp@intel.com \
--cc=perex@perex.cz \
--cc=tiwai@suse.com \
--cc=zuoqilin1@163.com \
--cc=zuoqilin@yulong.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.