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
Unknown Object (File)
Wed, Feb 7, 9:43 AM
Unknown Object (File)
Jan 15 2024, 6:44 AM
Unknown Object (File)
Dec 20 2023, 3:15 AM
Unknown Object (File)
Dec 16 2023, 3:38 PM
Unknown Object (File)
Dec 12 2023, 2:43 PM
Unknown Object (File)
Nov 26 2023, 10:56 AM
Unknown Object (File)
Nov 26 2023, 10:56 AM
Unknown Object (File)
Nov 26 2023, 3:13 AM
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...