[Mesa-dev] [PATCH] glsl : validate output types for shader stages

2014-08-11 Thread Tapani Pälli
Patch fixes Piglit test: arb_gpu_shader_fp64/preprocessor/fs-output-double.frag and adds additional validation for shader outputs. Signed-off-by: Tapani Pälli --- src/glsl/ast_to_hir.cpp | 47 +++ 1 file changed, 47 insertions(+) diff --git a/src/

Re: [Mesa-dev] [PATCH] hud: don't overrun malloced arrays

2014-08-11 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Mon, Aug 11, 2014 at 4:09 AM, Dave Airlie wrote: > From: Dave Airlie > > ==17630== Invalid read of size 4 > ==17630==at 0x400AE10: memcpy (in > /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so) > ==17630==by 0x49024A2: u_upload_data (u_upload_mgr.c:2

Re: [Mesa-dev] [PATCH] i965: Allow the blorp blit between BGR and RGB

2014-08-11 Thread Neil Roberts
Thanks for the review. I've pushed the Mesa patch and I'll have a look at adding more formats to the Piglit test. Regards, - Neil Kenneth Graunke writes: > On Tuesday, July 01, 2014 04:04:56 PM Neil Roberts wrote: >> > FWIW, I relaxed the format restrictions in >> > brw_blorp_copytexsubimage, s

Re: [Mesa-dev] [PATCH v2 1/4] gallium: add basic support for BPTC formats

2014-08-11 Thread Marek Olšák
Reviewed-by: Marek Olšák Marek On Mon, Aug 11, 2014 at 4:21 AM, Ilia Mirkin wrote: > Signed-off-by: Ilia Mirkin > --- > > v1 -> v2: > - add cases to util_format_srgb and util_format_linear > - remove special SRGBA case from util_format_fits_8unorm > > Not sure if I should rename the SRGBA on

[Mesa-dev] [PATCH] i965: Enable EWA anisotropic filtering algorithm for isotropic case

2014-08-11 Thread Pavel Popov
See the reasons in Bug 82463: https://bugs.freedesktop.org/show_bug.cgi?id=82463 Signed-off-by: Pavel Popov --- src/mesa/drivers/dri/i965/brw_sampler_state.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_sampler_state.c b/src/mesa/drivers/

Re: [Mesa-dev] [PATCH 7/8] mesa: fix assertion in _mesa_drawbuffers()

2014-08-11 Thread Brian Paul
I disagree. The assertion is doing sanity checking on all the buffer masks for buffer index > 0. -Brian On 08/10/2014 10:55 PM, Popov, Pavel E wrote: I also suggest removing the assertion "ASSERT(_mesa_bitcount(destMask[buf]) == 1);". It's no longer needed with this patch. - Pavel -Orig

Re: [Mesa-dev] [PATCH v2 1/4] gallium: add basic support for BPTC formats

2014-08-11 Thread Brian Paul
Patches 1-3 look OK to me. I don't have a strong opinion on the UNORM suffix. Reviewed-by: Brian Paul On 08/10/2014 08:21 PM, Ilia Mirkin wrote: Signed-off-by: Ilia Mirkin --- v1 -> v2: - add cases to util_format_srgb and util_format_linear - remove special SRGBA case from util_format

Re: [Mesa-dev] [PATCH 3/8] mesa: use PRIu64 for printing 64-bit uints

2014-08-11 Thread Brian Paul
On 08/08/2014 06:15 PM, Roland Scheidegger wrote: Am 08.08.2014 23:20, schrieb Brian Paul: Silences MinGW warnings: warning: unknown conversion type character ‘l’ in format [-Wformat] warning: too many arguments for format [-Wformat-extra-args] --- src/mesa/main/bufferobj.c | 33

[Mesa-dev] [PATCH] kms_flip: Improve the accuracy of out frame time calculation

2014-08-11 Thread ville . syrjala
From: Ville Syrjälä Don't use the rounded vrefresh info to predict the frame duration. Instead calculate if from the clock. --- tests/kms_flip.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/kms_flip.c b/tests/kms_flip.c index 227a176..f609ac0 100644 --- a/tests/k

[Mesa-dev] [PATCH 2/2] Use floating point for VTotal when calculating refresh rate

2014-08-11 Thread ville . syrjala
From: Foo Bar Interlaced modes generally have an odd VTotal, so we lose half a line from VTotal when we divide by two. That causes the final refresh rate to be slightly off. Make VTotal a double to avoid the problem. Signed-off-by: Foo Bar --- xrandr.c | 2 +- 1 file changed, 1 insertion(+), 1

[Mesa-dev] [PATCH 1/2] Use more decimal places when printing various rates

2014-08-11 Thread ville . syrjala
From: Foo Bar Using just one decimal place for dotclock and refresh rates loses quite a bit of information. When dealing with 60Hz and 59.94Hz modes for example it's useful to see at least two decimal places. Also for the dotclock in the same cases three decimal places seems quite a bit better th

Re: [Mesa-dev] [PATCH] kms_flip: Improve the accuracy of out frame time calculation

2014-08-11 Thread Ville Syrjälä
Sorry, ignore these. Shell history and fast fingers are a dangerous combination... -- Ville Syrjälä Intel OTC ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] gallium: remove support for predicates from TGSI

