Re: [Mesa-dev] [PATCH 30/53] mesa: don't always set _NEW_PROGRAM when linking

2017-01-03 Thread Marek Olšák
On Wed, Jan 4, 2017 at 12:56 AM, Timothy Arceri wrote: > On Wed, 2017-01-04 at 00:44 +0100, Marek Olšák wrote: >> On Wed, Jan 4, 2017 at 12:36 AM, Timothy Arceri >> wrote: >> > On Tue, 2017-01-03 at 23:55 +0100, Marek Olšák wrote: >> > > On Tue, Jan 3, 2017 at 11:14 PM, Timothy Arceri >> > > wro

Re: [Mesa-dev] [PATCH 30/53] mesa: don't always set _NEW_PROGRAM when linking

2017-01-03 Thread Timothy Arceri
On Wed, 2017-01-04 at 00:44 +0100, Marek Olšák wrote: > On Wed, Jan 4, 2017 at 12:36 AM, Timothy Arceri > wrote: > > On Tue, 2017-01-03 at 23:55 +0100, Marek Olšák wrote: > > > On Tue, Jan 3, 2017 at 11:14 PM, Timothy Arceri > > > wrote: > > > > On Tue, 2017-01-03 at 22:41 +0100, Marek Olšák wrot

Re: [Mesa-dev] [PATCH 30/53] mesa: don't always set _NEW_PROGRAM when linking

2017-01-03 Thread Marek Olšák
On Wed, Jan 4, 2017 at 12:36 AM, Timothy Arceri wrote: > On Tue, 2017-01-03 at 23:55 +0100, Marek Olšák wrote: >> On Tue, Jan 3, 2017 at 11:14 PM, Timothy Arceri >> wrote: >> > On Tue, 2017-01-03 at 22:41 +0100, Marek Olšák wrote: >> > > On Tue, Jan 3, 2017 at 10:14 PM, Timothy Arceri >> > > wro

Re: [Mesa-dev] [PATCH 30/53] mesa: don't always set _NEW_PROGRAM when linking

2017-01-03 Thread Timothy Arceri
On Tue, 2017-01-03 at 23:55 +0100, Marek Olšák wrote: > On Tue, Jan 3, 2017 at 11:14 PM, Timothy Arceri > wrote: > > On Tue, 2017-01-03 at 22:41 +0100, Marek Olšák wrote: > > > On Tue, Jan 3, 2017 at 10:14 PM, Timothy Arceri > > > wrote: > > > > On Tue, 2017-01-03 at 22:07 +0100, Marek Olšák wrot

Re: [Mesa-dev] [PATCH 30/53] mesa: don't always set _NEW_PROGRAM when linking

2017-01-03 Thread Marek Olšák
On Tue, Jan 3, 2017 at 11:14 PM, Timothy Arceri wrote: > On Tue, 2017-01-03 at 22:41 +0100, Marek Olšák wrote: >> On Tue, Jan 3, 2017 at 10:14 PM, Timothy Arceri >> wrote: >> > On Tue, 2017-01-03 at 22:07 +0100, Marek Olšák wrote: >> > > On Tue, Jan 3, 2017 at 9:57 PM, Timothy Arceri >> > > wrot

Re: [Mesa-dev] [PATCH 30/53] mesa: don't always set _NEW_PROGRAM when linking

2017-01-03 Thread Timothy Arceri
On Tue, 2017-01-03 at 22:41 +0100, Marek Olšák wrote: > On Tue, Jan 3, 2017 at 10:14 PM, Timothy Arceri > wrote: > > On Tue, 2017-01-03 at 22:07 +0100, Marek Olšák wrote: > > > On Tue, Jan 3, 2017 at 9:57 PM, Timothy Arceri > > > wrote: > > > > On Tue, 2017-01-03 at 20:47 +0100, Marek Olšák wrote

Re: [Mesa-dev] [PATCH 30/53] mesa: don't always set _NEW_PROGRAM when linking

2017-01-03 Thread Marek Olšák
On Tue, Jan 3, 2017 at 10:14 PM, Timothy Arceri wrote: > On Tue, 2017-01-03 at 22:07 +0100, Marek Olšák wrote: >> On Tue, Jan 3, 2017 at 9:57 PM, Timothy Arceri >> wrote: >> > On Tue, 2017-01-03 at 20:47 +0100, Marek Olšák wrote: >> > > On Tue, Jan 3, 2017 at 3:43 AM, Timothy Arceri >> > > wrote

Re: [Mesa-dev] [PATCH 30/53] mesa: don't always set _NEW_PROGRAM when linking

2017-01-03 Thread Timothy Arceri
On Tue, 2017-01-03 at 22:07 +0100, Marek Olšák wrote: > On Tue, Jan 3, 2017 at 9:57 PM, Timothy Arceri > wrote: > > On Tue, 2017-01-03 at 20:47 +0100, Marek Olšák wrote: > > > On Tue, Jan 3, 2017 at 3:43 AM, Timothy Arceri > > > wrote: > > > > We only need to set it when linking was successful an

Re: [Mesa-dev] [PATCH 30/53] mesa: don't always set _NEW_PROGRAM when linking

2017-01-03 Thread Marek Olšák
On Tue, Jan 3, 2017 at 9:57 PM, Timothy Arceri wrote: > On Tue, 2017-01-03 at 20:47 +0100, Marek Olšák wrote: >> On Tue, Jan 3, 2017 at 3:43 AM, Timothy Arceri >> wrote: >> > We only need to set it when linking was successful and the program >> > being linked is currently active. >> > >> > The pr

Re: [Mesa-dev] [PATCH 30/53] mesa: don't always set _NEW_PROGRAM when linking

2017-01-03 Thread Timothy Arceri
On Tue, 2017-01-03 at 20:47 +0100, Marek Olšák wrote: > On Tue, Jan 3, 2017 at 3:43 AM, Timothy Arceri > wrote: > > We only need to set it when linking was successful and the program > > being linked is currently active. > > > > The programs_in_use mask is just used as a flag for now but in > > a

Re: [Mesa-dev] [PATCH 30/53] mesa: don't always set _NEW_PROGRAM when linking

2017-01-03 Thread Marek Olšák
On Tue, Jan 3, 2017 at 3:43 AM, Timothy Arceri wrote: > We only need to set it when linking was successful and the program > being linked is currently active. > > The programs_in_use mask is just used as a flag for now but in > a following patch we will use it to update the CurrentProgram > array.

[Mesa-dev] [PATCH 30/53] mesa: don't always set _NEW_PROGRAM when linking

2017-01-02 Thread Timothy Arceri
We only need to set it when linking was successful and the program being linked is currently active. The programs_in_use mask is just used as a flag for now but in a following patch we will use it to update the CurrentProgram array. --- src/mesa/main/shaderapi.c | 22 +- 1 fil