[Mesa-dev] [PATCH 1/2] [v2] i965/skl: Add GT4 PCI IDs

2015-10-23 Thread Ben Widawsky
Like other gen8+ hardware, the hardware automatically scales up thread counts and URB sizes, so there is no need to do anything but add the PCI IDs. FINISHME: This patch still needs testing before merge. v2: Remove the PCI ID removal. That should be done as part of the next patch. Cc:

[Mesa-dev] [PATCH 2/2] [v2] i965/skl: PCI ID cleanup and brand strings

2015-10-23 Thread Ben Widawsky
A few new PCI ids are added here, and two are removed (0x190B, 0x192A) because it no longer seems to exist anywhere. v2: Update commit message to reflect the removal of 0x192a as well. Only use ascii characters (Ilia) Signed-off-by: Ben Widawsky ---

[Mesa-dev] [PATCH 2/2] intel: Cleanup SKL PCI ID definitions.

2015-10-23 Thread Ben Widawsky
This removes ones which aren't used 0x190b, 192a), and adds some new ones. I kept the original names where possible. Cc: Kristian Høgsberg Cc: Damien Lespiau Signed-off-by: Ben Widawsky --- intel/intel_chipset.h | 46

Re: [Mesa-dev] [PATCH 2/3] util: Add list_is_single() helper function

2015-10-23 Thread Eduardo Lima Mitev
On 10/23/2015 07:51 PM, Matt Turner wrote: > On Fri, Oct 23, 2015 at 8:55 AM, Eduardo Lima Mitev wrote: >> Returns true is the list has exactly one element, otherwise returns false. >> --- >> src/util/list.h | 8 >> 1 file changed, 8 insertions(+) >> >> diff --git

Re: [Mesa-dev] [PATCH 11/20] i965/fs: Append uniform entries to the gather table

2015-10-23 Thread Francisco Jerez
Abdiel Janulgue writes: > This patch generates the gather table entries for ordinary uniforms > if they are present. The uniform constants here will later be packed > together with UBO constants. > > Signed-off-by: Abdiel Janulgue

[Mesa-dev] [PATCH 2/2] [v3] i965/skl: PCI ID cleanup and brand strings

2015-10-23 Thread Ben Widawsky
A few new PCI ids are added here, and two are removed (0x190B, 0x192A) because it no longer seems to exist anywhere. v2: Update commit message to reflect the removal of 0x192a as well. Only use ascii characters (Ilia) v3: I misinterpreted the table. I don't actually know the brand strings for

[Mesa-dev] [Bug 91118] Skydrift (running in Wine) crashes on start

2015-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=91118 Béla Gyebrószki changed: What|Removed |Added Status|NEW |RESOLVED

[Mesa-dev] [PATCH 1/2] intel: Add SKL GT4 PCI IDs

2015-10-23 Thread Ben Widawsky
Cc: Kristian Høgsberg Cc: Damien Lespiau Signed-off-by: Ben Widawsky --- intel/intel_chipset.h | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-) diff --git a/intel/intel_chipset.h

Re: [Mesa-dev] [PATCH 2/3] util: Add list_is_single() helper function

2015-10-23 Thread Jason Ekstrand
On Fri, Oct 23, 2015 at 10:51 AM, Matt Turner wrote: > On Fri, Oct 23, 2015 at 8:55 AM, Eduardo Lima Mitev wrote: >> Returns true is the list has exactly one element, otherwise returns false. >> --- >> src/util/list.h | 8 >> 1 file changed, 8

Re: [Mesa-dev] [PATCH 2/3] util: Add list_is_single() helper function

2015-10-23 Thread Eduardo Lima Mitev
On 10/23/2015 08:00 PM, Jason Ekstrand wrote: > On Fri, Oct 23, 2015 at 10:51 AM, Matt Turner wrote: >> On Fri, Oct 23, 2015 at 8:55 AM, Eduardo Lima Mitev wrote: >>> Returns true is the list has exactly one element, otherwise returns false. >>> --- >>>

Re: [Mesa-dev] [RFC 18/21] mesa/extensions: Remove extra memsets on gl_extensions

2015-10-23 Thread Nanley Chery
On Thu, Oct 22, 2015 at 3:32 AM, Emil Velikov wrote: > On 19 October 2015 at 23:44, Nanley Chery wrote: > > From: Nanley Chery > > > > Aside from those modified in this commit, all gl_extensions structs are > >

[Mesa-dev] [Bug 92634] gallium's vl_mpeg12_decoder does not work with st/va

2015-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92634 Bug ID: 92634 Summary: gallium's vl_mpeg12_decoder does not work with st/va Product: Mesa Version: git Hardware: Other OS: Linux (All) Status: NEW

Re: [Mesa-dev] [PATCH 0/7] RadeonSI: Unbind shaders properly

2015-10-23 Thread Marek Olšák
On Fri, Oct 23, 2015 at 5:08 AM, Michel Dänzer wrote: > On 23.10.2015 08:12, Marek Olšák wrote: >> This series unbinds shaders properly in bind_(vs/fs/etc.)_state, because any >> context can release them now, therefore we can't rely on unbinding in >> delete_shader. This is

Re: [Mesa-dev] [PATCH 0/2] Nir: Allow CSE of SSBO loads

