From: kernel test robot <lkp@intel.com> To: Sakari Ailus <sakari.ailus@linux.intel.com> Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org, linux-kernel@vger.kernel.org, Thomas Zimmermann <tzimmermann@suse.de>, Petr Mladek <pmladek@suse.com>, Andy Shevchenko <andriy.shevchenko@linux.intel.com> Subject: drivers/media/v4l2-core/v4l2-ioctl.c:303:28: warning: taking address of packed member 'pixelformat' of class or structure 'v4l2_pix_format_mplane' may result in an unaligned pointer value Date: Fri, 5 Nov 2021 06:21:51 +0800 [thread overview] Message-ID: <202111050644.9HQjbpIQ-lkp@intel.com> (raw) [-- Attachment #1: Type: text/plain, Size: 6878 bytes --] tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: c1e2e0350ce37f633b5ce3ce1cdf4428513fc2a2 commit: e927e1e0f0dd3e353d5556503a71484008692c82 v4l: ioctl: Use %p4cc printk modifier to print FourCC codes date: 9 months ago config: mips-randconfig-r011-20211101 (attached as .config) compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 82ed106567063ea269c6d5669278b733e173a42f) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # install mips cross compiling tool for clang build # apt-get install binutils-mips-linux-gnu # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=e927e1e0f0dd3e353d5556503a71484008692c82 git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git git fetch --no-tags linus master git checkout e927e1e0f0dd3e353d5556503a71484008692c82 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=mips If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com> All warnings (new ones prefixed by >>): >> drivers/media/v4l2-core/v4l2-ioctl.c:303:28: warning: taking address of packed member 'pixelformat' of class or structure 'v4l2_pix_format_mplane' may result in an unaligned pointer value [-Waddress-of-packed-member] mp->width, mp->height, &mp->pixelformat, ^~~~~~~~~~~~~~~ include/linux/printk.h:385:26: note: expanded from macro 'pr_cont' printk(KERN_CONT fmt, ##__VA_ARGS__) ^~~~~~~~~~~ >> drivers/media/v4l2-core/v4l2-ioctl.c:347:37: warning: taking address of packed member 'pixelformat' of class or structure 'v4l2_sdr_format' may result in an unaligned pointer value [-Waddress-of-packed-member] pr_cont(", pixelformat=%p4cc\n", &sdr->pixelformat); ^~~~~~~~~~~~~~~~ include/linux/printk.h:385:26: note: expanded from macro 'pr_cont' printk(KERN_CONT fmt, ##__VA_ARGS__) ^~~~~~~~~~~ >> drivers/media/v4l2-core/v4l2-ioctl.c:353:5: warning: taking address of packed member 'dataformat' of class or structure 'v4l2_meta_format' may result in an unaligned pointer value [-Waddress-of-packed-member] &meta->dataformat, meta->buffersize); ^~~~~~~~~~~~~~~~ include/linux/printk.h:385:26: note: expanded from macro 'pr_cont' printk(KERN_CONT fmt, ##__VA_ARGS__) ^~~~~~~~~~~ 3 warnings generated. vim +303 drivers/media/v4l2-core/v4l2-ioctl.c 273 274 static void v4l_print_format(const void *arg, bool write_only) 275 { 276 const struct v4l2_format *p = arg; 277 const struct v4l2_pix_format *pix; 278 const struct v4l2_pix_format_mplane *mp; 279 const struct v4l2_vbi_format *vbi; 280 const struct v4l2_sliced_vbi_format *sliced; 281 const struct v4l2_window *win; 282 const struct v4l2_sdr_format *sdr; 283 const struct v4l2_meta_format *meta; 284 u32 planes; 285 unsigned i; 286 287 pr_cont("type=%s", prt_names(p->type, v4l2_type_names)); 288 switch (p->type) { 289 case V4L2_BUF_TYPE_VIDEO_CAPTURE: 290 case V4L2_BUF_TYPE_VIDEO_OUTPUT: 291 pix = &p->fmt.pix; 292 pr_cont(", width=%u, height=%u, pixelformat=%p4cc, field=%s, bytesperline=%u, sizeimage=%u, colorspace=%d, flags=0x%x, ycbcr_enc=%u, quantization=%u, xfer_func=%u\n", 293 pix->width, pix->height, &pix->pixelformat, 294 prt_names(pix->field, v4l2_field_names), 295 pix->bytesperline, pix->sizeimage, 296 pix->colorspace, pix->flags, pix->ycbcr_enc, 297 pix->quantization, pix->xfer_func); 298 break; 299 case V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE: 300 case V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE: 301 mp = &p->fmt.pix_mp; 302 pr_cont(", width=%u, height=%u, format=%p4cc, field=%s, colorspace=%d, num_planes=%u, flags=0x%x, ycbcr_enc=%u, quantization=%u, xfer_func=%u\n", > 303 mp->width, mp->height, &mp->pixelformat, 304 prt_names(mp->field, v4l2_field_names), 305 mp->colorspace, mp->num_planes, mp->flags, 306 mp->ycbcr_enc, mp->quantization, mp->xfer_func); 307 planes = min_t(u32, mp->num_planes, VIDEO_MAX_PLANES); 308 for (i = 0; i < planes; i++) 309 printk(KERN_DEBUG "plane %u: bytesperline=%u sizeimage=%u\n", i, 310 mp->plane_fmt[i].bytesperline, 311 mp->plane_fmt[i].sizeimage); 312 break; 313 case V4L2_BUF_TYPE_VIDEO_OVERLAY: 314 case V4L2_BUF_TYPE_VIDEO_OUTPUT_OVERLAY: 315 win = &p->fmt.win; 316 /* Note: we can't print the clip list here since the clips 317 * pointer is a userspace pointer, not a kernelspace 318 * pointer. */ 319 pr_cont(", wxh=%dx%d, x,y=%d,%d, field=%s, chromakey=0x%08x, clipcount=%u, clips=%p, bitmap=%p, global_alpha=0x%02x\n", 320 win->w.width, win->w.height, win->w.left, win->w.top, 321 prt_names(win->field, v4l2_field_names), 322 win->chromakey, win->clipcount, win->clips, 323 win->bitmap, win->global_alpha); 324 break; 325 case V4L2_BUF_TYPE_VBI_CAPTURE: 326 case V4L2_BUF_TYPE_VBI_OUTPUT: 327 vbi = &p->fmt.vbi; 328 pr_cont(", sampling_rate=%u, offset=%u, samples_per_line=%u, sample_format=%p4cc, start=%u,%u, count=%u,%u\n", 329 vbi->sampling_rate, vbi->offset, 330 vbi->samples_per_line, &vbi->sample_format, 331 vbi->start[0], vbi->start[1], 332 vbi->count[0], vbi->count[1]); 333 break; 334 case V4L2_BUF_TYPE_SLICED_VBI_CAPTURE: 335 case V4L2_BUF_TYPE_SLICED_VBI_OUTPUT: 336 sliced = &p->fmt.sliced; 337 pr_cont(", service_set=0x%08x, io_size=%d\n", 338 sliced->service_set, sliced->io_size); 339 for (i = 0; i < 24; i++) 340 printk(KERN_DEBUG "line[%02u]=0x%04x, 0x%04x\n", i, 341 sliced->service_lines[0][i], 342 sliced->service_lines[1][i]); 343 break; 344 case V4L2_BUF_TYPE_SDR_CAPTURE: 345 case V4L2_BUF_TYPE_SDR_OUTPUT: 346 sdr = &p->fmt.sdr; > 347 pr_cont(", pixelformat=%p4cc\n", &sdr->pixelformat); 348 break; 349 case V4L2_BUF_TYPE_META_CAPTURE: 350 case V4L2_BUF_TYPE_META_OUTPUT: 351 meta = &p->fmt.meta; 352 pr_cont(", dataformat=%p4cc, buffersize=%u\n", > 353 &meta->dataformat, meta->buffersize); 354 break; 355 } 356 } 357 --- 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: 32541 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com> To: kbuild-all@lists.01.org Subject: drivers/media/v4l2-core/v4l2-ioctl.c:303:28: warning: taking address of packed member 'pixelformat' of class or structure 'v4l2_pix_format_mplane' may result in an unaligned pointer value Date: Fri, 05 Nov 2021 06:21:51 +0800 [thread overview] Message-ID: <202111050644.9HQjbpIQ-lkp@intel.com> (raw) [-- Attachment #1: Type: text/plain, Size: 7015 bytes --] tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: c1e2e0350ce37f633b5ce3ce1cdf4428513fc2a2 commit: e927e1e0f0dd3e353d5556503a71484008692c82 v4l: ioctl: Use %p4cc printk modifier to print FourCC codes date: 9 months ago config: mips-randconfig-r011-20211101 (attached as .config) compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 82ed106567063ea269c6d5669278b733e173a42f) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # install mips cross compiling tool for clang build # apt-get install binutils-mips-linux-gnu # https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=e927e1e0f0dd3e353d5556503a71484008692c82 git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git git fetch --no-tags linus master git checkout e927e1e0f0dd3e353d5556503a71484008692c82 # save the attached .config to linux build tree COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=mips If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp@intel.com> All warnings (new ones prefixed by >>): >> drivers/media/v4l2-core/v4l2-ioctl.c:303:28: warning: taking address of packed member 'pixelformat' of class or structure 'v4l2_pix_format_mplane' may result in an unaligned pointer value [-Waddress-of-packed-member] mp->width, mp->height, &mp->pixelformat, ^~~~~~~~~~~~~~~ include/linux/printk.h:385:26: note: expanded from macro 'pr_cont' printk(KERN_CONT fmt, ##__VA_ARGS__) ^~~~~~~~~~~ >> drivers/media/v4l2-core/v4l2-ioctl.c:347:37: warning: taking address of packed member 'pixelformat' of class or structure 'v4l2_sdr_format' may result in an unaligned pointer value [-Waddress-of-packed-member] pr_cont(", pixelformat=%p4cc\n", &sdr->pixelformat); ^~~~~~~~~~~~~~~~ include/linux/printk.h:385:26: note: expanded from macro 'pr_cont' printk(KERN_CONT fmt, ##__VA_ARGS__) ^~~~~~~~~~~ >> drivers/media/v4l2-core/v4l2-ioctl.c:353:5: warning: taking address of packed member 'dataformat' of class or structure 'v4l2_meta_format' may result in an unaligned pointer value [-Waddress-of-packed-member] &meta->dataformat, meta->buffersize); ^~~~~~~~~~~~~~~~ include/linux/printk.h:385:26: note: expanded from macro 'pr_cont' printk(KERN_CONT fmt, ##__VA_ARGS__) ^~~~~~~~~~~ 3 warnings generated. vim +303 drivers/media/v4l2-core/v4l2-ioctl.c 273 274 static void v4l_print_format(const void *arg, bool write_only) 275 { 276 const struct v4l2_format *p = arg; 277 const struct v4l2_pix_format *pix; 278 const struct v4l2_pix_format_mplane *mp; 279 const struct v4l2_vbi_format *vbi; 280 const struct v4l2_sliced_vbi_format *sliced; 281 const struct v4l2_window *win; 282 const struct v4l2_sdr_format *sdr; 283 const struct v4l2_meta_format *meta; 284 u32 planes; 285 unsigned i; 286 287 pr_cont("type=%s", prt_names(p->type, v4l2_type_names)); 288 switch (p->type) { 289 case V4L2_BUF_TYPE_VIDEO_CAPTURE: 290 case V4L2_BUF_TYPE_VIDEO_OUTPUT: 291 pix = &p->fmt.pix; 292 pr_cont(", width=%u, height=%u, pixelformat=%p4cc, field=%s, bytesperline=%u, sizeimage=%u, colorspace=%d, flags=0x%x, ycbcr_enc=%u, quantization=%u, xfer_func=%u\n", 293 pix->width, pix->height, &pix->pixelformat, 294 prt_names(pix->field, v4l2_field_names), 295 pix->bytesperline, pix->sizeimage, 296 pix->colorspace, pix->flags, pix->ycbcr_enc, 297 pix->quantization, pix->xfer_func); 298 break; 299 case V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE: 300 case V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE: 301 mp = &p->fmt.pix_mp; 302 pr_cont(", width=%u, height=%u, format=%p4cc, field=%s, colorspace=%d, num_planes=%u, flags=0x%x, ycbcr_enc=%u, quantization=%u, xfer_func=%u\n", > 303 mp->width, mp->height, &mp->pixelformat, 304 prt_names(mp->field, v4l2_field_names), 305 mp->colorspace, mp->num_planes, mp->flags, 306 mp->ycbcr_enc, mp->quantization, mp->xfer_func); 307 planes = min_t(u32, mp->num_planes, VIDEO_MAX_PLANES); 308 for (i = 0; i < planes; i++) 309 printk(KERN_DEBUG "plane %u: bytesperline=%u sizeimage=%u\n", i, 310 mp->plane_fmt[i].bytesperline, 311 mp->plane_fmt[i].sizeimage); 312 break; 313 case V4L2_BUF_TYPE_VIDEO_OVERLAY: 314 case V4L2_BUF_TYPE_VIDEO_OUTPUT_OVERLAY: 315 win = &p->fmt.win; 316 /* Note: we can't print the clip list here since the clips 317 * pointer is a userspace pointer, not a kernelspace 318 * pointer. */ 319 pr_cont(", wxh=%dx%d, x,y=%d,%d, field=%s, chromakey=0x%08x, clipcount=%u, clips=%p, bitmap=%p, global_alpha=0x%02x\n", 320 win->w.width, win->w.height, win->w.left, win->w.top, 321 prt_names(win->field, v4l2_field_names), 322 win->chromakey, win->clipcount, win->clips, 323 win->bitmap, win->global_alpha); 324 break; 325 case V4L2_BUF_TYPE_VBI_CAPTURE: 326 case V4L2_BUF_TYPE_VBI_OUTPUT: 327 vbi = &p->fmt.vbi; 328 pr_cont(", sampling_rate=%u, offset=%u, samples_per_line=%u, sample_format=%p4cc, start=%u,%u, count=%u,%u\n", 329 vbi->sampling_rate, vbi->offset, 330 vbi->samples_per_line, &vbi->sample_format, 331 vbi->start[0], vbi->start[1], 332 vbi->count[0], vbi->count[1]); 333 break; 334 case V4L2_BUF_TYPE_SLICED_VBI_CAPTURE: 335 case V4L2_BUF_TYPE_SLICED_VBI_OUTPUT: 336 sliced = &p->fmt.sliced; 337 pr_cont(", service_set=0x%08x, io_size=%d\n", 338 sliced->service_set, sliced->io_size); 339 for (i = 0; i < 24; i++) 340 printk(KERN_DEBUG "line[%02u]=0x%04x, 0x%04x\n", i, 341 sliced->service_lines[0][i], 342 sliced->service_lines[1][i]); 343 break; 344 case V4L2_BUF_TYPE_SDR_CAPTURE: 345 case V4L2_BUF_TYPE_SDR_OUTPUT: 346 sdr = &p->fmt.sdr; > 347 pr_cont(", pixelformat=%p4cc\n", &sdr->pixelformat); 348 break; 349 case V4L2_BUF_TYPE_META_CAPTURE: 350 case V4L2_BUF_TYPE_META_OUTPUT: 351 meta = &p->fmt.meta; 352 pr_cont(", dataformat=%p4cc, buffersize=%u\n", > 353 &meta->dataformat, meta->buffersize); 354 break; 355 } 356 } 357 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org [-- Attachment #2: config.gz --] [-- Type: application/gzip, Size: 32541 bytes --]
next reply other threads:[~2021-11-04 22:22 UTC|newest] Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-11-04 22:21 kernel test robot [this message] 2021-11-04 22:21 ` drivers/media/v4l2-core/v4l2-ioctl.c:303:28: warning: taking address of packed member 'pixelformat' of class or structure 'v4l2_pix_format_mplane' may result in an unaligned pointer value kernel test robot 2021-11-05 10:21 ` Andy Shevchenko 2021-11-05 10:21 ` Andy Shevchenko -- strict thread matches above, loose matches on Subject: below -- 2022-01-17 3:40 kernel test robot 2022-01-17 3:40 ` kernel test robot 2022-01-08 10:33 kernel test robot 2022-01-08 10:33 ` kernel test robot 2022-01-09 14:26 ` Andy Shevchenko 2022-01-09 14:26 ` Andy Shevchenko 2022-01-10 19:44 ` Nick Desaulniers 2022-01-10 19:44 ` Nick Desaulniers 2022-01-10 20:49 ` Andy Shevchenko 2022-01-10 20:49 ` Andy Shevchenko 2022-01-10 21:22 ` Nick Desaulniers 2022-01-10 21:22 ` Nick Desaulniers 2021-07-14 17:55 kernel test robot 2021-07-14 17:55 ` kernel test robot 2021-07-14 19:45 ` Andy Shevchenko 2021-07-14 19:45 ` Andy Shevchenko 2021-07-16 11:41 ` Sakari Ailus 2021-07-16 11:41 ` Sakari Ailus 2021-07-16 12:12 ` Andy Shevchenko 2021-07-16 12:12 ` Andy Shevchenko 2021-08-19 8:10 ` Sakari Ailus 2021-08-19 8:10 ` Sakari Ailus 2021-08-26 13:22 ` Petr Mladek 2021-08-26 13:22 ` Petr Mladek
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=202111050644.9HQjbpIQ-lkp@intel.com \ --to=lkp@intel.com \ --cc=andriy.shevchenko@linux.intel.com \ --cc=kbuild-all@lists.01.org \ --cc=linux-kernel@vger.kernel.org \ --cc=llvm@lists.linux.dev \ --cc=pmladek@suse.com \ --cc=sakari.ailus@linux.intel.com \ --cc=tzimmermann@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: 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.