From: kbuild test robot <lkp@intel.com> To: Daniel Vetter <daniel.vetter@ffwll.ch> Cc: kbuild-all@01.org, "DRI Development" <dri-devel@lists.freedesktop.org>, "Bartlomiej Zolnierkiewicz" <b.zolnierkie@samsung.com>, "Daniel Vetter" <daniel.vetter@ffwll.ch>, "Intel Graphics Development" <intel-gfx@lists.freedesktop.org>, LKML <linux-kernel@vger.kernel.org>, "Michał Mirosław" <mirq-linux@rere.qmqm.pl>, "Yisheng Xie" <ysxie@foxmail.com>, "Mikulas Patocka" <mpatocka@redhat.com>, "Daniel Vetter" <daniel.vetter@intel.com>, "Peter Rosin" <peda@axentia.se> Subject: Re: [Intel-gfx] [PATCH 31/33] fbcon: Call con2fb_map functions directly Date: Tue, 21 May 2019 03:28:42 +0800 [thread overview] Message-ID: <201905210336.2fFkrqDM%lkp@intel.com> (raw) In-Reply-To: <20190520082216.26273-32-daniel.vetter@ffwll.ch> [-- Attachment #1: Type: text/plain, Size: 3059 bytes --] Hi Daniel, I love your patch! Yet something to improve: [auto build test ERROR on linus/master] [also build test ERROR on v5.2-rc1 next-20190520] [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/Daniel-Vetter/fbcon-notifier-begone/20190521-021841 config: sparc64-allyesconfig (attached as .config) compiler: sparc64-linux-gcc (GCC) 8.1.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree GCC_VERSION=8.1.0 make.cross ARCH=sparc64 If you fix the issue, kindly add following tag Reported-by: kbuild test robot <lkp@intel.com> All errors (new ones prefixed by >>): drivers/video/fbdev/core/fbcon.c: In function 'fbcon_set_con2fb_map_ioctl': >> drivers/video/fbdev/core/fbcon.c:3323:6: error: implicit declaration of function 'copy_from_user'; did you mean 'copy_creds'? [-Werror=implicit-function-declaration] if (copy_from_user(&con2fb, argp, sizeof(con2fb))) ^~~~~~~~~~~~~~ copy_creds drivers/video/fbdev/core/fbcon.c: In function 'fbcon_get_con2fb_map_ioctl': >> drivers/video/fbdev/core/fbcon.c:3356:9: error: implicit declaration of function 'copy_to_user'; did you mean 'cpu_to_mem'? [-Werror=implicit-function-declaration] return copy_to_user(argp, &con2fb, sizeof(con2fb)) ? -EFAULT : 0; ^~~~~~~~~~~~ cpu_to_mem cc1: some warnings being treated as errors vim +3323 drivers/video/fbdev/core/fbcon.c 3317 3318 int fbcon_set_con2fb_map_ioctl(void __user *argp) 3319 { 3320 struct fb_con2fbmap con2fb; 3321 int ret; 3322 > 3323 if (copy_from_user(&con2fb, argp, sizeof(con2fb))) 3324 return -EFAULT; 3325 if (con2fb.console < 1 || con2fb.console > MAX_NR_CONSOLES) 3326 return -EINVAL; 3327 if (con2fb.framebuffer >= FB_MAX) 3328 return -EINVAL; 3329 if (!registered_fb[con2fb.framebuffer]) 3330 request_module("fb%d", con2fb.framebuffer); 3331 if (!registered_fb[con2fb.framebuffer]) { 3332 return -EINVAL; 3333 } 3334 3335 console_lock(); 3336 ret = set_con2fb_map(con2fb.console - 1, 3337 con2fb.framebuffer, 1); 3338 console_unlock(); 3339 3340 return ret; 3341 } 3342 3343 int fbcon_get_con2fb_map_ioctl(void __user *argp) 3344 { 3345 struct fb_con2fbmap con2fb; 3346 3347 if (copy_from_user(&con2fb, argp, sizeof(con2fb))) 3348 return -EFAULT; 3349 if (con2fb.console < 1 || con2fb.console > MAX_NR_CONSOLES) 3350 return -EINVAL; 3351 3352 console_lock(); 3353 con2fb.framebuffer = con2fb_map[con2fb.console - 1]; 3354 console_unlock(); 3355 > 3356 return copy_to_user(argp, &con2fb, sizeof(con2fb)) ? -EFAULT : 0; 3357 } 3358 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 58532 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com> Cc: "Bartlomiej Zolnierkiewicz" <b.zolnierkie@samsung.com>, "Daniel Vetter" <daniel.vetter@ffwll.ch>, "Intel Graphics Development" <intel-gfx@lists.freedesktop.org>, LKML <linux-kernel@vger.kernel.org>, "DRI Development" <dri-devel@lists.freedesktop.org>, "Michał Mirosław" <mirq-linux@rere.qmqm.pl>, "Yisheng Xie" <ysxie@foxmail.com>, "Mikulas Patocka" <mpatocka@redhat.com>, kbuild-all@01.org, "Daniel Vetter" <daniel.vetter@intel.com>, "Peter Rosin" <peda@axentia.se> Subject: Re: [PATCH 31/33] fbcon: Call con2fb_map functions directly Date: Tue, 21 May 2019 03:28:42 +0800 [thread overview] Message-ID: <201905210336.2fFkrqDM%lkp@intel.com> (raw) In-Reply-To: <20190520082216.26273-32-daniel.vetter@ffwll.ch> [-- Attachment #1: Type: text/plain, Size: 3059 bytes --] Hi Daniel, I love your patch! Yet something to improve: [auto build test ERROR on linus/master] [also build test ERROR on v5.2-rc1 next-20190520] [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/Daniel-Vetter/fbcon-notifier-begone/20190521-021841 config: sparc64-allyesconfig (attached as .config) compiler: sparc64-linux-gcc (GCC) 8.1.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree GCC_VERSION=8.1.0 make.cross ARCH=sparc64 If you fix the issue, kindly add following tag Reported-by: kbuild test robot <lkp@intel.com> All errors (new ones prefixed by >>): drivers/video/fbdev/core/fbcon.c: In function 'fbcon_set_con2fb_map_ioctl': >> drivers/video/fbdev/core/fbcon.c:3323:6: error: implicit declaration of function 'copy_from_user'; did you mean 'copy_creds'? [-Werror=implicit-function-declaration] if (copy_from_user(&con2fb, argp, sizeof(con2fb))) ^~~~~~~~~~~~~~ copy_creds drivers/video/fbdev/core/fbcon.c: In function 'fbcon_get_con2fb_map_ioctl': >> drivers/video/fbdev/core/fbcon.c:3356:9: error: implicit declaration of function 'copy_to_user'; did you mean 'cpu_to_mem'? [-Werror=implicit-function-declaration] return copy_to_user(argp, &con2fb, sizeof(con2fb)) ? -EFAULT : 0; ^~~~~~~~~~~~ cpu_to_mem cc1: some warnings being treated as errors vim +3323 drivers/video/fbdev/core/fbcon.c 3317 3318 int fbcon_set_con2fb_map_ioctl(void __user *argp) 3319 { 3320 struct fb_con2fbmap con2fb; 3321 int ret; 3322 > 3323 if (copy_from_user(&con2fb, argp, sizeof(con2fb))) 3324 return -EFAULT; 3325 if (con2fb.console < 1 || con2fb.console > MAX_NR_CONSOLES) 3326 return -EINVAL; 3327 if (con2fb.framebuffer >= FB_MAX) 3328 return -EINVAL; 3329 if (!registered_fb[con2fb.framebuffer]) 3330 request_module("fb%d", con2fb.framebuffer); 3331 if (!registered_fb[con2fb.framebuffer]) { 3332 return -EINVAL; 3333 } 3334 3335 console_lock(); 3336 ret = set_con2fb_map(con2fb.console - 1, 3337 con2fb.framebuffer, 1); 3338 console_unlock(); 3339 3340 return ret; 3341 } 3342 3343 int fbcon_get_con2fb_map_ioctl(void __user *argp) 3344 { 3345 struct fb_con2fbmap con2fb; 3346 3347 if (copy_from_user(&con2fb, argp, sizeof(con2fb))) 3348 return -EFAULT; 3349 if (con2fb.console < 1 || con2fb.console > MAX_NR_CONSOLES) 3350 return -EINVAL; 3351 3352 console_lock(); 3353 con2fb.framebuffer = con2fb_map[con2fb.console - 1]; 3354 console_unlock(); 3355 > 3356 return copy_to_user(argp, &con2fb, sizeof(con2fb)) ? -EFAULT : 0; 3357 } 3358 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 58532 bytes --] [-- Attachment #3: Type: text/plain, Size: 159 bytes --] _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2019-05-20 19:29 UTC|newest] Thread overview: 112+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-05-20 8:21 [PATCH 00/33] fbcon notifier begone! Daniel Vetter 2019-05-20 8:21 ` [PATCH 01/33] dummycon: Sprinkle locking checks Daniel Vetter 2019-05-20 8:21 ` [PATCH 02/33] fbdev: locking check for fb_set_suspend Daniel Vetter 2019-05-20 8:21 ` [PATCH 03/33] vt: might_sleep() annotation for do_blank_screen Daniel Vetter 2019-05-20 8:21 ` [PATCH 04/33] vt: More locking checks Daniel Vetter 2019-05-20 8:21 ` Daniel Vetter 2019-05-20 8:21 ` [PATCH 05/33] fbdev/sa1100fb: Remove dead code Daniel Vetter 2019-05-20 8:21 ` Daniel Vetter 2019-05-20 8:21 ` [PATCH 06/33] fbdev/cyber2000: Remove struct display Daniel Vetter 2019-05-20 8:21 ` Daniel Vetter 2019-05-20 8:21 ` [PATCH 07/33] fbdev/aty128fb: Remove dead code Daniel Vetter 2019-05-20 8:21 ` Daniel Vetter 2019-05-20 8:21 ` [PATCH 08/33] fbcon: s/struct display/struct fbcon_display/ Daniel Vetter 2019-05-20 8:21 ` Daniel Vetter 2019-05-20 8:21 ` [PATCH 09/33] fbcon: Remove fbcon_has_exited Daniel Vetter 2019-05-20 8:21 ` Daniel Vetter 2019-05-21 14:23 ` [PATCH] " Daniel Vetter 2019-05-22 10:04 ` Bartlomiej Zolnierkiewicz 2019-05-22 10:04 ` Bartlomiej Zolnierkiewicz 2019-05-22 10:38 ` Daniel Vetter 2019-05-20 8:21 ` [PATCH 10/33] fbcon: call fbcon_fb_(un)registered directly Daniel Vetter 2019-05-20 8:21 ` Daniel Vetter 2019-05-20 8:33 ` Thomas Zimmermann 2019-05-20 8:33 ` Thomas Zimmermann 2019-05-20 8:37 ` Thomas Zimmermann 2019-05-20 8:37 ` Thomas Zimmermann 2019-05-21 15:09 ` Daniel Vetter 2019-05-21 15:09 ` Daniel Vetter 2019-05-20 17:08 ` Sam Ravnborg 2019-05-20 17:08 ` Sam Ravnborg 2019-05-20 17:08 ` Sam Ravnborg 2019-05-20 17:25 ` Daniel Vetter 2019-05-20 17:25 ` Daniel Vetter 2019-05-20 8:21 ` [PATCH 11/33] fbdev/sh_mobile: remove sh_mobile_lcdc_display_notify Daniel Vetter 2019-05-20 8:21 ` Daniel Vetter 2019-05-20 9:04 ` Geert Uytterhoeven 2019-05-20 8:21 ` [PATCH 12/33] fbdev/omap: sysfs files can't disappear before the device is gone Daniel Vetter 2019-05-20 8:21 ` [PATCH 13/33] fbdev: " Daniel Vetter 2019-05-20 8:21 ` Daniel Vetter 2019-05-20 8:21 ` [PATCH 14/33] staging/olpc: lock_fb_info can't fail Daniel Vetter 2019-05-20 8:21 ` Daniel Vetter 2019-05-20 8:21 ` [PATCH 15/33] fbdev/atyfb: " Daniel Vetter 2019-05-20 8:21 ` Daniel Vetter 2019-05-20 8:21 ` [PATCH 16/33] fbdev: lock_fb_info cannot fail Daniel Vetter 2019-05-20 8:21 ` Daniel Vetter 2019-05-20 8:22 ` [PATCH 17/33] fbcon: call fbcon_fb_bind directly Daniel Vetter 2019-05-20 8:22 ` Daniel Vetter 2019-05-20 8:22 ` Daniel Vetter 2019-05-20 8:22 ` [PATCH 18/33] fbdev: make unregister/unlink functions not fail Daniel Vetter 2019-05-20 8:22 ` Daniel Vetter 2019-05-20 8:22 ` Daniel Vetter 2019-05-20 19:08 ` [Intel-gfx] " kbuild test robot 2019-05-20 19:08 ` kbuild test robot 2019-05-20 19:08 ` kbuild test robot 2019-05-20 19:25 ` kbuild test robot 2019-05-20 19:25 ` kbuild test robot 2019-05-20 19:25 ` [Intel-gfx] " kbuild test robot 2019-05-20 21:45 ` kbuild test robot 2019-05-20 21:45 ` kbuild test robot 2019-05-20 21:45 ` kbuild test robot 2019-05-20 8:22 ` [PATCH 19/33] fbdev: unify unlink_framebufer paths Daniel Vetter 2019-05-21 10:52 ` Maarten Lankhorst 2019-05-20 8:22 ` [PATCH 20/33] fbdev/sh_mob: Remove fb notifier callback Daniel Vetter 2019-05-20 9:05 ` Geert Uytterhoeven 2019-05-20 9:19 ` Daniel Vetter 2019-05-20 8:22 ` [PATCH 21/33] fbdev: directly call fbcon_suspended/resumed Daniel Vetter 2019-05-20 8:22 ` Daniel Vetter 2019-05-20 19:24 ` kbuild test robot 2019-05-20 19:24 ` kbuild test robot 2019-05-20 19:24 ` kbuild test robot 2019-05-20 8:22 ` [PATCH 22/33] fbcon: Call fbcon_mode_deleted/new_modelist directly Daniel Vetter 2019-05-20 8:22 ` Daniel Vetter 2019-05-20 8:22 ` [PATCH 23/33] fbdev: Call fbcon_get_requirement directly Daniel Vetter 2019-05-20 8:22 ` Daniel Vetter 2019-05-20 8:22 ` [PATCH 24/33] Revert "backlight/fbcon: Add FB_EVENT_CONBLANK" Daniel Vetter 2019-05-20 8:22 ` Daniel Vetter 2019-05-20 8:22 ` [PATCH 25/33] fbcon: directly call fbcon_fb_blanked Daniel Vetter 2019-05-20 8:22 ` [PATCH 26/33] fbmem: pull fbcon_fb_blanked out of fb_blank Daniel Vetter 2019-05-20 8:22 ` [PATCH 27/33] fbdev: remove FBINFO_MISC_USEREVENT around fb_blank Daniel Vetter 2019-05-20 17:20 ` Sam Ravnborg 2019-05-20 17:20 ` Sam Ravnborg 2019-05-20 17:29 ` Daniel Vetter 2019-05-20 17:29 ` Daniel Vetter 2019-05-20 17:53 ` Sam Ravnborg 2019-05-20 8:22 ` [PATCH 28/33] fb: Flatten control flow in fb_set_var Daniel Vetter 2019-05-20 8:22 ` [PATCH 29/33] fbcon: replace FB_EVENT_MODE_CHANGE/_ALL with direct calls Daniel Vetter 2019-05-20 8:22 ` Daniel Vetter 2019-05-21 10:56 ` Maarten Lankhorst 2019-05-21 10:56 ` Maarten Lankhorst 2019-05-21 12:42 ` Daniel Vetter 2019-05-21 12:42 ` Daniel Vetter 2019-05-21 12:42 ` Daniel Vetter 2019-05-20 8:22 ` [PATCH 30/33] vgaswitcheroo: call fbcon_remap_all directly Daniel Vetter 2019-05-20 8:22 ` Daniel Vetter 2019-05-20 8:37 ` Lukas Wunner 2019-05-20 8:37 ` Lukas Wunner 2019-05-20 8:22 ` [PATCH 31/33] fbcon: Call con2fb_map functions directly Daniel Vetter 2019-05-20 19:28 ` kbuild test robot [this message] 2019-05-20 19:28 ` kbuild test robot 2019-05-20 19:34 ` [Intel-gfx] " kbuild test robot 2019-05-20 19:34 ` kbuild test robot 2019-05-20 8:22 ` [PATCH 32/33] fbcon: Document what I learned about fbcon locking Daniel Vetter 2019-05-21 11:13 ` Maarten Lankhorst 2019-05-20 8:22 ` [PATCH 33/33] staging/olpc_dcon: Add drm conversion to TODO Daniel Vetter 2019-05-20 8:30 ` ✗ Fi.CI.CHECKPATCH: warning for fbcon notifier begone! Patchwork 2019-05-20 8:41 ` ✗ Fi.CI.BAT: failure " Patchwork 2019-05-20 8:44 ` ✗ Fi.CI.SPARSE: warning " Patchwork 2019-05-21 17:08 ` ✗ Fi.CI.CHECKPATCH: warning for fbcon notifier begone! (rev2) Patchwork 2019-05-21 17:22 ` ✗ Fi.CI.SPARSE: " Patchwork 2019-05-21 17:29 ` ✗ Fi.CI.BAT: failure " Patchwork 2019-05-22 8:15 ` ✓ Fi.CI.BAT: success " Patchwork 2019-05-22 18:43 ` ✓ Fi.CI.IGT: " Patchwork
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=201905210336.2fFkrqDM%lkp@intel.com \ --to=lkp@intel.com \ --cc=b.zolnierkie@samsung.com \ --cc=daniel.vetter@ffwll.ch \ --cc=daniel.vetter@intel.com \ --cc=dri-devel@lists.freedesktop.org \ --cc=intel-gfx@lists.freedesktop.org \ --cc=kbuild-all@01.org \ --cc=linux-kernel@vger.kernel.org \ --cc=mirq-linux@rere.qmqm.pl \ --cc=mpatocka@redhat.com \ --cc=peda@axentia.se \ --cc=ysxie@foxmail.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: linkBe 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.