2015-10-23 Thread Francisco Jerez
Connor Abbott writes: > On Thu, Oct 22, 2015 at 10:37 AM, Francisco Jerez > wrote: >> Connor Abbott writes: >> >>> On Thu, Oct 22, 2015 at 7:21 AM, Iago Toral Quiroga >>> wrote: I implemented this first

Re: [Mesa-dev] [PATCH v2 5/8] i965/fs: Avoid scalar destinations in emit_uniformize()

2015-10-23 Thread Francisco Jerez
Kristian Høgsberg writes: > On Thu, Oct 22, 2015 at 12:24 PM, Matt Turner wrote: >> On Thu, Oct 22, 2015 at 11:37 AM, Kristian Høgsberg Kristensen >> wrote: >>> The scalar destination registers break copy propagation. Instead compute

Re: [Mesa-dev] [PATCH v2 8/8] i965/fs: Allow copy propagating into new surface access opcodes

2015-10-23 Thread Francisco Jerez
Kristian Høgsberg Kristensen writes: > Signed-off-by: Kristian Høgsberg Kristensen > --- > src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp | 15 +++ > 1 file changed, 15 insertions(+) > > diff --git

Re: [Mesa-dev] [PATCH 1/2] gbm: Add a flag to enable creation of rotated scanout buffers

2015-10-23 Thread Ville Syrjälä
On Fri, Oct 23, 2015 at 12:18:39PM +0900, Michel Dänzer wrote: > On 23.10.2015 10:44, Vivek Kasireddy wrote: > > For certain platforms that support rotated scanout buffers, currently, > > there is no way to create them with the GBM DRI interface. This flag > > will instruct the DRI driver to

[Mesa-dev] [Bug 92634] gallium's vl_mpeg12_decoder does not work with st/va

2015-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92634 --- Comment #5 from Julien Isorce --- (In reply to Christian König from comment #4) > Please submit "st/va: pass picture desc to begin and decode" to the mesa > mailing list. That looks like a bug to me which should be

Re: [Mesa-dev] [PATCH v3 5/7] radeonsi: Implement DCC fast clear.

2015-10-23 Thread Bas Nieuwenhuizen
On Thu, Oct 22, 2015 at 12:12 PM, Marek Olšák wrote: >> diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c >> b/src/gallium/drivers/radeonsi/si_descriptors.c >> index 5548cba3..a277fa5 100644 >> --- a/src/gallium/drivers/radeonsi/si_descriptors.c >> +++

Re: [Mesa-dev] [PATCH v3 5/7] radeonsi: Implement DCC fast clear.

2015-10-23 Thread Marek Olšák
On Fri, Oct 23, 2015 at 12:17 PM, Bas Nieuwenhuizen wrote: > On Thu, Oct 22, 2015 at 12:12 PM, Marek Olšák wrote: >>> diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c >>> b/src/gallium/drivers/radeonsi/si_descriptors.c >>> index

Re: [Mesa-dev] [PATCH v3 5/7] radeonsi: Implement DCC fast clear.

2015-10-23 Thread Marek Olšák
On Fri, Oct 23, 2015 at 1:30 PM, Bas Nieuwenhuizen wrote: > On Fri, Oct 23, 2015 at 12:50 PM, Marek Olšák wrote: >> On Fri, Oct 23, 2015 at 12:17 PM, Bas Nieuwenhuizen >> wrote: >>> On Thu, Oct 22, 2015 at 12:12 PM, Marek

Re: [Mesa-dev] [PATCH 02/20] i965: Enable gather push constants

2015-10-23 Thread Francisco Jerez
Abdiel Janulgue writes: > The 3DSTATE_GATHER_POOL_ALLOC is used to enable or disable the gather > push constants feature within a context. This patch provides the toggle > functionality of using gather push constants to program constant data > within a batch. > >

Re: [Mesa-dev] [PATCH v3 5/7] radeonsi: Implement DCC fast clear.

2015-10-23 Thread Marek Olšák
On Fri, Oct 23, 2015 at 1:57 PM, Bas Nieuwenhuizen wrote: > On Fri, Oct 23, 2015 at 1:52 PM, Marek Olšák wrote: >> On Fri, Oct 23, 2015 at 1:30 PM, Bas Nieuwenhuizen >> wrote: >>> On Fri, Oct 23, 2015 at 12:50 PM, Marek Olšák

[Mesa-dev] [Bug 92626] [BYT] piglit glx@glx-query-drawable subtests failing

2015-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92626 Jairo Miramontes changed: What|Removed |Added Component|GLX

Re: [Mesa-dev] [PATCH 03/20] i965: Allocate space on the gather pool for plain uniforms

2015-10-23 Thread Francisco Jerez
Abdiel Janulgue writes: > Reserve space in the gather pool where the gathered uniforms are flushed. > > Signed-off-by: Abdiel Janulgue > --- > src/mesa/drivers/dri/i965/gen6_vs_state.c | 8 > 1 file changed, 8

Re: [Mesa-dev] [PATCH 4/4] gallivm: fix tex offsets with mirror repeat linear

