tree:
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master
head: c8e31a0fc39797784e4b9acae3bd2e0c7bade36e
commit: 46b770f720bdd8a7de1c04a1cab5d4e9e21d6666 [11494/12088] ALSA: uapi: Fix sparse
warning
reproduce:
# apt-get install sparse
# sparse version: v0.6.1-154-g1dc00f87-dirty
git checkout 46b770f720bdd8a7de1c04a1cab5d4e9e21d6666
make ARCH=x86_64 allmodconfig
make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp(a)intel.com>
sparse warnings: (new ones prefixed by >>)
sound/core/pcm_native.c:558:51: sparse: sparse: incorrect type in assignment (different
base types)
sound/core/pcm_native.c:558:51: sparse: expected restricted snd_pcm_state_t
[usertype] state
sound/core/pcm_native.c:558:51: sparse: got int state
sound/core/pcm_native.c:748:38: sparse: sparse: incorrect type in argument 2 (different
base types)
sound/core/pcm_native.c:748:38: sparse: expected int state
sound/core/pcm_native.c:748:38: sparse: got restricted snd_pcm_state_t [usertype]
sound/core/pcm_native.c:760:38: sparse: sparse: incorrect type in argument 2 (different
base types)
sound/core/pcm_native.c:760:38: sparse: expected int state
sound/core/pcm_native.c:760:38: sparse: got restricted snd_pcm_state_t [usertype]
sound/core/pcm_native.c:814:38: sparse: sparse: incorrect type in argument 2 (different
base types)
sound/core/pcm_native.c:814:38: sparse: expected int state
sound/core/pcm_native.c:814:38: sparse: got restricted snd_pcm_state_t [usertype]
sound/core/pcm_native.c:1022:34: sparse: sparse: incorrect type in initializer
(different base types)
sound/core/pcm_native.c:1022:34: sparse: expected signed int [usertype] state
sound/core/pcm_native.c:1022:34: sparse: got restricted snd_pcm_state_t
[addressable] [assigned] [usertype] state
sound/core/pcm_native.c:1033:44: sparse: sparse: incorrect type in initializer
(different base types)
sound/core/pcm_native.c:1033:44: sparse: expected signed int [usertype]
suspended_state
sound/core/pcm_native.c:1033:44: sparse: got restricted snd_pcm_state_t
[addressable] [assigned] [usertype] suspended_state
sound/core/pcm_native.c:1335:32: sparse: sparse: incorrect type in assignment
(different base types)
sound/core/pcm_native.c:1335:32: sparse: expected restricted snd_pcm_state_t
[usertype] state
sound/core/pcm_native.c:1335:32: sparse: got int state
sound/core/pcm_native.c:1359:31: sparse: sparse: incorrect type in argument 3
(different base types)
sound/core/pcm_native.c:1359:31: sparse: expected int state
sound/core/pcm_native.c:1359:31: sparse: got restricted snd_pcm_state_t [usertype]
sound/core/pcm_native.c:1366:40: sparse: sparse: incorrect type in argument 3
(different base types)
sound/core/pcm_native.c:1366:40: sparse: expected int state
sound/core/pcm_native.c:1366:40: sparse: got restricted snd_pcm_state_t [usertype]
sound/core/pcm_native.c:1392:28: sparse: sparse: restricted snd_pcm_state_t degrades to
integer
sound/core/pcm_native.c:1394:40: sparse: sparse: incorrect type in assignment
(different base types)
sound/core/pcm_native.c:1394:40: sparse: expected restricted snd_pcm_state_t
[usertype] state
sound/core/pcm_native.c:1394:40: sparse: got int state
sound/core/pcm_native.c:1419:64: sparse: sparse: incorrect type in argument 3
(different base types)
sound/core/pcm_native.c:1419:64: sparse: expected int state
sound/core/pcm_native.c:1419:64: sparse: got restricted snd_pcm_state_t [usertype]
state
sound/core/pcm_native.c:1435:38: sparse: sparse: incorrect type in argument 3
(different base types)
sound/core/pcm_native.c:1435:38: sparse: expected int state
sound/core/pcm_native.c:1435:38: sparse: got restricted snd_pcm_state_t [usertype]
sound/core/pcm_native.c:1806:38: sparse: sparse: incorrect type in argument 2
(different base types)
sound/core/pcm_native.c:1806:38: sparse: expected int state
sound/core/pcm_native.c:1806:38: sparse: got restricted snd_pcm_state_t [usertype]
sound/core/pcm_native.c:1872:61: sparse: sparse: incorrect type in argument 2
(different base types)
sound/core/pcm_native.c:1872:61: sparse: expected int state
sound/core/pcm_native.c:1872:61: sparse: got restricted snd_pcm_state_t [usertype]
sound/core/pcm_native.c:1873:63: sparse: sparse: incorrect type in argument 2
(different base types)
sound/core/pcm_native.c:1873:63: sparse: expected int state
sound/core/pcm_native.c:1873:63: sparse: got restricted snd_pcm_state_t [usertype]
sound/core/pcm_native.c:1890:76: sparse: sparse: incorrect type in initializer
(different base types)
sound/core/pcm_native.c:1890:76: sparse: expected int new_state
sound/core/pcm_native.c:1890:76: sparse: got restricted snd_pcm_state_t
sound/core/pcm_native.c:2237:26: sparse: sparse: restricted snd_pcm_format_t degrades
to integer
sound/core/pcm_native.c:2241:54: sparse: sparse: incorrect type in argument 1
(different base types)
sound/core/pcm_native.c:2241:54: sparse: expected restricted snd_pcm_format_t
[usertype] format
sound/core/pcm_native.c:2241:54: sparse: got unsigned int [assigned] k
sound/core/pcm_native.c:2259:26: sparse: sparse: restricted snd_pcm_format_t degrades
to integer
sound/core/pcm_native.c:2263:54: sparse: sparse: incorrect type in argument 1
(different base types)
sound/core/pcm_native.c:2263:54: sparse: expected restricted snd_pcm_format_t
[usertype] format
sound/core/pcm_native.c:2263:54: sparse: got unsigned int [assigned] k
sound/core/pcm_native.c:2443:30: sparse: sparse: restricted snd_pcm_access_t degrades
to integer
sound/core/pcm_native.c:2445:30: sparse: sparse: restricted snd_pcm_access_t degrades
to integer
sound/core/pcm_native.c:2448:38: sparse: sparse: restricted snd_pcm_access_t degrades
to integer
sound/core/pcm_native.c:2450:38: sparse: sparse: restricted snd_pcm_access_t degrades
to integer
sound/core/pcm_native.c:2452:38: sparse: sparse: restricted snd_pcm_access_t degrades
to integer
sound/core/pcm_native.c:2462:86: sparse: sparse: restricted snd_pcm_subformat_t
degrades to integer
sound/core/pcm_native.c:2982:13: sparse: sparse: incorrect type in assignment
(different base types)
> sound/core/pcm_native.c:2982:13: sparse: expected signed int
__pu_val
> sound/core/pcm_native.c:2982:13: sparse: got restricted snd_pcm_state_t [assigned]
[usertype] state
sound/core/pcm_native.c:2986:13: sparse: sparse: incorrect type
in assignment (different base types)
sound/core/pcm_native.c:2986:13: sparse: expected signed int __pu_val
> sound/core/pcm_native.c:2986:13: sparse: got restricted
snd_pcm_state_t [assigned] [usertype] suspended_state
> sound/core/pcm_compat.c:207:32: sparse: sparse: incorrect type in initializer
(different base types)
> sound/core/pcm_compat.c:207:32: sparse: expected signed int [usertype] state
sound/core/pcm_compat.c:207:32: sparse: got restricted snd_pcm_state_t
[addressable] [assigned] [usertype] state
sound/core/pcm_compat.c:218:42: sparse: sparse: incorrect type in initializer
(different base types)
> sound/core/pcm_compat.c:218:42: sparse: expected signed int
[usertype] suspended_state
sound/core/pcm_compat.c:218:42: sparse: got
restricted snd_pcm_state_t [addressable] [assigned] [usertype] suspended_state
sound/core/pcm_compat.c:454:13: sparse: sparse: incorrect type in assignment (different
base types)
sound/core/pcm_compat.c:454:13: sparse: expected signed int __pu_val
sound/core/pcm_compat.c:454:13: sparse: got restricted snd_pcm_state_t [assigned]
[usertype] state
sound/core/pcm_compat.c:458:13: sparse: sparse: incorrect type in assignment (different
base types)
sound/core/pcm_compat.c:458:13: sparse: expected signed int __pu_val
> sound/core/pcm_compat.c:458:13: sparse: got restricted
snd_pcm_state_t [assigned] [usertype] suspended_state
sound/core/pcm_native.c:95:1: sparse: sparse: context imbalance in
'snd_pcm_group_lock' - different lock contexts for basic block
sound/core/pcm_native.c:96:1: sparse: sparse: context imbalance in
'snd_pcm_group_unlock' - unexpected unlock
sound/core/pcm_native.c:97:1: sparse: sparse: context imbalance in
'snd_pcm_group_lock_irq' - different lock contexts for basic block
sound/core/pcm_native.c:98:1: sparse: sparse: context imbalance in
'snd_pcm_group_unlock_irq' - unexpected unlock
sound/core/pcm_native.c:161:9: sparse: sparse: context imbalance in
'_snd_pcm_stream_lock_irqsave' - different lock contexts for basic block
sound/core/pcm_native.c:178:39: sparse: sparse: context imbalance in
'snd_pcm_stream_unlock_irqrestore' - unexpected unlock
sound/core/pcm_native.c:1157:52: sparse: sparse: context imbalance in
'snd_pcm_action_group' - unexpected unlock
sound/core/pcm_native.c:1228:37: sparse: sparse: context imbalance in
'snd_pcm_stream_group_ref' - different lock contexts for basic block
vim +2982 sound/core/pcm_native.c
09d94175dbeac1 Arnd Bergmann 2018-04-24 2936
09d94175dbeac1 Arnd Bergmann 2018-04-24 2937 static int
snd_pcm_ioctl_sync_ptr_compat(struct snd_pcm_substream *substream,
09d94175dbeac1 Arnd Bergmann 2018-04-24 2938 struct snd_pcm_sync_ptr32 __user
*src)
09d94175dbeac1 Arnd Bergmann 2018-04-24 2939 {
09d94175dbeac1 Arnd Bergmann 2018-04-24 2940 struct snd_pcm_runtime *runtime =
substream->runtime;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2941 volatile struct snd_pcm_mmap_status
*status;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2942 volatile struct snd_pcm_mmap_control
*control;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2943 u32 sflags;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2944 struct snd_pcm_mmap_control scontrol;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2945 struct snd_pcm_mmap_status sstatus;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2946 snd_pcm_uframes_t boundary;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2947 int err;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2948
09d94175dbeac1 Arnd Bergmann 2018-04-24 2949 if (snd_BUG_ON(!runtime))
09d94175dbeac1 Arnd Bergmann 2018-04-24 2950 return -EINVAL;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2951
09d94175dbeac1 Arnd Bergmann 2018-04-24 2952 if (get_user(sflags, &src->flags)
||
09d94175dbeac1 Arnd Bergmann 2018-04-24 2953 get_user(scontrol.appl_ptr,
&src->c.control.appl_ptr) ||
09d94175dbeac1 Arnd Bergmann 2018-04-24 2954 get_user(scontrol.avail_min,
&src->c.control.avail_min))
09d94175dbeac1 Arnd Bergmann 2018-04-24 2955 return -EFAULT;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2956 if (sflags &
SNDRV_PCM_SYNC_PTR_HWSYNC) {
09d94175dbeac1 Arnd Bergmann 2018-04-24 2957 err = snd_pcm_hwsync(substream);
09d94175dbeac1 Arnd Bergmann 2018-04-24 2958 if (err < 0)
09d94175dbeac1 Arnd Bergmann 2018-04-24 2959 return err;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2960 }
09d94175dbeac1 Arnd Bergmann 2018-04-24 2961 status = runtime->status;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2962 control = runtime->control;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2963 boundary = recalculate_boundary(runtime);
09d94175dbeac1 Arnd Bergmann 2018-04-24 2964 if (! boundary)
09d94175dbeac1 Arnd Bergmann 2018-04-24 2965 boundary = 0x7fffffff;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2966 snd_pcm_stream_lock_irq(substream);
09d94175dbeac1 Arnd Bergmann 2018-04-24 2967 /* FIXME: we should consider the boundary
for the sync from app */
09d94175dbeac1 Arnd Bergmann 2018-04-24 2968 if (!(sflags &
SNDRV_PCM_SYNC_PTR_APPL))
09d94175dbeac1 Arnd Bergmann 2018-04-24 2969 control->appl_ptr =
scontrol.appl_ptr;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2970 else
09d94175dbeac1 Arnd Bergmann 2018-04-24 2971 scontrol.appl_ptr = control->appl_ptr
% boundary;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2972 if (!(sflags &
SNDRV_PCM_SYNC_PTR_AVAIL_MIN))
09d94175dbeac1 Arnd Bergmann 2018-04-24 2973 control->avail_min =
scontrol.avail_min;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2974 else
09d94175dbeac1 Arnd Bergmann 2018-04-24 2975 scontrol.avail_min =
control->avail_min;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2976 sstatus.state = status->state;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2977 sstatus.hw_ptr = status->hw_ptr %
boundary;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2978 sstatus.tstamp = status->tstamp;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2979 sstatus.suspended_state =
status->suspended_state;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2980 sstatus.audio_tstamp =
status->audio_tstamp;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2981 snd_pcm_stream_unlock_irq(substream);
09d94175dbeac1 Arnd Bergmann 2018-04-24 @2982 if (put_user(sstatus.state,
&src->s.status.state) ||
09d94175dbeac1 Arnd Bergmann 2018-04-24 2983 put_user(sstatus.hw_ptr,
&src->s.status.hw_ptr) ||
80fe7430c70859 Arnd Bergmann 2018-04-24 2984 put_user(sstatus.tstamp.tv_sec,
&src->s.status.tstamp_sec) ||
80fe7430c70859 Arnd Bergmann 2018-04-24 2985 put_user(sstatus.tstamp.tv_nsec,
&src->s.status.tstamp_nsec) ||
09d94175dbeac1 Arnd Bergmann 2018-04-24 @2986 put_user(sstatus.suspended_state,
&src->s.status.suspended_state) ||
80fe7430c70859 Arnd Bergmann 2018-04-24 2987 put_user(sstatus.audio_tstamp.tv_sec,
&src->s.status.audio_tstamp_sec) ||
80fe7430c70859 Arnd Bergmann 2018-04-24 2988 put_user(sstatus.audio_tstamp.tv_nsec,
&src->s.status.audio_tstamp_nsec) ||
09d94175dbeac1 Arnd Bergmann 2018-04-24 2989 put_user(scontrol.appl_ptr,
&src->c.control.appl_ptr) ||
09d94175dbeac1 Arnd Bergmann 2018-04-24 2990 put_user(scontrol.avail_min,
&src->c.control.avail_min))
09d94175dbeac1 Arnd Bergmann 2018-04-24 2991 return -EFAULT;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2992
09d94175dbeac1 Arnd Bergmann 2018-04-24 2993 return 0;
09d94175dbeac1 Arnd Bergmann 2018-04-24 2994 }
80fe7430c70859 Arnd Bergmann 2018-04-24 2995 #define __SNDRV_PCM_IOCTL_SYNC_PTR32
_IOWR('A', 0x23, struct snd_pcm_sync_ptr32)
09d94175dbeac1 Arnd Bergmann 2018-04-24 2996
:::::: The code at line 2982 was first introduced by commit
:::::: 09d94175dbeac12d38b1599a02c7000a5e51b4cb ALSA: move snd_pcm_ioctl_sync_ptr_compat
into pcm_native.c
:::::: TO: Arnd Bergmann <arnd(a)arndb.de>
:::::: CC: Arnd Bergmann <arnd(a)arndb.de>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation