On Thu, Sep 10, 2015 at 08:41:37AM +0100, Chris Wilson wrote:
> http://people.freedesktop.org/~ickle/vbo/
> 
> ministat 63c4b7e_bench_synmark:OglBatch7 2fbce8b_bench_synmark:OglBatch7
> x 63c4b7e_bench_synmark:OglBatch7
> + 2fbce8b_bench_synmark:OglBatch7
> +------------------------------------------------------------------------------+
> |                                                    x               +        
>  |
> |                                                    x               +        
>  |
> |                                                    x   x           + +  +   
>  |
> |                                                    x   x           +++ +++  
>  |
> |                                               x  x x   x x         +++ +++  
>  |
> |                                               x  xxx  xx xx        +++++++  
>  |
> |                                              xx xxxxx xxxxx    +  
> ++++++++++ |
> |x                          x              x   xxxx**xx xxxxxx  x++ 
> +++++++++++|
> |                                          |________AM________|   |____A____| 
>  |
> +------------------------------------------------------------------------------+
>     N           Min           Max        Median           Avg        Stddev
> x  50       8.96925       10.4881       10.2213     10.203639    0.22548474
> +  50         10.16       10.8186       10.6612     10.649134    0.12242131
> Difference at 95.0% confidence
>       0.445495 +/- 0.0719896
>       4.36604% +/- 0.705529%
>       (Student's t, pooled s = 0.181425)
> 
> Looking at the total number of intel_update_state() vs brw_draw_prims()
> for OglBatch7 doesn't alone explain the change as there are nearly the
> same number of calls to both. (Though we do see similar changes in the
> INTEL_NO_HW versions of OglBatch0 and OglBatch7, but that data is much
> more noisy.)

The explanation for the improvement in the OglBatch7 case seems to lie
in the removal of the mutex dance from intel_update_state() and the
reduction of the post-resolve _mesa_update_state() recursions.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to