2015-10-23 Thread Jose Fonseca
On 22/10/15 23:42, srol...@vmware.com wrote: From: Roland Scheidegger Can't see why anyone would ever want to use this, but it was clearly broken. This fixes the piglit texwrap offset test using this combination. --- src/gallium/auxiliary/gallivm/lp_bld_sample_soa.c | 9

Re: [Mesa-dev] [PATCH 04/20] i965: Allocate space on the gather pool for UBO entries

2015-10-23 Thread Francisco Jerez
Francisco Jerez writes: > Abdiel Janulgue writes: > >> If there are UBO constant entries, append them to >> stage_state->push_const_size. >> The gather pool contains the combined entries of both ordinary uniforms >> and UBO constants. >>

[Mesa-dev] [PATCH 2/3] util: Add list_is_single() helper function

2015-10-23 Thread Eduardo Lima Mitev
Returns true is the list has exactly one element, otherwise returns false. --- src/util/list.h | 8 1 file changed, 8 insertions(+) diff --git a/src/util/list.h b/src/util/list.h index b98ce59..2b03461 100644 --- a/src/util/list.h +++ b/src/util/list.h @@ -99,6 +99,14 @@ static inline

[Mesa-dev] [PATCH 3/3] i965/nir/opt_peephole_ffma: Bypass fusion if any operand of fadd and fmul is a const

2015-10-23 Thread Eduardo Lima Mitev
When both fadd and fmul instructions have at least one operand that is a constant and it is only used once, the total number of instructions can be reduced from 3 (1 ffma + 2 load_const) to 2 (1 fmul + 1 fadd); because the constants will be progagated as immediate operands of fmul and fadd. This

Re: [Mesa-dev] [PATCH] i965/vec4: check opcode on vec4_instruction::reads_flag(channel)

2015-10-23 Thread Alejandro Piñeiro
On 23/10/15 17:46, Kenneth Graunke wrote: > On Friday, October 23, 2015 04:04:35 PM Alejandro Piñeiro wrote: >> Commit f17b78 added an alternative reads_flag(channel) that returned >> if the instruction was reading a specific channel flag. By mistake it >> only took into account the predicate, but

Re: [Mesa-dev] [PATCH 04/20] i965: Allocate space on the gather pool for UBO entries

2015-10-23 Thread Francisco Jerez
Abdiel Janulgue writes: > If there are UBO constant entries, append them to > stage_state->push_const_size. > The gather pool contains the combined entries of both ordinary uniforms > and UBO constants. > > Signed-off-by: Abdiel Janulgue

Re: [Mesa-dev] [PATCH] i965/vec4: check opcode on vec4_instruction::reads_flag(channel)

2015-10-23 Thread Kenneth Graunke
On Friday, October 23, 2015 04:04:35 PM Alejandro Piñeiro wrote: > Commit f17b78 added an alternative reads_flag(channel) that returned > if the instruction was reading a specific channel flag. By mistake it > only took into account the predicate, but when the opcode is >

Re: [Mesa-dev] [PATCH 3/3] i965/nir/opt_peephole_ffma: Bypass fusion if any operand of fadd and fmul is a const

2015-10-23 Thread Patrick Baggett
On Fri, Oct 23, 2015 at 10:55 AM, Eduardo Lima Mitev wrote: > When both fadd and fmul instructions have at least one operand that is a > constant and it is only used once, the total number of instructions can > be reduced from 3 (1 ffma + 2 load_const) to 2 (1 fmul + 1 fadd);

Re: [Mesa-dev] [PATCH 0/2] Nir: Allow CSE of SSBO loads

2015-10-23 Thread Jason Ekstrand
On Thu, Oct 22, 2015 at 11:13 PM, Iago Toral wrote: > On Thu, 2015-10-22 at 09:09 -0700, Jason Ekstrand wrote: >> On Thu, Oct 22, 2015 at 4:21 AM, Iago Toral Quiroga >> wrote: >> > I implemented this first as a separate optimization pass in GLSL IR [1], >>

Re: [Mesa-dev] [RFC 03/21] mesa/extensions: Wrap array entries in macros

2015-10-23 Thread Emil Velikov
On 22 October 2015 at 23:22, Nanley Chery wrote: > On Thu, Oct 22, 2015 at 3:06 AM, Emil Velikov > wrote: >> >> That aside, I'm in favour of keeping the comments as is. The editing >> comment does not apply imho. >> > > Since, there isn't a

Re: [Mesa-dev] [PATCH v2 7/8] i965/fs: Optimize ssbo stores

2015-10-23 Thread Francisco Jerez
Kristian Høgsberg Kristensen writes: > Write groups of enabled components together. > > Signed-off-by: Kristian Høgsberg Kristensen > --- > src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 56 > +++- > 1 file changed, 26

Re: [Mesa-dev] [PATCH 2/3] i965/fs: Make emit_uniformize a no-op for immediates

2015-10-23 Thread Francisco Jerez
Kristian Høgsberg writes: > On Tue, Oct 20, 2015 at 11:56 AM, Francisco Jerez > wrote: >> Kristian Høgsberg writes: >> >>> On Tue, Oct 20, 2015 at 3:16 AM, Francisco Jerez >>> wrote: Kristian Høgsberg

Re: [Mesa-dev] [PATCH] st/va: pass picture desc to begin and decode

