cats-effect icon indicating copy to clipboard operation
cats-effect copied to clipboard

Optimize `EpollSystem` on Native

Open armanbilge opened this issue 7 months ago • 2 comments
trafficstars

  • Replace @volatiles with loads/stores with precise memory orders
  • Replace TrieMap with ConcurrentHashMap ?
  • Replace unistd.{write,read} with bindings not marked @blocking

armanbilge avatar Apr 04 '25 02:04 armanbilge

I'm not sure what the performance characteristics of CHM are on native so it's hard to say if it's better or worse than TrieMap.

Speaking of which, we definitely need a benchmark harness for SN.

djspiewak avatar Apr 04 '25 02:04 djspiewak

Yeah, me neither. Also if TrieMap is better for now, there's a lot of places in Cats Effect we rely on CHM that maybe we should use TrieMap on Native for now.

armanbilge avatar Apr 04 '25 02:04 armanbilge