[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <201804261322.lA4etIjI%fengguang.wu@intel.com>
Date: Thu, 26 Apr 2018 14:20:52 +0800
From: kbuild test robot <lkp@...el.com>
To: Baolin Wang <baolin.wang@...aro.org>
Cc: kbuild-all@...org, perex@...ex.cz, tiwai@...e.com, arnd@...db.de,
baolin.wang@...aro.org, lgirdwood@...il.com, broonie@...nel.org,
o-takashi@...amocchi.jp, mingo@...nel.org,
elfring@...rs.sourceforge.net, dan.carpenter@...cle.com,
jeeja.kp@...el.com, vinod.koul@...el.com,
guneshwor.o.singh@...el.com, subhransu.s.prusty@...el.com,
bhumirks@...il.com, gudishax.kranthikumar@...el.com,
naveen.m@...el.com, hardik.t.shah@...el.com,
arvind.yadav.cs@...il.com, fabf@...net.be,
alsa-devel@...a-project.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 8/8] ALSA: add new 32-bit layout for
snd_pcm_mmap_status/control
Hi Arnd,
I love your patch! Yet something to improve:
[auto build test ERROR on v4.17-rc2]
[cannot apply to sound/for-next asoc/for-next arm-soc/for-next next-20180424]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/Baolin-Wang/Fix-year-2038-issue-for-sound-subsystem/20180426-010145
config: x86_64-acpi-redef (attached as .config)
compiler: gcc-7 (Debian 7.3.0-16) 7.3.0
reproduce:
# save the attached .config to linux build tree
make ARCH=x86_64
All errors (new ones prefixed by >>):
In file included from sound/core/pcm_native.c:3728:0:
sound/core/pcm_compat.c: In function 'snd_pcm_ioctl_compat':
>> sound/core/pcm_compat.c:580:2: error: duplicate case value
case SNDRV_PCM_IOCTL_SYNC_PTR_X32:
^~~~
sound/core/pcm_compat.c:547:2: note: previously used here
case __SNDRV_PCM_IOCTL_SYNC_PTR64:
^~~~
vim +580 sound/core/pcm_compat.c
^1da177e Linus Torvalds 2005-04-16 507
^1da177e Linus Torvalds 2005-04-16 508 static long snd_pcm_ioctl_compat(struct file *file, unsigned int cmd, unsigned long arg)
^1da177e Linus Torvalds 2005-04-16 509 {
877211f5 Takashi Iwai 2005-11-17 510 struct snd_pcm_file *pcm_file;
877211f5 Takashi Iwai 2005-11-17 511 struct snd_pcm_substream *substream;
^1da177e Linus Torvalds 2005-04-16 512 void __user *argp = compat_ptr(arg);
^1da177e Linus Torvalds 2005-04-16 513
^1da177e Linus Torvalds 2005-04-16 514 pcm_file = file->private_data;
^1da177e Linus Torvalds 2005-04-16 515 if (! pcm_file)
^1da177e Linus Torvalds 2005-04-16 516 return -ENOTTY;
^1da177e Linus Torvalds 2005-04-16 517 substream = pcm_file->substream;
^1da177e Linus Torvalds 2005-04-16 518 if (! substream)
^1da177e Linus Torvalds 2005-04-16 519 return -ENOTTY;
^1da177e Linus Torvalds 2005-04-16 520
^1da177e Linus Torvalds 2005-04-16 521 /*
^1da177e Linus Torvalds 2005-04-16 522 * When PCM is used on 32bit mode, we need to disable
53cdcc38 Arnd Bergmann 2018-04-24 523 * mmap of the old PCM status/control records because
53cdcc38 Arnd Bergmann 2018-04-24 524 * of the size incompatibility.
^1da177e Linus Torvalds 2005-04-16 525 */
548a648b Takashi Iwai 2006-07-31 526 pcm_file->no_compat_mmap = 1;
^1da177e Linus Torvalds 2005-04-16 527
^1da177e Linus Torvalds 2005-04-16 528 switch (cmd) {
^1da177e Linus Torvalds 2005-04-16 529 case SNDRV_PCM_IOCTL_PVERSION:
^1da177e Linus Torvalds 2005-04-16 530 case SNDRV_PCM_IOCTL_INFO:
5a7f2619 Takashi Iwai 2007-12-17 531 case SNDRV_PCM_IOCTL_TSTAMP:
6b587ef9 Takashi Iwai 2007-12-14 532 case SNDRV_PCM_IOCTL_TTSTAMP:
4b671f57 Takashi Iwai 2017-06-19 533 case SNDRV_PCM_IOCTL_USER_PVERSION:
^1da177e Linus Torvalds 2005-04-16 534 case SNDRV_PCM_IOCTL_HWSYNC:
^1da177e Linus Torvalds 2005-04-16 535 case SNDRV_PCM_IOCTL_PREPARE:
^1da177e Linus Torvalds 2005-04-16 536 case SNDRV_PCM_IOCTL_RESET:
^1da177e Linus Torvalds 2005-04-16 537 case SNDRV_PCM_IOCTL_START:
^1da177e Linus Torvalds 2005-04-16 538 case SNDRV_PCM_IOCTL_DROP:
^1da177e Linus Torvalds 2005-04-16 539 case SNDRV_PCM_IOCTL_DRAIN:
^1da177e Linus Torvalds 2005-04-16 540 case SNDRV_PCM_IOCTL_PAUSE:
^1da177e Linus Torvalds 2005-04-16 541 case SNDRV_PCM_IOCTL_HW_FREE:
^1da177e Linus Torvalds 2005-04-16 542 case SNDRV_PCM_IOCTL_RESUME:
^1da177e Linus Torvalds 2005-04-16 543 case SNDRV_PCM_IOCTL_XRUN:
^1da177e Linus Torvalds 2005-04-16 544 case SNDRV_PCM_IOCTL_LINK:
^1da177e Linus Torvalds 2005-04-16 545 case SNDRV_PCM_IOCTL_UNLINK:
53cdcc38 Arnd Bergmann 2018-04-24 546 case __SNDRV_PCM_IOCTL_SYNC_PTR32:
53cdcc38 Arnd Bergmann 2018-04-24 547 case __SNDRV_PCM_IOCTL_SYNC_PTR64:
67616fed Takashi Iwai 2017-08-30 548 return snd_pcm_common_ioctl(file, substream, cmd, argp);
^1da177e Linus Torvalds 2005-04-16 549 case SNDRV_PCM_IOCTL_HW_REFINE32:
^1da177e Linus Torvalds 2005-04-16 550 return snd_pcm_ioctl_hw_params_compat(substream, 1, argp);
^1da177e Linus Torvalds 2005-04-16 551 case SNDRV_PCM_IOCTL_HW_PARAMS32:
^1da177e Linus Torvalds 2005-04-16 552 return snd_pcm_ioctl_hw_params_compat(substream, 0, argp);
^1da177e Linus Torvalds 2005-04-16 553 case SNDRV_PCM_IOCTL_SW_PARAMS32:
^1da177e Linus Torvalds 2005-04-16 554 return snd_pcm_ioctl_sw_params_compat(substream, argp);
de41e437 Baolin Wang 2018-04-24 555 case SNDRV_PCM_IOCTL_STATUS_COMPAT32:
de41e437 Baolin Wang 2018-04-24 556 return snd_pcm_status_user32(substream, argp, false);
de41e437 Baolin Wang 2018-04-24 557 case SNDRV_PCM_IOCTL_STATUS_EXT_COMPAT32:
de41e437 Baolin Wang 2018-04-24 558 return snd_pcm_status_user32(substream, argp, true);
^1da177e Linus Torvalds 2005-04-16 559 case SNDRV_PCM_IOCTL_CHANNEL_INFO32:
^1da177e Linus Torvalds 2005-04-16 560 return snd_pcm_ioctl_channel_info_compat(substream, argp);
^1da177e Linus Torvalds 2005-04-16 561 case SNDRV_PCM_IOCTL_WRITEI_FRAMES32:
^1da177e Linus Torvalds 2005-04-16 562 return snd_pcm_ioctl_xferi_compat(substream, SNDRV_PCM_STREAM_PLAYBACK, argp);
^1da177e Linus Torvalds 2005-04-16 563 case SNDRV_PCM_IOCTL_READI_FRAMES32:
^1da177e Linus Torvalds 2005-04-16 564 return snd_pcm_ioctl_xferi_compat(substream, SNDRV_PCM_STREAM_CAPTURE, argp);
^1da177e Linus Torvalds 2005-04-16 565 case SNDRV_PCM_IOCTL_WRITEN_FRAMES32:
^1da177e Linus Torvalds 2005-04-16 566 return snd_pcm_ioctl_xfern_compat(substream, SNDRV_PCM_STREAM_PLAYBACK, argp);
^1da177e Linus Torvalds 2005-04-16 567 case SNDRV_PCM_IOCTL_READN_FRAMES32:
^1da177e Linus Torvalds 2005-04-16 568 return snd_pcm_ioctl_xfern_compat(substream, SNDRV_PCM_STREAM_CAPTURE, argp);
^1da177e Linus Torvalds 2005-04-16 569 case SNDRV_PCM_IOCTL_DELAY32:
^1da177e Linus Torvalds 2005-04-16 570 return snd_pcm_ioctl_delay_compat(substream, argp);
^1da177e Linus Torvalds 2005-04-16 571 case SNDRV_PCM_IOCTL_REWIND32:
^1da177e Linus Torvalds 2005-04-16 572 return snd_pcm_ioctl_rewind_compat(substream, argp);
^1da177e Linus Torvalds 2005-04-16 573 case SNDRV_PCM_IOCTL_FORWARD32:
^1da177e Linus Torvalds 2005-04-16 574 return snd_pcm_ioctl_forward_compat(substream, argp);
de41e437 Baolin Wang 2018-04-24 575 case SNDRV_PCM_IOCTL_STATUS_COMPAT64:
de41e437 Baolin Wang 2018-04-24 576 return snd_pcm_status_user_compat64(substream, argp, false);
de41e437 Baolin Wang 2018-04-24 577 case SNDRV_PCM_IOCTL_STATUS_EXT_COMPAT64:
de41e437 Baolin Wang 2018-04-24 578 return snd_pcm_status_user_compat64(substream, argp, true);
513ace79 Takashi Iwai 2016-02-28 579 #ifdef CONFIG_X86_X32
513ace79 Takashi Iwai 2016-02-28 @580 case SNDRV_PCM_IOCTL_SYNC_PTR_X32:
:::::: The code at line 580 was first introduced by commit
:::::: 513ace79b657e2022a592e77f24074e088681ecc ALSA: pcm: Fix ioctls for X32 ABI
:::::: TO: Takashi Iwai <tiwai@...e.de>
:::::: CC: Takashi Iwai <tiwai@...e.de>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
Download attachment ".config.gz" of type "application/gzip" (30366 bytes)
Powered by blists - more mailing lists