diff options
| author | Marek Olšák <[email protected]> | 2016-07-15 16:04:03 +0200 |
|---|---|---|
| committer | Marek Olšák <[email protected]> | 2016-07-19 23:45:06 +0200 |
| commit | ad70c3954be0d04ca5c61d3e465ea7622916102f (patch) | |
| tree | ca1d71e57b912a795e0c2683ecf7833c62220490 /src/gallium/drivers/radeonsi/si_perfcounter.c | |
| parent | 1a1cc67edd4c1ae08b739adaf78e014b828908de (diff) | |
radeonsi: really wait for the second EOP event and not the first one
Reviewed-by: Nicolai Hähnle <[email protected]>
Diffstat (limited to 'src/gallium/drivers/radeonsi/si_perfcounter.c')
| -rw-r--r-- | src/gallium/drivers/radeonsi/si_perfcounter.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/gallium/drivers/radeonsi/si_perfcounter.c b/src/gallium/drivers/radeonsi/si_perfcounter.c index 96007a523af..0ced617dbc8 100644 --- a/src/gallium/drivers/radeonsi/si_perfcounter.c +++ b/src/gallium/drivers/radeonsi/si_perfcounter.c @@ -595,16 +595,19 @@ static void si_pc_emit_stop(struct r600_common_context *ctx, /* Two EOP events are required to make all engines go idle * (and optional cache flushes executed) before the timestamp * is written. + * + * Write 1, because we need to wait for the second EOP event. */ radeon_emit(cs, PKT3(PKT3_EVENT_WRITE_EOP, 4, 0)); radeon_emit(cs, EVENT_TYPE(EVENT_TYPE_BOTTOM_OF_PIPE_TS) | EVENT_INDEX(5)); radeon_emit(cs, va); radeon_emit(cs, (va >> 32) | EOP_DATA_SEL(1)); - radeon_emit(cs, 0); /* immediate data */ + radeon_emit(cs, 1); /* immediate data */ radeon_emit(cs, 0); /* unused */ } + /* Write 0. */ radeon_emit(cs, PKT3(PKT3_EVENT_WRITE_EOP, 4, 0)); radeon_emit(cs, EVENT_TYPE(EVENT_TYPE_BOTTOM_OF_PIPE_TS) | EVENT_INDEX(5)); @@ -613,6 +616,7 @@ static void si_pc_emit_stop(struct r600_common_context *ctx, radeon_emit(cs, 0); /* immediate data */ radeon_emit(cs, 0); /* unused */ + /* Wait until the memory location is 0. */ radeon_emit(cs, PKT3(PKT3_WAIT_REG_MEM, 5, 0)); radeon_emit(cs, WAIT_REG_MEM_EQUAL | WAIT_REG_MEM_MEM_SPACE(1)); radeon_emit(cs, va); |