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)
Thu, Oct 2, 7:15 AM
Unknown Object (File)
Sep 10 2025, 5:57 PM
Unknown Object (File)
Sep 4 2025, 11:19 AM
Unknown Object (File)
Sep 4 2025, 9:29 AM
Unknown Object (File)
Sep 3 2025, 11:57 AM
Unknown Object (File)
Aug 14 2025, 12:34 AM
Unknown Object (File)
Aug 7 2025, 8:40 PM
Unknown Object (File)
Aug 2 2025, 6:37 PM
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