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>
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