systemc: Add a distinct async_request_update mechanism.
authorGabe Black <gabeblack@google.com>
Mon, 22 Apr 2019 22:24:40 +0000 (15:24 -0700)
committerGabe Black <gabeblack@google.com>
Tue, 30 Apr 2019 01:27:45 +0000 (01:27 +0000)
commitca3dd5087800a3400f60589ce2063d30ce602580
tree326f923a0d381900c9f808306ecc903d17373512
parent88fc141f72bea768fdf8d6e22611a89f135cfc10
systemc: Add a distinct async_request_update mechanism.

This mechanism had just been plumbed into the regular request_update,
but that doesn't have any thread safety which is the whole point of
async_request_update. This new mechanism puts async update requests
into their own list which is checked any time normal updates happen.

The delta cycle which triggers those updates must happen through some
other means which will usually be ok. The exact timing of the update
is undefined, so it would be legal for it to either not be recognized
before the impending end of the simulation, or for it to get picked up
by subsequent activity. If there isn't subsequent activity but the
simulation also doesn't end, for instance if there are only gem5 events
left, then that update could be lost. That is an unresolved issue.

It would be nice to schedule a "ready" event if async updates were
added which would ensure they wouldn't starve. Unfortunately that
requires the event queue lock, and in practice it's been found that a
systemc process might block, effectively holding the event queue lock,
while it waits for some asyncrhonous update to give it something to do.
This effectively deadlocks the system since the update is blocked on
the lock the main thread holds, and the main thread is blocked waiting
for the update.

Change-Id: I580303db01673faafc2e63545b6a69b3327a521c
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/18288
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/systemc/core/channel.cc
src/systemc/core/scheduler.cc
src/systemc/core/scheduler.hh