[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <202405141135.xZDS5YLg-lkp@intel.com>
Date: Tue, 14 May 2024 11:40:32 +0800
From: kernel test robot <lkp@...el.com>
To: zijianzhang@...edance.com, netdev@...r.kernel.org
Cc: oe-kbuild-all@...ts.linux.dev, edumazet@...gle.com,
willemdebruijn.kernel@...il.com, cong.wang@...edance.com,
xiaochun.lu@...edance.com, Zijian Zhang <zijianzhang@...edance.com>
Subject: Re: [PATCH net-next v4 2/3] sock: add MSG_ZEROCOPY notification
mechanism based on msg_control
Hi,
kernel test robot noticed the following build warnings:
[auto build test WARNING on net-next/main]
url: https://github.com/intel-lab-lkp/linux/commits/zijianzhang-bytedance-com/selftests-fix-OOM-problem-in-msg_zerocopy-selftest/20240514-051839
base: net-next/main
patch link: https://lore.kernel.org/r/20240513211755.2751955-3-zijianzhang%40bytedance.com
patch subject: [PATCH net-next v4 2/3] sock: add MSG_ZEROCOPY notification mechanism based on msg_control
config: openrisc-defconfig (https://download.01.org/0day-ci/archive/20240514/202405141135.xZDS5YLg-lkp@intel.com/config)
compiler: or1k-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240514/202405141135.xZDS5YLg-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@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202405141135.xZDS5YLg-lkp@intel.com/
All warnings (new ones prefixed by >>):
net/core/sock.c: In function '__sock_cmsg_send':
>> net/core/sock.c:2850:39: warning: unused variable 'tmp' [-Wunused-variable]
2850 | struct sk_buff *skb, *tmp;
| ^~~
vim +/tmp +2850 net/core/sock.c
2807
2808 int __sock_cmsg_send(struct sock *sk, struct cmsghdr *cmsg,
2809 struct sockcm_cookie *sockc)
2810 {
2811 u32 tsflags;
2812
2813 switch (cmsg->cmsg_type) {
2814 case SO_MARK:
2815 if (!ns_capable(sock_net(sk)->user_ns, CAP_NET_RAW) &&
2816 !ns_capable(sock_net(sk)->user_ns, CAP_NET_ADMIN))
2817 return -EPERM;
2818 if (cmsg->cmsg_len != CMSG_LEN(sizeof(u32)))
2819 return -EINVAL;
2820 sockc->mark = *(u32 *)CMSG_DATA(cmsg);
2821 break;
2822 case SO_TIMESTAMPING_OLD:
2823 case SO_TIMESTAMPING_NEW:
2824 if (cmsg->cmsg_len != CMSG_LEN(sizeof(u32)))
2825 return -EINVAL;
2826
2827 tsflags = *(u32 *)CMSG_DATA(cmsg);
2828 if (tsflags & ~SOF_TIMESTAMPING_TX_RECORD_MASK)
2829 return -EINVAL;
2830
2831 sockc->tsflags &= ~SOF_TIMESTAMPING_TX_RECORD_MASK;
2832 sockc->tsflags |= tsflags;
2833 break;
2834 case SCM_TXTIME:
2835 if (!sock_flag(sk, SOCK_TXTIME))
2836 return -EINVAL;
2837 if (cmsg->cmsg_len != CMSG_LEN(sizeof(u64)))
2838 return -EINVAL;
2839 sockc->transmit_time = get_unaligned((u64 *)CMSG_DATA(cmsg));
2840 break;
2841 /* SCM_RIGHTS and SCM_CREDENTIALS are semantically in SOL_UNIX. */
2842 case SCM_RIGHTS:
2843 case SCM_CREDENTIALS:
2844 break;
2845 case SCM_ZC_NOTIFICATION: {
2846 struct zc_info_elem zc_info_kern[SOCK_ZC_INFO_MAX];
2847 int cmsg_data_len, zc_info_elem_num;
2848 struct sk_buff_head *q, local_q;
2849 struct sock_exterr_skb *serr;
> 2850 struct sk_buff *skb, *tmp;
2851 void __user *usr_addr;
2852 unsigned long flags;
2853 int ret, i = 0;
2854
2855 if (!sock_flag(sk, SOCK_ZEROCOPY) || sk->sk_family == PF_RDS)
2856 return -EINVAL;
2857
2858 cmsg_data_len = cmsg->cmsg_len - sizeof(struct cmsghdr);
2859 if (cmsg_data_len % sizeof(struct zc_info_elem))
2860 return -EINVAL;
2861
2862 zc_info_elem_num = cmsg_data_len / sizeof(struct zc_info_elem);
2863 if (!zc_info_elem_num || zc_info_elem_num > SOCK_ZC_INFO_MAX)
2864 return -EINVAL;
2865
2866 if (in_compat_syscall())
2867 usr_addr = compat_ptr(*(compat_uptr_t *)CMSG_DATA(cmsg));
2868 else
2869 usr_addr = (void __user *)*(void **)CMSG_DATA(cmsg);
2870 if (!access_ok(usr_addr, cmsg_data_len))
2871 return -EFAULT;
2872
2873 q = &sk->sk_error_queue;
2874 skb_queue_head_init(&local_q);
2875 spin_lock_irqsave(&q->lock, flags);
2876 skb = skb_peek(q);
2877 while (skb && i < zc_info_elem_num) {
2878 struct sk_buff *skb_next = skb_peek_next(skb, q);
2879
2880 serr = SKB_EXT_ERR(skb);
2881 if (serr->ee.ee_errno == 0 &&
2882 serr->ee.ee_origin == SO_EE_ORIGIN_ZEROCOPY) {
2883 zc_info_kern[i].hi = serr->ee.ee_data;
2884 zc_info_kern[i].lo = serr->ee.ee_info;
2885 zc_info_kern[i].zerocopy = !(serr->ee.ee_code
2886 & SO_EE_CODE_ZEROCOPY_COPIED);
2887 __skb_unlink(skb, q);
2888 __skb_queue_tail(&local_q, skb);
2889 i++;
2890 }
2891 skb = skb_next;
2892 }
2893 spin_unlock_irqrestore(&q->lock, flags);
2894
2895 ret = copy_to_user(usr_addr,
2896 zc_info_kern,
2897 i * sizeof(struct zc_info_elem));
2898
2899 if (unlikely(ret)) {
2900 spin_lock_irqsave(&q->lock, flags);
2901 skb_queue_splice_init(&local_q, q);
2902 spin_unlock_irqrestore(&q->lock, flags);
2903 return -EFAULT;
2904 }
2905
2906 while ((skb = __skb_dequeue(&local_q)))
2907 consume_skb(skb);
2908 break;
2909 }
2910 default:
2911 return -EINVAL;
2912 }
2913 return 0;
2914 }
2915 EXPORT_SYMBOL(__sock_cmsg_send);
2916
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
Powered by blists - more mailing lists