Re: [Mesa-dev] [PATCH 2/6] i965/sync: Stop cacheing fence's signal status

2016-10-04 Thread Chad Versace
On Sun 02 Oct 2016, Kenneth Graunke wrote: > On Tuesday, September 27, 2016 11:51:20 PM PDT Chad Versace wrote: > > Cacheing the signal status complicates the code for questionable > > performance benefit. I added the cacheing long ago, and I now think it > > was the wrong decision. > > > > When w

Re: [Mesa-dev] [PATCH 2/6] i965/sync: Stop cacheing fence's signal status

2016-10-02 Thread Kenneth Graunke
On Tuesday, September 27, 2016 11:51:20 PM PDT Chad Versace wrote: > Cacheing the signal status complicates the code for questionable > performance benefit. I added the cacheing long ago, and I now think it > was the wrong decision. > > When we later add support for fences based on sync fds (that

[Mesa-dev] [PATCH 2/6] i965/sync: Stop cacheing fence's signal status

2016-09-27 Thread Chad Versace
Cacheing the signal status complicates the code for questionable performance benefit. I added the cacheing long ago, and I now think it was the wrong decision. When we later add support for fences based on sync fds (that is, a fd backed by struct sync_file in Linux 4.8), the cacheing becomes even