Re: [Mesa-dev] [PATCH v2] mesa: remove link validation that should be done elsewhere

2016-01-19 Thread Ian Romanick
On 01/05/2016 05:40 PM, Timothy Arceri wrote: > Even if re-linking fails rendering shouldn't fail as the previous > succesfully linked program will still be available. It also shouldn't > be possible to have an unlinked program as part of the current rendering > state. > > This fixes a subtest

Re: [Mesa-dev] [PATCH v2] mesa: remove link validation that should be done elsewhere

2016-01-19 Thread Timothy Arceri
On Fri, 2016-01-15 at 08:50 +0200, Tapani Pälli wrote: > > On 01/06/2016 03:40 AM, Timothy Arceri wrote: > > Even if re-linking fails rendering shouldn't fail as the previous > > succesfully linked program will still be available. It also > > shouldn't > > be possible to have an unlinked program

Re: [Mesa-dev] [PATCH v2] mesa: remove link validation that should be done elsewhere

2016-01-19 Thread Timothy Arceri
On Tue, 2016-01-19 at 14:09 -0800, Ian Romanick wrote: > On 01/05/2016 05:40 PM, Timothy Arceri wrote: > > Even if re-linking fails rendering shouldn't fail as the previous > > succesfully linked program will still be available. It also > > shouldn't > > be possible to have an unlinked program as

Re: [Mesa-dev] [PATCH v2] mesa: remove link validation that should be done elsewhere

2016-01-14 Thread Tapani Pälli
On 01/06/2016 03:40 AM, Timothy Arceri wrote: Even if re-linking fails rendering shouldn't fail as the previous succesfully linked program will still be available. It also shouldn't be possible to have an unlinked program as part of the current rendering state. This fixes a subtest in:

[Mesa-dev] [PATCH v2] mesa: remove link validation that should be done elsewhere

2016-01-05 Thread Timothy Arceri
Even if re-linking fails rendering shouldn't fail as the previous succesfully linked program will still be available. It also shouldn't be possible to have an unlinked program as part of the current rendering state. This fixes a subtest in: ES31-CTS.sepshaderobjs.StateInteraction This change