From: kernel test robot <lkp@intel.com> To: Tom Rix <trix@redhat.com>, nsaenz@kernel.org, bcm-kernel-feedback-list@broadcom.com, gregkh@linuxfoundation.org, stefan.wahren@i2se.com, gascoar@gmail.com, ojaswin98@gmail.com Cc: kbuild-all@lists.01.org, linux-rpi-kernel@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Tom Rix <trix@redhat.com> Subject: Re: [PATCH v2] staging: vchiq_arm: change vchiq_platform_init from global to static Date: Tue, 3 May 2022 11:07:38 +0800 [thread overview] Message-ID: <202205031150.FRdwOUbC-lkp@intel.com> (raw) In-Reply-To: <20220419133616.544255-1-trix@redhat.com> Hi Tom, I love your patch! Perhaps something to improve: [auto build test WARNING on staging/staging-testing] url: https://github.com/intel-lab-lkp/linux/commits/Tom-Rix/staging-vchiq_arm-change-vchiq_platform_init-from-global-to-static/20220419-213849 base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git bed6d200f8ca38e1ecbdd8fb7e0564884002abd1 config: openrisc-buildonly-randconfig-r003-20220501 (https://download.01.org/0day-ci/archive/20220503/202205031150.FRdwOUbC-lkp@intel.com/config) compiler: or1k-linux-gcc (GCC) 11.3.0 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 # https://github.com/intel-lab-lkp/linux/commit/36525447e8cdf4a25d5217bffc548768c0f7cec1 git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Tom-Rix/staging-vchiq_arm-change-vchiq_platform_init-from-global-to-static/20220419-213849 git checkout 36525447e8cdf4a25d5217bffc548768c0f7cec1 # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=openrisc SHELL=/bin/bash drivers/staging/vc04_services/ 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 >>): In file included from drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c:16: In function 'memcpy_to_page', inlined from 'free_pagelist' at drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c:434:4: >> include/linux/highmem.h:353:9: warning: argument 2 null where non-null expected [-Wnonnull] 353 | memcpy(to + offset, from, len); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ In file included from include/linux/string.h:20, from include/linux/bitmap.h:11, from include/linux/cpumask.h:12, from include/linux/mm_types_task.h:14, from include/linux/mm_types.h:5, from include/linux/buildid.h:5, from include/linux/module.h:14, from drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c:8: drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c: In function 'free_pagelist': arch/openrisc/include/asm/string.h:9:14: note: in a call to function 'memcpy' declared 'nonnull' 9 | extern void *memcpy(void *dest, __const void *src, __kernel_size_t n); | ^~~~~~ In file included from drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c:16: In function 'memcpy_to_page', inlined from 'free_pagelist' at drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c:441:4: >> include/linux/highmem.h:353:9: warning: 'memcpy' offset 0 is out of the bounds [0, 0] [-Warray-bounds] 353 | memcpy(to + offset, from, len); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ vim +353 include/linux/highmem.h bb90d4bc7b6a53 Ira Weiny 2021-02-09 346 bb90d4bc7b6a53 Ira Weiny 2021-02-09 347 static inline void memcpy_to_page(struct page *page, size_t offset, bb90d4bc7b6a53 Ira Weiny 2021-02-09 348 const char *from, size_t len) bb90d4bc7b6a53 Ira Weiny 2021-02-09 349 { 61b205f579911a Ira Weiny 2021-02-09 350 char *to = kmap_local_page(page); bb90d4bc7b6a53 Ira Weiny 2021-02-09 351 ca18f6ea012bf3 Ira Weiny 2021-02-10 352 VM_BUG_ON(offset + len > PAGE_SIZE); bb90d4bc7b6a53 Ira Weiny 2021-02-09 @353 memcpy(to + offset, from, len); 8dad53a11f8d94 Christoph Hellwig 2021-07-23 354 flush_dcache_page(page); 61b205f579911a Ira Weiny 2021-02-09 355 kunmap_local(to); bb90d4bc7b6a53 Ira Weiny 2021-02-09 356 } bb90d4bc7b6a53 Ira Weiny 2021-02-09 357 -- 0-DAY CI Kernel Test Service https://01.org/lkp
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com> To: Tom Rix <trix@redhat.com>, nsaenz@kernel.org, bcm-kernel-feedback-list@broadcom.com, gregkh@linuxfoundation.org, stefan.wahren@i2se.com, gascoar@gmail.com, ojaswin98@gmail.com Cc: kbuild-all@lists.01.org, linux-rpi-kernel@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org, Tom Rix <trix@redhat.com> Subject: Re: [PATCH v2] staging: vchiq_arm: change vchiq_platform_init from global to static Date: Tue, 3 May 2022 11:07:38 +0800 [thread overview] Message-ID: <202205031150.FRdwOUbC-lkp@intel.com> (raw) In-Reply-To: <20220419133616.544255-1-trix@redhat.com> Hi Tom, I love your patch! Perhaps something to improve: [auto build test WARNING on staging/staging-testing] url: https://github.com/intel-lab-lkp/linux/commits/Tom-Rix/staging-vchiq_arm-change-vchiq_platform_init-from-global-to-static/20220419-213849 base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git bed6d200f8ca38e1ecbdd8fb7e0564884002abd1 config: openrisc-buildonly-randconfig-r003-20220501 (https://download.01.org/0day-ci/archive/20220503/202205031150.FRdwOUbC-lkp@intel.com/config) compiler: or1k-linux-gcc (GCC) 11.3.0 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 # https://github.com/intel-lab-lkp/linux/commit/36525447e8cdf4a25d5217bffc548768c0f7cec1 git remote add linux-review https://github.com/intel-lab-lkp/linux git fetch --no-tags linux-review Tom-Rix/staging-vchiq_arm-change-vchiq_platform_init-from-global-to-static/20220419-213849 git checkout 36525447e8cdf4a25d5217bffc548768c0f7cec1 # save the config file mkdir build_dir && cp config build_dir/.config COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.3.0 make.cross W=1 O=build_dir ARCH=openrisc SHELL=/bin/bash drivers/staging/vc04_services/ 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 >>): In file included from drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c:16: In function 'memcpy_to_page', inlined from 'free_pagelist' at drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c:434:4: >> include/linux/highmem.h:353:9: warning: argument 2 null where non-null expected [-Wnonnull] 353 | memcpy(to + offset, from, len); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ In file included from include/linux/string.h:20, from include/linux/bitmap.h:11, from include/linux/cpumask.h:12, from include/linux/mm_types_task.h:14, from include/linux/mm_types.h:5, from include/linux/buildid.h:5, from include/linux/module.h:14, from drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c:8: drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c: In function 'free_pagelist': arch/openrisc/include/asm/string.h:9:14: note: in a call to function 'memcpy' declared 'nonnull' 9 | extern void *memcpy(void *dest, __const void *src, __kernel_size_t n); | ^~~~~~ In file included from drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c:16: In function 'memcpy_to_page', inlined from 'free_pagelist' at drivers/staging/vc04_services/interface/vchiq_arm/vchiq_arm.c:441:4: >> include/linux/highmem.h:353:9: warning: 'memcpy' offset 0 is out of the bounds [0, 0] [-Warray-bounds] 353 | memcpy(to + offset, from, len); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ vim +353 include/linux/highmem.h bb90d4bc7b6a53 Ira Weiny 2021-02-09 346 bb90d4bc7b6a53 Ira Weiny 2021-02-09 347 static inline void memcpy_to_page(struct page *page, size_t offset, bb90d4bc7b6a53 Ira Weiny 2021-02-09 348 const char *from, size_t len) bb90d4bc7b6a53 Ira Weiny 2021-02-09 349 { 61b205f579911a Ira Weiny 2021-02-09 350 char *to = kmap_local_page(page); bb90d4bc7b6a53 Ira Weiny 2021-02-09 351 ca18f6ea012bf3 Ira Weiny 2021-02-10 352 VM_BUG_ON(offset + len > PAGE_SIZE); bb90d4bc7b6a53 Ira Weiny 2021-02-09 @353 memcpy(to + offset, from, len); 8dad53a11f8d94 Christoph Hellwig 2021-07-23 354 flush_dcache_page(page); 61b205f579911a Ira Weiny 2021-02-09 355 kunmap_local(to); bb90d4bc7b6a53 Ira Weiny 2021-02-09 356 } bb90d4bc7b6a53 Ira Weiny 2021-02-09 357 -- 0-DAY CI Kernel Test Service https://01.org/lkp _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2022-05-03 3:08 UTC|newest] Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-04-19 13:36 [PATCH v2] staging: vchiq_arm: change vchiq_platform_init from global to static Tom Rix 2022-04-19 13:36 ` Tom Rix 2022-04-19 14:19 ` Greg KH 2022-04-19 14:19 ` Greg KH 2022-05-03 3:07 ` kernel test robot [this message] 2022-05-03 3:07 ` kernel test robot
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=202205031150.FRdwOUbC-lkp@intel.com \ --to=lkp@intel.com \ --cc=bcm-kernel-feedback-list@broadcom.com \ --cc=gascoar@gmail.com \ --cc=gregkh@linuxfoundation.org \ --cc=kbuild-all@lists.01.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-rpi-kernel@lists.infradead.org \ --cc=linux-staging@lists.linux.dev \ --cc=nsaenz@kernel.org \ --cc=ojaswin98@gmail.com \ --cc=stefan.wahren@i2se.com \ --cc=trix@redhat.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.