2015-10-23 Thread Christian König
On 23.10.2015 14:25, Julien Isorce wrote: At least vl_mpeg12_decoder uses the picture desc in begin_frame and decode_bitstream. https://bugs.freedesktop.org/show_bug.cgi?id=92634 Signed-off-by: Julien Isorce Reviewed-by: Christian König ---

[Mesa-dev] [PATCH] i965/vec4: check opcode on vec4_instruction::reads_flag(channel)

2015-10-23 Thread Alejandro Piñeiro
Commit f17b78 added an alternative reads_flag(channel) that returned if the instruction was reading a specific channel flag. By mistake it only took into account the predicate, but when the opcode is VS_OPCODE_UNPACK_FLAGS_SIMD4X2 there isn't any predicate, but the flag are used. That mistake

[Mesa-dev] [Bug 92366] [BSW SKL] Regression: glx@glx-swap-event_async failed

2015-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92366 cprigent changed: What|Removed |Added Summary|[BSW] Regression: |[BSW SKL]

[Mesa-dev] [PATCH] gallivm: Explicitly disable unsupported CPU features.

2015-10-23 Thread Jose Fonseca
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=92214 --- src/gallium/auxiliary/gallivm/lp_bld_misc.cpp | 69 --- 1 file changed, 31 insertions(+), 38 deletions(-) diff --git a/src/gallium/auxiliary/gallivm/lp_bld_misc.cpp

Re: [Mesa-dev] [PATCH v2 6/8] i965/fs: Drop offset_reg temporary in ssbo load

2015-10-23 Thread Francisco Jerez
Kristian Høgsberg Kristensen writes: > Now that we don't read each component one-by-one, we don't need the > temoprary vgrf for the offset. More importantly, this register was type > UD while the nir source was type D. This broke copy propagation and left > a redundant MOV in

[Mesa-dev] [Bug 92634] gallium's vl_mpeg12_decoder does not work with st/va

2015-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92634 --- Comment #4 from Christian König --- Please submit "st/va: pass picture desc to begin and decode" to the mesa mailing list. That looks like a bug to me which should be fixed asap. Regarding "vl_mpeg12_decoder:

Re: [Mesa-dev] [PATCH v3 5/7] radeonsi: Implement DCC fast clear.

2015-10-23 Thread Bas Nieuwenhuizen
On Fri, Oct 23, 2015 at 12:50 PM, Marek Olšák wrote: > On Fri, Oct 23, 2015 at 12:17 PM, Bas Nieuwenhuizen > wrote: >> On Thu, Oct 22, 2015 at 12:12 PM, Marek Olšák wrote: diff --git

Re: [Mesa-dev] [PATCH v2 0/8] SSBO optimizations

2015-10-23 Thread Francisco Jerez
Kristian Høgsberg Kristensen writes: > Here's an updated and expanded ssbo optimization series. I found a bit > of low-hanging fruit around dynamic ssbo array indexing. I removed the > IMM shortcut in emit_uniformize() and added the constant propagation > for the read and

Re: [Mesa-dev] [RFC 20/21] mesa: Remove gl_extensions::dummy

2015-10-23 Thread Emil Velikov
On 23 October 2015 at 00:34, Nanley Chery wrote: > On Thu, Oct 22, 2015 at 3:40 AM, Emil Velikov > wrote: >> >> On 19 October 2015 at 23:44, Nanley Chery wrote: >> > From: Nanley Chery >> > >> >

[Mesa-dev] [Bug 92634] gallium's vl_mpeg12_decoder does not work with st/va

2015-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92634 --- Comment #1 from Julien Isorce --- Created attachment 119138 --> https://bugs.freedesktop.org/attachment.cgi?id=119138=edit build: enable st/va with ilo aka i965 driver -- You are receiving this mail because: You

[Mesa-dev] [Bug 92634] gallium's vl_mpeg12_decoder does not work with st/va

2015-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92634 --- Comment #2 from Julien Isorce --- Created attachment 119139 --> https://bugs.freedesktop.org/attachment.cgi?id=119139=edit st/va: pass picture desc to begin and decode -- You are receiving this mail because: You

[Mesa-dev] [Bug 92634] gallium's vl_mpeg12_decoder does not work with st/va

2015-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92634 --- Comment #3 from Julien Isorce --- Created attachment 119140 --> https://bugs.freedesktop.org/attachment.cgi?id=119140=edit vl_mpeg12_decoder: support st/va where intra_matrix can be set after begin_frame -- You

Re: [Mesa-dev] [PATCH] glsl: add buffer block number information to atomic counter variables.

2015-10-23 Thread Timothy Arceri
On Fri, 2015-10-23 at 08:57 +0200, Samuel Iglesias Gonsalvez wrote: > Atomic counter variables can have a 'binding' layout modifier. > Unfortunately, > the atomic counter buffers are not sorted by binding value in > gl_shader_program's AtomicBuffers. > > Save the atomic counter buffer block index

[Mesa-dev] [Bug 92625] [BYT] piglit glx_arb_sync_control@timing subtests fail

2015-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92625 Emil Velikov changed: What|Removed |Added Component|GLX

Re: [Mesa-dev] [PATCH] glsl: add buffer block number information to atomic counter variables.

