[Mesa-dev] [PATCH 1/2] mesa: Ensure that transform feedback refers to the correct program.

Kenneth Graunke kenneth at whitecape.org
Tue Jan 28 14:09:33 PST 2014


On 01/28/2014 01:26 PM, Paul Berry wrote:
> On 28 January 2014 13:15, Carl Worth <cworth at cworth.org
> <mailto:cworth at cworth.org>> wrote:
> 
>     Paul Berry <stereotype441 at gmail.com
>     <mailto:stereotype441 at gmail.com>> writes:
>     > Previous to this patch, the _mesa_{Begin,Resume}TransformFeedback
>     > functions were using ctx->Shader.CurrentProgram[MESA_SHADER_VERTEX]
> 
>     I see that this commit was nominated for the stable branch when it was
>     committed.
> 
>     However, it appears to depend at least on commit b22146dc714 which is on
> 
> 
> I think you mean 3b22146dc714 (mesa: Replace
> ctx->Shader.Current{Vertex,Fragment,Geometry}Program with an array.).
>  
> 
>     master, but is not on the 10.0 branch. So this patch cannot be applied
>     as-is to 10.0.
> 
>     My current inclination is to simply drop this patch from the list of
>     candidates for 10.0. If you think differently, please recommend an
>     alternate plan.
> 
>     -Carl
> 
> 
> (Also from IRC: <cworth> stereotype441: In my recent mail, I should have
> included 43e77215b13 as well. (It's another patch I plan to drop from
> stable unless you guide me differently.))
> 
> In principle we could make versions of these patches that don't depend
> on commit 3b22146dc714, but I don't think it's worth it--the bugs I've
> fixed in these patches are unlikely to arise until
> ARB_separate_shader_objects is supported.
> 
> So yeah, I'm ok with dropping these two patches from the list of
> candidates for 10.0.
> 
> I believe it was Ken who initially suggested marking these patches as
> candidates for stable.  Cc'ing him in case he wants to express a
> dissenting opinion.

Given that we discovered the only use of "shader" is a NULL-check to
determine if it's GLSL or ARB programs...I don't think it's nearly as
important as I thought.  I'd just drop it from stable.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20140128/8fd4c0b0/attachment-0001.pgp>


More information about the mesa-dev mailing list