Page MenuHomeFreeBSD

tcp: Make hostcache.cache_count MPSAFE by using a counter_u64_t
ClosedPublic

Authored by rscheff on Mar 31 2021, 10:53 AM.
Tags
None
Referenced Files
F113986110: D29510.id86604.diff
Sun, Apr 6, 1:52 PM
Unknown Object (File)
Fri, Mar 14, 9:07 AM
Unknown Object (File)
Feb 25 2025, 1:31 PM
Unknown Object (File)
Feb 25 2025, 8:46 AM
Unknown Object (File)
Feb 25 2025, 4:40 AM
Unknown Object (File)
Feb 24 2025, 3:00 PM
Unknown Object (File)
Feb 10 2025, 9:49 PM
Unknown Object (File)
Feb 5 2025, 5:56 PM
Subscribers

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Mar 31 2021, 1:59 PM

FWIW, I disagree with this change. I think we should instead use atomic operations here.

Counters have the property that they are cheap to update, but expensive to read. Moreover, if you are doing a lot of reads, you will be pulling in cachelines from the per-CPU space into other CPUs (which is fine, but uses cache space).

For something which will be written often, but only read occasionally, counters are a big performance win. (Statistics, for example, are a great example of things for which counters are ideal.) For things which are read with any regularity (and, especially, from critical paths), atomics should perform better.

I suggest reverting this change and replacing it with atomic operations.

In D29510#661680, @jtl wrote:

FWIW, I disagree with this change. I think we should instead use atomic operations here.

Counters have the property that they are cheap to update, but expensive to read. Moreover, if you are doing a lot of reads, you will be pulling in cachelines from the per-CPU space into other CPUs (which is fine, but uses cache space).

For something which will be written often, but only read occasionally, counters are a big performance win. (Statistics, for example, are a great example of things for which counters are ideal.) For things which are read with any regularity (and, especially, from critical paths), atomics should perform better.

I suggest reverting this change and replacing it with atomic operations.

Thanks for the explanation. I actually suggested to Richard to use the counter API instead of atomic operations. So it is my mistake...