Page MenuHomeFreeBSD

ena: exponential backoff exp limit
Needs ReviewPublic

Authored by osamaabb_amazon.com on Aug 20 2024, 8:45 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 28, 3:17 AM
Unknown Object (File)
Wed, Nov 26, 9:17 PM
Unknown Object (File)
Fri, Nov 21, 2:12 AM
Unknown Object (File)
Nov 5 2025, 9:26 AM
Unknown Object (File)
Oct 30 2025, 5:25 AM
Unknown Object (File)
Oct 20 2025, 2:58 AM
Unknown Object (File)
Oct 20 2025, 2:58 AM
Unknown Object (File)
Oct 20 2025, 2:58 AM
Subscribers

Details

Reviewers
cperciva
Summary

This patch limits the exponent in the exponential backoff
mechanism in order to avoid the value overflowing.

Approved by: cperciva
MFC after: 2 weeks
Sponsored by: Amazon, Inc.

Diff Detail

Repository
rG FreeBSD src repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 59059
Build 55946: arc lint + arc unit

Event Timeline

Dumb question, but where does the delay_us value which is passed to this come from? It looks like it come from ena_min_poll_delay_us which... as far as I can tell is never set? Is this just vestigal code or am I missing something?

Dumb question, but where does the delay_us value which is passed to this come from? It looks like it come from ena_min_poll_delay_us which... as far as I can tell is never set? Is this just vestigal code or am I missing something?

You're correct, added such a patch in [1]
[1] https://reviews.freebsd.org/D46763