On 7 December 2011 13:16, Eric Anholt <<a href="mailto:eric@anholt.net">eric@anholt.net</a>> wrote: <div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"> <div class="im">On Wed, 7 Dec 2011 11:09:12 -0800, Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>> wrote: > From: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>> > > We never filled this in before because we didn't care. > > I'm skeptical these are correct; my sources indicate that both the VS > and GS # of entries are 256 on both GT1 and GT2. > > I'm also loathe to change it and break stuff. </div>My stale specs say GT2 VS: [24, 256] GT1 VS: [24, 128] GT2 GS: [0, 256] GT1 GS: [0, 254]. So these look like the docs our old VS numbers came from. The current docs I see say: GT1 VS: [24, 256] GT1 GS: [0, 256] with the mention of gt2 removed. The changelog doesn't show anything obvious about "update DevSNB 3DSTATE_URB", though there are things for other hardware that look like they might have stomped the old information in the way it looks stomped. So, my guess is we should stick with our old values for VS (as is done in this change), and use the old values for GS (GT2 256, GT1 254), and go complain to doc authors for clarification. </blockquote></div> I will try this and rerun piglit tests to see if anything breaks.