2014-08-11 Thread Roland Scheidegger
Am 09.08.2014 05:24, schrieb Connor Abbott: > On Wed, Aug 6, 2014 at 6:29 PM, Marek Olšák wrote: >> What IR? A flatland GLSL IR? A replacement for Mesa IR? Something else? > > It's a flatland IR, similar to TGSI/Direct3D style with enough GLSL > IR-like stuff to get existing things working now an

Re: [Mesa-dev] [PATCH] mesa/st: add support for dynamic sampler offsets

2014-08-11 Thread Roland Scheidegger
Am 09.08.2014 17:25, schrieb Ilia Mirkin: > On Sat, Aug 9, 2014 at 11:12 AM, Roland Scheidegger > wrote: >> Am 09.08.2014 16:33, schrieb Ilia Mirkin: >>> On Sat, Aug 9, 2014 at 10:14 AM, Roland Scheidegger >>> wrote: On closer look, it looks to me like it wouldn't be all that difficult to

Re: [Mesa-dev] [PATCH] mesa/st: add support for dynamic sampler offsets

2014-08-11 Thread Ilia Mirkin
On Mon, Aug 11, 2014 at 10:56 AM, Roland Scheidegger wrote: > Am 09.08.2014 17:25, schrieb Ilia Mirkin: >> On Sat, Aug 9, 2014 at 11:12 AM, Roland Scheidegger >> wrote: >>> Am 09.08.2014 16:33, schrieb Ilia Mirkin: On Sat, Aug 9, 2014 at 10:14 AM, Roland Scheidegger wrote: > On c

Re: [Mesa-dev] [PATCH 3/8] mesa: use PRIu64 for printing 64-bit uints

