> -----Original Message----- > From: Eric Dumazet <[email protected]> > Sent: Thursday, August 21, 2025 2:56 PM > To: Chia-Yu Chang (Nokia) <[email protected]> > Cc: [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; Koen De Schepper (Nokia) > <[email protected]>; [email protected]; > [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected] > Subject: Re: [PATCH v15 net-next 12/14] tcp: accecn: AccECN option failure > handling > > > CAUTION: This is an external email. Please be very careful when clicking > links or opening attachments. See the URL nok.it/ext for additional > information. > > > > On Fri, Aug 15, 2025 at 1:40 AM <[email protected]> wrote: > > > > From: Chia-Yu Chang <[email protected]> > > > > AccECN option may fail in various way, handle these: > > - Attempt to negotiate the use of AccECN on the 1st retransmitted SYN > > - From the 2nd retransmitted SYN, stop AccECN negotiation > > - Remove option from SYN/ACK rexmits to handle blackholes > > - If no option arrives in SYN/ACK, assume Option is not usable > > - If an option arrives later, re-enabled > > - If option is zeroed, disable AccECN option processing > > > > This patch use existing padding bits in tcp_request_sock and holes in > > tcp_sock without increasing the size. > > > > Signed-off-by: Ilpo Järvinen <[email protected]> > > Signed-off-by: Chia-Yu Chang <[email protected]> > > Truly invasive changes, what can I say.... > > Not sure why you CC so many folks which will very unlikely give any feedback. > > What is the purpose of WARN_ON_ONCE() in tcp_accecn_option_init() ? > > Just to feed syzbot ?
Hi Eric, WARN_ON_ONCE() exists in both tcp_accecn_option_init() and tcp_accecn_process_option(). And the one in tcp_accecn_option_init() is for AccECN option handling during the handshake. Or is it not preferred to put WARN_ON_ONCE() during TCP handshake? Chia-Yu
