Page MenuHomeFreeBSD

libsa: Add MAXWAIT to net for establishing max total timeout
ClosedPublic

Authored by kevans on Feb 15 2018, 9:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 29, 10:42 AM
Unknown Object (File)
Dec 20 2023, 1:47 AM
Unknown Object (File)
Jun 29 2023, 12:46 PM
Unknown Object (File)
Jun 29 2023, 12:45 PM
Unknown Object (File)
Jun 29 2023, 12:43 PM
Unknown Object (File)
Jun 29 2023, 12:36 PM
Unknown Object (File)
May 6 2023, 5:53 PM
Unknown Object (File)
Mar 18 2023, 9:14 AM
Subscribers

Details

Summary

Current timeout behavior is to progress in timeout values from MINTMO to MAXTMO exponentially before finally timing out. This results in a fairly long time before operations finally timeout, which may not be ideal for some use-cases.

Add MAXWAIT that may be configured along with MINTMO/MAXTMO. If we attempt to start our send/recv cycle over again but MAXWAIT > 0 and MAXWAIT seconds have already passed, then go ahead and timeout.

This is intended for those that just want to say "timeout after 180 seconds" rather than calculate and tweak MINTMO/MAXTMO to get their desired timeout. The default is 0, or "progress from MINTMO to MAXTMO with no exception."

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

sbruno requested changes to this revision.Feb 15 2018, 9:53 PM
sbruno added a subscriber: sbruno.

Hmmm ... I'm up to five minutes waiting for a timeout here and I don't even see the twiddle moving.

This revision now requires changes to proceed.Feb 15 2018, 9:53 PM

After *READING* the review, I tested with a MAXWAIT of 20 and this works as advertised.

shipit

This revision is now accepted and ready to land.Feb 15 2018, 10:08 PM

looks good to me, but I've not deeply studied the network code to know for sure it's totally cool.

This revision was automatically updated to reflect the committed changes.