Re: [PATCH v2 1/2] media: replace setting of bytesused with vb2_set_plane_payload
by kernel test robot
Hi Dafna,
I love your patch! Yet something to improve:
[auto build test ERROR on media-tree/master]
[also build test ERROR on v5.16-rc1 next-20211115]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Dafna-Hirschfeld/media-videobuf2...
base: git://linuxtv.org/media_tree.git master
config: microblaze-randconfig-r013-20211115 (attached as .config)
compiler: microblaze-linux-gcc (GCC) 11.2.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/0day-ci/linux/commit/6934ed79a3550b00a9f9a6ae14a283a59...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Dafna-Hirschfeld/media-videobuf2-make-sure-bytesused-is-smaller-than-the-buffer-size/20211111-232720
git checkout 6934ed79a3550b00a9f9a6ae14a283a59a10f566
# save the attached .config to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=microblaze SHELL=/bin/bash drivers/staging/media/meson/vdec/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
drivers/staging/media/meson/vdec/vdec_helpers.c: In function 'dst_buf_done':
>> drivers/staging/media/meson/vdec/vdec_helpers.c:279:43: error: incompatible type for argument 1 of 'vb2_set_plane_payload'
279 | vb2_set_plane_payload(vbuf->vb2_buf, 0, output_size);
| ~~~~^~~~~~~~~
| |
| struct vb2_buffer
In file included from include/media/videobuf2-v4l2.h:16,
from include/media/v4l2-mem2mem.h:16,
from drivers/staging/media/meson/vdec/vdec_helpers.c:8:
include/media/videobuf2-core.h:1155:61: note: expected 'struct vb2_buffer *' but argument is of type 'struct vb2_buffer'
1155 | static inline void vb2_set_plane_payload(struct vb2_buffer *vb,
| ~~~~~~~~~~~~~~~~~~~^~
drivers/staging/media/meson/vdec/vdec_helpers.c:280:43: error: incompatible type for argument 1 of 'vb2_set_plane_payload'
280 | vb2_set_plane_payload(vbuf->vb2_buf, 1, output_size / 2);
| ~~~~^~~~~~~~~
| |
| struct vb2_buffer
In file included from include/media/videobuf2-v4l2.h:16,
from include/media/v4l2-mem2mem.h:16,
from drivers/staging/media/meson/vdec/vdec_helpers.c:8:
include/media/videobuf2-core.h:1155:61: note: expected 'struct vb2_buffer *' but argument is of type 'struct vb2_buffer'
1155 | static inline void vb2_set_plane_payload(struct vb2_buffer *vb,
| ~~~~~~~~~~~~~~~~~~~^~
drivers/staging/media/meson/vdec/vdec_helpers.c:283:43: error: incompatible type for argument 1 of 'vb2_set_plane_payload'
283 | vb2_set_plane_payload(vbuf->vb2_buf, 0, output_size);
| ~~~~^~~~~~~~~
| |
| struct vb2_buffer
In file included from include/media/videobuf2-v4l2.h:16,
from include/media/v4l2-mem2mem.h:16,
from drivers/staging/media/meson/vdec/vdec_helpers.c:8:
include/media/videobuf2-core.h:1155:61: note: expected 'struct vb2_buffer *' but argument is of type 'struct vb2_buffer'
1155 | static inline void vb2_set_plane_payload(struct vb2_buffer *vb,
| ~~~~~~~~~~~~~~~~~~~^~
drivers/staging/media/meson/vdec/vdec_helpers.c:284:43: error: incompatible type for argument 1 of 'vb2_set_plane_payload'
284 | vb2_set_plane_payload(vbuf->vb2_buf, 1, output_size / 4);
| ~~~~^~~~~~~~~
| |
| struct vb2_buffer
In file included from include/media/videobuf2-v4l2.h:16,
from include/media/v4l2-mem2mem.h:16,
from drivers/staging/media/meson/vdec/vdec_helpers.c:8:
include/media/videobuf2-core.h:1155:61: note: expected 'struct vb2_buffer *' but argument is of type 'struct vb2_buffer'
1155 | static inline void vb2_set_plane_payload(struct vb2_buffer *vb,
| ~~~~~~~~~~~~~~~~~~~^~
drivers/staging/media/meson/vdec/vdec_helpers.c:285:43: error: incompatible type for argument 1 of 'vb2_set_plane_payload'
285 | vb2_set_plane_payload(vbuf->vb2_buf, 2, output_size / 4);
| ~~~~^~~~~~~~~
| |
| struct vb2_buffer
In file included from include/media/videobuf2-v4l2.h:16,
from include/media/v4l2-mem2mem.h:16,
from drivers/staging/media/meson/vdec/vdec_helpers.c:8:
include/media/videobuf2-core.h:1155:61: note: expected 'struct vb2_buffer *' but argument is of type 'struct vb2_buffer'
1155 | static inline void vb2_set_plane_payload(struct vb2_buffer *vb,
| ~~~~~~~~~~~~~~~~~~~^~
vim +/vb2_set_plane_payload +279 drivers/staging/media/meson/vdec/vdec_helpers.c
268
269 static void dst_buf_done(struct amvdec_session *sess,
270 struct vb2_v4l2_buffer *vbuf,
271 u32 field, u64 timestamp,
272 struct v4l2_timecode timecode, u32 flags)
273 {
274 struct device *dev = sess->core->dev_dec;
275 u32 output_size = amvdec_get_output_size(sess);
276
277 switch (sess->pixfmt_cap) {
278 case V4L2_PIX_FMT_NV12M:
> 279 vb2_set_plane_payload(vbuf->vb2_buf, 0, output_size);
280 vb2_set_plane_payload(vbuf->vb2_buf, 1, output_size / 2);
281 break;
282 case V4L2_PIX_FMT_YUV420M:
283 vb2_set_plane_payload(vbuf->vb2_buf, 0, output_size);
284 vb2_set_plane_payload(vbuf->vb2_buf, 1, output_size / 4);
285 vb2_set_plane_payload(vbuf->vb2_buf, 2, output_size / 4);
286 break;
287 }
288
289 vbuf->vb2_buf.timestamp = timestamp;
290 vbuf->sequence = sess->sequence_cap++;
291 vbuf->flags = flags;
292 vbuf->timecode = timecode;
293
294 if (sess->should_stop &&
295 atomic_read(&sess->esparser_queued_bufs) <= 1) {
296 const struct v4l2_event ev = { .type = V4L2_EVENT_EOS };
297
298 dev_dbg(dev, "Signaling EOS, sequence_cap = %u\n",
299 sess->sequence_cap - 1);
300 v4l2_event_queue_fh(&sess->fh, &ev);
301 vbuf->flags |= V4L2_BUF_FLAG_LAST;
302 } else if (sess->status == STATUS_NEEDS_RESUME) {
303 /* Mark LAST for drained show frames during a source change */
304 vbuf->flags |= V4L2_BUF_FLAG_LAST;
305 sess->sequence_cap = 0;
306 } else if (sess->should_stop)
307 dev_dbg(dev, "should_stop, %u bufs remain\n",
308 atomic_read(&sess->esparser_queued_bufs));
309
310 dev_dbg(dev, "Buffer %u done, ts = %llu, flags = %08X\n",
311 vbuf->vb2_buf.index, timestamp, flags);
312 vbuf->field = field;
313 v4l2_m2m_buf_done(vbuf, VB2_BUF_STATE_DONE);
314
315 /* Buffer done probably means the vififo got freed */
316 schedule_work(&sess->esparser_queue_work);
317 }
318
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week
gpio-rockchip.c:undefined reference to `irq_gc_mask_set_bit'
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 8ab774587903771821b59471cc723bba6d893942
commit: 936ee2675eee1faca0dcdfa79165c7990422e0fc gpio/rockchip: add driver for rockchip gpio
date: 3 months ago
config: arm64-randconfig-r023-20211019 (attached as .config)
compiler: aarch64-linux-gcc (GCC) 11.2.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://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout 936ee2675eee1faca0dcdfa79165c7990422e0fc
# save the attached .config to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=arm64 SHELL=/bin/bash
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
aarch64-linux-ld: Unexpected GOT/PLT entries detected!
aarch64-linux-ld: Unexpected run-time procedure linkages detected!
aarch64-linux-ld: drivers/gpio/gpio-rockchip.o: in function `rockchip_irq_disable':
>> gpio-rockchip.c:(.text+0x454): undefined reference to `irq_gc_mask_set_bit'
aarch64-linux-ld: drivers/gpio/gpio-rockchip.o: in function `rockchip_irq_enable':
>> gpio-rockchip.c:(.text+0x478): undefined reference to `irq_gc_mask_clr_bit'
aarch64-linux-ld: drivers/gpio/gpio-rockchip.o: in function `rockchip_interrupts_register':
>> gpio-rockchip.c:(.text+0x518): undefined reference to `irq_generic_chip_ops'
aarch64-linux-ld: drivers/gpio/gpio-rockchip.o: relocation R_AARCH64_ADR_PREL_PG_HI21 against symbol `irq_generic_chip_ops' which may bind externally can not be used when making a shared object; recompile with -fPIC
gpio-rockchip.c:(.text+0x518): dangerous relocation: unsupported relocation
>> aarch64-linux-ld: gpio-rockchip.c:(.text+0x528): undefined reference to `irq_generic_chip_ops'
>> aarch64-linux-ld: gpio-rockchip.c:(.text+0x594): undefined reference to `__irq_alloc_domain_generic_chips'
>> aarch64-linux-ld: gpio-rockchip.c:(.text+0x5cc): undefined reference to `irq_get_domain_generic_chip'
>> aarch64-linux-ld: gpio-rockchip.c:(.text+0x5e0): undefined reference to `irq_gc_ack_set_bit'
aarch64-linux-ld: drivers/gpio/gpio-rockchip.o: relocation R_AARCH64_ADR_PREL_PG_HI21 against symbol `irq_gc_ack_set_bit' which may bind externally can not be used when making a shared object; recompile with -fPIC
gpio-rockchip.c:(.text+0x5e0): dangerous relocation: unsupported relocation
>> aarch64-linux-ld: gpio-rockchip.c:(.text+0x5e4): undefined reference to `irq_gc_mask_set_bit'
aarch64-linux-ld: drivers/gpio/gpio-rockchip.o: relocation R_AARCH64_ADR_PREL_PG_HI21 against symbol `irq_gc_mask_set_bit' which may bind externally can not be used when making a shared object; recompile with -fPIC
gpio-rockchip.c:(.text+0x5e4): dangerous relocation: unsupported relocation
aarch64-linux-ld: gpio-rockchip.c:(.text+0x5e8): undefined reference to `irq_gc_ack_set_bit'
aarch64-linux-ld: gpio-rockchip.c:(.text+0x5ec): undefined reference to `irq_gc_mask_set_bit'
>> aarch64-linux-ld: gpio-rockchip.c:(.text+0x5f0): undefined reference to `irq_gc_mask_clr_bit'
aarch64-linux-ld: drivers/gpio/gpio-rockchip.o: relocation R_AARCH64_ADR_PREL_PG_HI21 against symbol `irq_gc_mask_clr_bit' which may bind externally can not be used when making a shared object; recompile with -fPIC
gpio-rockchip.c:(.text+0x5f0): dangerous relocation: unsupported relocation
aarch64-linux-ld: gpio-rockchip.c:(.text+0x5f8): undefined reference to `irq_gc_mask_clr_bit'
>> aarch64-linux-ld: gpio-rockchip.c:(.text+0x604): undefined reference to `irq_gc_set_wake'
aarch64-linux-ld: drivers/gpio/gpio-rockchip.o: relocation R_AARCH64_ADR_PREL_PG_HI21 against symbol `irq_gc_set_wake' which may bind externally can not be used when making a shared object; recompile with -fPIC
gpio-rockchip.c:(.text+0x604): dangerous relocation: unsupported relocation
aarch64-linux-ld: gpio-rockchip.c:(.text+0x60c): undefined reference to `irq_gc_set_wake'
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week
Re: [PATCH v5] Add payload to be 32-bit aligned to fix dropped packets
by kernel test robot
Hi Kumar,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on linus/master]
[also build test WARNING on v5.16-rc1 next-20211115]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Kumar-Thangavel/Add-payload-to-b...
base: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 8ab774587903771821b59471cc723bba6d893942
config: riscv-randconfig-r026-20211115 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project fbe72e41b99dc7994daac300d208a955be3e4a0a)
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 riscv cross compiling tool for clang build
# apt-get install binutils-riscv64-linux-gnu
# https://github.com/0day-ci/linux/commit/dc2cde90f4de825ff92f0ad81bc70e132...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Kumar-Thangavel/Add-payload-to-be-32-bit-aligned-to-fix-dropped-packets/20211115-224945
git checkout dc2cde90f4de825ff92f0ad81bc70e132c6ad6ec
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=riscv
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> net/ncsi/ncsi-cmd.c:298:9: warning: comparison of distinct pointer types ('typeof (payload) *' (aka 'int *') and 'typeof (padding_bytes) *' (aka 'const unsigned short *')) [-Wcompare-distinct-pointer-types]
len += max(payload, padding_bytes);
^~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/minmax.h:52:19: note: expanded from macro 'max'
#define max(x, y) __careful_cmp(x, y, >)
^~~~~~~~~~~~~~~~~~~~~~
include/linux/minmax.h:36:24: note: expanded from macro '__careful_cmp'
__builtin_choose_expr(__safe_cmp(x, y), \
^~~~~~~~~~~~~~~~
include/linux/minmax.h:26:4: note: expanded from macro '__safe_cmp'
(__typecheck(x, y) && __no_side_effects(x, y))
^~~~~~~~~~~~~~~~~
include/linux/minmax.h:20:28: note: expanded from macro '__typecheck'
(!!(sizeof((typeof(x) *)1 == (typeof(y) *)1)))
~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~
1 warning generated.
vim +298 net/ncsi/ncsi-cmd.c
273
274 static struct ncsi_request *ncsi_alloc_command(struct ncsi_cmd_arg *nca)
275 {
276 struct ncsi_dev_priv *ndp = nca->ndp;
277 struct ncsi_dev *nd = &ndp->ndev;
278 struct net_device *dev = nd->dev;
279 int hlen = LL_RESERVED_SPACE(dev);
280 int tlen = dev->needed_tailroom;
281 int payload;
282 int len = hlen + tlen;
283 struct sk_buff *skb;
284 struct ncsi_request *nr;
285
286 nr = ncsi_alloc_request(ndp, nca->req_flags);
287 if (!nr)
288 return NULL;
289
290 /* NCSI command packet has 16-bytes header, payload, 4 bytes checksum.
291 * Payload needs padding so that the checksum field following payload is
292 * aligned to 32-bit boundary.
293 * The packet needs padding if its payload is less than 26 bytes to
294 * meet 64 bytes minimal ethernet frame length.
295 */
296 len += sizeof(struct ncsi_cmd_pkt_hdr) + 4;
297 payload = ALIGN(nca->payload, 4);
> 298 len += max(payload, padding_bytes);
299
300 /* Allocate skb */
301 skb = alloc_skb(len, GFP_ATOMIC);
302 if (!skb) {
303 ncsi_free_request(nr);
304 return NULL;
305 }
306
307 nr->cmd = skb;
308 skb_reserve(skb, hlen);
309 skb_reset_network_header(skb);
310
311 skb->dev = dev;
312 skb->protocol = htons(ETH_P_NCSI);
313
314 return nr;
315 }
316
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week
arch/mips/boot/compressed/../../lib/bswapsi.c:5:22: warning: no previous prototype for '__bswapsi2'
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 8ab774587903771821b59471cc723bba6d893942
commit: f78b25ee922ef6faf59a258af1b9388ca894cfd9 mips: decompressor: do not copy source files while building
date: 5 days ago
config: mips-loongson1b_defconfig (attached as .config)
compiler: mipsel-linux-gcc (GCC) 11.2.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://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit...
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout f78b25ee922ef6faf59a258af1b9388ca894cfd9
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross ARCH=mips
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
In file included from arch/mips/boot/compressed/bswapsi.c:2:
>> arch/mips/boot/compressed/../../lib/bswapsi.c:5:22: warning: no previous prototype for '__bswapsi2' [-Wmissing-prototypes]
5 | unsigned int notrace __bswapsi2(unsigned int u)
| ^~~~~~~~~~
--
In file included from arch/mips/boot/compressed/ashldi3.c:2:
>> arch/mips/boot/compressed/../../../../lib/ashldi3.c:9:19: warning: no previous prototype for '__ashldi3' [-Wmissing-prototypes]
9 | long long notrace __ashldi3(long long u, word_type b)
| ^~~~~~~~~
vim +/__bswapsi2 +5 arch/mips/boot/compressed/../../lib/bswapsi.c
1ee3630a3e57f3 Ralf Baechle 2015-09-29 4
aedcfbe06558a9 Harvey Hunt 2016-05-25 @5 unsigned int notrace __bswapsi2(unsigned int u)
1ee3630a3e57f3 Ralf Baechle 2015-09-29 6 {
1ee3630a3e57f3 Ralf Baechle 2015-09-29 7 return (((u) & 0xff000000) >> 24) |
1ee3630a3e57f3 Ralf Baechle 2015-09-29 8 (((u) & 0x00ff0000) >> 8) |
1ee3630a3e57f3 Ralf Baechle 2015-09-29 9 (((u) & 0x0000ff00) << 8) |
1ee3630a3e57f3 Ralf Baechle 2015-09-29 10 (((u) & 0x000000ff) << 24);
1ee3630a3e57f3 Ralf Baechle 2015-09-29 11 }
1ee3630a3e57f3 Ralf Baechle 2015-09-29 12
:::::: The code at line 5 was first introduced by commit
:::::: aedcfbe06558a9f53002e82d5be64c6c94687726 MIPS: lib: Mark intrinsics notrace
:::::: TO: Harvey Hunt <harvey.hunt(a)imgtec.com>
:::::: CC: Ralf Baechle <ralf(a)linux-mips.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week
[toke:xdp-traffic-gen-01 3/4] (.text+0xf80): multiple definition of `page_pool_alloc_pages'; drivers/scsi/libfc/fc_libfc.o:(.text+0x340): first defined here
by kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/toke/linux.git xdp-traffic-gen-01
head: fe25e608904cabc4e9453734eb34c20c2c5b3533
commit: 9e4b84b16e4445480899d2b2827d65431bf779b6 [3/4] bpf: Add XDP_REDIRECT support to XDP for bpf_prog_run()
config: x86_64-buildonly-randconfig-r005-20211115 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
# https://git.kernel.org/pub/scm/linux/kernel/git/toke/linux.git/commit/?id...
git remote add toke https://git.kernel.org/pub/scm/linux/kernel/git/toke/linux.git
git fetch --no-tags toke xdp-traffic-gen-01
git checkout 9e4b84b16e4445480899d2b2827d65431bf779b6
# save the attached .config to linux build tree
mkdir build_dir
make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
ld: drivers/scsi/libfc/fc_disc.o: in function `page_pool_alloc_pages':
>> (.text+0xf80): multiple definition of `page_pool_alloc_pages'; drivers/scsi/libfc/fc_libfc.o:(.text+0x340): first defined here
ld: drivers/scsi/libfc/fc_exch.o: in function `page_pool_alloc_pages':
(.text+0x2c40): multiple definition of `page_pool_alloc_pages'; drivers/scsi/libfc/fc_libfc.o:(.text+0x340): first defined here
ld: drivers/scsi/libfc/fc_elsct.o: in function `page_pool_alloc_pages':
(.text+0x1b80): multiple definition of `page_pool_alloc_pages'; drivers/scsi/libfc/fc_libfc.o:(.text+0x340): first defined here
ld: drivers/scsi/libfc/fc_frame.o: in function `page_pool_alloc_pages':
(.text+0x140): multiple definition of `page_pool_alloc_pages'; drivers/scsi/libfc/fc_libfc.o:(.text+0x340): first defined here
ld: drivers/scsi/libfc/fc_lport.o: in function `page_pool_alloc_pages':
(.text+0x2c40): multiple definition of `page_pool_alloc_pages'; drivers/scsi/libfc/fc_libfc.o:(.text+0x340): first defined here
ld: drivers/scsi/libfc/fc_rport.o: in function `page_pool_alloc_pages':
(.text+0x3500): multiple definition of `page_pool_alloc_pages'; drivers/scsi/libfc/fc_libfc.o:(.text+0x340): first defined here
ld: drivers/scsi/libfc/fc_fcp.o: in function `page_pool_alloc_pages':
(.text+0x37c0): multiple definition of `page_pool_alloc_pages'; drivers/scsi/libfc/fc_libfc.o:(.text+0x340): first defined here
ld: drivers/scsi/libfc/fc_npiv.o: in function `page_pool_alloc_pages':
(.text+0x2c0): multiple definition of `page_pool_alloc_pages'; drivers/scsi/libfc/fc_libfc.o:(.text+0x340): first defined here
--
ld: drivers/scsi/fcoe/fcoe_transport.o: in function `page_pool_alloc_pages':
>> (.text+0x1380): multiple definition of `page_pool_alloc_pages'; drivers/scsi/fcoe/fcoe_ctlr.o:(.text+0x4d40): first defined here
ld: drivers/scsi/fcoe/fcoe_sysfs.o: in function `page_pool_alloc_pages':
(.text+0x1480): multiple definition of `page_pool_alloc_pages'; drivers/scsi/fcoe/fcoe_ctlr.o:(.text+0x4d40): first defined here
--
ld: drivers/usb/gadget/function/rndis.o: in function `page_pool_alloc_pages':
>> (.text+0xec0): multiple definition of `page_pool_alloc_pages'; drivers/usb/gadget/function/f_rndis.o:(.text+0x1340): first defined here
--
ld: net/bluetooth/hci_core.o: in function `page_pool_alloc_pages':
>> (.text+0x3f00): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/hci_conn.o: in function `page_pool_alloc_pages':
(.text+0x1000): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/hci_event.o: in function `page_pool_alloc_pages':
hci_event.c:(.text+0x77c0): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/mgmt.o: in function `page_pool_alloc_pages':
mgmt.c:(.text+0xd540): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/hci_sock.o: in function `page_pool_alloc_pages':
(.text+0x1240): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/hci_sysfs.o: in function `page_pool_alloc_pages':
hci_sysfs.c:(.text+0xc0): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/l2cap_core.o: in function `page_pool_alloc_pages':
(.text+0x4700): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/l2cap_sock.o: in function `page_pool_alloc_pages':
(.text+0x2c40): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/smp.o: in function `page_pool_alloc_pages':
smp.c:(.text+0x3bc0): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/lib.o: in function `page_pool_alloc_pages':
(.text+0x1c0): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/hci_request.o: in function `page_pool_alloc_pages':
(.text+0x4c0): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/mgmt_util.o: in function `page_pool_alloc_pages':
mgmt_util.c:(.text+0x100): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/mgmt_config.o: in function `page_pool_alloc_pages':
mgmt_config.c:(.text+0x0): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/hci_codec.o: in function `page_pool_alloc_pages':
hci_codec.c:(.text+0x280): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/eir.o: in function `page_pool_alloc_pages':
eir.c:(.text+0x0): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/sco.o: in function `page_pool_alloc_pages':
sco.c:(.text+0x2580): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/leds.o: in function `page_pool_alloc_pages':
leds.c:(.text+0x40): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/msft.o: in function `page_pool_alloc_pages':
msft.c:(.text+0x840): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
ld: net/bluetooth/selftest.o: in function `page_pool_alloc_pages':
selftest.c:(.text+0x80): multiple definition of `page_pool_alloc_pages'; net/bluetooth/af_bluetooth.o:(.text+0x1340): first defined here
--
ld: net/bluetooth/rfcomm/sock.o: in function `page_pool_alloc_pages':
>> sock.c:(.text+0x18c0): multiple definition of `page_pool_alloc_pages'; net/bluetooth/rfcomm/core.o:core.c:(.text+0x1000): first defined here
--
ld: drivers/bus/mhi/core/main.o: in function `page_pool_alloc_pages':
>> (.text+0x2c0): multiple definition of `page_pool_alloc_pages'; drivers/bus/mhi/core/init.o:(.text+0x540): first defined here
ld: drivers/bus/mhi/core/pm.o: in function `page_pool_alloc_pages':
(.text+0x300): multiple definition of `page_pool_alloc_pages'; drivers/bus/mhi/core/init.o:(.text+0x540): first defined here
ld: drivers/bus/mhi/core/boot.o: in function `page_pool_alloc_pages':
(.text+0x540): multiple definition of `page_pool_alloc_pages'; drivers/bus/mhi/core/init.o:(.text+0x540): first defined here
ld: drivers/bus/mhi/core/debugfs.o: in function `page_pool_alloc_pages':
debugfs.c:(.text+0xbc0): multiple definition of `page_pool_alloc_pages'; drivers/bus/mhi/core/init.o:(.text+0x540): first defined here
--
ld: drivers/staging/ks7010/ks_wlan_net.o: in function `page_pool_alloc_pages':
>> ks_wlan_net.c:(.text+0x2c80): multiple definition of `page_pool_alloc_pages'; drivers/staging/ks7010/ks_hostif.o:ks_hostif.c:(.text+0x24c0): first defined here
ld: drivers/staging/ks7010/ks7010_sdio.o: in function `page_pool_alloc_pages':
ks7010_sdio.c:(.text+0xdc0): multiple definition of `page_pool_alloc_pages'; drivers/staging/ks7010/ks_hostif.o:ks_hostif.c:(.text+0x24c0): first defined here
--
ld: drivers/bluetooth/btmrvl_debugfs.o: in function `page_pool_alloc_pages':
>> btmrvl_debugfs.c:(.text+0x300): multiple definition of `page_pool_alloc_pages'; drivers/bluetooth/btmrvl_main.o:(.text+0x1440): first defined here
--
ld: net/mac802154/rx.o: in function `page_pool_alloc_pages':
>> (.text+0x40): multiple definition of `page_pool_alloc_pages'; net/mac802154/main.o:(.text+0x480): first defined here
ld: net/mac802154/tx.o: in function `page_pool_alloc_pages':
tx.c:(.text+0x100): multiple definition of `page_pool_alloc_pages'; net/mac802154/main.o:(.text+0x480): first defined here
ld: net/mac802154/mac_cmd.o: in function `page_pool_alloc_pages':
mac_cmd.c:(.text+0x380): multiple definition of `page_pool_alloc_pages'; net/mac802154/main.o:(.text+0x480): first defined here
ld: net/mac802154/mib.o: in function `page_pool_alloc_pages':
mib.c:(.text+0x0): multiple definition of `page_pool_alloc_pages'; net/mac802154/main.o:(.text+0x480): first defined here
ld: net/mac802154/iface.o: in function `page_pool_alloc_pages':
iface.c:(.text+0xd00): multiple definition of `page_pool_alloc_pages'; net/mac802154/main.o:(.text+0x480): first defined here
ld: net/mac802154/llsec.o: in function `page_pool_alloc_pages':
llsec.c:(.text+0x980): multiple definition of `page_pool_alloc_pages'; net/mac802154/main.o:(.text+0x480): first defined here
ld: net/mac802154/util.o: in function `page_pool_alloc_pages':
(.text+0x100): multiple definition of `page_pool_alloc_pages'; net/mac802154/main.o:(.text+0x480): first defined here
ld: net/mac802154/cfg.o: in function `page_pool_alloc_pages':
cfg.c:(.text+0x7c0): multiple definition of `page_pool_alloc_pages'; net/mac802154/main.o:(.text+0x480): first defined here
ld: net/mac802154/trace.o: in function `page_pool_alloc_pages':
trace.c:(.text+0x0): multiple definition of `page_pool_alloc_pages'; net/mac802154/main.o:(.text+0x480): first defined here
--
ld: net/appletalk/ddp.o: in function `page_pool_alloc_pages':
>> (.text+0x2ac0): multiple definition of `page_pool_alloc_pages'; net/appletalk/aarp.o:(.text+0xe80): first defined here
ld: net/appletalk/dev.o: in function `page_pool_alloc_pages':
(.text+0xc0): multiple definition of `page_pool_alloc_pages'; net/appletalk/aarp.o:(.text+0xe80): first defined here
--
ld: net/hsr/hsr_framereg.o: in function `page_pool_alloc_pages':
>> hsr_framereg.c:(.text+0x1c0): multiple definition of `page_pool_alloc_pages'; net/hsr/hsr_main.o:(.text+0x380): first defined here
ld: net/hsr/hsr_device.o: in function `page_pool_alloc_pages':
(.text+0x940): multiple definition of `page_pool_alloc_pages'; net/hsr/hsr_main.o:(.text+0x380): first defined here
ld: net/hsr/hsr_netlink.o: in function `page_pool_alloc_pages':
hsr_netlink.c:(.text+0x980): multiple definition of `page_pool_alloc_pages'; net/hsr/hsr_main.o:(.text+0x380): first defined here
ld: net/hsr/hsr_slave.o: in function `page_pool_alloc_pages':
hsr_slave.c:(.text+0x180): multiple definition of `page_pool_alloc_pages'; net/hsr/hsr_main.o:(.text+0x380): first defined here
ld: net/hsr/hsr_forward.o: in function `page_pool_alloc_pages':
hsr_forward.c:(.text+0xc0): multiple definition of `page_pool_alloc_pages'; net/hsr/hsr_main.o:(.text+0x380): first defined here
ld: net/hsr/hsr_debugfs.o: in function `page_pool_alloc_pages':
hsr_debugfs.c:(.text+0x200): multiple definition of `page_pool_alloc_pages'; net/hsr/hsr_main.o:(.text+0x380): first defined here
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week
[intel-lts:5.10/yocto 17552/17586] arch/x86/include/asm/pgtable.h:1323:13: error: invalid output size for constraint '+a'
by kernel test robot
tree: https://github.com/intel/linux-intel-lts.git 5.10/yocto
head: 2202f30edc3f9f14646647cd48c4bd4bfd12e9b7
commit: a18c5b9b95c27d092a997152c4e640dd6c235b82 [17552/17586] x86/mm: Update ptep_set_wrprotect() and pmdp_set_wrprotect() for transition from _PAGE_DIRTY to _PAGE_COW
config: i386-randconfig-a013-20211115 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project fbe72e41b99dc7994daac300d208a955be3e4a0a)
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/linux-intel-lts/commit/a18c5b9b95c27d092a997152c...
git remote add intel-lts https://github.com/intel/linux-intel-lts.git
git fetch --no-tags intel-lts 5.10/yocto
git checkout a18c5b9b95c27d092a997152c4e640dd6c235b82
# save the attached .config to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=i386 prepare
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
In file included from arch/x86/kernel/asm-offsets.c:13:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:28:
In file included from include/linux/cgroup-defs.h:22:
In file included from include/linux/bpf-cgroup.h:5:
In file included from include/linux/bpf.h:21:
In file included from include/linux/kallsyms.h:12:
In file included from include/linux/mm.h:33:
In file included from include/linux/pgtable.h:6:
>> arch/x86/include/asm/pgtable.h:1323:13: error: invalid output size for constraint '+a'
} while (!try_cmpxchg(&ptep->pte, &old_pte.pte, new_pte.pte));
^
arch/x86/include/asm/cmpxchg.h:225:2: note: expanded from macro 'try_cmpxchg'
__try_cmpxchg((ptr), (pold), (new), sizeof(*(ptr)))
^
arch/x86/include/asm/cmpxchg.h:222:2: note: expanded from macro '__try_cmpxchg'
__raw_try_cmpxchg((ptr), (pold), (new), (size), LOCK_PREFIX)
^
arch/x86/include/asm/cmpxchg.h:172:23: note: expanded from macro '__raw_try_cmpxchg'
[old] "+a" (__old) \
^
In file included from arch/x86/kernel/asm-offsets.c:13:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:28:
In file included from include/linux/cgroup-defs.h:22:
In file included from include/linux/bpf-cgroup.h:5:
In file included from include/linux/bpf.h:21:
In file included from include/linux/kallsyms.h:12:
In file included from include/linux/mm.h:33:
In file included from include/linux/pgtable.h:6:
>> arch/x86/include/asm/pgtable.h:1323:13: error: invalid output size for constraint '+a'
arch/x86/include/asm/cmpxchg.h:225:2: note: expanded from macro 'try_cmpxchg'
__try_cmpxchg((ptr), (pold), (new), sizeof(*(ptr)))
^
arch/x86/include/asm/cmpxchg.h:222:2: note: expanded from macro '__try_cmpxchg'
__raw_try_cmpxchg((ptr), (pold), (new), (size), LOCK_PREFIX)
^
arch/x86/include/asm/cmpxchg.h:184:23: note: expanded from macro '__raw_try_cmpxchg'
[old] "+a" (__old) \
^
In file included from arch/x86/kernel/asm-offsets.c:13:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:28:
In file included from include/linux/cgroup-defs.h:22:
In file included from include/linux/bpf-cgroup.h:5:
In file included from include/linux/bpf.h:21:
In file included from include/linux/kallsyms.h:12:
In file included from include/linux/mm.h:33:
In file included from include/linux/pgtable.h:6:
>> arch/x86/include/asm/pgtable.h:1323:13: error: invalid output size for constraint '+a'
arch/x86/include/asm/cmpxchg.h:225:2: note: expanded from macro 'try_cmpxchg'
__try_cmpxchg((ptr), (pold), (new), sizeof(*(ptr)))
^
arch/x86/include/asm/cmpxchg.h:222:2: note: expanded from macro '__try_cmpxchg'
__raw_try_cmpxchg((ptr), (pold), (new), (size), LOCK_PREFIX)
^
arch/x86/include/asm/cmpxchg.h:196:23: note: expanded from macro '__raw_try_cmpxchg'
[old] "+a" (__old) \
^
In file included from arch/x86/kernel/asm-offsets.c:13:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:28:
In file included from include/linux/cgroup-defs.h:22:
In file included from include/linux/bpf-cgroup.h:5:
In file included from include/linux/bpf.h:21:
In file included from include/linux/kallsyms.h:12:
In file included from include/linux/mm.h:33:
In file included from include/linux/pgtable.h:6:
>> arch/x86/include/asm/pgtable.h:1323:13: error: invalid output size for constraint '+a'
arch/x86/include/asm/cmpxchg.h:225:2: note: expanded from macro 'try_cmpxchg'
__try_cmpxchg((ptr), (pold), (new), sizeof(*(ptr)))
^
arch/x86/include/asm/cmpxchg.h:222:2: note: expanded from macro '__try_cmpxchg'
__raw_try_cmpxchg((ptr), (pold), (new), (size), LOCK_PREFIX)
^
arch/x86/include/asm/cmpxchg.h:208:23: note: expanded from macro '__raw_try_cmpxchg'
[old] "+a" (__old) \
^
In file included from arch/x86/kernel/asm-offsets.c:13:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:28:
In file included from include/linux/cgroup-defs.h:22:
In file included from include/linux/bpf-cgroup.h:5:
In file included from include/linux/bpf.h:21:
In file included from include/linux/kallsyms.h:12:
In file included from include/linux/mm.h:33:
In file included from include/linux/pgtable.h:6:
arch/x86/include/asm/pgtable.h:1385:13: error: invalid output size for constraint '+a'
} while (!try_cmpxchg((pmdval_t *)pmdp, (pmdval_t *)&old_pmd, pmd_val(new_pmd)));
^
arch/x86/include/asm/cmpxchg.h:225:2: note: expanded from macro 'try_cmpxchg'
__try_cmpxchg((ptr), (pold), (new), sizeof(*(ptr)))
^
arch/x86/include/asm/cmpxchg.h:222:2: note: expanded from macro '__try_cmpxchg'
__raw_try_cmpxchg((ptr), (pold), (new), (size), LOCK_PREFIX)
^
arch/x86/include/asm/cmpxchg.h:172:23: note: expanded from macro '__raw_try_cmpxchg'
[old] "+a" (__old) \
^
In file included from arch/x86/kernel/asm-offsets.c:13:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:28:
In file included from include/linux/cgroup-defs.h:22:
In file included from include/linux/bpf-cgroup.h:5:
In file included from include/linux/bpf.h:21:
In file included from include/linux/kallsyms.h:12:
In file included from include/linux/mm.h:33:
In file included from include/linux/pgtable.h:6:
arch/x86/include/asm/pgtable.h:1385:13: error: invalid output size for constraint '+a'
arch/x86/include/asm/cmpxchg.h:225:2: note: expanded from macro 'try_cmpxchg'
__try_cmpxchg((ptr), (pold), (new), sizeof(*(ptr)))
^
arch/x86/include/asm/cmpxchg.h:222:2: note: expanded from macro '__try_cmpxchg'
__raw_try_cmpxchg((ptr), (pold), (new), (size), LOCK_PREFIX)
^
arch/x86/include/asm/cmpxchg.h:184:23: note: expanded from macro '__raw_try_cmpxchg'
[old] "+a" (__old) \
^
In file included from arch/x86/kernel/asm-offsets.c:13:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:28:
In file included from include/linux/cgroup-defs.h:22:
In file included from include/linux/bpf-cgroup.h:5:
In file included from include/linux/bpf.h:21:
In file included from include/linux/kallsyms.h:12:
In file included from include/linux/mm.h:33:
In file included from include/linux/pgtable.h:6:
arch/x86/include/asm/pgtable.h:1385:13: error: invalid output size for constraint '+a'
arch/x86/include/asm/cmpxchg.h:225:2: note: expanded from macro 'try_cmpxchg'
__try_cmpxchg((ptr), (pold), (new), sizeof(*(ptr)))
^
arch/x86/include/asm/cmpxchg.h:222:2: note: expanded from macro '__try_cmpxchg'
__raw_try_cmpxchg((ptr), (pold), (new), (size), LOCK_PREFIX)
^
arch/x86/include/asm/cmpxchg.h:196:23: note: expanded from macro '__raw_try_cmpxchg'
[old] "+a" (__old) \
^
In file included from arch/x86/kernel/asm-offsets.c:13:
In file included from include/linux/suspend.h:5:
In file included from include/linux/swap.h:9:
In file included from include/linux/memcontrol.h:13:
In file included from include/linux/cgroup.h:28:
In file included from include/linux/cgroup-defs.h:22:
In file included from include/linux/bpf-cgroup.h:5:
In file included from include/linux/bpf.h:21:
In file included from include/linux/kallsyms.h:12:
In file included from include/linux/mm.h:33:
In file included from include/linux/pgtable.h:6:
arch/x86/include/asm/pgtable.h:1385:13: error: invalid output size for constraint '+a'
arch/x86/include/asm/cmpxchg.h:225:2: note: expanded from macro 'try_cmpxchg'
__try_cmpxchg((ptr), (pold), (new), sizeof(*(ptr)))
^
arch/x86/include/asm/cmpxchg.h:222:2: note: expanded from macro '__try_cmpxchg'
__raw_try_cmpxchg((ptr), (pold), (new), (size), LOCK_PREFIX)
^
arch/x86/include/asm/cmpxchg.h:208:23: note: expanded from macro '__raw_try_cmpxchg'
[old] "+a" (__old) \
^
8 errors generated.
make[2]: *** [scripts/Makefile.build:117: arch/x86/kernel/asm-offsets.s] Error 1
make[2]: Target '__build' not remade because of errors.
make[1]: *** [Makefile:1215: prepare0] Error 2
make[1]: Target 'prepare' not remade because of errors.
vim +1323 arch/x86/include/asm/pgtable.h
1304
1305 #define __HAVE_ARCH_PTEP_SET_WRPROTECT
1306 static inline void ptep_set_wrprotect(struct mm_struct *mm,
1307 unsigned long addr, pte_t *ptep)
1308 {
1309 /*
1310 * If Shadow Stack is enabled, pte_wrprotect() moves _PAGE_DIRTY
1311 * to _PAGE_COW (see comments at pte_wrprotect()).
1312 * When a thread reads a RW=1, Dirty=0 PTE and before changing it
1313 * to RW=0, Dirty=0, another thread could have written to the page
1314 * and the PTE is RW=1, Dirty=1 now. Use try_cmpxchg() to detect
1315 * PTE changes and update old_pte, then try again.
1316 */
1317 if (cpu_feature_enabled(X86_FEATURE_SHSTK)) {
1318 pte_t old_pte, new_pte;
1319
1320 old_pte = READ_ONCE(*ptep);
1321 do {
1322 new_pte = pte_wrprotect(old_pte);
> 1323 } while (!try_cmpxchg(&ptep->pte, &old_pte.pte, new_pte.pte));
1324
1325 return;
1326 }
1327 clear_bit(_PAGE_BIT_RW, (unsigned long *)&ptep->pte);
1328 }
1329
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week
[xilinx-xlnx:master 10436/12535] drivers/net/ethernet/xilinx/xilinx_tsn_ep.c:370:11: error: 'struct axienet_local' has no member named 'qbv_regs'
by kernel test robot
tree: https://github.com/Xilinx/linux-xlnx master
head: 0a88ef03d3015782318b4bc94ceb20dca375a01b
commit: 1a67745a47fc9601447f7d8aff40ddb1023848d5 [10436/12535] net: xilinx: Access qdv address and associated properties separately
config: sparc64-randconfig-r021-20211103 (attached as .config)
compiler: sparc64-linux-gcc (GCC) 11.2.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/Xilinx/linux-xlnx/commit/1a67745a47fc9601447f7d8aff40d...
git remote add xilinx-xlnx https://github.com/Xilinx/linux-xlnx
git fetch --no-tags xilinx-xlnx master
git checkout 1a67745a47fc9601447f7d8aff40ddb1023848d5
# save the attached .config to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=sparc64 SHELL=/bin/bash
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
drivers/net/ethernet/xilinx/xilinx_tsn_ep.c: In function 'tsn_ep_open':
drivers/net/ethernet/xilinx/xilinx_tsn_ep.c:57:23: error: implicit declaration of function 'axienet_mcdma_rx_q_init'; did you mean 'axienet_dma_q_init'? [-Werror=implicit-function-declaration]
57 | ret = axienet_mcdma_rx_q_init(ndev, q);
| ^~~~~~~~~~~~~~~~~~~~~~~
| axienet_dma_q_init
drivers/net/ethernet/xilinx/xilinx_tsn_ep.c:60:46: error: 'axienet_mcdma_rx_irq' undeclared (first use in this function); did you mean 'axienet_rx_irq'?
60 | ret = request_irq(q->rx_irq, axienet_mcdma_rx_irq,
| ^~~~~~~~~~~~~~~~~~~~
| axienet_rx_irq
drivers/net/ethernet/xilinx/xilinx_tsn_ep.c:60:46: note: each undeclared identifier is reported only once for each function it appears in
drivers/net/ethernet/xilinx/xilinx_tsn_ep.c:66:30: error: 'axienet_mcdma_err_handler' undeclared (first use in this function); did you mean 'axienet_dma_err_handler'?
66 | axienet_mcdma_err_handler,
| ^~~~~~~~~~~~~~~~~~~~~~~~~
| axienet_dma_err_handler
drivers/net/ethernet/xilinx/xilinx_tsn_ep.c:74:23: error: implicit declaration of function 'axienet_mcdma_tx_q_init'; did you mean 'axienet_dma_q_init'? [-Werror=implicit-function-declaration]
74 | ret = axienet_mcdma_tx_q_init(ndev, q);
| ^~~~~~~~~~~~~~~~~~~~~~~
| axienet_dma_q_init
drivers/net/ethernet/xilinx/xilinx_tsn_ep.c:76:46: error: 'axienet_mcdma_tx_irq' undeclared (first use in this function); did you mean 'axienet_tx_irq'?
76 | ret = request_irq(q->tx_irq, axienet_mcdma_tx_irq,
| ^~~~~~~~~~~~~~~~~~~~
| axienet_tx_irq
drivers/net/ethernet/xilinx/xilinx_tsn_ep.c: In function 'tsn_mcdma_probe':
drivers/net/ethernet/xilinx/xilinx_tsn_ep.c:270:9: error: implicit declaration of function 'axienet_mcdma_rx_probe'; did you mean 'axienet_tsn_probe'? [-Werror=implicit-function-declaration]
270 | axienet_mcdma_rx_probe(pdev, lp, ndev);
| ^~~~~~~~~~~~~~~~~~~~~~
| axienet_tsn_probe
drivers/net/ethernet/xilinx/xilinx_tsn_ep.c:271:9: error: implicit declaration of function 'axienet_mcdma_tx_probe'; did you mean 'axienet_tsn_probe'? [-Werror=implicit-function-declaration]
271 | axienet_mcdma_tx_probe(pdev, np, lp);
| ^~~~~~~~~~~~~~~~~~~~~~
| axienet_tsn_probe
drivers/net/ethernet/xilinx/xilinx_tsn_ep.c: In function 'tsn_ep_probe':
>> drivers/net/ethernet/xilinx/xilinx_tsn_ep.c:370:11: error: 'struct axienet_local' has no member named 'qbv_regs'
370 | lp->qbv_regs = lp->regs;
| ^~
>> drivers/net/ethernet/xilinx/xilinx_tsn_ep.c:374:9: error: implicit declaration of function 'axienet_qbv_init'; did you mean 'axienet_dma_q_init'? [-Werror=implicit-function-declaration]
374 | axienet_qbv_init(ndev);
| ^~~~~~~~~~~~~~~~
| axienet_dma_q_init
drivers/net/ethernet/xilinx/xilinx_tsn_ep.c: In function 'tsn_ep_remove':
>> drivers/net/ethernet/xilinx/xilinx_tsn_ep.c:392:9: error: implicit declaration of function 'axienet_qbv_remove'; did you mean 'axienet_bd_free'? [-Werror=implicit-function-declaration]
392 | axienet_qbv_remove(ndev);
| ^~~~~~~~~~~~~~~~~~
| axienet_bd_free
cc1: some warnings being treated as errors
Kconfig warnings: (for reference only)
WARNING: unmet direct dependencies detected for PTP_1588_CLOCK
Depends on NET && POSIX_TIMERS
Selected by
- XILINX_TSN_PTP && NETDEVICES && ETHERNET && NET_VENDOR_XILINX && XILINX_TSN
Selected by
- XILINX_AXI_EMAC_HWTSTAMP && NETDEVICES && ETHERNET && NET_VENDOR_XILINX && XILINX_AXI_EMAC
vim +370 drivers/net/ethernet/xilinx/xilinx_tsn_ep.c
281
282 /**
283 * tsn_ep_probe - TSN ep pointer probe function.
284 * @pdev: Pointer to platform device structure.
285 *
286 * Return: 0, on success
287 * Non-zero error value on failure.
288 *
289 * This is the probe routine for TSN endpoint driver.
290 */
291 static int tsn_ep_probe(struct platform_device *pdev)
292 {
293 int ret = 0;
294 struct axienet_local *lp;
295 struct net_device *ndev;
296 struct resource *ethres;
297 const void *mac_addr;
298 u16 num_tc = 0;
299 char irq_name[32];
300
301 ndev = alloc_netdev(sizeof(*lp), "ep", NET_NAME_UNKNOWN, ether_setup);
302 if (!ndev)
303 return -ENOMEM;
304
305 platform_set_drvdata(pdev, ndev);
306
307 SET_NETDEV_DEV(ndev, &pdev->dev);
308 ndev->flags &= ~IFF_MULTICAST; /* clear multicast */
309 ndev->features = NETIF_F_SG;
310 ndev->netdev_ops = &ep_netdev_ops;
311
312 /* MTU range: 64 - 9000 */
313 ndev->min_mtu = 64;
314 ndev->max_mtu = XAE_JUMBO_MTU;
315
316 lp = netdev_priv(ndev);
317 lp->ndev = ndev;
318 lp->dev = &pdev->dev;
319 lp->options = XAE_OPTION_DEFAULTS;
320 lp->tx_bd_num = TX_BD_NUM_DEFAULT;
321 lp->rx_bd_num = RX_BD_NUM_DEFAULT;
322
323 /* TODO
324 * there are two temacs or two slaves to ep
325 * get this infor from design?
326 */
327 lp->slaves[0] = NULL;
328 lp->slaves[1] = NULL;
329
330 lp->axienet_config = &tsn_endpoint_cfg;
331
332 lp->max_frm_size = XAE_MAX_VLAN_FRAME_SIZE;
333
334 /* Setup checksum offload, but default to off if not specified */
335 lp->features = 0;
336
337 lp->eth_hasnobuf = of_property_read_bool(pdev->dev.of_node,
338 "xlnx,eth-hasnobuf");
339
340 /* Retrieve the MAC address */
341 mac_addr = of_get_mac_address(pdev->dev.of_node);
342 if (!mac_addr) {
343 dev_err(&pdev->dev, "could not find MAC address\n");
344 goto free_netdev;
345 }
346 if (mac_addr)
347 ether_addr_copy(ndev->dev_addr, mac_addr);
348 if (!is_valid_ether_addr(ndev->dev_addr))
349 eth_hw_addr_random(ndev);
350
351 ret = tsn_mcdma_probe(pdev, lp, ndev);
352 if (ret) {
353 dev_err(&pdev->dev, "Getting MCDMA resource failed\n");
354 goto free_netdev;
355 }
356
357 ret = of_property_read_u16(
358 pdev->dev.of_node, "xlnx,num-tc", &num_tc);
359 if (ret || (num_tc != 2 && num_tc != 3))
360 lp->num_tc = XAE_MAX_TSN_TC;
361 else
362 lp->num_tc = num_tc;
363 /* Map device registers */
364 ethres = platform_get_resource(pdev, IORESOURCE_MEM, 0);
365 lp->regs = devm_ioremap_resource(&pdev->dev, ethres);
366 if (IS_ERR(lp->regs)) {
367 ret = PTR_ERR(lp->regs);
368 goto free_netdev;
369 }
> 370 lp->qbv_regs = lp->regs;
371
372 sprintf(irq_name, "tsn_ep_scheduler_irq");
373 lp->qbv_irq = platform_get_irq_byname(pdev, irq_name);
> 374 axienet_qbv_init(ndev);
375
376 ret = register_netdev(lp->ndev);
377 if (ret)
378 dev_err(lp->dev, "register_netdev() error (%i)\n", ret);
379
380 return ret;
381
382 free_netdev:
383 free_netdev(ndev);
384
385 return ret;
386 }
387
388 static int tsn_ep_remove(struct platform_device *pdev)
389 {
390 struct net_device *ndev = platform_get_drvdata(pdev);
391
> 392 axienet_qbv_remove(ndev);
393 unregister_netdev(ndev);
394
395 free_netdev(ndev);
396
397 return 0;
398 }
399
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week
drivers/md/bcache/util.c:280:16: warning: comparison of distinct pointer types ('typeof (size) *' (aka 'unsigned int *') and 'typeof ((1UL << 16) - offset) *' (aka 'unsigned long *'))
by kernel test robot
tree: https://github.com/0day-ci/linux/commits/UPDATE-20211105-042601/Coly-Li/b...
head: d9060ab757bd509ac3746469f23bf3e22a53f8ea
commit: d9060ab757bd509ac3746469f23bf3e22a53f8ea bcache: Revert "bcache: use bvec_virt"
date: 11 days ago
config: hexagon-randconfig-r045-20211105 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 847a6807332b13f43704327c2d30103ec0347c77)
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/0day-ci/linux/commit/d9060ab757bd509ac3746469f23bf3e22...
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review UPDATE-20211105-042601/Coly-Li/bcache-Revert-bcache-use-bvec_virt/20211103-231427
git checkout d9060ab757bd509ac3746469f23bf3e22a53f8ea
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 ARCH=hexagon
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All warnings (new ones prefixed by >>):
>> drivers/md/bcache/util.c:280:16: warning: comparison of distinct pointer types ('typeof (size) *' (aka 'unsigned int *') and 'typeof ((1UL << 16) - offset) *' (aka 'unsigned long *')) [-Wcompare-distinct-pointer-types]
size_t len = min(size, PAGE_SIZE - offset);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/minmax.h:45:19: note: expanded from macro 'min'
#define min(x, y) __careful_cmp(x, y, <)
^~~~~~~~~~~~~~~~~~~~~~
include/linux/minmax.h:36:24: note: expanded from macro '__careful_cmp'
__builtin_choose_expr(__safe_cmp(x, y), \
^~~~~~~~~~~~~~~~
include/linux/minmax.h:26:4: note: expanded from macro '__safe_cmp'
(__typecheck(x, y) && __no_side_effects(x, y))
^~~~~~~~~~~~~~~~~
include/linux/minmax.h:20:28: note: expanded from macro '__typecheck'
(!!(sizeof((typeof(x) *)1 == (typeof(y) *)1)))
~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~
1 warning generated.
vim +280 drivers/md/bcache/util.c
257
258 /**
259 * bch_bio_alloc_pages - allocates a single page for each bvec in a bio
260 * @bio: bio to allocate pages for
261 * @data: if non-NULL copy data from here into the newly allocate pages
262 * @size: size to allocate
263 * @gfp_mask: flags for allocation
264 *
265 * Allocates pages up to @bio->bi_vcnt.
266 *
267 * Returns 0 on success, -ENOMEM on failure. On failure, any allocated pages are
268 * freed.
269 */
270 int bch_bio_alloc_pages(struct bio *bio, void *data, size_t size, gfp_t gfp)
271 {
272 struct bvec_iter iter;
273 struct bio_vec bv;
274
275 BUG_ON(bio->bi_vcnt);
276
277 while (size) {
278 struct page *page = alloc_page(gfp);
279 unsigned int offset = offset_in_page(page);
> 280 size_t len = min(size, PAGE_SIZE - offset);
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
10 months, 1 week