On Fri, Oct 4, 2013 at 9:17 PM, Chris Forbes wrote:
> This series adds support for ARB_texture_query_levels on i965 Gen6+. The
> fourth channel of the resinfo message (used for textureSize()) includes the
> correct value for textureQueryLevels().
The series looks good to me. With the one small
Mesa 9.1.7 has been released. Mesa 9.1.6 is a bug fix release which
fixes bugs fixed since the 9.1.6 release.
The tag in the git repository for Mesa 9.1.7 is 'mesa-9.1.7'.
Note: This is the last anticipated release of the 9.1 series. Future
effort for stable releases will be focused on the 9.2 se
On Fri, Oct 4, 2013 at 9:17 PM, Chris Forbes wrote:
> Theoretically would work on Gen5 as well but requires GLSL 1.30, which
> is not (yet) enabled by default there.
In that case, just do
ctx->Extensions.ARB_texture_query_levels = ctx->Const.GLSLVersion >= 130;
and put it in the if (brw->gen >=
Theoretically would work on Gen5 as well but requires GLSL 1.30, which
is not (yet) enabled by default there.
Signed-off-by: Chris Forbes
---
src/mesa/drivers/dri/i965/intel_extensions.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/drivers/dri/i965/intel_extensions.c
b/src/mesa/
Signed-off-by: Chris Forbes
---
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 15 ++-
1 file changed, 14 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
b/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
index 9e6cc78..1112b23 100644
--- a/s
Signed-off-by: Chris Forbes
---
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 20 +++-
1 file changed, 19 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
index a8ad659..6141009 100644
--- a/sr
Previously we special-cased textureSize() but this is the more correct
condition.
Signed-off-by: Chris Forbes
---
src/mesa/drivers/dri/i965/brw_cubemap_normalize.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_cubemap_normalize.cpp
b/src/mes
Signed-off-by: Chris Forbes
---
src/glsl/builtin_functions.cpp | 56 +
src/glsl/glcpp/glcpp-parse.y| 3 ++
src/glsl/glsl_parser_extras.cpp | 1 +
src/glsl/glsl_parser_extras.h | 2 ++
src/glsl/ir.cpp | 4 +--
src/g
Signed-off-by: Chris Forbes
---
src/mesa/main/extensions.c | 1 +
src/mesa/main/mtypes.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c
index c0f17c5..2507fdf 100644
--- a/src/mesa/main/extensions.c
+++ b/src/mesa/main/extensions.
This series adds support for ARB_texture_query_levels on i965 Gen6+. The fourth
channel of the resinfo message (used for textureSize()) includes the correct
value for textureQueryLevels().
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http:/
Mesa 9.2.1 has been released. Mesa 9.2.1 is a bug fix release which
fixes bugs fixed since the 9.2 release.
The tag in the git repository for Mesa 9.2.1 is 'mesa-9.2.1'.
Mesa 9.2.1 is available for download at
ftp://freedesktop.org/pub/mesa/9.2.1/
md5sums:
e6cdfa84dfddd86e3d36ec7ff4b6478a Mesa
From: Roland Scheidegger
Not used since ages, and it wouldn't work at all with explicit derivatives now
(not that it did before as it ignored them but now the code would just use
the derivs pre-projected which would be quite random numbers).
v2: also get rid of 3 helper functions no longer used.
From: Roland Scheidegger
There's two reasons for this:
1) even when ignoring rho approximation for cube maps, the result is still
not correct, but it's better as the max error at edges is now sqrt(2) instead
of 2 (which was a full mip level), same as it is for ordinary 2d maps when
doing rho appr
From: Roland Scheidegger
They need some special handling. Quite complicated.
Additionally, use the same code for implicit derivatives too if no_rho_approx
and no_quad_lod is set, because it seems while generally it should be ok
to use per quad lod for implicit derivatives there's at least some te
https://bugs.freedesktop.org/show_bug.cgi?id=70150
klondike changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
Matt Turner writes:
> On Fri, Oct 4, 2013 at 2:19 PM, Paul Berry wrote:
>> On 4 October 2013 13:51, Paul Berry wrote:
>>>
>>> On 3 October 2013 10:59, Matt Turner wrote:
v2: Check fixed_hw_reg.{file,nr} instead of dst.reg.
v3: Store the bool emitted_addc_or_subb in the class, no
https://bugs.freedesktop.org/show_bug.cgi?id=70150
--- Comment #2 from Tom Stellard ---
The type sizes are supposed to be controlled by the TargetInfo class.
TargetInfo::setForcedLangOptions() in clang/lib/basic/TargetInfo.cpp is where
the type sizes are set for OpenCL.
R600's TargetInfo is def
On Fri, 04 Oct 2013 13:31:11 -0600
Brian Paul wrote:
> The series look OK to me. Do you want to add/update any docs/ files?
>
> Reviewed-by: Brian Paul
>
> Do you need me to push this for you?
>
Nah, I can push it.
I'll see if I can add anything meaningful to the docs.
Thanks!
-- Alex
Note that the dropped comment in brw_context.h is mostly (better written)
in brw_binding_table.c as well.
---
src/mesa/drivers/dri/i965/brw_context.h | 51
src/mesa/drivers/dri/i965/brw_vec4.cpp | 29 +++---
src/mesa/drivers/dri/i965/brw_vec4_gen
Now that both vec4 and fs are dynamically assigning offsets, a lot of the
code is the same.
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 33 ++
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 3 ++
src/mesa/drivers/dri/i965/brw_shader.cpp | 47 ++
It would be nice to be able to pack our binding table so that programs
that use 1 render target don't upload an extra BRW_MAX_DRAW_BUFFERS - 1
binding table entries. To do that, we need the compiled program to have
information on where its surfaces go.
---
I consistently named the structure membe
---
src/mesa/drivers/dri/i965/brw_context.h| 41 --
src/mesa/drivers/dri/i965/brw_fs.cpp | 35 ++
src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 2 +-
3 files changed, 35 insertions(+), 43 deletions(-)
diff --git a/src/mesa/drivers/
vec4 already had it, so put it in the FS, too.
---
src/mesa/drivers/dri/i965/brw_fs_fp.cpp | 14 +++---
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 13 +++--
src/mesa/drivers/dri/i965/brw_shader.h | 1 +
src/mesa/drivers/dri/i965/brw_vec4.h | 1 -
4 files ch
---
src/mesa/drivers/dri/i965/brw_context.h | 3 ---
1 file changed, 3 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_context.h
b/src/mesa/drivers/dri/i965/brw_context.h
index d4ace98..eb9d93c 100644
--- a/src/mesa/drivers/dri/i965/brw_context.h
+++ b/src/mesa/drivers/dri/i965/brw_conte
---
src/mesa/drivers/dri/i965/brw_context.h | 3 +--
src/mesa/drivers/dri/i965/brw_state_cache.c | 4 +---
src/mesa/drivers/dri/i965/brw_vec4_gs.c | 3 +--
src/mesa/drivers/dri/i965/brw_vec4_gs.h | 3 +--
src/mesa/drivers/dri/i965/brw_vs.c | 3 +--
src/mesa/drivers/dri/i965/br
Here's a series to pack our binding tables so that you (for example) don't
waste BRW_MAX_DRAWBUFFERS-1 indices in your simple texturing shaders.
I think it cleans up the code, though the net LOC change is approximately
0. Performance data for GLB2.7:
IVB: No SSPD (n=754)
gm45: -0.0323403% +/- 0.
On 4 October 2013 14:24, Matt Turner wrote:
> On Fri, Oct 4, 2013 at 2:19 PM, Paul Berry
> wrote:
> > On 4 October 2013 13:51, Paul Berry wrote:
> >>
> >> On 3 October 2013 10:59, Matt Turner wrote:
> >>>
> >>> v2: Check fixed_hw_reg.{file,nr} instead of dst.reg.
> >>> v3: Store the bool emitt
On Fri, Oct 4, 2013 at 2:19 PM, Paul Berry wrote:
> On 4 October 2013 13:51, Paul Berry wrote:
>>
>> On 3 October 2013 10:59, Matt Turner wrote:
>>>
>>> v2: Check fixed_hw_reg.{file,nr} instead of dst.reg.
>>> v3: Store the bool emitted_addc_or_subb in the class, not static.
>>> ---
>>> src/mes
On 3 October 2013 11:00, Matt Turner wrote:
> v2: Check fixed_hw_reg.{file,nr} instead of dst.reg.
> v3: Store the bool emitted_addc_or_subb in the class, not static.
> ---
> src/mesa/drivers/dri/i965/brw_vec4.h | 3 +
> src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 104
>
On 4 October 2013 13:51, Paul Berry wrote:
> On 3 October 2013 10:59, Matt Turner wrote:
>
>> v2: Check fixed_hw_reg.{file,nr} instead of dst.reg.
>> v3: Store the bool emitted_addc_or_subb in the class, not static.
>> ---
>> src/mesa/drivers/dri/i965/brw_fs.h | 3 +
>> src/mesa/dri
Giulio Camuffo writes:
> eglSwapBuffers() should result in a call to wl_display_flush since it
> may be called by a rendering thread, and it cannot rely on the main
> loop to call it.
...
> Cc: mesa-sta...@lists.freedesktop.org
Hi Giulio,
I'm the maintainer of Mesa's stable branches, so I'm goin
I've pushed this patch. I'll let somebody else review the other one.
Marek
On Thu, Oct 3, 2013 at 12:12 AM, Grigori Goronzy wrote:
> All texture instructions can use offsets, not just TXF. Offsets into
> the literals array were wrong, too.
> ---
> src/gallium/drivers/r600/r600_shader.c | 20 +++
On 3 October 2013 10:59, Matt Turner wrote:
> v2: Check fixed_hw_reg.{file,nr} instead of dst.reg.
> v3: Store the bool emitted_addc_or_subb in the class, not static.
> ---
> src/mesa/drivers/dri/i965/brw_fs.h | 3 +
> src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 104
>
https://bugs.freedesktop.org/show_bug.cgi?id=70150
--- Comment #1 from klondike ---
I have been checking a bit around this issue, whilst modifying clc/clctypes.h
to add "typedef unsigned __INT64_TYPE__ ulong;" works, doing the same with long
doesn't, quite likely because it is a standard C type.
On 27 September 2013 18:17, Matt Turner wrote:
> v2: Set destination register using brw_null_reg().
>
Reviewed-by: Paul Berry
> ---
> src/mesa/drivers/dri/i965/brw_fs.cpp | 15 ++-
> src/mesa/drivers/dri/i965/brw_vec4.cpp | 15 ++-
> 2 files changed, 28 insertions(+
Now, one can do the following to generate and read the i965 Doxygen:
cd $MESA_TOP/doxygen
make
firefox i965/index.html
CC: Frank Henigman
Signed-off-by: Chad Versace
---
doxygen/.gitignore | 1 +
doxygen/Makefile | 3 ++-
doxygen/i965.doxy | 50 ++
https://bugs.freedesktop.org/show_bug.cgi?id=31800
David "okias" Heidelberger changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution
https://bugs.freedesktop.org/show_bug.cgi?id=70150
Priority: medium
Bug ID: 70150
Assignee: mesa-dev@lists.freedesktop.org
Summary: Mesa downsizes to i32 the types ulong and long
Severity: normal
Classification: Unclassified
On 10/04/2013 11:36 AM, Alexander von Gluck IV wrote:
* This in essence means that Mesa would be
taking control of Haiku's OpenGL kit.
* This works by dispatching renderers from the
OpenGL add-ons directory
---
src/gallium/SConscript | 1 +
src/gallium/targe
On 10/04/2013 10:03 AM, Ian Romanick wrote:
> On 09/20/2013 07:28 PM, Francisco Jerez wrote:
>> In the array-of-struct case, recursion() takes the row_major flag for
>> each iteration from 't->fields.structure[i]', but 't' is not a record
>> type. Inherit the array declaration row_major flag inste
http://llvm.org/viewvc/llvm-project?view=revision&revision=191922
---
src/gallium/state_trackers/clover/llvm/invocation.cpp | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/clover/llvm/invocation.cpp
b/src/gallium/state_trackers/clover/llvm/invoc
On Fri, Oct 04, 2013 at 10:01:32AM -0700, Owen Anderson wrote:
>
> On Oct 3, 2013, at 5:36 PM, Tom Stellard wrote:
>
> > Unfortunately, we cannot enable the machine verifier on tests that have
> > branches, because the way IF and ELSE instructions are selected leaves us
> > with a copy instructi
On Oct 3, 2013, at 5:36 PM, Tom Stellard wrote:
> Unfortunately, we cannot enable the machine verifier on tests that have
> branches, because the way IF and ELSE instructions are selected leaves us
> with a copy instruction following the IF and ELSE terminators, which
> violates one of the verif
"Rogovin, Kevin" writes:
> Additionally, I really, really, want the text in doxygen/doxy_text to
> eventually land in src/ and a policy put in place that as patches are
> added the corresponding doxy_text files are updated to reflect the
> patch. This will keep the documentation up to date and use
* This shared library gets parsed by the
system as a system "add-on"
---
.../targets/haiku-softpipe/GalliumContext.cpp | 528 +
.../targets/haiku-softpipe/GalliumContext.h| 83
.../targets/haiku-softpipe/GalliumFramebuffer.cpp | 116 +
.../targets/haik
---
scons/gallium.py | 2 ++
src/gallium/targets/haiku-softpipe/SConscript | 1 -
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/scons/gallium.py b/scons/gallium.py
index 2e341e9..5b20f3f 100755
--- a/scons/gallium.py
+++ b/scons/gallium.py
@@ -506,6 +50
* It isn't needed any longer as we're
moving in the code that called it.
* The winsys code is C, so make sure
we include the header in the extern C
---
.../targets/haiku-softpipe/GalliumContext.cpp | 2 +-
src/gallium/targets/haiku-softpipe/SConscript | 1 -
.../targets/haiku-softp
* Thanks for the help xexaxo!
---
src/gallium/targets/haiku-softpipe/GalliumContext.cpp | 16 ++--
1 file changed, 6 insertions(+), 10 deletions(-)
diff --git a/src/gallium/targets/haiku-softpipe/GalliumContext.cpp
b/src/gallium/targets/haiku-softpipe/GalliumContext.cpp
index 1c08fb1
* Use LoadableModule vs SharedLibrary
---
src/gallium/targets/haiku-softpipe/SConscript | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/targets/haiku-softpipe/SConscript
b/src/gallium/targets/haiku-softpipe/SConscript
index 0a99976..6e12dc4 100644
--- a/src/gallium
* This in essence means that Mesa would be
taking control of Haiku's OpenGL kit.
* This works by dispatching renderers from the
OpenGL add-ons directory
---
src/gallium/SConscript | 1 +
src/gallium/targets/libgl-haiku/GLDispatcher.cpp | 72 +++
src/gallium/tar
On 09/20/2013 07:28 PM, Francisco Jerez wrote:
> In the array-of-struct case, recursion() takes the row_major flag for
> each iteration from 't->fields.structure[i]', but 't' is not a record
> type. Inherit the array declaration row_major flag instead.
>
> This mistake was found by running piglit
On Fri, Oct 4, 2013 at 10:02 AM, Marek Olšák wrote:
> Wouldn't it be more flexible for you to just change the depth buffer
> format internally by reallocating the depth buffer in the driver
> whenever it's incompatible with the colorbuffer? I think you should be
> able to do that easily without ch
On 10/04/2013 07:30 AM, Adam Jackson wrote:
Signed-off-by: Adam Jackson
---
src/glx/indirect_glx.c | 16 +---
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/src/glx/indirect_glx.c b/src/glx/indirect_glx.c
index f8944a7..d0457fe 100644
--- a/src/glx/indirect_glx.c
++
Signed-off-by: Adam Jackson
---
src/glx/indirect_glx.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/src/glx/indirect_glx.c b/src/glx/indirect_glx.c
index d0457fe..d27b019 100644
--- a/src/glx/indirect_glx.c
+++ b/src/glx/indirect_glx.c
@@ -132,6 +132,7 @@ indirect_bi
Wouldn't it be more flexible for you to just change the depth buffer
format internally by reallocating the depth buffer in the driver
whenever it's incompatible with the colorbuffer? I think you should be
able to do that easily without changing the pipe_resource structure.
If a depth buffer cannot
For a few reasons.
1: In the (current) common case, these conditionals are never true. All
we're doing by checking them is slowing down MakeCurrent. The server
does these checks already anyway.
2: GLX >= 3.0 contexts may legally be made current without a bound
framebuffer.
This does not fix pig
Signed-off-by: Adam Jackson
---
src/glx/indirect_glx.c | 16 +---
1 file changed, 9 insertions(+), 7 deletions(-)
diff --git a/src/glx/indirect_glx.c b/src/glx/indirect_glx.c
index f8944a7..d0457fe 100644
--- a/src/glx/indirect_glx.c
+++ b/src/glx/indirect_glx.c
@@ -59,8 +59,9 @@ sta
https://bugs.freedesktop.org/show_bug.cgi?id=70123
Priority: medium
Bug ID: 70123
Assignee: mesa-dev@lists.freedesktop.org
Summary: Freeze caused by 'winsys/radeon: remove
cs_queue_empty' commit
Severity: normal
Classific
This should finally fix RT_FORMAT errors due to weird fb configurations.
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/nouveau/nv30/nv30_screen.c | 20 +++-
1 file changed, 19 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/nouveau/nv30/nv30_screen.c
b/src/ga
This is now completely handled by the new is_fb_format_supported
callback.
Signed-off-by: Ilia Mirkin
---
src/gallium/docs/source/screen.rst | 2 --
src/gallium/drivers/freedreno/freedreno_screen.c | 1 -
src/gallium/drivers/i915/i915_screen.c | 1 -
src/gallium/driver
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/nouveau/nv30/nv30_screen.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/drivers/nouveau/nv30/nv30_screen.c
b/src/gallium/drivers/nouveau/nv30/nv30_screen.c
index 807100e..c546c43 100644
--- a/src/gallium/drivers/nouveau/nv30/n
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/softpipe/sp_screen.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/drivers/softpipe/sp_screen.c
b/src/gallium/drivers/softpipe/sp_screen.c
index 47ef20e..5dca415 100644
--- a/src/gallium/drivers/softpipe/sp_screen.c
+++ b/src/g
Only do this in the !r500 case, same as how
PIPE_CAP_MIXED_COLORBUFFER_FORMATS is set.
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/r300/r300_screen.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/drivers/r300/r300_screen.c
b/src/gallium/drivers/r300/r300_screen.c
inde
Signed-off-by: Ilia Mirkin
---
src/gallium/drivers/i915/i915_screen.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/drivers/i915/i915_screen.c
b/src/gallium/drivers/i915/i915_screen.c
index 77607d0..78800ac 100644
--- a/src/gallium/drivers/i915/i915_screen.c
+++ b/src/gallium/d
Signed-off-by: Ilia Mirkin
---
src/gallium/auxiliary/util/u_format.c | 20
src/gallium/auxiliary/util/u_format.h | 11 +++
2 files changed, 31 insertions(+)
diff --git a/src/gallium/auxiliary/util/u_format.c
b/src/gallium/auxiliary/util/u_format.c
index a8aa571..8b9
Extract format parameters in st_validate_framebuffer and call the new
optional pipe->is_fb_format_supported function to check that parameters
are supported by the driver.
Signed-off-by: Ilia Mirkin
---
I'm a bit weirded out that no attempt is made to check if the att->Type ==
GL_TEXTURE for cbuf
NV30 is blessed with all manners of hardware restrictions. One of them is that
the render target format's color and depth outputs need to be the same
bit-ness, i.e. either both 16 or both 32 bits. (In addition to all color
attachments needing to be the same, and everything needing to be the same
si
When PIPE_CAP_MIXED_FRAMEBUFFER_SIZES is not provided, parts of
ARB_framebuffer_object can't be supported, such as on NV30.
Signed-off-by: Ilia Mirkin
---
src/mesa/state_tracker/st_extensions.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/mesa/state_tracker/st_e
This CAP will determine whether ARB_framebuffer_object can be enabled.
The nv30 driver does not allow mixing swizzled and linear zsbuf/cbuf
textures.
Signed-off-by: Ilia Mirkin
---
src/gallium/docs/source/screen.rst | 3 +++
src/gallium/drivers/freedreno/freedreno_screen.c | 1 +
s
This fixes issues where get_rt_format would see a 0 format because the
nouveau_surface had not been properly initialized. Fixes crash on
supertuxkart startup (which still fails due to out-of-vram issues).
Signed-off-by: Ilia Mirkin
---
src/mesa/drivers/dri/nouveau/nouveau_texture.c | 9 +
Signed-off-by: Emilio Pozuelo Monfort
---
src/mapi/glapi/gen/gl_API.xml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mapi/glapi/gen/gl_API.xml b/src/mapi/glapi/gen/gl_API.xml
index 3ffa817..48fce36 100644
--- a/src/mapi/glapi/gen/gl_API.xml
+++ b/src/mapi/glapi/gen/gl_A
71 matches
Mail list logo