On Wednesday, January 21, 2015 08:17:35 PM ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrj...@linux.intel.com>
> 
> According to BSpec the correct number for min_vs_entries is 34 for CHV.
> 
> Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
> ---
>  src/mesa/drivers/dri/i965/brw_device_info.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_device_info.c 
> b/src/mesa/drivers/dri/i965/brw_device_info.c
> index 2c3af66..bdef42b 100644
> --- a/src/mesa/drivers/dri/i965/brw_device_info.c
> +++ b/src/mesa/drivers/dri/i965/brw_device_info.c
> @@ -244,7 +244,7 @@ static const struct brw_device_info brw_device_info_chv = 
> {
>     .max_wm_threads = 128,
>     .urb = {
>        .size = 128,
> -      .min_vs_entries = 64,
> +      .min_vs_entries = 34,
>        .max_vs_entries = 640,
>        .max_gs_entries = 256,
>     }
> 

Yes, it is.  Thanks!  Given that we always try to allocate the VS as much
space as possible, I doubt this will have much practical impact - and we're
overallocating now - so I don't think it's a stable candidate.

Definitely worth fixing though.

Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>

Attachment: signature.asc
Description: This is a digitally signed message part.

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to