2015-10-23 Thread Samuel Iglesias Gonsálvez
On 23/10/15 12:11, Timothy Arceri wrote: > On Fri, 2015-10-23 at 08:57 +0200, Samuel Iglesias Gonsalvez wrote: >> Atomic counter variables can have a 'binding' layout modifier. >> Unfortunately, >> the atomic counter buffers are not sorted by binding value in >> gl_shader_program's

Re: [Mesa-dev] [PATCH v3 5/7] radeonsi: Implement DCC fast clear.

2015-10-23 Thread Bas Nieuwenhuizen
On Fri, Oct 23, 2015 at 1:52 PM, Marek Olšák wrote: > On Fri, Oct 23, 2015 at 1:30 PM, Bas Nieuwenhuizen > wrote: >> On Fri, Oct 23, 2015 at 12:50 PM, Marek Olšák wrote: >>> On Fri, Oct 23, 2015 at 12:17 PM, Bas Nieuwenhuizen >>>

[Mesa-dev] [Bug 92634] gallium's vl_mpeg12_decoder does not work with st/va

2015-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92634 Julien Isorce changed: What|Removed |Added Attachment #119139|0 |1 is

[Mesa-dev] [PATCH] st/va: pass picture desc to begin and decode

2015-10-23 Thread Julien Isorce
At least vl_mpeg12_decoder uses the picture desc in begin_frame and decode_bitstream. https://bugs.freedesktop.org/show_bug.cgi?id=92634 Signed-off-by: Julien Isorce --- src/gallium/state_trackers/va/picture.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

Re: [Mesa-dev] [PATCH 06/20] i965: Assign hw-binding table index for each UBO constant buffer.

2015-10-23 Thread Francisco Jerez
Abdiel Janulgue writes: > To be able to refer to a constant buffer, the resource streamer needs > to index it with a hardware binding table entry. This blankets the ubo > buffers with hardware binding table indices. > > Gather constants hardware fetches in

[Mesa-dev] [PATCH 1/2] i965/skl: Add GT4 PCI IDs

2015-10-23 Thread Ben Widawsky
Like other gen8+ hardware, the hardware automatically scales up thread counts and URB sizes, so there is no need to do anything but add the PCI IDs. FINISHME: This patch still needs testing before merge. Cc: mesa-sta...@lists.freedesktop.org Signed-off-by: Ben Widawsky

[Mesa-dev] [PATCH 2/2] i965/skl: PCI ID cleanup and brand strings

2015-10-23 Thread Ben Widawsky
A few new PCI ids are added here, and one is removed (0x190B) because it no longer seems to exist anywhere. Signed-off-by: Ben Widawsky --- include/pci_ids/i965_pci_ids.h | 40 ++-- 1 file changed, 22 insertions(+), 18

Re: [Mesa-dev] [RFC 05/21] mesa/extensions: Add extension::version

2015-10-23 Thread Nanley Chery
On Thu, Oct 22, 2015 at 11:02 AM, Chad Versace wrote: > On Mon 19 Oct 2015, Nanley Chery wrote: > > From: Nanley Chery > > > > Enable limiting advertised extension support by context version with > > finer granularity. GLuint is chosen over

Re: [Mesa-dev] [PATCH 1/2] i965/skl: Add GT4 PCI IDs

2015-10-23 Thread Ben Widawsky
On Fri, Oct 23, 2015 at 10:37:29AM -0700, Ben Widawsky wrote: > Like other gen8+ hardware, the hardware automatically scales up thread counts > and URB sizes, so there is no need to do anything but add the PCI IDs. > > FINISHME: This patch still needs testing before merge. > > Cc:

Re: [Mesa-dev] [PATCH 3/3] i965/nir/opt_peephole_ffma: Bypass fusion if any operand of fadd and fmul is a const

2015-10-23 Thread Eduardo Lima Mitev
On 10/23/2015 06:22 PM, Patrick Baggett wrote: > > > On Fri, Oct 23, 2015 at 10:55 AM, Eduardo Lima Mitev > wrote: > > When both fadd and fmul instructions have at least one operand that is a > constant and it is only used once, the total

Re: [Mesa-dev] [PATCH 05/20] i965: Store gather table information in the program data

2015-10-23 Thread Francisco Jerez
Abdiel Janulgue writes: > The resource streamer is able to gather and pack sparsely-located > constant data from any buffer object by a referring to a gather table > This patch adds support for keeping track of these constant data > fetches into a gather table. >

Re: [Mesa-dev] [PATCH 07/20] i965: Assign hw-binding table index for uniform constant buffer block

2015-10-23 Thread Francisco Jerez
Abdiel Janulgue writes: > Assign the uploaded uniform block with hardware binding table indices. > This is indexed by the resource streamer to fetch the constant buffers > referred to by our gather table entries. > > Signed-off-by: Abdiel Janulgue

Re: [Mesa-dev] [RFC 10/21] mesa: Remove equality check in helper functions

2015-10-23 Thread Nanley Chery
On Wed, Oct 21, 2015 at 11:43 PM, Erik Faye-Lund wrote: > On Tue, Oct 20, 2015 at 12:44 AM, Nanley Chery > wrote: > > From: Nanley Chery > > > > Since the version numbers being compared are integral and we don't ever > >

