All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Paul Chaignon <paul.chaignon@gmail.com>
Cc: oe-kbuild-all@lists.linux.dev
Subject: Re: [RFC PATCH bpf-next] bpf: Support L4 csum update for IPv6 address changes
Date: Wed, 21 May 2025 21:32:02 +0800	[thread overview]
Message-ID: <202505212155.GD0kA7Jj-lkp@intel.com> (raw)
In-Reply-To: <aCz84JU60wd8etiT@mail.gmail.com>

Hi Paul,

[This is a private test report for your RFC patch.]
kernel test robot noticed the following build warnings:

[auto build test WARNING on bpf-next/master]

url:    https://github.com/intel-lab-lkp/linux/commits/Paul-Chaignon/bpf-Support-L4-csum-update-for-IPv6-address-changes/20250521-060705
base:   https://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next.git master
patch link:    https://lore.kernel.org/r/aCz84JU60wd8etiT%40mail.gmail.com
patch subject: [RFC PATCH bpf-next] bpf: Support L4 csum update for IPv6 address changes
config: arc-randconfig-001-20250521 (https://download.01.org/0day-ci/archive/20250521/202505212155.GD0kA7Jj-lkp@intel.com/config)
compiler: arc-linux-gcc (GCC) 10.5.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250521/202505212155.GD0kA7Jj-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/202505212155.GD0kA7Jj-lkp@intel.com/

All warnings (new ones prefixed by >>):

   net/core/filter.c: In function '____bpf_l4_csum_replace':
>> net/core/filter.c:1990:40: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
    1990 |    inet_proto_csum_replace16(ptr, skb, (void *)from, (void *)to, is_pseudo);
         |                                        ^
   net/core/filter.c:1990:54: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
    1990 |    inet_proto_csum_replace16(ptr, skb, (void *)from, (void *)to, is_pseudo);
         |                                                      ^


vim +1990 net/core/filter.c

  1964	
  1965	BPF_CALL_5(bpf_l4_csum_replace, struct sk_buff *, skb, u32, offset,
  1966		   u64, from, u64, to, u64, flags)
  1967	{
  1968		bool is_pseudo    = flags & BPF_F_PSEUDO_HDR;
  1969		bool is_mmzero    = flags & BPF_F_MARK_MANGLED_0;
  1970		bool do_mforce    = flags & BPF_F_MARK_ENFORCE;
  1971		bool is_ipv6_addr = flags & BPF_F_IPV6_ADDR;
  1972		__sum16 *ptr;
  1973	
  1974		if (unlikely(flags & ~(BPF_F_MARK_MANGLED_0 | BPF_F_MARK_ENFORCE |
  1975				       BPF_F_PSEUDO_HDR | BPF_F_HDR_FIELD_MASK |
  1976				       BPF_F_IPV6_ADDR)))
  1977			return -EINVAL;
  1978		if (unlikely(offset > 0xffff || offset & 1))
  1979			return -EFAULT;
  1980		if (unlikely(bpf_try_make_writable(skb, offset + sizeof(*ptr))))
  1981			return -EFAULT;
  1982	
  1983		ptr = (__sum16 *)(skb->data + offset);
  1984		if (is_mmzero && !do_mforce && !*ptr)
  1985			return 0;
  1986	
  1987		switch (flags & BPF_F_HDR_FIELD_MASK) {
  1988		case 0:
  1989			if (is_ipv6_addr) {
> 1990				inet_proto_csum_replace16(ptr, skb, (void *)from, (void *)to, is_pseudo);
  1991			} else {
  1992				if (unlikely(from != 0))
  1993					return -EINVAL;
  1994				inet_proto_csum_replace_by_diff(ptr, skb, to, is_pseudo);
  1995			}
  1996			break;
  1997		case 2:
  1998			inet_proto_csum_replace2(ptr, skb, from, to, is_pseudo);
  1999			break;
  2000		case 4:
  2001			inet_proto_csum_replace4(ptr, skb, from, to, is_pseudo);
  2002			break;
  2003		default:
  2004			return -EINVAL;
  2005		}
  2006	
  2007		if (is_mmzero && !*ptr)
  2008			*ptr = CSUM_MANGLED_0;
  2009		return 0;
  2010	}
  2011	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

      parent reply	other threads:[~2025-05-21 13:33 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-05-20 22:06 [RFC PATCH bpf-next] bpf: Support L4 csum update for IPv6 address changes Paul Chaignon
2025-05-21  1:07 ` Alexei Starovoitov
2025-05-21 21:19   ` Paul Chaignon
2025-05-22 19:01     ` Paul Chaignon
2025-05-21 13:32 ` kernel test robot [this message]

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=202505212155.GD0kA7Jj-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=paul.chaignon@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 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.