diff options
| author | Tvrtko Ursulin <tvrtko.ursulin@intel.com> | 2017-05-11 13:00:51 +0100 |
|---|---|---|
| committer | Tvrtko Ursulin <tvrtko.ursulin@intel.com> | 2017-05-16 08:02:33 +0100 |
| commit | feaf779de5adb587214faa3f935360e0e4aa0c25 (patch) | |
| tree | 23177929250f9a279e3e35c61938e99cf87368cf /benchmarks/gem_wsim.c | |
| parent | 93215b00db06ba568c34a6cad194f4749bbfbc9d (diff) | |
gem_wsim: Two small tidies
Tidy last in the array presumed offset setting even though this
code path is not used at the moment.
Also use READ_ONCE on all fields we are trying to read from the
status page.
Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Diffstat (limited to 'benchmarks/gem_wsim.c')
| -rw-r--r-- | benchmarks/gem_wsim.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/benchmarks/gem_wsim.c b/benchmarks/gem_wsim.c index 58a428ab..e09ef257 100644 --- a/benchmarks/gem_wsim.c +++ b/benchmarks/gem_wsim.c @@ -888,8 +888,8 @@ static void get_rt_depth(struct workload *wrk, do { latch = READ_ONCE(wrk->status_page[idx + 3]); - rt->submitted = wrk->status_page[idx + 1]; - rt->completed = wrk->status_page[idx + 2]; + rt->submitted = READ_ONCE(wrk->status_page[idx + 1]); + rt->completed = READ_ONCE(wrk->status_page[idx + 2]); rt->seqno = READ_ONCE(wrk->status_page[idx]); } while (latch != rt->seqno); } @@ -1094,6 +1094,7 @@ update_bb_rt(struct w_step *w, enum intel_engine_id engine, uint32_t seqno) if (!(w->eb.flags & I915_EXEC_NO_RELOC)) { w->reloc[1].presumed_offset = -1; w->reloc[2].presumed_offset = -1; + w->reloc[3].presumed_offset = -1; } } |