From: Maciej Fijalkowski <maciej.fijalkowski@intel.com>
To: kernel test robot <lkp@intel.com>
Cc: <bpf@vger.kernel.org>, <ast@kernel.org>, <daniel@iogearbox.net>,
<andrii@kernel.org>, <oe-kbuild-all@lists.linux.dev>,
<netdev@vger.kernel.org>, <magnus.karlsson@intel.com>,
<stfomichev@gmail.com>, <kerneljasonxing@gmail.com>,
Eryk Kubanski <e.kubanski@partner.samsung.com>,
Stanislav Fomichev <sdf@fomichev.me>
Subject: Re: [PATCH v8 bpf] xsk: fix immature cq descriptor production
Date: Thu, 4 Sep 2025 11:31:44 +0200 [thread overview]
Message-ID: <aLlcgDC7s3Dh0NdX@boxer> (raw)
In-Reply-To: <202509031029.iL7rCVvQ-lkp@intel.com>
On Wed, Sep 03, 2025 at 10:29:02AM +0800, kernel test robot wrote:
> Hi Maciej,
>
> kernel test robot noticed the following build warnings:
>
> [auto build test WARNING on bpf/master]
>
> url: https://github.com/intel-lab-lkp/linux/commits/Maciej-Fijalkowski/xsk-fix-immature-cq-descriptor-production/20250903-060850
> base: https://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf.git master
> patch link: https://lore.kernel.org/r/20250902220613.2331265-1-maciej.fijalkowski%40intel.com
> patch subject: [PATCH v8 bpf] xsk: fix immature cq descriptor production
> config: riscv-randconfig-001-20250903 (https://download.01.org/0day-ci/archive/20250903/202509031029.iL7rCVvQ-lkp@intel.com/config)
> compiler: riscv32-linux-gcc (GCC) 8.5.0
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250903/202509031029.iL7rCVvQ-lkp@intel.com/reproduce)
>
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <lkp@intel.com>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202509031029.iL7rCVvQ-lkp@intel.com/
>
> All warnings (new ones prefixed by >>):
>
> net/xdp/xsk.c: In function 'xsk_cq_submit_addr_locked':
> >> net/xdp/xsk.c:572:38: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
> xskq_prod_write_addr(pool->cq, idx, (u64)skb_shinfo(skb)->destructor_arg);
> ^
> net/xdp/xsk.c: In function 'xsk_set_destructor_arg':
> >> net/xdp/xsk.c:625:36: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
> skb_shinfo(skb)->destructor_arg = (void *)addr;
Meh, I suppose uintptr_t casting will help here.
So I'll send v9 in the evening :)
> ^
>
>
> vim +572 net/xdp/xsk.c
>
> 560
> 561 static void xsk_cq_submit_addr_locked(struct xsk_buff_pool *pool,
> 562 struct sk_buff *skb)
> 563 {
> 564 struct xsk_addr_node *pos, *tmp;
> 565 u32 descs_processed = 0;
> 566 unsigned long flags;
> 567 u32 idx;
> 568
> 569 spin_lock_irqsave(&pool->cq_lock, flags);
> 570 idx = xskq_get_prod(pool->cq);
> 571
> > 572 xskq_prod_write_addr(pool->cq, idx, (u64)skb_shinfo(skb)->destructor_arg);
> 573 descs_processed++;
> 574
> 575 if (unlikely(XSKCB(skb)->num_descs > 1)) {
> 576 list_for_each_entry_safe(pos, tmp, &XSKCB(skb)->addrs_list, addr_node) {
> 577 xskq_prod_write_addr(pool->cq, idx + descs_processed,
> 578 pos->addr);
> 579 descs_processed++;
> 580 list_del(&pos->addr_node);
> 581 kmem_cache_free(xsk_tx_generic_cache, pos);
> 582 }
> 583 }
> 584 xskq_prod_submit_n(pool->cq, descs_processed);
> 585 spin_unlock_irqrestore(&pool->cq_lock, flags);
> 586 }
> 587
> 588 static void xsk_cq_cancel_locked(struct xsk_buff_pool *pool, u32 n)
> 589 {
> 590 unsigned long flags;
> 591
> 592 spin_lock_irqsave(&pool->cq_lock, flags);
> 593 xskq_prod_cancel_n(pool->cq, n);
> 594 spin_unlock_irqrestore(&pool->cq_lock, flags);
> 595 }
> 596
> 597 static void xsk_inc_num_desc(struct sk_buff *skb)
> 598 {
> 599 XSKCB(skb)->num_descs++;
> 600 }
> 601
> 602 static u32 xsk_get_num_desc(struct sk_buff *skb)
> 603 {
> 604 return XSKCB(skb)->num_descs;
> 605 }
> 606
> 607 static void xsk_destruct_skb(struct sk_buff *skb)
> 608 {
> 609 struct xsk_tx_metadata_compl *compl = &skb_shinfo(skb)->xsk_meta;
> 610
> 611 if (compl->tx_timestamp) {
> 612 /* sw completion timestamp, not a real one */
> 613 *compl->tx_timestamp = ktime_get_tai_fast_ns();
> 614 }
> 615
> 616 xsk_cq_submit_addr_locked(xdp_sk(skb->sk)->pool, skb);
> 617 sock_wfree(skb);
> 618 }
> 619
> 620 static void xsk_set_destructor_arg(struct sk_buff *skb, u64 addr)
> 621 {
> 622 BUILD_BUG_ON(sizeof(struct xsk_addr_head) > sizeof(skb->cb));
> 623 INIT_LIST_HEAD(&XSKCB(skb)->addrs_list);
> 624 XSKCB(skb)->num_descs = 0;
> > 625 skb_shinfo(skb)->destructor_arg = (void *)addr;
> 626 }
> 627
>
> --
> 0-DAY CI Kernel Test Service
> https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2025-09-04 9:32 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-02 22:06 [PATCH v8 bpf] xsk: fix immature cq descriptor production Maciej Fijalkowski
2025-09-03 2:29 ` kernel test robot
2025-09-04 9:31 ` Maciej Fijalkowski [this message]
2025-09-04 11:31 ` Alexander Lobakin
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=aLlcgDC7s3Dh0NdX@boxer \
--to=maciej.fijalkowski@intel.com \
--cc=andrii@kernel.org \
--cc=ast@kernel.org \
--cc=bpf@vger.kernel.org \
--cc=daniel@iogearbox.net \
--cc=e.kubanski@partner.samsung.com \
--cc=kerneljasonxing@gmail.com \
--cc=lkp@intel.com \
--cc=magnus.karlsson@intel.com \
--cc=netdev@vger.kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=sdf@fomichev.me \
--cc=stfomichev@gmail.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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox