Re: [Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

On Thu, Dec 13, 2012 at 10:59 AM, Stéphane Marchesin wrote: > On Wed, Dec 12, 2012 at 3:49 PM, Roland Scheidegger > wrote: >> This is interesting, personally I'm fine with getting this merged. >> That said, there was a i965g driver upstream before (though it never >> worked right IIRC and was pr

Re: [Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

Hi Marek, On Thu, Dec 13, 2012 at 8:58 AM, Marek Olšák wrote: > If you want a piece of advice... it would be great to extend gallium > to allow the GLSL IR to be passed to drivers directly. Once that's > done, I think you can copy their shader codegen as-is with all > optimizations or even share

Re: [Mesa-dev] llvmpipe and transform feedback of position

On Thu, Dec 13, 2012 at 2:48 PM, Dave Airlie wrote: >> Hi, >> >> So I've been playing around a bit more with transform feedback with >> llvmpipe, and it seems the position output from the vertex shader is >> different from what softpipe produces. >> >> using from piglit ./bin/ext_transform_feedbac

Re: [Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

On Thu, Dec 13, 2012 at 7:49 AM, Roland Scheidegger wrote: > This is interesting, personally I'm fine with getting this merged. > That said, there was a i965g driver upstream before (though it never > worked right IIRC and was probably gen4 or maybe gen5 too only due to > its age), the problem is

Re: [Mesa-dev] llvmpipe and transform feedback of position

> Hi, > > So I've been playing around a bit more with transform feedback with > llvmpipe, and it seems the position output from the vertex shader is > different from what softpipe produces. > > using from piglit ./bin/ext_transform_feedback-builtin-varyings gl_Position > > the position with softpip

[Mesa-dev] llvmpipe and transform feedback of position

Hi, So I've been playing around a bit more with transform feedback with llvmpipe, and it seems the position output from the vertex shader is different from what softpipe produces. using from piglit ./bin/ext_transform_feedback-builtin-varyings gl_Position the position with softpipe for the first

[Mesa-dev] [PATCH] r600: silent warnings

GCC complains about mixed code and declarations and unhandled enum cases when compiling r600. This patch correct it : >From 25b5d12eec732ed582a8a3e146c0285819806ba8 Mon Sep 17 00:00:00 2001 From: Maxence Le Dore Date: Thu, 13 Dec 2012 05:17:35 +0100 Subject: [PATCH] r600: silent warnings --- sr

[Mesa-dev] [PATCH] draw: add support for later transform feedback extensions

This adds support to draw for the new features of transform feedback. a) fix count_from_stream_output, using max_index+1 for now but it looks like it should be valid as its derived from the vertex elements/vbo. b) fix striding and dst offsets in output buffers - was just wrong before. c) fix cra