Re: [Mesa-dev] [PATCH 2/3] util: Add list_is_single() helper function

2015-10-23 Thread Matt Turner
On Fri, Oct 23, 2015 at 8:55 AM, Eduardo Lima Mitev wrote: > Returns true is the list has exactly one element, otherwise returns false. > --- > src/util/list.h | 8 > 1 file changed, 8 insertions(+) > > diff --git a/src/util/list.h b/src/util/list.h > index

Re: [Mesa-dev] [PATCH 10/20] i965: Include UBO parameter sizes in push constant parameters

2015-10-23 Thread Francisco Jerez
Abdiel Janulgue writes: > Now that we consider UBO constants as push constants, we need to include > the sizes of the UBO's constant slots in the visitor's uniform slot sizes. > This information is needed to properly pack vector constants tightly next to > each

Re: [Mesa-dev] [PATCH 2/2] i965/skl: PCI ID cleanup and brand strings

2015-10-23 Thread Ben Widawsky
On Fri, Oct 23, 2015 at 01:44:38PM -0400, Ilia Mirkin wrote: > On Fri, Oct 23, 2015 at 1:37 PM, Ben Widawsky > wrote: > > A few new PCI ids are added here, and one is removed (0x190B) because it no > > longer seems to exist anywhere. > > > > Signed-off-by: Ben

Re: [Mesa-dev] [PATCH 2/3] i965/fs: Make emit_uniformize a no-op for immediates

2015-10-23 Thread Kristian Høgsberg
On Fri, Oct 23, 2015 at 5:38 AM, Francisco Jerez wrote: > Kristian Høgsberg writes: > >> On Tue, Oct 20, 2015 at 11:56 AM, Francisco Jerez >> wrote: >>> Kristian Høgsberg writes: >>> On Tue, Oct 20,

Re: [Mesa-dev] [PATCH 2/2] i965/skl: PCI ID cleanup and brand strings

2015-10-23 Thread Ilia Mirkin
On Fri, Oct 23, 2015 at 1:37 PM, Ben Widawsky wrote: > A few new PCI ids are added here, and one is removed (0x190B) because it no > longer seems to exist anywhere. > > Signed-off-by: Ben Widawsky > --- > include/pci_ids/i965_pci_ids.h |

[Mesa-dev] [PATCH 1/2] radeonsi: add support for Stoney asics (v3)

2015-10-23 Thread Alex Deucher
From: Samuel Li v2 (agd): rebase on mesa master, split pci ids to separate commit v3 (agd): use carrizo for llvm processor name for llvm 3.7 and older Signed-off-by: Samuel Li --- src/gallium/drivers/radeon/r600_pipe_common.c | 6 ++

Re: [Mesa-dev] [PATCH v3 5/7] radeonsi: Implement DCC fast clear.

2015-10-23 Thread Bas Nieuwenhuizen
On Fri, Oct 23, 2015 at 4:57 PM, Marek Olšák wrote: > On Fri, Oct 23, 2015 at 1:57 PM, Bas Nieuwenhuizen > wrote: >> On Fri, Oct 23, 2015 at 1:52 PM, Marek Olšák wrote: >>> On Fri, Oct 23, 2015 at 1:30 PM, Bas Nieuwenhuizen >>>

Re: [Mesa-dev] [RFC 05/21] mesa/extensions: Add extension::version

2015-10-23 Thread Nanley Chery
On Thu, Oct 22, 2015 at 3:17 AM, Emil Velikov wrote: > On 19 October 2015 at 23:36, Nanley Chery wrote: > > From: Nanley Chery > > > > Enable limiting advertised extension support by context version with > > finer

Re: [Mesa-dev] [PATCH 2/2] i965: Set Y-tiling for qualified rotated scanout buffers

2015-10-23 Thread Ben Widawsky
On Fri, Oct 23, 2015 at 06:19:02PM -0700, Vivek Kasireddy wrote: > On Thu, 22 Oct 2015 21:12:02 -0700 > Ben Widawsky wrote: > > > On Thu, Oct 22, 2015 at 06:44:53PM -0700, Vivek Kasireddy wrote: > > > On newer hardware platforms that support rotation, if the gbm > > >

[Mesa-dev] [PATCH 2/4] radeon/uvd: implement and add flag for VAAPI HEVC decode

2015-10-23 Thread Leo Liu
From: Boyuan Zhang Signed-off-by: Boyuan Zhang Reviewed-by: Christian König Reviewed-by: Leo Liu --- src/gallium/drivers/radeon/radeon_uvd.c | 7 +++ src/gallium/drivers/radeon/radeon_uvd.h | 9

Re: [Mesa-dev] [PATCH 11/20] i965/fs: Append uniform entries to the gather table

2015-10-23 Thread Francisco Jerez
Matt Turner writes: > On Fri, Oct 23, 2015 at 11:23 AM, Francisco Jerez > wrote: >> Abdiel Janulgue writes: >> >>> This patch generates the gather table entries for ordinary uniforms >>> if they are present. The

Re: [Mesa-dev] [PATCH 11/20] i965/fs: Append uniform entries to the gather table

