Page MenuHomeFreeBSD

Implement optional table entry limits of if_llatbl.
ClosedPublic

Authored by bz on Dec 6 2019, 10:14 PM.
Tags
None
Referenced Files
F102974371: D22713.diff
Tue, Nov 19, 9:24 AM
Unknown Object (File)
Sun, Nov 3, 2:35 AM
Unknown Object (File)
Tue, Oct 22, 11:08 PM
Unknown Object (File)
Sep 30 2024, 3:43 AM
Unknown Object (File)
Sep 29 2024, 7:21 PM
Unknown Object (File)
Sep 29 2024, 7:57 AM
Unknown Object (File)
Sep 29 2024, 1:36 AM
Unknown Object (File)
Sep 28 2024, 5:18 AM
Subscribers

Details

Summary

Implement counting of table entries linked on a per-table base.
Add an optional (if set > 0) limit of the maximum number of table
entries.

For that the public lltable_link_entry() and lltable_unlink_entry()
functions as well as the internal function pointers change from void
to having an int return type.

For as long as we do not set llt_maxentries this is can be committed
on its own. The moment we make use of the table limits, the callers
of the link function must check the return value as it can change.

Adjustments for IPv6 (and possibly IPv4) will follow in a separate
review.

Sponsored by: Netflix (originally)

This is extracted from D22447 and adjusted based on review comments.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 28017
Build 26174: arc lint + arc unit

Event Timeline

melifaro added inline comments.
sys/net/if_llatbl.h
165

Just for the sake of history: this change makes llt_lookup() the last pointer in the cache line (given 64B size). Eventually we will need to reshuffle fields here a bit.

This revision is now accepted and ready to land.Dec 7 2019, 8:00 PM
This revision was automatically updated to reflect the committed changes.