Re: [Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

On Wed, Dec 12, 2012 at 3:49 PM, Roland Scheidegger wrote: > This is interesting, personally I'm fine with getting this merged. > That said, there was a i965g driver upstream before (though it never > worked right IIRC and was probably gen4 or maybe gen5 too only due to > its age), the problem is

Re: [Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

If you want a piece of advice... it would be great to extend gallium to allow the GLSL IR to be passed to drivers directly. Once that's done, I think you can copy their shader codegen as-is with all optimizations or even share it. Also, I don't think Intel will move to Gallium if they can't use the

Re: [Mesa-dev] [PATCH 08/10] glsl: Add a lowering pass for packing varyings.

On 12 December 2012 14:49, Eric Anholt wrote: > Paul Berry writes: > > diff --git a/src/glsl/lower_packed_varyings.cpp > b/src/glsl/lower_packed_varyings.cpp > > new file mode 100644 > > index 000..4cb9066 > > --- /dev/null > > +++ b/src/glsl/lower_packed_varyings.cpp > > @@ -0,0 +1,371 @@ >

Re: [Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

This is interesting, personally I'm fine with getting this merged. That said, there was a i965g driver upstream before (though it never worked right IIRC and was probably gen4 or maybe gen5 too only due to its age), the problem is unless you manage to get the intel guys interested you'll always end

[Mesa-dev] [PATCH 6/6] mesa: Fix framebuffer blitting to GL_COLOR_ATTACHMENTi when i!=0

This patch fixes a case when blitting to a framebuffer with renderbuffers/textures attached to GL_COLOR_ATTACHMENT{1, 2, ...}. Earlier we were incorrectly blitting to GL_COLOR_ATTACHMENT0 by default. It also fixes a blitting case when drawAttachment->Texture == readAttachment->Texture. This was ca

[Mesa-dev] [PATCH 5/6] mesa: Fix GL error generation in _mesa_GetFramebufferAttachmentParameteriv()

This allows query on default framebuffer in glGetFramebufferAttachmentParameteriv() for gles3. Fixes unexpected GL errors in gles3 conformance test case: framebuffer_blit_functionality_multisampled_to_singlesampled_blit Signed-off-by: Anuj Phogat --- src/mesa/main/fbobject.c |4 +++- 1 file

[Mesa-dev] [PATCH 4/6] intel: Fix framebuffer blitting to GL_COLOR_ATTACHMENTi when i!=0

This patch fixes a case when blitting to a framebuffer with renderbuffers/textures attached to GL_COLOR_ATTACHMENT{1, 2, ...}. Earlier we were incorrectly blitting to GL_COLOR_ATTACHMENT0 by default. Signed-off-by: Anuj Phogat --- src/mesa/drivers/dri/i965/brw_blorp_blit.cpp |5 +- src/mesa/

[Mesa-dev] [PATCH 3/6] meta: Add functionality to do _mesa_meta_BlitFrameBuffer() using glsl

This is required by glBlitFrameBuffer() in gles3. This patch, along with other patches in this series, make failing framebuffer_blit test cases in gles3 conformancen pass. Signed-off-by: Anuj Phogat --- src/mesa/drivers/common/meta.c | 313 +++- 1 files chang

[Mesa-dev] [PATCH 2/6] mesa: Fix GL error generation in glBlitFramebuffer()

Signed-off-by: Anuj Phogat --- src/mesa/main/fbobject.c | 10 +- 1 files changed, 9 insertions(+), 1 deletions(-) diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c index 4b7d4ab..9e32836 100644 --- a/src/mesa/main/fbobject.c +++ b/src/mesa/main/fbobject.c @@ -2856,7 +285

[Mesa-dev] [PATCH 1/6] mesa: Allow GL_BACK in glDrawBuffers() for gles3

Makes framebuffer_blit_coverage_default_draw_buffer_binding.test in gles3 conformance pass. Signed-off-by: Anuj Phogat --- src/mesa/main/buffers.c | 11 ++- 1 files changed, 10 insertions(+), 1 deletions(-) diff --git a/src/mesa/main/buffers.c b/src/mesa/main/buffers.c index c2a02c0..

[Mesa-dev] [PATCH 0/6] gles3: Fix glBlitFramebuffer()

This series is based on gles3 branch of mesa. All patches collectively fixes around 17 failing test cases in gles3 conformance framebuffer_blit and framebuffer_srgb_default_encoding.test. Observed no regressions in gles3 conformance. Anuj Phogat (6): mesa: Allow GL_BACK in glDrawBuffers() for gl

Re: [Mesa-dev] [PATCH 08/10] glsl: Add a lowering pass for packing varyings.

Paul Berry writes: > diff --git a/src/glsl/lower_packed_varyings.cpp > b/src/glsl/lower_packed_varyings.cpp > new file mode 100644 > index 000..4cb9066 > --- /dev/null > +++ b/src/glsl/lower_packed_varyings.cpp > @@ -0,0 +1,371 @@ > +/* > + * Copyright © 2011 Intel Corporation

[Mesa-dev] new i965g pipe driver for Intel GEN6 (and later)

Hi list, I've been working on i965g, a new pipe driver for Intel GEN6 (and later), for a while now. I would like to know if there is any interest in it and if it can be merged upstream. The code is currently available here https://github.com/olvaffe/mesa/tree/i965g The project was started fo

Re: [Mesa-dev] [PATCH 10/10] glsl/linker: Pack between varyings.

On 12 December 2012 14:06, Eric Anholt wrote: > Paul Berry writes: > > > This patch implements varying packing between varyings. > > > > Previously, each varying occupied components 0 through N-1 of its > > assigned varying slot, so there was no way to pack two varyings into > > the same slot.

Re: [Mesa-dev] [PATCH 09/10] glsl/linker: Pack within compound varyings.

On 12 December 2012 14:08, Eric Anholt wrote: > Paul Berry writes: > > > This patch implements varying packing within varyings that are > > composed of multiple vectors of size less than 4 (e.g. arrays of > > vec2's, or matrices with height less than 4). > > > > Previously, such varyings used up

Re: [Mesa-dev] [PATCH 09/10] glsl/linker: Pack within compound varyings.

Paul Berry writes: > This patch implements varying packing within varyings that are > composed of multiple vectors of size less than 4 (e.g. arrays of > vec2's, or matrices with height less than 4). > > Previously, such varyings used up a full 4-wide varying slot for each > constituent vector, me

Re: [Mesa-dev] [PATCH 10/10] glsl/linker: Pack between varyings.

Paul Berry writes: > This patch implements varying packing between varyings. > > Previously, each varying occupied components 0 through N-1 of its > assigned varying slot, so there was no way to pack two varyings into > the same slot. For example, if the varyings were a float, a vec2, a > vec3,

Re: [Mesa-dev] [PATCH] gallivm: fix texel fetch for array textures

I was wrong that there is no test for this in piglit. spec/glsl-1.30/execution/fs-texelFetch-2D.c (and the offset version) only test 2d samplers, but texturing/shaders/texelFetch.c can test all samplers, in both fs and vs. It currently crashes though with llvmpipe (using glsl and GL version overrid

Re: [Mesa-dev] [PATCH 05/10] glsl/linker: Defer recording transform feedback locations.

On 12 December 2012 13:32, Eric Anholt wrote: > Paul Berry writes: > > > This patch subdivides the loop that assigns varying locations into two > > phases: one phase to match up varyings between shader stages (and > > assign them varying locations), and a second phase to record the > > varying a

Re: [Mesa-dev] [PATCH 05/10] glsl/linker: Defer recording transform feedback locations.

Paul Berry writes: > This patch subdivides the loop that assigns varying locations into two > phases: one phase to match up varyings between shader stages (and > assign them varying locations), and a second phase to record the > varying assignments for use by transform feedback. > -bool > -tfee

Re: [Mesa-dev] [PATCH 00/10] glsl: Implement varying packing.

On 12 December 2012 12:53, Marek Olšák wrote: > On Wed, Dec 12, 2012 at 9:21 PM, Eric Anholt wrote: > > Marek Olšák writes: > > > >> On Wed, Dec 12, 2012 at 5:06 PM, Paul Berry > wrote: > >>> On 11 December 2012 23:49, Aras Pranckevicius > wrote: > Not sure if relevant for Mesa, but e.g.

Re: [Mesa-dev] [PATCH 00/10] glsl: Implement varying packing.

On 12 December 2012 12:44, Aras Pranckevicius wrote: > > Not sure if relevant for Mesa, but e.g. on PowerVR SGX it's really bad to >>> pack two vec2 texture coordinates into a single vec4. That's because var.xy >>> texture read can be "prefetched", whereas var.zw texture read is not >>> prefetche

[Mesa-dev] [Bug 57899] Compilation fails (libgl): No rule to make target `../program/libdricore_program.la', needed by `libdricore9.1.0.la'

https://bugs.freedesktop.org/show_bug.cgi?id=57899 pejakm changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

Re: [Mesa-dev] [PATCH 00/10] glsl: Implement varying packing.

On Wed, Dec 12, 2012 at 9:21 PM, Eric Anholt wrote: > Marek Olšák writes: > >> On Wed, Dec 12, 2012 at 5:06 PM, Paul Berry wrote: >>> On 11 December 2012 23:49, Aras Pranckevicius wrote: Not sure if relevant for Mesa, but e.g. on PowerVR SGX it's really bad to pack two vec2 texture co

Re: [Mesa-dev] [PATCH] configure.ac: Disable compiler optimizations when --enable-debug is set

On 12/12/2012 09:52 AM, Dan Nicholson wrote: > On Dec 12, 2012 8:30 AM, "Emil Velikov" > wrote: >> >> Signed-off-by: Emil Velikov > > > Looks good. > > Reviewed-by: Dan Nicholson mailto:dbn.li...@gmail.com>> Thanks. Patch is com

Re: [Mesa-dev] [PATCH] gallivm: fix texel fetch for array textures

Looks good AFAICT. Maybe we should add a test for this in piglit? Jose - Original Message - > From: Roland Scheidegger > > Since we don't call lp_build_sample_common() in the texel fetch path > we missed > the layer fixup code. If someone would have tried to do texelFetch > with array

Re: [Mesa-dev] [PATCH 00/10] glsl: Implement varying packing.

> Not sure if relevant for Mesa, but e.g. on PowerVR SGX it's really bad to >> pack two vec2 texture coordinates into a single vec4. That's because var.xy >> texture read can be "prefetched", whereas var.zw texture read is not >> prefetched (essentially treated as a dependent texture read), and oft

[Mesa-dev] [Bug 58210] New: vmwgfx freezes system when starting Xorg

https://bugs.freedesktop.org/show_bug.cgi?id=58210 Priority: medium Bug ID: 58210 Assignee: mesa-dev@lists.freedesktop.org Summary: vmwgfx freezes system when starting Xorg Severity: major Classification: Unclassified OS: Lin

Re: [Mesa-dev] [PATCH 00/10] glsl: Implement varying packing.

Marek Olšák writes: > On Wed, Dec 12, 2012 at 5:06 PM, Paul Berry wrote: >> On 11 December 2012 23:49, Aras Pranckevicius wrote: >>> Not sure if relevant for Mesa, but e.g. on PowerVR SGX it's really bad to >>> pack two vec2 texture coordinates into a single vec4. That's because var.xy >>> text

Re: [Mesa-dev] [PATCH 00/10] glsl: Implement varying packing.

On Wed, Dec 12, 2012 at 7:51 PM, Paul Berry wrote: > On 12 December 2012 10:21, Paul Berry wrote: >> >> On 12 December 2012 09:09, Marek Olšák wrote: >>> >>> R300 and R400 support 4 texture indirections (as defined by >>> ARB_fragment_program). Adding ALU instructions before the first TEX >>> in

[Mesa-dev] [PATCH] llvmpipe: Fix creation of shared and scanout textures.

NOTE: This is a candidate for the stable branches. --- src/gallium/drivers/llvmpipe/lp_texture.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/llvmpipe/lp_texture.c b/src/gallium/drivers/llvmpipe/lp_texture.c index 2e9c6bf..f17a04a 100644 --- a/src/gal

Re: [Mesa-dev] [PATCH] i965: Fix disassembly of jump targets on Gen7.

Kenneth Graunke writes: > Gen7 stores the JIP/UIP bits in different places. Reviewed-by: Eric Anholt pgpnqrtiFdjEP.pgp Description: PGP signature ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/

Re: [Mesa-dev] [PATCH 00/10] glsl: Implement varying packing.

On 12 December 2012 10:21, Paul Berry wrote: > On 12 December 2012 09:09, Marek Olšák wrote: > >> R300 and R400 support 4 texture indirections (as defined by >> ARB_fragment_program). Adding ALU instructions before the first TEX >> instruction increases the number of texture indirections by 1, w

Re: [Mesa-dev] [PATCH 00/10] glsl: Implement varying packing.

On 12 December 2012 09:09, Marek Olšák wrote: > R300 and R400 support 4 texture indirections (as defined by > ARB_fragment_program). Adding ALU instructions before the first TEX > instruction increases the number of texture indirections by 1, which > might make some shaders not be executable on t

Re: [Mesa-dev] [PATCH 00/10] glsl: Implement varying packing.

On 12 December 2012 09:51, Eric Anholt wrote: > Paul Berry writes: > > > On 12 December 2012 00:02, Eric Anholt wrote: > > > >> Paul Berry writes: > >> > >> > This patch series adds varying packing to Mesa, so that we can handle > >> > varyings composed of things other than vec4's without usin

Re: [Mesa-dev] [PATCH 00/10] glsl: Implement varying packing.

Paul Berry writes: > On 12 December 2012 00:02, Eric Anholt wrote: > >> Paul Berry writes: >> >> > This patch series adds varying packing to Mesa, so that we can handle >> > varyings composed of things other than vec4's without using up extra >> > varying components. >> >> Results for glbenchma

Re: [Mesa-dev] [PATCH 00/10] glsl: Implement varying packing.

On Wed, Dec 12, 2012 at 5:06 PM, Paul Berry wrote: > On 11 December 2012 23:49, Aras Pranckevicius wrote: >> >> >>> For the initial implementation I've chosen a strategy that operates >>> exclusively at the GLSL IR level, so that it doesn't require the >>> cooperation of the driver back-ends. >>

Re: [Mesa-dev] [PATCH 00/10] glsl: Implement varying packing.

On 12 December 2012 00:02, Eric Anholt wrote: > Paul Berry writes: > > > This patch series adds varying packing to Mesa, so that we can handle > > varyings composed of things other than vec4's without using up extra > > varying components. > > Results for glbenchmark 2.7 at 320x240 (units of fps

Re: [Mesa-dev] [PATCH 00/10] glsl: Implement varying packing.

On 11 December 2012 23:49, Aras Pranckevicius wrote: > > For the initial implementation I've chosen a strategy that operates >> exclusively at the GLSL IR level, so that it doesn't require the >> cooperation of the driver back-ends. > > > Wouldn't this negatively affect performance of some GPUs?

Re: [Mesa-dev] [PATCH] configure.ac: Disable compiler optimizations when --enable-debug is set

On Dec 12, 2012 8:30 AM, "Emil Velikov" wrote: > > Signed-off-by: Emil Velikov Looks good. Reviewed-by: Dan Nicholson ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Re: [Mesa-dev] [PATCH] llvmpipe: remove unneeded draw_flush() call

Sounds good to me. Jose - Original Message - > This is redundant since we're calling draw_bind_fragment_shader() > which already does a flush. > > v2: the redundant flush in llvmpipe_set_constant_buffer() has > already been removed by commit > 3427466e6dbbb8db7c1ecda6b3859ca1cc5827a3 > -

[Mesa-dev] [PATCH] configure.ac: Disable compiler optimizations when --enable-debug is set

Signed-off-by: Emil Velikov --- configure.ac | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/configure.ac b/configure.ac index a133902..1294740 100644 --- a/configure.ac +++ b/configure.ac @@ -341,18 +341,18 @@ AC_ARG_ENABLE([debug], if test "x$enable_debug" = xyes; t

Re: [Mesa-dev] [PATCH 00/10] glsl: Implement varying packing.

On 12 December 2012 00:02, Eric Anholt wrote: > Paul Berry writes: > > > This patch series adds varying packing to Mesa, so that we can handle > > varyings composed of things other than vec4's without using up extra > > varying components. > > Results for glbenchmark 2.7 at 320x240 (units of fps

Re: [Mesa-dev] [PATCH] mesa: Fix computation of default vertex attrib stride for 2_10_10_10 formats.

On 12/11/2012 09:25 PM, Paul Berry wrote: Previously, if the client program didn't specify a stride when setting up a vertex attribute, we used _mesa_sizeof_type() to compute the size of the type, and multiplied it by the number of components. This didn't work for the 2_10_10_10 formats, since _

[Mesa-dev] [Bug 57899] Compilation fails (libgl): No rule to make target `../program/libdricore_program.la', needed by `libdricore9.1.0.la'

https://bugs.freedesktop.org/show_bug.cgi?id=57899 --- Comment #1 from pejakm --- Sorry for bumping up, I still get this error. Anyone care to take a look? -- You are receiving this mail because: You are the assignee for the bug. ___ mesa-dev mailing

Re: [Mesa-dev] [PATCH 1/3] build: Require X11 pkg-config files

On Tue, 14 Aug 2012 12:59:42 -0700 Matt Turner wrote: > Cc: Jeremy Huddleston > --- > Jeremy, does OS X ship the pkg-config files? Haiku doesn't... Mesa should have a path to properly handle a system not having X11 installed. ( Mesa *is* the only open source GL project in town... lets keep a le

[Mesa-dev] [PATCH] i965: Fix disassembly of jump targets on Gen7.

Gen7 stores the JIP/UIP bits in different places. --- src/mesa/drivers/dri/i965/brw_disasm.c | 13 + 1 file changed, 9 insertions(+), 4 deletions(-) Otherwise you get awful junk like "endif 8207". Clearly rubbish. diff --git a/src/mesa/drivers/dri/i965/brw_disasm.c b/src/mesa/drive

Re: [Mesa-dev] [PATCH 00/10] glsl: Implement varying packing.

Paul Berry writes: > This patch series adds varying packing to Mesa, so that we can handle > varyings composed of things other than vec4's without using up extra > varying components. Results for glbenchmark 2.7 at 320x240 (units of fps): N Min MaxMedian