mmtk / mmtk-core

Memory Management ToolKit
https://www.mmtk.io
Other
377 stars 69 forks source link

Allow multiple instances of MMTk to co-exist #43

Closed qinsoon closed 4 years ago

qinsoon commented 4 years ago

In GitLab by @qinsoon on Nov 29, 2019, 17:48

qinsoon commented 4 years ago

In GitLab by @qinsoon on Jan 7, 2020, 11:20

assigned to @qinsoon

qinsoon commented 4 years ago

In GitLab by @qinsoon on Jan 7, 2020, 11:28

I am refactoring global variables/states into an MMTK struct, which is an instance of MMTk. To start with, I have a singleton MMTK instance as a global variable. When the refactoring is done, we can remove the global MMTK instance, and allow multiple instances (we need to spend some time dealing with lifetime declarations and our MMTk API).

Progress for removing global states for MMTk:

qinsoon commented 4 years ago

In GitLab by @qinsoon on Jan 15, 2020, 18:40

mentioned in merge request !19

qinsoon commented 4 years ago

In GitLab by @qinsoon on Jan 16, 2020, 11:50

mentioned in merge request !20

qinsoon commented 4 years ago

In GitLab by @qinsoon on Jan 19, 2020, 18:58

mentioned in merge request !21

qinsoon commented 4 years ago

In GitLab by @qinsoon on Jan 23, 2020, 13:23

mentioned in merge request !22

qinsoon commented 4 years ago

In GitLab by @qinsoon on Jan 26, 2020, 19:10

mentioned in merge request !23

qinsoon commented 4 years ago

In GitLab by @qinsoon on Jan 31, 2020, 13:16

mentioned in merge request !24

qinsoon commented 4 years ago

In GitLab by @qinsoon on Feb 4, 2020, 14:28

mentioned in merge request !26

qinsoon commented 4 years ago

Closing this as listed tasks were all done. We have removed global states annd our API now supports multiple MMTk instances. However, we haven't verified that the internal implementation can properly support multiple instances. I am creating a new issue to track this.