Thread (22 messages) 22 messages, 4 authors, 2023-05-24

Re: [PATCH net-next 3/3] net: tcp: handle window shrink properly

From: kernel test robot <hidden>
Date: 2023-05-17 16:55:43
Also in: lkml, oe-kbuild-all

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/menglong8-dong-gmail-com/net-tcp-add-sysctl-for-controling-tcp-window-shrink/20230517-204436
base:   net-next/main
patch link:    https://lore.kernel.org/r/20230517124201.441634-4-imagedong%40tencent.com
patch subject: [PATCH net-next 3/3] net: tcp: handle window shrink properly
config: m68k-allyesconfig
compiler: m68k-linux-gcc (GCC) 12.1.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/intel-lab-lkp/linux/commit/3712ab843c151a10d6570ded9d8c77ad831a7a41
        git remote add linux-review https://github.com/intel-lab-lkp/linux
        git fetch --no-tags linux-review menglong8-dong-gmail-com/net-tcp-add-sysctl-for-controling-tcp-window-shrink/20230517-204436
        git checkout 3712ab843c151a10d6570ded9d8c77ad831a7a41
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=m68k olddefconfig
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=m68k SHELL=/bin/bash net/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot [off-list ref]
| Closes: https://lore.kernel.org/oe-kbuild-all/202305180050.jwEMlDOS-lkp@intel.com/ (local)

All warnings (new ones prefixed by >>):
quoted
net/ipv4/tcp_input.c:3477: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
    * This function is called only when there are packets in the rtx queue,


vim +3477 net/ipv4/tcp_input.c

  3475	
  3476	/**
3477	 * This function is called only when there are packets in the rtx queue,
  3478	 * which means that the packets out is not 0.
  3479	 *
  3480	 * NOTE: we only handle window shrink case in this part.
  3481	 */
  3482	static void tcp_ack_probe_shrink(struct sock *sk)
  3483	{
  3484		struct inet_connection_sock *icsk = inet_csk(sk);
  3485		unsigned long when;
  3486	
  3487		if (!sysctl_tcp_wnd_shrink)
  3488			return;
  3489	
  3490		if (tcp_rtx_overflow(sk)) {
  3491			when = tcp_probe0_when(sk, TCP_RTO_MAX);
  3492	
  3493			when = tcp_clamp_probe0_to_user_timeout(sk, when);
  3494			tcp_reset_xmit_timer(sk, ICSK_TIME_PROBE0, when, TCP_RTO_MAX);
  3495		} else {
  3496			/* check if recover from window shrink */
  3497			if (icsk->icsk_pending != ICSK_TIME_PROBE0)
  3498				return;
  3499	
  3500			icsk->icsk_backoff = 0;
  3501			icsk->icsk_probes_tstamp = 0;
  3502			inet_csk_clear_xmit_timer(sk, ICSK_TIME_PROBE0);
  3503			if (!tcp_rtx_queue_empty(sk))
  3504				tcp_retransmit_timer(sk);
  3505		}
  3506	}
  3507	

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

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help