https://bugs.freedesktop.org/show_bug.cgi?id=42128
--- Comment #5 from Thor 2011-10-27 02:53:52 UTC ---
Created attachment 52818
--> https://bugs.freedesktop.org/attachment.cgi?id=52818
The output of glxinfo
--
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
--- Yo
If there are no other concerns, I'll push this on Saturday, including
the nv20 fix from Francisco Jerez.
http://cgit.freedesktop.org/~mareko/mesa/log/?h=nv-fog-distance
Marek
On Tue, Sep 20, 2011 at 7:50 AM, Nicholas Miell wrote:
> On 09/19/2011 10:23 AM, Ian Romanick wrote:
>>
>> I guess the b
I only see implementation for ffvertex_prog.c
Wasn't (or will soon be) the FF->Mesa IR replaced with FF->GLSL IR? If so, can
you truly enable unconditionally on gallium/st?
Jose
- Original Message -
> If there are no other concerns, I'll push this on Saturday, including
> the nv20 fix
From what I can see, the fog distance is computed per-vertex and
stored in the FogCoord vertex shader output. There is no fragment
shader change required. The fixed-function vertex program hasn't been
converted to the GLSL IR yet, only the fragment program, which is
irrelevant here.
Gallium only u
- Original Message -
> From what I can see, the fog distance is computed per-vertex and
> stored in the FogCoord vertex shader output. There is no fragment
> shader change required. The fixed-function vertex program hasn't been
> converted to the GLSL IR yet, only the fragment program, whic
On Wed, 26 Oct 2011 18:42:52 -0700, Paul Berry wrote:
> The implementation of ir_binop_nequal in constant_expression_value()
> appears to have been copy-and-pasted from the implementation of
> ir_binop_equal, but with all instances of '==' changed to '!='. This
> is correct except for one minor f
https://bugs.freedesktop.org/show_bug.cgi?id=42313
Bug #: 42313
Summary: software rasterizer: ignore glTexCoordPointer if
GL_TEXTURE_2D is disabled
Classification: Unclassified
Product: Mesa
Version: 7.11
Platform: x86
Hi,
On Saturday, October 22, 2011 13:36:37 Mathias Fröhlich wrote:
> The attached patch reduces the amount of pipe flushes for r600g.
> I am not exactly sure if we could skip this flush entirely because of the
> internal r600g winsys flush logic.
> But what we can do is the attached patch:
>
> R
Looks good to me. Ideally even texture_barrier should not be needed.
set_framebuffer_state() and flush() should flush caches automatically.
If they don't, there is a bug.
Marek
2011/10/27 Mathias Fröhlich :
>
> Hi,
>
> On Saturday, October 22, 2011 13:36:37 Mathias Fröhlich wrote:
>> The attached
On Thursday 27 October 2011, Mathias Fröhlich wrote:
>
> Hi,
>
> On Saturday, October 22, 2011 13:36:37 Mathias Fröhlich wrote:
> > The attached patch reduces the amount of pipe flushes for r600g.
> > I am not exactly sure if we could skip this flush entirely because of the
> > internal r600g win
On 10/26/2011 03:28 PM, Marek Olšák wrote:
> This adds a function that takes an array of varyings from
> glTranformFeedbackVaryingsEXT and generates gl_transform_feedback_info,
> which is supposed to be consumed by drivers. Useful for ir_to_mesa
> and glsl_to_tgsi.
>
> With Dan McCabe's patch, I t
On 10/26/2011 11:11 AM, Brian Paul wrote:
I think the linker is mis-counting gl_TexCoord[] varying vars when linking.
For example, if we have this vertex/fragment shader combination:
// vs
void main()
{
gl_Position = ftransform();
gl_TexCoord[6] = gl_MultiTexCoord[0];
}
// fs
void main()
{
gl
On 24 October 2011 14:17, Eric Anholt wrote:
> This rearranges the code a bit, and makes the upload of the binding
> table take only as many surfaces as there are in use.
> ---
> src/mesa/drivers/dri/i965/brw_vs_surface_state.c | 62
> +-
> 1 files changed, 25 insertions(+)
On 24 October 2011 14:16, Eric Anholt wrote:
> I did do some testing by making libdrm claim a smaller aperture size
> and thus trigger the rollback code, which caught one bug. Generally,
> piglit doesn't trigger rollback at all, which is a bit concerning from
> a testing coverage standpoint.
>
>
These are effectively doing type->get_base_type()->base_type, which is
equivalent to type->base_type. Just use that, as it's simpler.
Signed-off-by: Kenneth Graunke
---
src/glsl/ir_print_visitor.cpp |4 +---
src/glsl/ir_reader.cpp|6 ++
2 files changed, 3 insertions(+), 7 de
The idea here is to set up the message header with the Sampler State
pointer which the hardware provides as part of the PS Thread Payload in
register g0.
Unfortunately, the existing code
fs_reg(GRF, 0, BRW_REGISTER_TYPE_UD))
actually references "virtual GRF 0" rather than the hardware g0. Th
On 26 October 2011 13:00, Eric Anholt wrote:
> From page 22 (28 of PDF) of GLSL 1.30 spec:
>It is an error to provide a literal integer whose magnitude is too
>large to store in a variable of matching signed or unsigned type.
>
>Unsigned integers have exactly 32 bits of precision. Si
On 10/26/2011 01:00 PM, Eric Anholt wrote:
From page 22 (28 of PDF) of GLSL 1.30 spec:
It is an error to provide a literal integer whose magnitude is too
large to store in a variable of matching signed or unsigned type.
Unsigned integers have exactly 32 bits of precision. Signed
On 10/27/2011 10:48 AM, Kenneth Graunke wrote:
These are effectively doing type->get_base_type()->base_type, which is
equivalent to type->base_type. Just use that, as it's simpler.
Signed-off-by: Kenneth Graunke
Reviewed-by: Ian Romanick
---
src/glsl/ir_print_visitor.cpp |4 +---
sr
On 10/26/2011 06:42 PM, Paul Berry wrote:
The implementation of ir_binop_nequal in constant_expression_value()
appears to have been copy-and-pasted from the implementation of
ir_binop_equal, but with all instances of '==' changed to '!='. This
is correct except for one minor flaw: one of those '
Margan,
The code in the diff has wrapped-around and can't be applied automatically.
Please re-send the patch as an attachment.
Jose
- Original Message -
> Here is the modified patch. I just checked if it works but doesn't
> have a good test case for it. But I will write one for sure!
>
Fixes Coverity resource leak defect.
---
src/mesa/main/pack.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/src/mesa/main/pack.c b/src/mesa/main/pack.c
index ecdeaf5..539a06c 100644
--- a/src/mesa/main/pack.c
+++ b/src/mesa/main/pack.c
@@ -1983,6 +1983,7 @@ _mesa_pack_rg
From: Ian Romanick
Signed-off-by: Ian Romanick
---
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp |4 +---
1 files changed, 1 insertions(+), 3 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
b/src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
index 02ecdaf..1312c0d 1
From: Ian Romanick
Previously check_resources could fail, but we'd still try to optimize
the shader, do device-specific code generation, etc. In some cases,
this could explode (especially in the device-specific code
generation). I haven't found that I could trigger this with the
current code.
See the commit meesage for patch 5/5.
Chad Versace (5):
intel: Temporarily disable HiZ for textures
intel: Define intel_miptree_create_for_renderbuffer()
intel: Kill intel_framebuffer_get_hiz_region()
intel: Move inline functions from intel_fbo.h to intel_fbo.c
intel: Replace intel_rende
A great refactor thrashing begins after this commit. Removing code for
texture HiZ will make that refactoring easier.
HiZ was broken for textures anyway, so there's no regression here.
Signed-off-by: Chad Versace
---
src/mesa/drivers/dri/intel/intel_fbo.c | 62 ++--
This function creates a miptree that is suitable as the storage for
a non-texture renderbuffer.
This commit is prerequisite for replacing intel_renderbuffer::region with
a miptree.
Signed-off-by: Chad Versace
---
src/mesa/drivers/dri/intel/intel_mipmap_tree.c | 21 +
src/m
The only client of intel_framebuffer_get_hiz_region() was
intel_framebuffer_has_hiz(). So I folded the body of the former into the
latter.
Signed-off-by: Chad Versace
---
src/mesa/drivers/dri/intel/intel_fbo.h | 20 +++-
1 files changed, 3 insertions(+), 17 deletions(-)
diff -
Move the following inline functions:
intel_get_rb_region
intel_framebuffer_has_hiz
A future commit will replace the renderbuffer's region with a miptree.
This small refactor will eliminate the need for intel_fbo.h to include
intel_mipmap_tree.h on that commit. I'd like to avoid the situati
This is in preparation for 1) fixing the implementation of
glFramebufferTexture(GL_DEPTH_STENCIL) for separate stencil and
2) supporting HiZ for miptrees. As a nice benefit, this lays down some
preliminary groundwork for easily texturing from any renderbuffer, even
those of the window system.
With
On 10/26/2011 06:42 PM, Paul Berry wrote:
> In order to implement the GLSL 1.30 isinf() function, it will be
> necessary to be able to represent infinity in the GLSL IR s-expression
> format. This patch extends the s-expression parser so that it treats
> the string "#inf" as a floating point value
On 10/27/2011 02:59 PM, Ian Romanick wrote:
> From: Ian Romanick
>
> Previously check_resources could fail, but we'd still try to optimize
> the shader, do device-specific code generation, etc. In some cases,
> this could explode (especially in the device-specific code
> generation). I haven't
32 matches
Mail list logo