2015-10-23 Thread Matt Turner
On Fri, Oct 23, 2015 at 12:38 PM, Francisco Jerez wrote: > Matt Turner writes: > >> On Fri, Oct 23, 2015 at 11:23 AM, Francisco Jerez >> wrote: >>> Abdiel Janulgue writes: >>> This patch

[Mesa-dev] [PATCH 3/4] st/va: add VAAPI HEVC decode support

2015-10-23 Thread Leo Liu
From: Boyuan Zhang Signed-off-by: Boyuan Zhang Reviewed-by: Christian König Reviewed-by: Leo Liu --- configure.ac | 2 +- src/gallium/state_trackers/va/config.c | 2 +-

[Mesa-dev] [PATCH 0/4] HEVC support for VA-API

2015-10-23 Thread Leo Liu
The series support HEVC codec for VA-API and fix for VDPAU they are tested with the latest ffmpeg and gstreamer-vaapi plugins. Boyuan Zhang (4): vl: add RefPicList defines for VAAPI HEVC decode radeon/uvd: implement and add flag for VAAPI HEVC decode st/va: add VAAPI HEVC decode support

[Mesa-dev] [PATCH 1/4] vl: add RefPicList defines for VAAPI HEVC decode

2015-10-23 Thread Leo Liu
From: Boyuan Zhang Signed-off-by: Boyuan Zhang Reviewed-by: Christian König Reviewed-by: Leo Liu --- src/gallium/include/pipe/p_video_state.h | 2 ++ 1 file changed, 2 insertions(+) diff --git

[Mesa-dev] [PATCH 4/4] st/vdpau: disable RefPicList for Vdpau HEVC

2015-10-23 Thread Leo Liu
From: Boyuan Zhang Signed-off-by: Boyuan Zhang Reviewed-by: Christian König Reviewed-by: Leo Liu --- src/gallium/state_trackers/vdpau/decode.c | 1 + 1 file changed, 1 insertion(+) diff --git

Re: [Mesa-dev] [PATCH 1/2] gbm: Add a flag to enable creation of rotated scanout buffers

2015-10-23 Thread Vivek Kasireddy
On Fri, 23 Oct 2015 12:18:39 +0900 Michel Dänzer wrote: > On 23.10.2015 10:44, Vivek Kasireddy wrote: > > For certain platforms that support rotated scanout buffers, > > currently, there is no way to create them with the GBM DRI > > interface. This flag will instruct the DRI

Re: [Mesa-dev] [PATCH 2/2] i965: Set Y-tiling for qualified rotated scanout buffers

2015-10-23 Thread Vivek Kasireddy
On Thu, 22 Oct 2015 21:12:02 -0700 Ben Widawsky wrote: > On Thu, Oct 22, 2015 at 06:44:53PM -0700, Vivek Kasireddy wrote: > > On newer hardware platforms that support rotation, if the gbm > > interface requests to create a rotated scanout buffer via the > > flag

Re: [Mesa-dev] [PATCH 1/2] gbm: Add a flag to enable creation of rotated scanout buffers

2015-10-23 Thread Vivek Kasireddy
On Fri, 23 Oct 2015 15:29:08 +0300 Ville Syrjälä wrote: > On Fri, Oct 23, 2015 at 12:18:39PM +0900, Michel Dänzer wrote: > > On 23.10.2015 10:44, Vivek Kasireddy wrote: > > > For certain platforms that support rotated scanout buffers, > > > currently, there is no

[Mesa-dev] [PATCH 3/4] i965/vec4: Move vec4_generator class definition into the .cpp file.

2015-10-23 Thread Kenneth Graunke
The public API for the generator is brw_vec4_generate_code(); nobody actually needs to use the class. This means we can extend it without triggering the recompiles associated with altering brw_vec4.h. Signed-off-by: Kenneth Graunke ---

[Mesa-dev] [PATCH 2/4] i965/vec4: Wrap vec4_generator in a C function.

2015-10-23 Thread Kenneth Graunke
vec4_generator is a class for convenience, but only exports a single method as its public API. It makes much more sense to just export a single function. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_vec4.cpp| 6 +++---

[Mesa-dev] [PATCH 1/4] i965/vec4: Convert src_reg/dst_reg to brw_reg at the end of the visitor.

2015-10-23 Thread Kenneth Graunke
This patch makes the visitor convert registers to the HW_REG file at the very end, after register allocation, post-RA scheduling, and dependency control flagging. After that, everything is in fixed brw_regs. This simplifies the code generator, as it can just use the hardware registers rather

[Mesa-dev] [PATCH] i965/vec4: Eliminate the vec4_generator class altogether.

2015-10-23 Thread Kenneth Graunke
We really weren't taking advantage of vec4_generator being a class. By adding a "p" parameter to the helper methods, and "prog_data" to ones which need binding table information, we can convert everything to static functions. Signed-off-by: Kenneth Graunke ---

[Mesa-dev] [Bug 92366] [BSW SKL] Regression: glx@glx-swap-event_async failed

2015-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92366 Ben Widawsky changed: What|Removed |Added Status|NEW |NEEDINFO

Re: [Mesa-dev] [PATCH 1/2] radeonsi: add support for Stoney asics (v2)

