Quoting Mika Kuoppala (2020-05-18 16:07:47) > Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes: > > + cs = emit_timestamp_store(cs, ce, offset + i * sizeof(u32)); > > Is the dual writes here so that when you kick the semaphore, you get the > latest no matter which side you were on? We wait on the first write in the CPU before releasing the semaphore. It was easier to copy the code, but now it can be an emit_store_dw() to make it clearer that we are not using it as timestamp -- and avoid the infinite wait if we hit CS_TIMESTAMP == 0. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
