Review Board 2.0.15


arch: don't call *Timing functions from *Atomic versions

Review Request #3267 - Created Dec. 31, 2015 and submitted

Information
Steve Reinhardt
gem5
default
Reviewers
Default
Changeset 11285:d2fbd02314de
---------------------------
arch: don't call *Timing functions from *Atomic versions

The readMemAtomic/writeMemAtomic helper functions were calling
readMemTiming/writeMemTiming respectively.  This is functionally
correct, since the *Timing functions are doing the same access
initiation operation as the *Atomic functions (just that the
*Atomic versions also complete the access in line).  It also
provides for some (very minimal) code reuse.  Unfortunately,
it's potentially pretty confusing, since it makes it look like
the atomic accesses are somehow being converted to timing
accesses.  It also gets in the way of specializing the timing
interface (as will be done in a future patch).

   
Review request changed
Updated (Jan. 17, 2016, 7:07 p.m.)

Status: Closed (submitted)