-
Notifications
You must be signed in to change notification settings - Fork 173
WIP: Parallel SystemC Module #118
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
WIP: Parallel SystemC Module #118
Conversation
Todo: wrap calls to b_transport/nb_transport etc such that they are called on the right simcontext. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So far, PR looks good. As you you mention WIP in title, I would suggest that you mark it explicitly as draft in the PR's properties.
src/sysc/kernel/sc_event.h
Outdated
sc_time time; | ||
|
||
void stage_callback(const sc_core::sc_stage& stage) { | ||
mutex.lock(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fix indentation
8c4a2e2
to
97c6717
Compare
…imcontext::init Signed-off-by: Mark Burton <mburton@quicinc.com>
Signed-off-by: Mark Burton <mburton@quicinc.com>
…d nb_transport thread safe, other tlm APIs are NOT handled Signed-off-by: Mark Burton <mburton@quicinc.com>
Signed-off-by: Mark Burton <mburton@quicinc.com>
97c6717
to
c2d4c18
Compare
Still an issue to do with prim_channels - created due to dynamic event creation after start of simulation, which generates a new 'name' - putting that into the name hierarchy is not thread safe, and causes things to go wrong if you run for long enough..... |
THANKS @maehne ! |
Signed-off-by: Mark Burton <mburton@quicinc.com>
5feda24
to
69f610b
Compare
Signed-off-by: Mark Burton <mburton@quicinc.com>
69f610b
to
c60e62c
Compare
No description provided.