2015-10-23 Thread Marek Olšák
On Fri, Oct 23, 2015 at 5:11 PM, Alex Deucher wrote: > On Thu, Oct 22, 2015 at 10:44 PM, Michel Dänzer wrote: >> On 23.10.2015 01:09, Alex Deucher wrote: >>> From: Samuel Li >>> >>> v2 (agd): rebase on mesa master, split pci ids to

[Mesa-dev] [PATCH] i965/vec4: select predicate based on writemask for sel emissions

2015-10-23 Thread Alejandro Piñeiro
Equivalent to commit 4eebeb but with sel operations. In this case we select the PredCtrl based on the writemask. This change allows cmod propagation to optimize out several instructions. Shader-db numbers: total instructions in shared programs: 6235835 -> 6228008 (-0.13%) instructions in

Re: [Mesa-dev] [PATCH 1/2] radeonsi: add support for Stoney asics (v2)

2015-10-23 Thread Alex Deucher
On Thu, Oct 22, 2015 at 10:44 PM, Michel Dänzer wrote: > On 23.10.2015 01:09, Alex Deucher wrote: >> From: Samuel Li >> >> v2 (agd): rebase on mesa master, split pci ids to >> separate commit >> >> Signed-off-by: Samuel Li > > [...] > >>

[Mesa-dev] [Bug 92634] gallium's vl_mpeg12_decoder does not work with st/va

2015-10-23 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=92634 --- Comment #6 from Christian König --- (In reply to Julien Isorce from comment #5) > Just to be sure I understand correctly, do you mean the fix should actually > be done in st/va directly, right ? Yes, calling

Re: [Mesa-dev] [PATCH 03/20] i965: Allocate space on the gather pool for plain uniforms

2015-10-23 Thread Francisco Jerez
Francisco Jerez writes: > Abdiel Janulgue writes: > >> Reserve space in the gather pool where the gathered uniforms are flushed. >> >> Signed-off-by: Abdiel Janulgue >> --- >>

[Mesa-dev] [PATCH] nvc0: respect edgeflag attribute width

2015-10-23 Thread Ilia Mirkin
The edgeflag comes in as ubyte with glEdgeFlagPointer but as float with plain immediate glEdgeFlag. Avoid reading bytes that weren't meant for the edgeflag in the pointer case. Fixes intermittent failures with gl-2.0-edgeflag piglit (and valgrind complaints about reading uninitialized memory).

[Mesa-dev] [PATCH] glsl: add buffer block number information to atomic counter variables.

2015-10-23 Thread Samuel Iglesias Gonsalvez
Atomic counter variables can have a 'binding' layout modifier. Unfortunately, the atomic counter buffers are not sorted by binding value in gl_shader_program's AtomicBuffers. Save the atomic counter buffer block index into the variable, so we can use it inside the drivers. Fixes 298

Re: [Mesa-dev] [PATCH] glsl: remove excess location qualifier validation

2015-10-23 Thread Timothy Arceri
On Fri, 2015-10-23 at 08:01 +0300, Tapani Pälli wrote: > On 10/22/2015 11:07 AM, Timothy Arceri wrote: > > On Thu, 2015-10-22 at 09:56 +0300, Tapani Pälli wrote: > > > On 10/22/2015 09:41 AM, Timothy Arceri wrote: > > > > On Thu, 2015-10-22 at 08:55 +0300, Tapani Pälli wrote: > > > > > On

Re: [Mesa-dev] [PATCH 1/2] main: Remove interface block array index for doing the name comparison

2015-10-23 Thread Samuel Iglesias Gonsálvez
On 22/10/15 13:19, Samuel Iglesias Gonsálvez wrote: > > > On 22/10/15 13:08, Tapani Pälli wrote: >> On 10/22/2015 12:01 PM, Samuel Iglesias Gonsalvez wrote: >>> From ARB_program_query_interface spec: >>> >>> "uint GetProgramResourceIndex(uint program, enum programInterface, >>>

Re: [Mesa-dev] [PATCH 0/2] Nir: Allow CSE of SSBO loads

2015-10-23 Thread Iago Toral
On Thu, 2015-10-22 at 09:09 -0700, Jason Ekstrand wrote: > On Thu, Oct 22, 2015 at 4:21 AM, Iago Toral Quiroga wrote: > > I implemented this first as a separate optimization pass in GLSL IR [1], but > > Curro pointed out that this being pretty much a restricted form of a CSE >

Re: [Mesa-dev] [PATCH 1/2] main: Remove interface block array index for doing the name comparison

2015-10-23 Thread Samuel Iglesias Gonsálvez
On 22/10/15 13:06, Timothy Arceri wrote: > On Thu, 2015-10-22 at 11:01 +0200, Samuel Iglesias Gonsalvez wrote: >> From ARB_program_query_interface spec: >> >> "uint GetProgramResourceIndex(uint program, enum programInterface, >>const char *name); >> [...] >>

Re: [Mesa-dev] [PATCH 2/2] radeonsi: add Stoney pci ids

2015-10-23 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Thu, Oct 22, 2015 at 6:09 PM, Alex Deucher wrote: > From: Samuel Li > > Signed-off-by: Samuel Li > --- > include/pci_ids/radeonsi_pci_ids.h | 2 ++ > 1 file changed, 2

  1   2   >