Reviewed-by: Jordan Justen
On Mon, Mar 18, 2013 at 6:57 PM, Kenneth Graunke wrote:
> This series improves our basic GLSL 1.50 support. First, it makes 1.50
> shaders using built-in functions work. Secondly, it adds the new types
> and built-ins from the ARB_texture_multisample extension (which
From: Dave Airlie
This is for a specific use-case for the QXL driver (which wraps
llvmpipe/softpipe). Texture it get via TFP are inverted, so
we need to use a negative stride to sample from them. So
this converts a bunch of cases where we use unsigned stride
to a signed value. I can't see the sin
From: Dave Airlie
We have cases on virtual GPU hw that renders using a wrapped llvmpipe,
that we can't produce the normal DRI2 semantics having the kernel/hw
enforce synchronisation semantics. It is also suboptimal to flush
all surfaces on the server side in the block handler. So there is
already
So we have this virtual GPU with nothing approaching a 3D engine,
so we are currently running llvmpipe with drisw on it. However
this incurs some overheads that now that we have a kernel driver,
I believe we can remove.
The main overheads are putimage for all rendering from
a 3D compositor and get
Kenneth Graunke writes:
> On 03/18/2013 03:56 PM, Eric Anholt wrote:
>> On an INTEL_DEBUG=perf piglit run on IVB, reduces the instances of "HW
>> workaround: blit" (the printouts from the misaligned-depth workaround
>> blits) from 725 to 684
>>
>> It doesn't totally eliminate the workaround blit,
On Mon, Mar 18, 2013 at 01:45:26PM -0700, Ian Romanick wrote:
> On 03/13/2013 10:11 AM, Tom Stellard wrote:
> > Hi,
> >
> > It's time again for Google Summer of Code, so we need to start updating
> > the X.Org ideas page (http://www.x.org/wiki/SummerOfCodeIdeas) with new
> > ideas. Since there hav
This makes basic built-in functions work in GLSL 1.50. It supports
everything except the new Geometry Shader functions.
The new 150.glsl file is 140.glsl plus ARB_texture_multisample.glsl;
150.frag is identical to 140.frag except for the #version bump.
Signed-off-by: Kenneth Graunke
---
src/gl
GLSL 1.50 includes support for the new sampler types introduced by
the ARB_texture_multisample extension.
Signed-off-by: Kenneth Graunke
---
src/glsl/glsl_types.cpp | 12 +++-
src/glsl/glsl_types.h | 1 +
2 files changed, 12 insertions(+), 1 deletion(-)
diff --git a/src/glsl/glsl_typ
The version bumps are necessary in order to compile built-ins for 1.50.
Signed-off-by: Kenneth Graunke
---
src/glsl/builtins/tools/generate_builtins.py | 4 ++--
src/glsl/main.cpp| 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/glsl/builtins/
This series improves our basic GLSL 1.50 support. First, it makes 1.50
shaders using built-in functions work. Secondly, it adds the new types
and built-ins from the ARB_texture_multisample extension (which are now
part of core).
It also makes it possible to run Unigine Valley and Heaven 4.0 with
On Fri, Mar 01, 2013 at 02:06:03PM +0200, Ander Conselvan de Oliveira wrote:
> On 02/28/2013 03:52 AM, Kristian Høgsberg wrote:
> >diff --git a/src/egl/drivers/dri2/platform_wayland.c
> >b/src/egl/drivers/dri2/platform_wayland.c
> >index b5cd04a..1b42a98 100644
> >--- a/src/egl/drivers/dri2/platfo
On Fri, Mar 01, 2013 at 02:03:20PM +0200, Ander Conselvan de Oliveira wrote:
> On 02/28/2013 03:52 AM, Kristian Høgsberg wrote:
> >---
> > include/GL/internal/dri_interface.h| 14 +++-
> > src/mesa/drivers/dri/intel/intel_regions.c | 33 +++
> > src/mesa/drivers/dri/int
On 03/18/2013 03:56 PM, Eric Anholt wrote:
On an INTEL_DEBUG=perf piglit run on IVB, reduces the instances of "HW
workaround: blit" (the printouts from the misaligned-depth workaround
blits) from 725 to 684
It doesn't totally eliminate the workaround blit, because we still have
problems with Y o
Verify that interface blocks match when linking separate shader
stages into a program.
Fixes piglit glsl-1.50 tests:
* linker/interface-blocks-vs-fs-member-count-mismatch.shader_test
* linker/interface-blocks-vs-fs-member-order-mismatch.shader_test
Signed-off-by: Jordan Justen
---
src/glsl/inte
Verify that interface blocks match when combining compilation
units at the same stage. (For example, when merging all vertex
shaders.)
Fixes piglit glsl-1.50 test:
* linker/interface-blocks-multiple-vs-member-count-mismatch.shader_test
Signed-off-by: Jordan Justen
---
src/glsl/Makefile.sources
With this change we now support interface block arrays.
For example, cases like this:
out block_name {
float f;
} block_instance[2];
This allows Mesa to pass the piglit glsl-1.50 test:
* execution/interface-blocks-complex-vs-fs.shader_test
Signed-off-by: Jordan Justen
---
src/glsl/lower_na
Signed-off-by: Jordan Justen
---
src/glsl/link_varyings.cpp | 33 +
1 file changed, 29 insertions(+), 4 deletions(-)
diff --git a/src/glsl/link_varyings.cpp b/src/glsl/link_varyings.cpp
index 04c9fdd..b374049 100644
--- a/src/glsl/link_varyings.cpp
+++ b/src/gls
Signed-off-by: Jordan Justen
---
src/glsl/linker.cpp | 13 +
1 file changed, 13 insertions(+)
diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp
index 29856b0..acbdd2f 100644
--- a/src/glsl/linker.cpp
+++ b/src/glsl/linker.cpp
@@ -478,6 +478,7 @@ cross_validate_globals(struct g
Convert interface blocks with instance names into flat
interface blocks without an instance name.
Signed-off-by: Jordan Justen
---
src/glsl/Makefile.sources |1 +
src/glsl/ir_optimization.h|1 +
src/glsl/linker.cpp |6 +
src/glsl/
Signed-off-by: Jordan Justen
---
src/glsl/ir.h |6 ++
1 file changed, 6 insertions(+)
diff --git a/src/glsl/ir.h b/src/glsl/ir.h
index bbfec69..ee27dea 100644
--- a/src/glsl/ir.h
+++ b/src/glsl/ir.h
@@ -120,6 +120,7 @@ public:
virtual class ir_dereference * as_dereference()
Signed-off-by: Jordan Justen
---
src/glsl/ast_to_hir.cpp | 17 ++---
1 file changed, 14 insertions(+), 3 deletions(-)
diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
index 40f3188..ee54c70 100644
--- a/src/glsl/ast_to_hir.cpp
+++ b/src/glsl/ast_to_hir.cpp
@@ -4293,6
Previously uniform blocks allowed for the 'uniform' keyword
to be used with members of a uniform blocks. With interface
blocks 'in' can be used on 'in' interface block members and
'out' can be used on 'out' interface block members.
The basic_interface_block rule will verify that the same
qualifier
An interface block member may specify the type:
in {
in vec4 in_var_with_qualifier;
};
In this case it must match the interface block type.
It can also omit the qualifier:
uniform {
vec4 uniform_var_without_qualifier;
};
In this case, it should use the same type as the interface block.
Interface blocks in GLSL 150 allow an instance name to be used.
Signed-off-by: Jordan Justen
---
src/glsl/glsl_parser.yy | 15 ++-
1 file changed, 10 insertions(+), 5 deletions(-)
diff --git a/src/glsl/glsl_parser.yy b/src/glsl/glsl_parser.yy
index 8e6b04d..1fd8cc2 100644
--- a/sr
Signed-off-by: Jordan Justen
---
src/glsl/glsl_parser.yy | 14 ++
1 file changed, 14 insertions(+)
diff --git a/src/glsl/glsl_parser.yy b/src/glsl/glsl_parser.yy
index 7adc06d..8e6b04d 100644
--- a/src/glsl/glsl_parser.yy
+++ b/src/glsl/glsl_parser.yy
@@ -1935,6 +1935,20 @@ basic_i
Previously only 'uniform' was allowed for uniform blocks.
Now, in/out can be parsed, but it will only be allowed for
GLSL >= 150.
Signed-off-by: Jordan Justen
---
src/glsl/glsl_parser.yy | 60 ++-
1 file changed, 49 insertions(+), 11 deletions(-)
d
Signed-off-by: Jordan Justen
---
src/glsl/glsl_parser.yy | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/src/glsl/glsl_parser.yy b/src/glsl/glsl_parser.yy
index cd33078..33b74ea 100644
--- a/src/glsl/glsl_parser.yy
+++ b/src/glsl/glsl_parser.yy
@@
Signed-off-by: Jordan Justen
---
src/glsl/ast.h |4 ++--
src/glsl/ast_to_hir.cpp |6 +++---
src/glsl/glsl_parser.yy | 14 +++---
3 files changed, 12 insertions(+), 12 deletions(-)
diff --git a/src/glsl/ast.h b/src/glsl/ast.h
index fcc6b45..49c3939 100644
--- a/src/glsl
git://people.freedesktop.org/~jljusten/mesa interface-blocks-v2
v2:
* 3 new patches added to series
* Add support for interface block instance arrays
* Add support for rejecting unmatched interface blocks
during the linking phase.
* Known issue: fails new piglit glsl-1.50 test:
execution
On Sun, Mar 17, 2013 at 12:57 AM, Ritvik Sharma
wrote:
> Also in the file mapi_tmp.h I am getting #include error for MAPI_ABI_HEADER.
Look at how the Makefiles define MAPI_ABI_HEADER (e.g., in
src/mapi/es2api/Makefile.am).
___
mesa-dev mailing list
me
On an INTEL_DEBUG=perf piglit run on IVB, reduces the instances of "HW
workaround: blit" (the printouts from the misaligned-depth workaround
blits) from 725 to 684
It doesn't totally eliminate the workaround blit, because we still have
problems with Y offsets that we can't fix (since texturing can
On Wed, Mar 13, 2013 at 10:11 AM, Tom Stellard wrote:
> - A brief description of the project
> - A difficulty rating (e.g. easy, medium, hard)
> - The skills / programming languages required
GL/GLSL tests for GL 3.2, 3.3 in the piglit test suite
Difficulty: Easy-Medium
Skills Required: C
Useful s
Kenneth Graunke writes:
> This partially fixes a performance regression since commit 33599433c7.
> More work is required to fully fix it in all cases. This at least helps
> Warsow.
Thanks, Ken.
Reviewed-by: Carl Worth
-Carl
pgpEBf6hWs_2H.pgp
Description: PGP signature
__
On 03/13/2013 10:11 AM, Tom Stellard wrote:
Hi,
It's time again for Google Summer of Code, so we need to start updating
the X.Org ideas page (http://www.x.org/wiki/SummerOfCodeIdeas) with new
ideas. Since there have been a few issues with the wikis lately, if you
have any ideas please respond t
https://bugs.freedesktop.org/show_bug.cgi?id=61821
--- Comment #7 from Mike Stroyan ---
Created attachment 76708
--> https://bugs.freedesktop.org/attachment.cgi?id=76708&action=edit
configure.ac: Build dricommon for gallium swrast
The 07f2dee7 commit added setting HAVE_COMMON_DRI in gallium_ch
Test case?
Chad
On Mar 16, 2013, at 9:45 AM, Vincent Lejeune wrote:
> It allows the backend to generate reads to constant cache which
> are faster that VTX_READ.
> ---
> lib/Target/R600/R600ISelLowering.cpp | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/Target/R60
Chris Forbes writes:
> The new entrypoints will come later, but this adds the actual logic for
> supporting immutable multisample textures:
>
> - The immutability flag is set as desired.
> - Attempting to modify an immutable multisample texture produces
> INVALID_OPERATION.
>
> Note: The extens
Chris Forbes writes:
> The new entrypoints will come later, but this adds the actual logic for
> supporting immutable multisample textures:
>
> - The immutability flag is set as desired.
> - Attempting to modify an immutable multisample texture produces
> INVALID_OPERATION.
>
> Note: The extens
Paul Berry writes:
> Since apps typically begin rendering with a call to glClear(), it is
> likely that when brw_workaround_depthstencil_alignment() moves a
> miplevel to a temporary buffer, it can avoid doing a blit, since the
> contents of the miplevel are about to be erased.
>
> This patch add
Since half of ir_validate uses asserts() (the other using printf() then
abort()), there's not much use to calling it in a release build. Cuts
6.3% of the startup time of TF2.
NOTE: This is a candidate for the stable branches.
---
src/glsl/ir_validate.cpp |6 ++
src/glsl/linker.cpp |
Chris Forbes writes:
> ARB_texture_storage_multisample allows texture parameters to be
> queried for TEXTURE_2D_MULTISAMPLE and TEXTURE_2D_MULTISAMPLE_ARRAY
> targets.
>
> Some parameters may also be set, with the following exceptions:
>
> - TEXTURE_BASE_LEVEL may not be set to a nonzero value; g
- Original Message -
>
>
> Hi,
>
> I am receiving the following error while compiling the code in the mapi
> directory. I am using mesa 7.5.
If you're compiling with MSVC I'd recommend using a recent Mesa release and
save your self a world of trouble. It's known to build well there.
I
https://bugs.freedesktop.org/show_bug.cgi?id=62482
Priority: medium
Bug ID: 62482
Keywords: regression
CC: jfons...@vmware.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: [softpipe] piglit amd_seamless_cubemap_per_texture
https://bugs.freedesktop.org/show_bug.cgi?id=61827
Anuj Phogat changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On 03/15/2013 05:33 PM, jupiter wrote:
Hi Brian,
On 3/15/13, Brian Paul wrote:
On 03/15/2013 05:39 AM, jupiter wrote:
Thanks Brian and Matt.
On 3/15/13, Matt Turner wrote:
On Thu, Mar 14, 2013 at 6:29 AM, Brian Paul wrote:
Hmm, I guess autoconf still has some unneeded dependencies on D
https://bugs.freedesktop.org/show_bug.cgi?id=62425
--- Comment #1 from Brian Paul ---
This sounds the same as fedora bug 918661:
http://bugzilla.redhat.com/show_bug.cgi?id=918661
Can you see if the patch from that bug report helps?
--
You are receiving this mail because:
You are the assignee
Looks good to me. Please add all necessary test cases to cover all these code
paths.
Good stuff guys. Very subtle code indeed.
Jose
- Original Message -
> From: Roland Scheidegger
>
> If we're in some conditional or loop we must not return, or the code
> after the condition is never
2013/3/18 Maarten Lankhorst :
> This is one of the 2 patches used in ubuntu for decreasing size of mesa build.
>
> The other one is more hacky, and links libmesagallium into libgallium,
> and then links libgallium against libdricore too for minimal duplication.
>
> This might mess up with static ll
This is a hack ubuntu uses to reduce the size of mesa, it links libgallium
against libmesagallium and libdricore.
I think it's too ugly to commit in its current form, but it works with the
flags ubuntu passes to configure.
On 9.0 branch I had a similar patch, without shared libgallium and witho
Am 18.03.2013 15:12, schrieb Roland Scheidegger:
Am 18.03.2013 14:36, schrieb Christian König:
From: Christian König
To further improve the optimization of source and destination
indirect addressing we need the ability to store a reference
to the declaration of the addressed operands.
Since m
This is one of the 2 patches used in ubuntu for decreasing size of mesa build.
The other one is more hacky, and links libmesagallium into libgallium,
and then links libgallium against libdricore too for minimal duplication.
This might mess up with static llvm, iirc static llvm is built wrong and
Am 18.03.2013 14:36, schrieb Christian König:
> From: Christian König
>
> To further improve the optimization of source and destination
> indirect addressing we need the ability to store a reference
> to the declaration of the addressed operands.
>
> Since most of the fields in tgsi_src_register
Hi,
I am receiving the following error while compiling the code in the mapi
directory. I am using mesa 7.5.
(On compiling glapi_dispatch.c)
OpenGL\mapi/glapi/glapitemp.h(86) : error C2375: 'glNewList' : redefinition;
different linkage
c:\poc\edk2\AppPkg\Include\GL/gl.h(944) : see decl
From: Christian König
v2: further improve the text with comments from Christoph Bumiller.
Signed-off-by: Christian König
---
src/gallium/docs/source/tgsi.rst | 18 ++
1 file changed, 18 insertions(+)
diff --git a/src/gallium/docs/source/tgsi.rst b/src/gallium/docs/source/tgs
From: Christian König
To further improve the optimization of source and destination
indirect addressing we need the ability to store a reference
to the declaration of the addressed operands.
Since most of the fields in tgsi_src_register doesn't apply for
an indirect addressing operand replace it
From: Christian König
Remember which declarations are declared as "arrays" and so
can be indirectly addressed. ArrayIDs start at 1, cause for
compatibility reasons zero is treaded as no array present.
Signed-off-by: Christian König
---
src/gallium/auxiliary/tgsi/tgsi_dump.c |6 ++
From: Christian König
Nobody seems to be using it, and only nv50 had a partial implementation.
Signed-off-by: Christian König
---
src/gallium/auxiliary/tgsi/tgsi_build.c| 19 -
src/gallium/auxiliary/tgsi/tgsi_dump.c | 38 --
src/gallium/auxiliary/tgsi/tg
From: Christian König
They shouldn't be necessary any more.
Signed-off-by: Christian König
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 36 +++-
1 file changed, 3 insertions(+), 33 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/s
From: Christian König
Instead of allocating everything as temporaries, use the
new array allocation functions.
v2: fix bug in simplify_cmp, declare arrays on demand
Signed-off-by: Christian König
---
src/mesa/main/mtypes.h |1 +
src/mesa/state_tracker/st_glsl_to_tgsi.c
From: Christian König
Signed-off-by: Christian König
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 23 ++-
1 file changed, 10 insertions(+), 13 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
index 0cef
From: Christian König
Don't bother with free temporaries, just allocate them at
the end and also emit them in their own declaration.
Signed-off-by: Christian König
---
src/gallium/auxiliary/tgsi/tgsi_ureg.c | 55
src/gallium/auxiliary/tgsi/tgsi_ureg.h | 38
From: Christian König
Instead of emitting each temporary separately, emit them in a chunk.
v2: keep separate function for emitting temps
Signed-off-by: Christian König
---
src/gallium/auxiliary/tgsi/tgsi_ureg.c | 30 +-
1 file changed, 13 insertions(+), 17 deleti
On Thu, Mar 14, 2013 at 3:23 PM, Brian Paul wrote:
> On 03/14/2013 08:18 AM, Marek Olšák wrote:
>>
>> Z24X8 actually does save memory, because Radeon HD 5000 and later
>> cards don't support combined depth stencil formats, therefore Z24S8 is
>> allocated internally as two separate buffers Z24X8 an
Am 17.03.2013 20:59, schrieb Christoph Bumiller:
On 17.03.2013 18:04, Christoph Bumiller wrote:
On 17.03.2013 16:30, Christian König wrote:
Am 15.03.2013 18:58, schrieb Christoph Bumiller:
On 15.03.2013 13:08, Christian König wrote:
Am 14.03.2013 15:53, schrieb Christoph Bumiller:
On 14.03.2
On Mon, 2013-03-18 at 01:06 +0100, Marek Olšák wrote:
>
> ([rant]We should really have an unmappable portion of VRAM as a
> feature, because unmappable memory is cool: the memory can be
> reclaimed immediately after texture deletion even if the texture is
> busy, because the CPU can't access it,
65 matches
Mail list logo
