Re: [Mesa-dev] [PATCH v2 1/2] i965: require post sync operation prior to ISP disable

2018-05-09 Thread Lionel Landwerlin
Pushed with the title changed. Thanks again. On 09/05/18 15:39, Rafael Antognolli wrote: In the summary log, you should drop the mention to post sync operation too, and mention stall on pixel scoreboard, right? With that, this and the next patches are Reviewed-by: Rafael Antognolli On Wed,

Re: [Mesa-dev] [PATCH v2 1/2] i965: require post sync operation prior to ISP disable

2018-05-09 Thread Lionel Landwerlin
Duh! Thanks a lot :) - Lionel On 09/05/18 15:39, Rafael Antognolli wrote: In the summary log, you should drop the mention to post sync operation too, and mention stall on pixel scoreboard, right? With that, this and the next patches are Reviewed-by: Rafael Antognolli On Wed, May 09, 2018 at

Re: [Mesa-dev] [PATCH v2 1/2] i965: require post sync operation prior to ISP disable

2018-05-09 Thread Rafael Antognolli
In the summary log, you should drop the mention to post sync operation too, and mention stall on pixel scoreboard, right? With that, this and the next patches are Reviewed-by: Rafael Antognolli On Wed, May 09, 2018 at 12:39:51AM +0100, Lionel Landwerlin wrote: > Invalidating the indirect state

[Mesa-dev] [PATCH v2 1/2] i965: require post sync operation prior to ISP disable

2018-05-08 Thread Lionel Landwerlin
Invalidating the indirect state pointers might affect a previously scheduled & still running 3DPRIMITIVE (causing page fault). So stall on pixel scoreboard before that. v2: Fix compile issue :( v3: Stall on pixel scoreboard v4: Drop the post sync operation (Lionel) Signed-off-by: Lionel Landwer