2014-08-11 Thread Roland Scheidegger
Am 11.08.2014 15:29, schrieb Brian Paul: > On 08/08/2014 06:15 PM, Roland Scheidegger wrote: >> Am 08.08.2014 23:20, schrieb Brian Paul: >>> Silences MinGW warnings: >>> warning: unknown conversion type character ‘l’ in format [-Wformat] >>> warning: too many arguments for format [-Wformat-extr

[Mesa-dev] [PATCH] mesa/st: emit sampler declarations as arrays when indirect accesses are done

2014-08-11 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin --- This applies on top of my previous patch to allow dynamic sampler offsets. Roland, I believe this should address your concerns. The generated code looks like: FRAG PROPERTY FS_COLOR0_WRITES_ALL_CBUFS 1 DCL IN[0], GENERIC[0], PERSPECTIVE DCL OUT[0], COLOR DCL SAMP[

Re: [Mesa-dev] [PATCH 3/8] mesa: use PRIu64 for printing 64-bit uints

2014-08-11 Thread Brian Paul
On 08/11/2014 09:10 AM, Roland Scheidegger wrote: Am 11.08.2014 15:29, schrieb Brian Paul: On 08/08/2014 06:15 PM, Roland Scheidegger wrote: Am 08.08.2014 23:20, schrieb Brian Paul: Silences MinGW warnings: warning: unknown conversion type character ‘l’ in format [-Wformat] warning: too

[Mesa-dev] [PATCH 2/2] st/mesa: use PRId64 for printing 64-bit ints

2014-08-11 Thread Brian Paul
v2: use signed types/formats Reviewed-by: Ian Romanick --- src/mesa/state_tracker/st_cb_bufferobjects.c |5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/mesa/state_tracker/st_cb_bufferobjects.c b/src/mesa/state_tracker/st_cb_bufferobjects.c index 3b4d28d..e0cb979 1

[Mesa-dev] [PATCH 1/2] mesa: use PRId64 for printing 64-bit ints

2014-08-11 Thread Brian Paul
Silences MinGW warnings: warning: unknown conversion type character ‘l’ in format [-Wformat] warning: too many arguments for format [-Wformat-extra-args] v2: use signed types/formats Reviewed-by: Ian Romanick --- src/mesa/main/bufferobj.c | 33 + src/mesa/mai

Re: [Mesa-dev] [PATCH 1/2] mesa: use PRId64 for printing 64-bit ints

2014-08-11 Thread Roland Scheidegger
Both patches Reviewed-by: Roland Scheidegger Am 11.08.2014 17:24, schrieb Brian Paul: > Silences MinGW warnings: > warning: unknown conversion type character ‘l’ in format [-Wformat] > warning: too many arguments for format [-Wformat-extra-args] > > v2: use signed types/formats > > Reviewed-b

Re: [Mesa-dev] [PATCH v2 1/4] gallium: add basic support for BPTC formats

2014-08-11 Thread Roland Scheidegger
Am 11.08.2014 04:21, schrieb Ilia Mirkin: > Signed-off-by: Ilia Mirkin > --- > > v1 -> v2: > - add cases to util_format_srgb and util_format_linear > - remove special SRGBA case from util_format_fits_8unorm > > Not sure if I should rename the SRGBA one to not have a _UNORM ending... Might be a

Re: [Mesa-dev] [PATCH] i965: Enable EWA anisotropic filtering algorithm for isotropic case

2014-08-11 Thread Kenneth Graunke
On Monday, August 11, 2014 07:05:36 PM Pavel Popov wrote: > See the reasons in Bug 82463: > https://bugs.freedesktop.org/show_bug.cgi?id=82463 > > Signed-off-by: Pavel Popov > --- > src/mesa/drivers/dri/i965/brw_sampler_state.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > dif

Re: [Mesa-dev] [PATCH] mesa/st: emit sampler declarations as arrays when indirect accesses are done

2014-08-11 Thread Roland Scheidegger
Am 11.08.2014 17:18, schrieb Ilia Mirkin: > Signed-off-by: Ilia Mirkin > --- > > This applies on top of my previous patch to allow dynamic sampler offsets. > > Roland, I believe this should address your concerns. The generated code looks > like: > > FRAG > PROPERTY FS_COLOR0_WRITES_ALL_CBUFS 1

Re: [Mesa-dev] [PATCH] mesa/st: emit sampler declarations as arrays when indirect accesses are done

2014-08-11 Thread Ilia Mirkin
On Mon, Aug 11, 2014 at 12:43 PM, Roland Scheidegger wrote: > Am 11.08.2014 17:18, schrieb Ilia Mirkin: >> Signed-off-by: Ilia Mirkin >> --- >> >> This applies on top of my previous patch to allow dynamic sampler offsets. >> >> Roland, I believe this should address your concerns. The generated co

Re: [Mesa-dev] [PATCH] mesa/st: emit sampler declarations as arrays when indirect accesses are done

2014-08-11 Thread Roland Scheidegger
Am 11.08.2014 18:56, schrieb Ilia Mirkin: > On Mon, Aug 11, 2014 at 12:43 PM, Roland Scheidegger > wrote: >> Am 11.08.2014 17:18, schrieb Ilia Mirkin: >>> Signed-off-by: Ilia Mirkin >>> --- >>> >>> This applies on top of my previous patch to allow dynamic sampler offsets. >>> >>> Roland, I belie

Re: [Mesa-dev] [PATCH] gallium: remove support for predicates from TGSI

2014-08-11 Thread Marek Olšák
On Sat, Aug 9, 2014 at 5:24 AM, Connor Abbott wrote: > On Wed, Aug 6, 2014 at 6:29 PM, Marek Olšák wrote: >> What IR? A flatland GLSL IR? A replacement for Mesa IR? Something else? > > It's a flatland IR, similar to TGSI/Direct3D style with enough GLSL > IR-like stuff to get existing things worki

Re: [Mesa-dev] [PATCH 7/8] mesa: fix assertion in _mesa_drawbuffers()

2014-08-11 Thread Roland Scheidegger
Am 11.08.2014 00:51, schrieb Brian Paul: > On 08/08/2014 07:43 PM, Roland Scheidegger wrote: >> Am 08.08.2014 23:20, schrieb Brian Paul: >>> Fixes failed assertion when _mesa_update_draw_buffers() was called >>> with GL_DRAW_BUFFER == GL_FRONT_AND_BACK. The piglit gl30basic hit >>> this. >>> >>> C

[Mesa-dev] [Bug 53608] lp_test_conv crashes on Sandy Bridge

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=53608 Roland Scheidegger changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

Re: [Mesa-dev] [PATCH 4/5] i965: Add support for ARB_copy_image

2014-08-11 Thread Neil Roberts
Jason Ekstrand writes: > + /* If we are on the same miptree, same level, and same slice, then > +* intel_miptree_map won't let us map it twice. We have to do things a > +* big differently. In particular, we do a single map large There's a small typo here. Otherwise looks great to me

[Mesa-dev] [Bug 41162] util/u_draw.c:71:util_draw_max_index: Assertion `buffer_size - buffer->buffer_offset <= buffer_size' failed.

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=41162 --- Comment #2 from Roland Scheidegger --- Is this still happening with recent mesa? -- You are receiving this mail because: You are the assignee for the bug. ___ mesa-dev mailing list mesa-dev@lists.

[Mesa-dev] [Bug 77023] [llvmpipe] SIGSEGV src/gallium/drivers/llvmpipe/lp_texture.c:600

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=77023 Roland Scheidegger changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 52209] lp_test_format regression

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=52209 Roland Scheidegger changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 58326] [llvmpipe] piglit glean depthStencil regression

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=58326 Roland Scheidegger changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [PATCH] i965: Return NONE from brw_swap_cmod on unknown input.

2014-08-11 Thread Matt Turner
Comparing ~0u with a packed enum (i.e., 1 byte) always evaluates to false. Shouldn't gcc warn about this? Reported-by: Connor Abbott --- src/mesa/drivers/dri/i965/brw_eu.c | 2 +- src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp | 2 +- src/mesa/drivers/dri/i965/brw_v

[Mesa-dev] [Bug 65224] piglit arb_uniform_buffer_object-maxuniformblocksize fs regression

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=65224 Roland Scheidegger changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 79039] [TRACKER] Mesa 10.2 release tracker

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=79039 Bug 79039 depends on bug 65224, which changed state. Bug 65224 Summary: piglit arb_uniform_buffer_object-maxuniformblocksize fs regression https://bugs.freedesktop.org/show_bug.cgi?id=65224 What|Removed |Added

Re: [Mesa-dev] [PATCH 4/5] i965: Add support for ARB_copy_image

2014-08-11 Thread Jason Ekstrand
Neil, Thanks for reviewing. Pushed. --Jason On Mon, Aug 11, 2014 at 10:59 AM, Neil Roberts wrote: > Jason Ekstrand writes: > > > + /* If we are on the same miptree, same level, and same slice, then > > +* intel_miptree_map won't let us map it twice. We have to do > things a > > +*

Re: [Mesa-dev] [PATCH 02/20] i965: Add and use foreach_block macro.

2014-08-11 Thread Matt Turner
On Mon, Aug 4, 2014 at 6:08 AM, Pohjolainen, Topi wrote: > Line overflowing here also. Fixed all of these locally. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH 10/20] i965: Add a basic-block aware backend_instruction::remove method.

2014-08-11 Thread Matt Turner
On Tue, Aug 5, 2014 at 10:31 AM, Pohjolainen, Topi wrote: > On Thu, Jul 24, 2014 at 07:54:17PM -0700, Matt Turner wrote: >> --- >> src/mesa/drivers/dri/i965/brw_cfg.h | 1 + >> src/mesa/drivers/dri/i965/brw_shader.cpp | 32 >> >> src/mesa/drivers/dri/i965/b

[Mesa-dev] [Bug 72572] [llvmpipe] no error on draw call without vbo set

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=72572 Roland Scheidegger changed: What|Removed |Added Component|Other |Mesa core --- Comment #1 from Rolan

Re: [Mesa-dev] [PATCH 13/20] i965: Use basic-block aware insertion/removal functions.

2014-08-11 Thread Matt Turner
On Thu, Aug 7, 2014 at 7:55 AM, Pohjolainen, Topi wrote: > On Thu, Jul 24, 2014 at 07:54:20PM -0700, Matt Turner wrote: >> To avoid invalidating and recreating the control flow graph. Also stop >> invalidating the CFG in places we didn't add or remove an instruction. >> >> cfg calculations: 20

[Mesa-dev] [Bug 82471] New: [swrast] piglit fp-condition_codes-01 regression

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=82471 Priority: medium Bug ID: 82471 Keywords: bisected, regression CC: e...@anholt.net, kenn...@whitecape.org Assignee: mesa-dev@lists.freedesktop.org Summary: [swrast] piglit fp-cond

[Mesa-dev] [PATCH 1/5] mesa: update GL headers to Khronos revision 27684

2014-08-11 Thread Ilia Mirkin
This includes GL 4.5, which is necessary to support the new extensions. Signed-off-by: Ilia Mirkin --- include/GL/glext.h | 380 +++- include/GL/glxext.h | 11 +- include/GL/wglext.h | 11 +- 3 files changed, 396 insertions(+), 6 deletions(-) d

[Mesa-dev] [PATCH 2/5] mesa: regenerate mangle file

2014-08-11 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin --- include/GL/gl_mangle.h | 139 + 1 file changed, 139 insertions(+) diff --git a/include/GL/gl_mangle.h b/include/GL/gl_mangle.h index b3a9c88..e04cf66 100644 --- a/include/GL/gl_mangle.h +++ b/include/GL/gl_mangle.h @@

[Mesa-dev] [PATCH 5/5] docs: add GL4.5 section, mark ARB_texture_barrier as done

2014-08-11 Thread Ilia Mirkin
Signed-off-by: Ilia Mirkin --- Not 100% sure about the KHR_robust_buffer_access_behavior -- is that part of GL4.5? docs/GL3.txt | 16 1 file changed, 16 insertions(+) diff --git a/docs/GL3.txt b/docs/GL3.txt index 7f18cd7..f204652 100644 --- a/docs/GL3.txt +++ b/docs/GL3.txt @

[Mesa-dev] [PATCH 3/5] mesa: update mangle script to force locale

2014-08-11 Thread Ilia Mirkin
The sorting is different with LC_ALL=C. To avoid different people's locale settings from fighting, force it to the one that was used to generate the current file. Signed-off-by: Ilia Mirkin --- Not sure if there's a better way of doing it... perhaps forcing to C and just regenerating with that m

[Mesa-dev] [PATCH 4/5] mesa: add ARB_texture_barrier support

2014-08-11 Thread Ilia Mirkin
The extension is identical to NV_texture_barrier except for the function name. Signed-off-by: Ilia Mirkin --- I know these are normally split up a bit more, but here the implementation is *trivial*. src/mapi/glapi/gen/ARB_texture_barrier.xml | 13 + src/mapi/glapi/gen/Makefile.am

Re: [Mesa-dev] [PATCH 11/20] i965: Add basic-block aware backend_instruction::insert_* methods.

2014-08-11 Thread Matt Turner
On Wed, Aug 6, 2014 at 6:06 AM, Pohjolainen, Topi wrote: > On Thu, Jul 24, 2014 at 07:54:18PM -0700, Matt Turner wrote: >> --- >> src/mesa/drivers/dri/i965/brw_shader.cpp | 80 >> >> src/mesa/drivers/dri/i965/brw_shader.h | 5 ++ >> 2 files changed, 85 inserti

[Mesa-dev] [Bug 82472] New: piglit 16385-consecutive-chars regression

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=82472 Priority: medium Bug ID: 82472 Keywords: bisected, regression CC: cwo...@cworth.org, i...@freedesktop.org Assignee: mesa-dev@lists.freedesktop.org Summary: piglit 16385-consecuti

Re: [Mesa-dev] [PATCH 4/5] mesa: add ARB_texture_barrier support

2014-08-11 Thread Ilia Mirkin
On Mon, Aug 11, 2014 at 3:07 PM, Ilia Mirkin wrote: > The extension is identical to NV_texture_barrier except for the function > name. > > Signed-off-by: Ilia Mirkin > --- > > I know these are normally split up a bit more, but here the implementation is > *trivial*. Err I realized that I for

[Mesa-dev] [Bug 82475] New: [llvmpipe] [softpipe] piglit draw-pixels regression

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=82475 Priority: medium Bug ID: 82475 Keywords: bisected, regression CC: bri...@vmware.com, ja...@jlekstrand.net Assignee: mesa-dev@lists.freedesktop.org Summary: [llvmpipe] [softpipe]

[Mesa-dev] [PATCH v2 4/5] mesa: add ARB_texture_barrier support

2014-08-11 Thread Ilia Mirkin
This extension is identical to NV_texture_barrier. Alias glTextureBarrier to the existing glTextureBarrierNV and use the existing NV_texture_barrier extension bit. Signed-off-by: Ilia Mirkin --- v1 -> v2: - Add the actual extension string - Remove separate (and missing dlist bits) TextureBarri

[Mesa-dev] [PATCH 2/2] r600g: set IA_MULTI_VGT_PARAM like radeonsi on Cayman

2014-08-11 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/r600/evergreen_state.c | 2 -- src/gallium/drivers/r600/r600_pipe.h | 2 +- src/gallium/drivers/r600/r600_state_common.c | 21 + src/gallium/drivers/r600/r600d.h | 11 +++ 4 files changed, 33 insertions

[Mesa-dev] [PATCH 1/2] radeonsi: turn off PARTIAL_VS_WAVE and bump PRIMGROUP_SIZE, do the same for SI

2014-08-11 Thread Marek Olšák
From: Marek Olšák These are recommended values. Some additional tweeks will be needed for tessellation. --- src/gallium/drivers/radeonsi/si_state.c | 6 - src/gallium/drivers/radeonsi/si_state_draw.c | 33 2 files changed, 19 insertions(+), 20 deletions(-)

Re: [Mesa-dev] [PATCH 5/5] docs: add GL4.5 section, mark ARB_texture_barrier as done

2014-08-11 Thread Roland Scheidegger
Am 11.08.2014 21:07, schrieb Ilia Mirkin: > Signed-off-by: Ilia Mirkin > --- > > Not 100% sure about the KHR_robust_buffer_access_behavior -- is that part of > GL4.5? Well this is mostly the same as ARB_robust_buffer_access_behavior which is core in 4.3. Except ARB_robust_buffer_access_behavior i

[Mesa-dev] [PATCH 1/4] r600g: clear constant buffer sizes at the beginning of CS

2014-08-11 Thread Marek Olšák
From: Marek Olšák --- src/gallium/drivers/r600/evergreen_state.c | 104 ++--- src/gallium/drivers/r600/evergreend.h | 1 + src/gallium/drivers/r600/r600_state.c | 31 - 3 files changed, 49 insertions(+), 87 deletions(-) diff --git a/src/gallium/drive

[Mesa-dev] [PATCH 4/4] radeonsi: fix buffer invalidation of unbound texture buffer objects

2014-08-11 Thread Marek Olšák
From: Marek Olšák This maintains a list of all TBOs in a pipe_context. --- src/gallium/drivers/radeonsi/si_descriptors.c | 13 - src/gallium/drivers/radeonsi/si_pipe.h| 1 + src/gallium/drivers/radeonsi/si_state.c | 10 -- 3 files changed, 17 insertions(+), 7 d

[Mesa-dev] [PATCH 3/4] r600g: implement invalidation of texture buffer objects

2014-08-11 Thread Marek Olšák
From: Marek Olšák This fixes piglit spec/ARB_texture_buffer_object/data-sync. --- src/gallium/drivers/r600/evergreen_state.c| 9 +-- src/gallium/drivers/r600/r600_pipe.h | 1 + src/gallium/drivers/r600/r600_state_common.c | 39 --- src/gallium/drivers/

[Mesa-dev] [PATCH 2/4] r600g: fix constant buffer fetches

2014-08-11 Thread Marek Olšák
From: Marek Olšák Somebody forgot to do this. It was uncovered by recent st/mesa changes. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=82139 Cc: mesa-sta...@lists.freedesktop.org --- src/gallium/drivers/r600/r600_shader.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium

Re: [Mesa-dev] [PATCH 5/5] docs: add GL4.5 section, mark ARB_texture_barrier as done

2014-08-11 Thread Ilia Mirkin
On Mon, Aug 11, 2014 at 4:34 PM, Roland Scheidegger wrote: > Am 11.08.2014 21:07, schrieb Ilia Mirkin: >> Signed-off-by: Ilia Mirkin >> --- >> >> Not 100% sure about the KHR_robust_buffer_access_behavior -- is that part of >> GL4.5? > Well this is mostly the same as ARB_robust_buffer_access_behav

[Mesa-dev] [Bug 68296] Using old viewport value after a window resize (content is clipped)

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=68296 --- Comment #14 from U. Artie Eoff --- Created attachment 104458 --> https://bugs.freedesktop.org/attachment.cgi?id=104458&action=edit modified weston-simple-egl to trigger the issue too -- You are receiving this mail because: You are the ass

Re: [Mesa-dev] [PATCH] gallium: remove support for predicates from TGSI

2014-08-11 Thread Ilia Mirkin
On Mon, Aug 11, 2014 at 1:31 PM, Marek Olšák wrote: > On Sat, Aug 9, 2014 at 5:24 AM, Connor Abbott wrote: >> On Wed, Aug 6, 2014 at 6:29 PM, Marek Olšák wrote: >>> What IR? A flatland GLSL IR? A replacement for Mesa IR? Something else? >> >> It's a flatland IR, similar to TGSI/Direct3D style wi

[Mesa-dev] [Bug 68296] Using old viewport value after a window resize (content is clipped)

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=68296 U. Artie Eoff changed: What|Removed |Added Assignee|mesa-dev@lists.freedesktop. |wayland-bugs@lists.freedesk

[Mesa-dev] [Bug 68296] Using old viewport value after a window resize (content is clipped)

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=68296 U. Artie Eoff changed: What|Removed |Added Attachment #104458|modified weston-simple-egl |modified weston-simple-egl descri

[Mesa-dev] [Bug 82477] New: [softpipe] piglit fp-long-alu regression

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=82477 Priority: medium Bug ID: 82477 Keywords: bisected, have-backtrace, regression CC: e...@anholt.net, kenn...@whitecape.org Assignee: mesa-dev@lists.freedesktop.org Summary: [softpi

[Mesa-dev] [Bug 82477] [softpipe] piglit fp-long-alu regression

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=82477 --- Comment #1 from Marek Olšák --- The Mesa IR optimizer creates a lot of immediates when it evaluates constant expressions, but doesn't eliminate them. You can easily get hundreds of unused immediates, which is what is happening with fp-long-al

Re: [Mesa-dev] [PATCH 5/5] docs: add GL4.5 section, mark ARB_texture_barrier as done

2014-08-11 Thread Roland Scheidegger
Am 11.08.2014 22:56, schrieb Ilia Mirkin: > On Mon, Aug 11, 2014 at 4:34 PM, Roland Scheidegger > wrote: >> Am 11.08.2014 21:07, schrieb Ilia Mirkin: >>> Signed-off-by: Ilia Mirkin >>> --- >>> >>> Not 100% sure about the KHR_robust_buffer_access_behavior -- is that part of >>> GL4.5? >> Well thi

[Mesa-dev] [Bug 82475] [llvmpipe] [softpipe] piglit draw-pixels regression

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=82475 Jason Ekstrand changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] New stable-branch 10.2 candidate pushed

2014-08-11 Thread Carl Worth
I've just made my pass over the stable queue and pushed out a new candidate branch for the upcoming 10.2.6 release. You can see the 25 patches included (and the 42 still awaiting review) here: http://cworth.org/~cworth/mesa-stable-queue/ As usual, I plan to make the next stable release on

[Mesa-dev] [Bug 82483] New: format_srgb.h:145: undefined reference to `util_format_srgb_to_linear_8unorm_table'

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=82483 Priority: medium Bug ID: 82483 Keywords: regression CC: ja...@jlekstrand.net Assignee: mesa-dev@lists.freedesktop.org Summary: format_srgb.h:145: undefined reference to

[Mesa-dev] [PATCH] glxgears_pixmap: destroy correct pixmap id

2014-08-11 Thread Nathan Kidd
Previously we'd get a GLXBadPixmap error. Signed-off-by: Nathan Kidd --- src/xdemos/glxgears_pixmap.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/xdemos/glxgears_pixmap.c b/src/xdemos/glxgears_pixmap.c index f2bb013..afd5828 100644 --- a/src/xdemos/glxgears_pixmap

Re: [Mesa-dev] [PATCH] glxgears_pixmap: destroy correct pixmap id

2014-08-11 Thread Brian Paul
On 08/11/2014 04:51 PM, Nathan Kidd wrote: Previously we'd get a GLXBadPixmap error. Signed-off-by: Nathan Kidd --- src/xdemos/glxgears_pixmap.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/xdemos/glxgears_pixmap.c b/src/xdemos/glxgears_pixmap.c index f2bb013..a

Re: [Mesa-dev] [PATCH] glxgears_pixmap: destroy correct pixmap id

2014-08-11 Thread Nathan Kidd
On 11/08/14 06:56 PM, Brian Paul wrote: On 08/11/2014 04:51 PM, Nathan Kidd wrote: Previously we'd get a GLXBadPixmap error. Signed-off-by: Nathan Kidd --- src/xdemos/glxgears_pixmap.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/xdemos/glxgears_pixmap.c b/src/

Re: [Mesa-dev] [PATCH 5/5] docs: add GL4.5 section, mark ARB_texture_barrier as done

2014-08-11 Thread Fredrik Höglund
On Monday 11 August 2014, Ilia Mirkin wrote: > Signed-off-by: Ilia Mirkin > --- > > Not 100% sure about the KHR_robust_buffer_access_behavior -- is that part of > GL4.5? > > docs/GL3.txt | 16 > 1 file changed, 16 insertions(+) > > diff --git a/docs/GL3.txt b/docs/GL3.txt > in

Re: [Mesa-dev] [PATCH] gallium: remove support for predicates from TGSI

2014-08-11 Thread Eric Anholt
Connor Abbott writes: > On Wed, Aug 6, 2014 at 6:29 PM, Marek Olšák wrote: >> What IR? A flatland GLSL IR? A replacement for Mesa IR? Something else? > > It's a flatland IR, similar to TGSI/Direct3D style with enough GLSL > IR-like stuff to get existing things working now and enable us to > even

[Mesa-dev] [PATCH 00/12] i965: Fast clear using meta

2014-08-11 Thread Kristian Høgsberg
This patch series implements the fast clear feature using meta operations. This series consists of a series of patches that add bits and pieces of infrastructure to control the custom state and the final patch uses all those to implement color fast clear and resolve. The big ugly problem with doin

[Mesa-dev] [PATCH 02/12] i965: Add an option to not generate the SIMD8 fragment shader

2014-08-11 Thread Kristian Høgsberg
For now, this can only be triggered with a new 'no8' INTEL_DEBUG option and a new context flag. We'll use the context flag later, but introducing it now lets us bisect to this commit if it breaks something. Signed-off-by: Kristian Høgsberg --- src/mesa/drivers/dri/i965/brw_context.h | 2 ++

[Mesa-dev] [PATCH 04/12] i965: Add a mechanism for sending native primitives into the driver

2014-08-11 Thread Kristian Høgsberg
The brw_draw_prims() function is the draw entry point into the driver, and takes struct _mesa_prim for input. We want to be able to feed native primitives into the driver, and to that end we introduce BRW_PRIM_OFFSET, which lets use describe geometry using the native GEN primitive types. Signed-o

[Mesa-dev] [PATCH 01/12] i965: Assign PS kernel start pointers when we decide which kernels to use

2014-08-11 Thread Kristian Høgsberg
Right now we decide which kernels to use and the GRF start offsets in one place and emit the kernel pointers later. The logic of how to map 8, 16 and 32 kernels to kernel start pointers follows the same logic as which GRF start offsets to use, so lets figure out these two things in one place. Sig

[Mesa-dev] [PATCH 06/12] i965: Add optimization pass to let us use the replicate data message

2014-08-11 Thread Kristian Høgsberg
The data port has a SIMD16 'replicate data' message, which lets us write the same color for all 16 pixels by sending the four floats in the lower half of a register instead of sending 4 times 16 identical component values in 8 registers. The message comes with a lot of restrictions and could be ma

[Mesa-dev] [PATCH 08/12] i965: Rename intelValidateState to intel_update_state

2014-08-11 Thread Kristian Høgsberg
This matches the name of the dd hook. Also convert a couple of nearby dd implementations to lowercase + underscore as is now the standard. Signed-off-by: Kristian Høgsberg --- src/mesa/drivers/dri/i965/brw_context.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git

[Mesa-dev] [PATCH 09/12] i965: Move pre-draw resolve buffers to dd::UpdateState

2014-08-11 Thread Kristian Høgsberg
No functional change except for glBegin/glEnd style rendering, where we now do the resolves at glBegin time instead of FLUSH_VERTICES time. This is also the reason for this change, so that when we later switch fast clear resolve to use meta, we won't be doing meta operations in the middle of a beg

[Mesa-dev] [PATCH 03/12] i965: Add context flag to disable the viewport transform

2014-08-11 Thread Kristian Høgsberg
This lets us disable the viewport transform, which will be useful for emitting 3DPRIM_RECTLIST. Signed-off-by: Kristian Høgsberg --- src/mesa/drivers/dri/i965/brw_context.c | 1 + src/mesa/drivers/dri/i965/brw_context.h | 1 + src/mesa/drivers/dri/i965/gen6_sf_state.c | 5 +++-- src/mesa/dri

[Mesa-dev] [PATCH 11/12] meta: Export _mesa_meta_drawbuffers_from_bitfield()

2014-08-11 Thread Kristian Høgsberg
We'll use this in the i965 fast clear implementation. Signed-off-by: Kristian Høgsberg --- src/mesa/drivers/common/meta.c | 6 +++--- src/mesa/drivers/common/meta.h | 3 +++ 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/me

[Mesa-dev] [PATCH 10/12] mesa: Use _mesa_lock_context_textures in _mesa_GetTexParameterfv()

2014-08-11 Thread Kristian Høgsberg
GetTexParamterfv() doesnt change texture state, so instead of _mesa_lock_texture() we can use _mesa_lock_context_textures(), which doesn't increase the texture stamp. With this change, _mesa_update_state_locked() is now only called from under _mesa_lock_context_textures(), which is right thing to

[Mesa-dev] [PATCH 12/12] i965: Implement fast color clears using meta operations

2014-08-11 Thread Kristian Høgsberg
This patch uses the infrastructure put in place by previous patches to implement fast color clears and replicated color clears in terms of meta operations. This works all the way back to gen7 where fast clear was introduced and adds support for fast clear on gen8. It replaces the blorp path compl

[Mesa-dev] [PATCH 07/12] i965: Provide a context flag to let us enable fast clear

2014-08-11 Thread Kristian Høgsberg
GEN7+ has the fast clear functionality, which lets us clear the color buffers using the MCS and a scaled down rectangle. To enable this we have to set the appropriate bits in the 3DSTATE_PS package. Signed-off-by: Kristian Høgsberg --- src/mesa/drivers/dri/i965/brw_context.h | 1 + src/mesa/d

[Mesa-dev] [PATCH 05/12] i965: Disable clipping when rendering 3DPRIM_RECTLIST primitives

2014-08-11 Thread Kristian Høgsberg
The clipper doesn't support clipping 3DPRIM_RECTLIST primitives and must be turned off when we use them. Signed-off-by: Kristian Høgsberg --- src/mesa/drivers/dri/i965/gen6_clip_state.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/gen6_clip

Re: [Mesa-dev] [PATCH 12/12] i965: Implement fast color clears using meta operations

2014-08-11 Thread Ilia Mirkin
On Mon, Aug 11, 2014 at 8:29 PM, Kristian Høgsberg wrote: > diff --git a/src/mesa/drivers/dri/i965/intel_tex_copy.c > b/src/mesa/drivers/dri/i965/intel_tex_copy.c > index 97f1569..2456080 100644 > --- a/src/mesa/drivers/dri/i965/intel_tex_copy.c > +++ b/src/mesa/drivers/dri/i965/intel_tex_copy.c

[Mesa-dev] [PATCH 10/20] i965: Add a basic-block aware backend_instruction::remove method.

2014-08-11 Thread Matt Turner
--- src/mesa/drivers/dri/i965/brw_shader.cpp | 46 src/mesa/drivers/dri/i965/brw_shader.h | 4 +++ 2 files changed, 50 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp b/src/mesa/drivers/dri/i965/brw_shader.cpp index 0deb090..53545de 100644

[Mesa-dev] [PATCH 11/20] i965: Add basic-block aware backend_instruction::insert_* methods.

2014-08-11 Thread Matt Turner
--- src/mesa/drivers/dri/i965/brw_shader.cpp | 47 src/mesa/drivers/dri/i965/brw_shader.h | 5 2 files changed, 52 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_shader.cpp b/src/mesa/drivers/dri/i965/brw_shader.cpp index 53545de..3de7ad9 100644

[Mesa-dev] [Bug 81680] [r600g] Firefox crashes with hardware acceleration turned on

2014-08-11 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=81680 Michel Dänzer changed: What|Removed |Added Assignee|dri-devel@lists.freedesktop |mesa-dev@lists.freedesktop.

Re: [Mesa-dev] [PATCH] gallium: remove support for predicates from TGSI

2014-08-11 Thread Connor Abbott
On Mon, Aug 11, 2014 at 4:32 PM, Eric Anholt wrote: > Connor Abbott writes: > >> On Wed, Aug 6, 2014 at 6:29 PM, Marek Olšák wrote: >>> What IR? A flatland GLSL IR? A replacement for Mesa IR? Something else? >> >> It's a flatland IR, similar to TGSI/Direct3D style with enough GLSL >> IR-like stu

Re: [Mesa-dev] [PATCH] i965: Enable EWA anisotropic filtering algorithm for isotropic case

2014-08-11 Thread Popov, Pavel E
I agree. I just sent this patch to focus attention on this issue (Bug 82463). - Pavel -Original Message- From: Kenneth Graunke [mailto:kenn...@whitecape.org] Sent: Monday, August 11, 2014 11:32 PM To: mesa-dev@lists.freedesktop.org Cc: Popov, Pavel E Subject: Re: [Mesa-dev] [PATCH] i965:

Re: [Mesa-dev] [PATCH 1/2] i965: Bail on vec4 copy propagation for scratch writes with source modifiers

2014-08-11 Thread Anuj Phogat
On Tue, Jul 29, 2014 at 3:08 PM, Anuj Phogat wrote: > On Tue, Jul 29, 2014 at 7:18 AM, Matt Turner wrote: > > > > On Mon, Jul 28, 2014 at 8:47 PM, Anuj Phogat > wrote: > > > Fixes Khronos GLES3 CTS test: > > > dynamic_expression_array_access_vertex > > > > > > Cc: > > > Signed-off-by: Anuj Pho

Re: [Mesa-dev] [PATCH 01/12] i965: Assign PS kernel start pointers when we decide which kernels to use

2014-08-11 Thread Ben Widawsky
On Mon, Aug 11, 2014 at 05:29:31PM -0700, Kristian Høgsberg wrote: > Right now we decide which kernels to use and the GRF start offsets in > one place and emit the kernel pointers later. The logic of how to map > 8, 16 and 32 kernels to kernel start pointers follows the same logic as which > GRF s

Re: [Mesa-dev] [PATCH 02/12] i965: Add an option to not generate the SIMD8 fragment shader

2014-08-11 Thread Ben Widawsky
On Mon, Aug 11, 2014 at 05:29:32PM -0700, Kristian Høgsberg wrote: > For now, this can only be triggered with a new 'no8' INTEL_DEBUG option > and a new context flag. We'll use the context flag later, but introducing > it now lets us bisect to this commit if it breaks something. > > Signed-off-by

Re: [Mesa-dev] [PATCH 04/12] i965: Add a mechanism for sending native primitives into the driver

2014-08-11 Thread Ben Widawsky
On Mon, Aug 11, 2014 at 05:29:34PM -0700, Kristian Høgsberg wrote: > The brw_draw_prims() function is the draw entry point into the driver, > and takes struct _mesa_prim for input. We want to be able to feed > native primitives into the driver, and to that end we introduce > BRW_PRIM_OFFSET, which

Re: [Mesa-dev] [PATCH 1/2] i965: Bail on vec4 copy propagation for scratch writes with source modifiers

2014-08-11 Thread Matt Turner
On Mon, Aug 11, 2014 at 7:39 PM, Anuj Phogat wrote: > Matt, should I consider both of these r-b you? Yes. ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

  1   2   >