search

mmtk / mmtk-core

Memory Management ToolKit
https://www.mmtk.io
Other
373 stars 67 forks source link

OOM lock necessary? #18

Closed qinsoon closed 4 years ago

qinsoon commented 4 years ago

In GitLab by @pzakopaylo01 on Mar 19, 2018, 13:45

Given that the only action being perfomred in the critical section is a store, I'm pretty sure it can be replaced with a sequentially consistent store.

Old MMTk reference: Allocator.java:268

qinsoon commented 4 years ago

In GitLab by @pzakopaylo01 on Mar 19, 2018, 13:51

Potentially a CMPXCHG is needed as well, see :280. Still, an explicit lock should not be necessary.

qinsoon commented 4 years ago

Duplicate of https://github.com/mmtk/mmtk-core/issues/59. Closing this now.