Re: [Mesa-dev] [PATCH 04/10] i965: Emit full-length PIPE_CONTROLs for (non-write) flushes.

2013-12-13 Thread Daniel Vetter
On Thu, Dec 12, 2013 at 01:26:36AM -0800, Kenneth Graunke wrote: > The PIPE_CONTROL packet actually has 5 DWords on Gen6+: > 1. Header > 2. Flags > 3. Address > 4. Immediate Data: Lower DWord > 5. Immediate Data: Upper DWord > > We just never emitted the last one. While it appears to work, it's >

[Mesa-dev] [PATCH 04/10] i965: Emit full-length PIPE_CONTROLs for (non-write) flushes.

2013-12-12 Thread Kenneth Graunke
The PIPE_CONTROL packet actually has 5 DWords on Gen6+: 1. Header 2. Flags 3. Address 4. Immediate Data: Lower DWord 5. Immediate Data: Upper DWord We just never emitted the last one. While it appears to work, it's probably safer to emit the entire thing. Signed-off-by: Kenneth Graunke --- src