All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Abhishek Chauhan <quic_abchauha@quicinc.com>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev
Subject: Re: [RFC PATCH bpf-next v3 1/2] net: Rename mono_delivery_time to tstamp_type for scalabilty
Date: Mon, 15 Apr 2024 16:30:19 +0800	[thread overview]
Message-ID: <202404151604.8WpyG3jW-lkp@intel.com> (raw)
In-Reply-To: <20240412210125.1780574-2-quic_abchauha@quicinc.com>

Hi Abhishek,

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

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

url:    https://github.com/intel-lab-lkp/linux/commits/Abhishek-Chauhan/net-Rename-mono_delivery_time-to-tstamp_type-for-scalabilty/20240415-094415
base:   https://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next.git master
patch link:    https://lore.kernel.org/r/20240412210125.1780574-2-quic_abchauha%40quicinc.com
patch subject: [RFC PATCH bpf-next v3 1/2] net: Rename mono_delivery_time to tstamp_type for scalabilty
config: s390-allmodconfig (https://download.01.org/0day-ci/archive/20240415/202404151604.8WpyG3jW-lkp@intel.com/config)
compiler: clang version 19.0.0git (https://github.com/llvm/llvm-project 8b3b4a92adee40483c27f26c478a384cd69c6f05)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240415/202404151604.8WpyG3jW-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/202404151604.8WpyG3jW-lkp@intel.com/

All errors (new ones prefixed by >>):

   In file included from net/ieee802154/6lowpan/reassembly.c:12:
   In file included from include/linux/net.h:24:
   In file included from include/linux/mm.h:2208:
   include/linux/vmstat.h:508:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion]
     508 |         return vmstat_text[NR_VM_ZONE_STAT_ITEMS +
         |                            ~~~~~~~~~~~~~~~~~~~~~ ^
     509 |                            item];
         |                            ~~~~
   include/linux/vmstat.h:515:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion]
     515 |         return vmstat_text[NR_VM_ZONE_STAT_ITEMS +
         |                            ~~~~~~~~~~~~~~~~~~~~~ ^
     516 |                            NR_VM_NUMA_EVENT_ITEMS +
         |                            ~~~~~~~~~~~~~~~~~~~~~~
   include/linux/vmstat.h:522:36: warning: arithmetic between different enumeration types ('enum node_stat_item' and 'enum lru_list') [-Wenum-enum-conversion]
     522 |         return node_stat_name(NR_LRU_BASE + lru) + 3; // skip "nr_"
         |                               ~~~~~~~~~~~ ^ ~~~
   include/linux/vmstat.h:527:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion]
     527 |         return vmstat_text[NR_VM_ZONE_STAT_ITEMS +
         |                            ~~~~~~~~~~~~~~~~~~~~~ ^
     528 |                            NR_VM_NUMA_EVENT_ITEMS +
         |                            ~~~~~~~~~~~~~~~~~~~~~~
   include/linux/vmstat.h:536:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion]
     536 |         return vmstat_text[NR_VM_ZONE_STAT_ITEMS +
         |                            ~~~~~~~~~~~~~~~~~~~~~ ^
     537 |                            NR_VM_NUMA_EVENT_ITEMS +
         |                            ~~~~~~~~~~~~~~~~~~~~~~
   In file included from net/ieee802154/6lowpan/reassembly.c:14:
   In file included from include/linux/netdevice.h:38:
   In file included from include/net/net_namespace.h:43:
   In file included from include/linux/skbuff.h:28:
   In file included from include/linux/dma-mapping.h:11:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:78:
   include/asm-generic/io.h:547:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     547 |         val = __raw_readb(PCI_IOBASE + addr);
         |                           ~~~~~~~~~~ ^
   include/asm-generic/io.h:560:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     560 |         val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
         |                                                         ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:37:59: note: expanded from macro '__le16_to_cpu'
      37 | #define __le16_to_cpu(x) __swab16((__force __u16)(__le16)(x))
         |                                                           ^
   include/uapi/linux/swab.h:102:54: note: expanded from macro '__swab16'
     102 | #define __swab16(x) (__u16)__builtin_bswap16((__u16)(x))
         |                                                      ^
   In file included from net/ieee802154/6lowpan/reassembly.c:14:
   In file included from include/linux/netdevice.h:38:
   In file included from include/net/net_namespace.h:43:
   In file included from include/linux/skbuff.h:28:
   In file included from include/linux/dma-mapping.h:11:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:78:
   include/asm-generic/io.h:573:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     573 |         val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
         |                                                         ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:35:59: note: expanded from macro '__le32_to_cpu'
      35 | #define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
         |                                                           ^
   include/uapi/linux/swab.h:115:54: note: expanded from macro '__swab32'
     115 | #define __swab32(x) (__u32)__builtin_bswap32((__u32)(x))
         |                                                      ^
   In file included from net/ieee802154/6lowpan/reassembly.c:14:
   In file included from include/linux/netdevice.h:38:
   In file included from include/net/net_namespace.h:43:
   In file included from include/linux/skbuff.h:28:
   In file included from include/linux/dma-mapping.h:11:
   In file included from include/linux/scatterlist.h:9:
   In file included from arch/s390/include/asm/io.h:78:
   include/asm-generic/io.h:584:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     584 |         __raw_writeb(value, PCI_IOBASE + addr);
         |                             ~~~~~~~~~~ ^
   include/asm-generic/io.h:594:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     594 |         __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
         |                                                       ~~~~~~~~~~ ^
   include/asm-generic/io.h:604:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     604 |         __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
         |                                                       ~~~~~~~~~~ ^
   include/asm-generic/io.h:692:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     692 |         readsb(PCI_IOBASE + addr, buffer, count);
         |                ~~~~~~~~~~ ^
   include/asm-generic/io.h:700:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     700 |         readsw(PCI_IOBASE + addr, buffer, count);
         |                ~~~~~~~~~~ ^
   include/asm-generic/io.h:708:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     708 |         readsl(PCI_IOBASE + addr, buffer, count);
         |                ~~~~~~~~~~ ^
   include/asm-generic/io.h:717:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     717 |         writesb(PCI_IOBASE + addr, buffer, count);
         |                 ~~~~~~~~~~ ^
   include/asm-generic/io.h:726:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     726 |         writesw(PCI_IOBASE + addr, buffer, count);
         |                 ~~~~~~~~~~ ^
   include/asm-generic/io.h:735:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     735 |         writesl(PCI_IOBASE + addr, buffer, count);
         |                 ~~~~~~~~~~ ^
>> net/ieee802154/6lowpan/reassembly.c:133:8: error: no member named 'mono_delivery_time' in 'struct inet_frag_queue'
     133 |         fq->q.mono_delivery_time = skb->mono_delivery_time;
         |         ~~~~~ ^
>> net/ieee802154/6lowpan/reassembly.c:133:34: error: no member named 'mono_delivery_time' in 'struct sk_buff'
     133 |         fq->q.mono_delivery_time = skb->mono_delivery_time;
         |                                    ~~~  ^
   17 warnings and 2 errors generated.


vim +133 net/ieee802154/6lowpan/reassembly.c

7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28   83  
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28   84  static int lowpan_frag_queue(struct lowpan_frag_queue *fq,
72a5e6bb5120d6 net/ieee802154/6lowpan/reassembly.c Alexander Aring     2015-09-02   85  			     struct sk_buff *skb, u8 frag_type)
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28   86  {
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14   87  	struct sk_buff *prev_tail;
f4606583fd8dcf net/ieee802154/6lowpan/reassembly.c Alexander Aring     2015-09-02   88  	struct net_device *ldev;
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14   89  	int end, offset, err;
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14   90  
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14   91  	/* inet_frag_queue_* functions use skb->cb; see struct ipfrag_skb_cb
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14   92  	 * in inet_fragment.c
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14   93  	 */
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14   94  	BUILD_BUG_ON(sizeof(struct lowpan_802154_cb) > sizeof(struct inet_skb_parm));
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14   95  	BUILD_BUG_ON(sizeof(struct lowpan_802154_cb) > sizeof(struct inet6_skb_parm));
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28   96  
06aa8b8a0345c7 net/ieee802154/reassembly.c         Nikolay Aleksandrov 2014-08-01   97  	if (fq->q.flags & INET_FRAG_COMPLETE)
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28   98  		goto err;
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28   99  
72a5e6bb5120d6 net/ieee802154/6lowpan/reassembly.c Alexander Aring     2015-09-02  100  	offset = lowpan_802154_cb(skb)->d_offset << 3;
72a5e6bb5120d6 net/ieee802154/6lowpan/reassembly.c Alexander Aring     2015-09-02  101  	end = lowpan_802154_cb(skb)->d_size;
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  102  
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  103  	/* Is this the final fragment? */
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  104  	if (offset + skb->len == end) {
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  105  		/* If we already have some bits beyond end
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  106  		 * or have different end, the segment is corrupted.
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  107  		 */
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  108  		if (end < fq->q.len ||
06aa8b8a0345c7 net/ieee802154/reassembly.c         Nikolay Aleksandrov 2014-08-01  109  		    ((fq->q.flags & INET_FRAG_LAST_IN) && end != fq->q.len))
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  110  			goto err;
06aa8b8a0345c7 net/ieee802154/reassembly.c         Nikolay Aleksandrov 2014-08-01  111  		fq->q.flags |= INET_FRAG_LAST_IN;
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  112  		fq->q.len = end;
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  113  	} else {
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  114  		if (end > fq->q.len) {
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  115  			/* Some bits beyond end -> corruption. */
06aa8b8a0345c7 net/ieee802154/reassembly.c         Nikolay Aleksandrov 2014-08-01  116  			if (fq->q.flags & INET_FRAG_LAST_IN)
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  117  				goto err;
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  118  			fq->q.len = end;
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  119  		}
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  120  	}
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  121  
f4606583fd8dcf net/ieee802154/6lowpan/reassembly.c Alexander Aring     2015-09-02  122  	ldev = skb->dev;
f4606583fd8dcf net/ieee802154/6lowpan/reassembly.c Alexander Aring     2015-09-02  123  	if (ldev)
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  124  		skb->dev = NULL;
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14  125  	barrier();
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14  126  
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14  127  	prev_tail = fq->q.fragments_tail;
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14  128  	err = inet_frag_queue_insert(&fq->q, skb, offset, end);
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14  129  	if (err)
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14  130  		goto err;
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  131  
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  132  	fq->q.stamp = skb->tstamp;
335c8cf3b53760 net/ieee802154/6lowpan/reassembly.c Martin KaFai Lau    2022-03-02 @133  	fq->q.mono_delivery_time = skb->mono_delivery_time;
72a5e6bb5120d6 net/ieee802154/6lowpan/reassembly.c Alexander Aring     2015-09-02  134  	if (frag_type == LOWPAN_DISPATCH_FRAG1)
06aa8b8a0345c7 net/ieee802154/reassembly.c         Nikolay Aleksandrov 2014-08-01  135  		fq->q.flags |= INET_FRAG_FIRST_IN;
72a5e6bb5120d6 net/ieee802154/6lowpan/reassembly.c Alexander Aring     2015-09-02  136  
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  137  	fq->q.meat += skb->len;
6ce3b4dcee4f96 net/ieee802154/6lowpan/reassembly.c Eric Dumazet        2019-05-24  138  	add_frag_mem_limit(fq->q.fqdir, skb->truesize);
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  139  
06aa8b8a0345c7 net/ieee802154/reassembly.c         Nikolay Aleksandrov 2014-08-01  140  	if (fq->q.flags == (INET_FRAG_FIRST_IN | INET_FRAG_LAST_IN) &&
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  141  	    fq->q.meat == fq->q.len) {
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  142  		int res;
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  143  		unsigned long orefdst = skb->_skb_refdst;
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  144  
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  145  		skb->_skb_refdst = 0UL;
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14  146  		res = lowpan_frag_reasm(fq, skb, prev_tail, ldev);
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  147  		skb->_skb_refdst = orefdst;
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  148  		return res;
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  149  	}
254c5dbe15d44c net/ieee802154/6lowpan/reassembly.c Peter Oskolkov      2019-02-14  150  	skb_dst_drop(skb);
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  151  
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  152  	return -1;
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  153  err:
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  154  	kfree_skb(skb);
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  155  	return -1;
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  156  }
7240cdec60b136 net/ieee802154/reassembly.c         Alexander Aring     2014-02-28  157  

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

  parent reply	other threads:[~2024-04-15  8:30 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-04-12 21:01 [RFC PATCH bpf-next v3 0/2] Replace mono_delivery_time with tstamp_type Abhishek Chauhan
2024-04-12 21:01 ` [RFC PATCH bpf-next v3 1/2] net: Rename mono_delivery_time to tstamp_type for scalabilty Abhishek Chauhan
2024-04-13  0:37   ` Abhishek Chauhan (ABC)
2024-04-13 18:54   ` Willem de Bruijn
2024-04-15 20:27     ` Abhishek Chauhan (ABC)
2024-04-15 20:46       ` Willem de Bruijn
2024-04-15 21:06         ` Abhishek Chauhan (ABC)
2024-04-15 21:22           ` Willem de Bruijn
2024-04-15 21:26             ` Abhishek Chauhan (ABC)
2024-04-15  5:52   ` kernel test robot
2024-04-15  8:30   ` kernel test robot [this message]
2024-04-12 21:01 ` [RFC PATCH bpf-next v3 2/2] net: Add additional bit to support userspace timestamp type Abhishek Chauhan
2024-04-13 19:07   ` Willem de Bruijn
2024-04-15 20:00     ` Martin KaFai Lau
2024-04-16 23:40       ` Abhishek Chauhan (ABC)

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=202404151604.8WpyG3jW-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=llvm@lists.linux.dev \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=quic_abchauha@quicinc.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.