In that case move `map_count` above `map` which should move it to a separate cache line and update the doc strings as needed. Reviewed-by: Ian Forbes <ian.forbes@xxxxxxxxxxxx> On Tue, Aug 13, 2024 at 12:40 PM Zack Rusin <zack.rusin@xxxxxxxxxxxx> wrote: > > On Tue, Aug 13, 2024 at 1:29 PM Ian Forbes <ian.forbes@xxxxxxxxxxxx> wrote: > > > > Remove `busy_places` now that it's unused. There's also probably a > > better place to put `map_count` in the struct layout to avoid false > > sharing with `cpu_writers`. I'd repack the whole struct if we're going > > to be adding and removing fields. > > Those are not related to this change. They'd be two seperate changes. > One to remove other unused members and third to relayout the struct. > > z
