https://bugs.freedesktop.org/show_bug.cgi?id=91747
Benjamin Bellec changed:
What|Removed |Added
Status|NEW |NEEDINFO
--
You are receiving this ma
On 28/08/15 08:48, Jordan Justen wrote:
> On 2015-08-27 23:07:26, Samuel Iglesias Gonsálvez wrote:
>>
>> On 28/08/15 08:06, Samuel Iglesias Gonsálvez wrote:
>>>
>>> On 28/08/15 02:21, Jordan Justen wrote:
On 2015-08-05 01:30:12, Iago Toral Quiroga wrote:
> From: Samuel Iglesias Gonsalvez
On Thu, Aug 27, 2015 at 10:05:14AM -0700, Ben Widawsky wrote:
> On Thu, Aug 27, 2015 at 10:51:59AM +0300, Pohjolainen, Topi wrote:
> > On Wed, Aug 26, 2015 at 03:46:05PM -0700, Ben Widawsky wrote:
> > > This reverts commit 1bba29ed403e735ba0bf04ed8aa2e571884fcaaf
> > > Author: Topi Pohjolainen
> >
On 2015-08-27 23:07:26, Samuel Iglesias Gonsálvez wrote:
>
> On 28/08/15 08:06, Samuel Iglesias Gonsálvez wrote:
> >
> > On 28/08/15 02:21, Jordan Justen wrote:
> >> On 2015-08-05 01:30:12, Iago Toral Quiroga wrote:
> >>> From: Samuel Iglesias Gonsalvez
> >>>
> >>> Notice that Skylake needs to i
On Thu, Aug 27, 2015 at 11:42 PM, Jason Ekstrand wrote:
> From: Brian Paul
>
> Previously, core Mesa's _mesa_CopyImageSubData() created temporary textures
> to wrap renderbuffer sources/destinations. This caused a bit of a mess in
> the Mesa/gallium state tracker because we had to basically undo
From: Brian Paul
Previously, core Mesa's _mesa_CopyImageSubData() created temporary textures
to wrap renderbuffer sources/destinations. This caused a bit of a mess in
the Mesa/gallium state tracker because we had to basically undo that
wrapping.
Instead, change ctx->Driver.CopyImageSubData() to
On Fri, Aug 28, 2015 at 2:25 AM, Matt Turner wrote:
> On Tue, Aug 25, 2015 at 9:39 AM, Ilia Mirkin wrote:
>> This greatly improves generated code, especially for the snorm variants,
>> since it is able to get rid of the lshift/rshift for sext, as well as
>> replacing each shift + mask with a sing
On Fri, Aug 28, 2015 at 2:20 AM, Matt Turner wrote:
> On Thu, Aug 20, 2015 at 6:00 PM, Ilia Mirkin wrote:
>> It is fairly tricky to detect the proper conditions for using bitfield
>> insert, but easy to just use it up front. This removes a lot of
>> instructions on nvc0 when invoking the packing
On Tue, Aug 25, 2015 at 9:39 AM, Ilia Mirkin wrote:
> This greatly improves generated code, especially for the snorm variants,
> since it is able to get rid of the lshift/rshift for sext, as well as
> replacing each shift + mask with a single op.
>
> Signed-off-by: Ilia Mirkin
> ---
>
> v1 -> v2:
https://bugs.freedesktop.org/show_bug.cgi?id=91785
Vinson Lee changed:
What|Removed |Added
Keywords||bisected
CC|
On Thursday, August 27, 2015 11:50:52 AM Ben Widawsky wrote:
> Gen9 changes the meaning of this to coarse LOD quality mode. Although that's a
> desirable thing to be setting, it doesn't match the gen8 behavior and this was
> unintentional.
>
> This doesn't fix, or regress anything on SKL (AFAICT).
On Thu, Aug 20, 2015 at 6:00 PM, Ilia Mirkin wrote:
> It is fairly tricky to detect the proper conditions for using bitfield
> insert, but easy to just use it up front. This removes a lot of
> instructions on nvc0 when invoking the packing builtins.
>
> Signed-off-by: Ilia Mirkin
> ---
>
> Not su
On Thursday, August 27, 2015 11:50:51 AM Ben Widawsky wrote:
> As far as I can tell, the behavior is preserved from the previous generations.
> Before we set a single bit to tell the FS whether or not we'll be using an
> input
> coverage mask. Now we have some options which are implementing variou
On 28/08/15 08:06, Samuel Iglesias Gonsálvez wrote:
>
>
> On 28/08/15 02:21, Jordan Justen wrote:
>> On 2015-08-05 01:30:12, Iago Toral Quiroga wrote:
>>> From: Samuel Iglesias Gonsalvez
>>>
>>> Notice that Skylake needs to include a header in the sampler message
>>> so it will need some tweak
On 28/08/15 02:21, Jordan Justen wrote:
> On 2015-08-05 01:30:12, Iago Toral Quiroga wrote:
>> From: Samuel Iglesias Gonsalvez
>>
>> Notice that Skylake needs to include a header in the sampler message
>> so it will need some tweaks to work there.
>>
>> Signed-off-by: Samuel Iglesias Gonsalvez
On 28/08/15 02:04, Jordan Justen wrote:
> On 2015-08-05 01:30:12, Iago Toral Quiroga wrote:
>> From: Samuel Iglesias Gonsalvez
>>
>> Notice that Skylake needs to include a header in the sampler message
>> so it will need some tweaks to work there.
>>
>> Signed-off-by: Samuel Iglesias Gonsalvez
On 27/08/15 21:24, Jordan Justen wrote:
> On 2015-08-05 01:30:10, Iago Toral Quiroga wrote:
>> From: Samuel Iglesias Gonsalvez
>>
>> v2:
>> - Reduce the number of lines over 80 character line width
>> limit. (Thomas Hellan)
>>
>> Signed-off-by: Samuel Iglesias Gonsalvez
>> ---
>> src/glsl/lo
On 27/08/15 21:26, Jordan Justen wrote:
> On 2015-08-05 01:30:10, Iago Toral Quiroga wrote:
>> From: Samuel Iglesias Gonsalvez
>>
>> v2:
>> - Reduce the number of lines over 80 character line width
>> limit. (Thomas Hellan)
>>
>> Signed-off-by: Samuel Iglesias Gonsalvez
>> ---
>> src/glsl/lo
On Fri, Aug 28, 2015 at 1:46 AM, Kenneth Graunke wrote:
> On Thursday, August 27, 2015 11:48:36 PM Ilia Mirkin wrote:
>> Signed-off-by: Ilia Mirkin
>> ---
>> src/glsl/builtin_functions.cpp | 48
>> +++---
>> 1 file changed, 45 insertions(+), 3 deletions(-)
>>
https://bugs.freedesktop.org/show_bug.cgi?id=91785
Bug ID: 91785
Summary: make check DispatchSanity_test.GLES31 regression
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: All
Status: NEW
Keywords:
On Thursday, August 27, 2015 11:48:35 PM Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
> ---
> src/mesa/drivers/dri/i965/brw_defines.h | 3 +++
> src/mesa/drivers/dri/i965/brw_disasm.c | 1 +
> src/mesa/drivers/dri/i965/brw_fs.cpp | 7 +++
> src/mesa/driver
On Thursday, August 27, 2015 11:48:36 PM Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
> ---
> src/glsl/builtin_functions.cpp | 48
> +++---
> 1 file changed, 45 insertions(+), 3 deletions(-)
>
> diff --git a/src/glsl/builtin_functions.cpp b/src/glsl/builti
On Aug 27, 2015 10:33 PM, "Kenneth Graunke" wrote:
>
> On Thursday, August 27, 2015 10:17:24 PM Matt Turner wrote:
> > Noticed when debugging things that lead to the next patch.
> >
> > On G45 (and presumably ILK) this helps register coalescing:
> >
> > total instructions in shared programs: 40773
On Thursday, August 27, 2015 10:17:24 PM Matt Turner wrote:
> Noticed when debugging things that lead to the next patch.
>
> On G45 (and presumably ILK) this helps register coalescing:
>
> total instructions in shared programs: 4077373 -> 4077340 (-0.00%)
> instructions in affected programs:
No need to walk through instructions in blocks we know don't contain our
registers' live ranges.
---
View with git diff -w for easier review.
.../drivers/dri/i965/brw_fs_register_coalesce.cpp | 34 +-
1 file changed, 20 insertions(+), 14 deletions(-)
diff --git a/src/mesa/dr
start_to -> dst_start
end_to-> dst_end
start_from-> src_start
end_from -> src_end
var_to-> dst_var
var_from -> src_var
reg_to-> dst_reg
reg_to_offset -> dst_reg_offset
reg_from -> src_reg
Not sure how these made sense to me
No shader-db changes on g4x, snb, hsw, or bdw.
---
opt_peephole_sel() still offers some small benefits.
src/mesa/drivers/dri/i965/brw_fs.h | 1 -
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 2 -
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 89
3 files
Noticed when debugging things that lead to the next patch.
On G45 (and presumably ILK) this helps register coalescing:
total instructions in shared programs: 4077373 -> 4077340 (-0.00%)
instructions in affected programs: 43751 -> 43718 (-0.08%)
helped:52
HURT:
I always thought that the is_control_flow() -> return false check was a
bad hack, and some previous attempts to remove it have failed and have
been reverted.
The previous two patches fix some problems that caused register
coalescing to not notice some interference between registers, which the
is_c
equals() returns false for registers with different types, using it
isn't appropriate to determine whether an is overwriting a register.
---
src/mesa/drivers/dri/i965/brw_fs_register_coalesce.cpp | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_
Just an observation... this was previously only done for EG+, while
now R600/R700 will get this too.
On Fri, Aug 28, 2015 at 12:31 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> The geom shader rings require a VGT FLUSH, but up until now
> it was just hacked into the function, add proper support
From: Dave Airlie
The geom shader rings require a VGT FLUSH, but up until now
it was just hacked into the function, add proper support for it.
Signed-off-by: Dave Airlie
---
src/gallium/drivers/r600/evergreen_state.c | 9 +
src/gallium/drivers/r600/r600_hw_context.c | 5 +
src/gall
Will be used for textureSamples()
Signed-off-by: Ilia Mirkin
---
src/glsl/ir.cpp| 5 +++--
src/glsl/ir.h | 3 ++-
src/glsl/ir_clone.cpp | 1 +
src/glsl/ir_equals.cpp | 1 +
src/glsl/ir_hv_accept.cpp
Signed-off-by: Ilia Mirkin
---
src/glsl/nir/glsl_to_nir.cpp | 5 +
src/glsl/nir/nir.h | 4 +++-
src/glsl/nir/nir_print.c | 3 +++
3 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/src/glsl/nir/glsl_to_nir.cpp b/src/glsl/nir/glsl_to_nir.cpp
index 5fb4ee2..ed1a91e 100
Signed-off-by: Ilia Mirkin
---
src/glsl/builtin_functions.cpp | 48 +++---
1 file changed, 45 insertions(+), 3 deletions(-)
diff --git a/src/glsl/builtin_functions.cpp b/src/glsl/builtin_functions.cpp
index 3fdda32..b3982f4 100644
--- a/src/glsl/builtin_functi
Signed-off-by: Ilia Mirkin
---
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/drivers/dri/i965/intel_extensions.c
index c9eb24f..997f359 100644
--- a/src/mesa/drivers/dri/i965/intel_extensio
This should include everything. I sent a test for textureSamples to
piglit a while ago, not sure how to test imageSamples -- apparently ms
images aren't entirely supported on i965? But I'm not sure via what
feat that happens.
I'll send some tests for compiler tests though.
Ilia Mirkin (10):
mes
Signed-off-by: Ilia Mirkin
---
src/glsl/builtin_functions.cpp | 31 +++
1 file changed, 31 insertions(+)
diff --git a/src/glsl/builtin_functions.cpp b/src/glsl/builtin_functions.cpp
index 5e05199..3fdda32 100644
--- a/src/glsl/builtin_functions.cpp
+++ b/src/glsl/buil
Signed-off-by: Ilia Mirkin
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index 172e29f..0b13e74 100644
--- a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
+++ b/
Signed-off-by: Ilia Mirkin
---
src/glsl/nir/glsl_to_nir.cpp | 6 +-
src/glsl/nir/nir_intrinsics.h | 2 ++
2 files changed, 7 insertions(+), 1 deletion(-)
diff --git a/src/glsl/nir/glsl_to_nir.cpp b/src/glsl/nir/glsl_to_nir.cpp
index ed1a91e..fb90ade 100644
--- a/src/glsl/nir/glsl_to_nir.cpp
Signed-off-by: Ilia Mirkin
---
src/glsl/glcpp/glcpp-parse.y| 3 +++
src/glsl/glsl_parser_extras.cpp | 1 +
src/glsl/glsl_parser_extras.h | 2 ++
3 files changed, 6 insertions(+)
diff --git a/src/glsl/glcpp/glcpp-parse.y b/src/glsl/glcpp/glcpp-parse.y
index 2d631f0..118333d 100644
--- a/src
Signed-off-by: Ilia Mirkin
---
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 a57d5ba..1f7d542 100644
--- a/src/mesa/main/extensions.c
+++ b/src/mesa/main/extensions.c
Signed-off-by: Ilia Mirkin
---
src/mesa/drivers/dri/i965/brw_defines.h | 3 +++
src/mesa/drivers/dri/i965/brw_disasm.c | 1 +
src/mesa/drivers/dri/i965/brw_fs.cpp | 7 +++
src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 4
src/mesa/drivers/dri/i965/b
On Thu, 2015-08-27 at 23:08 -0400, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
Reviewed-by: Timothy Arceri
> ---
> src/glsl/builtin_functions.cpp | 5 +
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
> diff --git a/src/glsl/builtin_functions.cpp b/src/glsl/builtin_functions.cpp
>
On Thu, Aug 27, 2015 at 11:14 PM, Timothy Arceri wrote:
> On Thu, 2015-08-27 at 15:43 -0400, Ilia Mirkin wrote:
>> Cube maps are special in that they have separate teximages for each
>> face. We handled that by copying the data to them separately, but in
>> case zoffset != 0 or depth != 6 we would
On Thu, 2015-08-27 at 15:43 -0400, Ilia Mirkin wrote:
> Cube maps are special in that they have separate teximages for each
> face. We handled that by copying the data to them separately, but in
> case zoffset != 0 or depth != 6 we would read off the end of the client
> array or modify the wrong im
Signed-off-by: Ilia Mirkin
---
src/glsl/builtin_functions.cpp | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/src/glsl/builtin_functions.cpp b/src/glsl/builtin_functions.cpp
index 4092d68..5e05199 100644
--- a/src/glsl/builtin_functions.cpp
+++ b/src/glsl/builtin_functions
From: Dave Airlie
This code we broken by the tess merge, and I totally missed it
until now. I'm not sure this fixes anything but it stops the assert.
Cc: "11.0"
Signed-off-by: Dave Airlie
---
src/gallium/drivers/r600/r600_pipe.h | 31 ---
1 file changed, 16 inserti
On Thu, Aug 27, 2015 at 4:56 PM, Ilia Mirkin wrote:
> The ffsll patch seems dodgy... among other things the same logic is in
> imports.c and thus would probably best refactored to src/util, but
> additionally it's only defined if !_GNU_SOURCE -- is that OK? Would
> e.g. a 32-bit build have ffsll?
On Thu, Aug 27, 2015 at 8:19 PM, Brian Paul wrote:
> On 08/27/2015 06:01 PM, Ilia Mirkin wrote:
>>
>> Are you sure you want to enable it in spite of the fact that it
>> doesn't handle swizzling (which you've slightly papered over by
>> switching some of the format orders around, but as I pointed o
On 2015-08-05 01:30:12, Iago Toral Quiroga wrote:
> From: Samuel Iglesias Gonsalvez
>
> Notice that Skylake needs to include a header in the sampler message
> so it will need some tweaks to work there.
>
> Signed-off-by: Samuel Iglesias Gonsalvez
> ---
> src/mesa/drivers/dri/i965/brw_defines.h
On 08/27/2015 06:01 PM, Ilia Mirkin wrote:
Are you sure you want to enable it in spite of the fact that it
doesn't handle swizzling (which you've slightly papered over by
switching some of the format orders around, but as I pointed out not
completely)?
As I said before, I'd like to get as much
On 08/27/2015 05:59 PM, Ilia Mirkin wrote:
You need to rip out the st_CopyImageSubData I already added in
st_cb_texture.c (I think).
Yes, I created my feature branch before you added that code. I'll have
to rebase onto master. A patch to remove your code will be trivial.
-Brian
On Thu,
On 08/27/2015 05:56 PM, Ilia Mirkin wrote:
The ffsll patch seems dodgy... among other things the same logic is in
imports.c and thus would probably best refactored to src/util,
I can do that in a follow-on someday. There was very little in
src/util/ back when I added that code to u_math.h s
On 2015-08-05 01:30:12, Iago Toral Quiroga wrote:
> From: Samuel Iglesias Gonsalvez
>
> Notice that Skylake needs to include a header in the sampler message
> so it will need some tweaks to work there.
>
> Signed-off-by: Samuel Iglesias Gonsalvez
> ---
> src/mesa/drivers/dri/i965/brw_defines.h
Are you sure you want to enable it in spite of the fact that it
doesn't handle swizzling (which you've slightly papered over by
switching some of the format orders around, but as I pointed out not
completely)?
On Thu, Aug 27, 2015 at 7:48 PM, Brian Paul wrote:
> This enables GL_ARB_copy_image for
From: Dave Airlie
Otherwise this will crash on 32-bit, and it gets rid of
warnings building on 32-bit.
Signed-off-by: Dave Airlie
---
src/gallium/drivers/r600/compute_memory_pool.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/r600/compute_memo
From: Dave Airlie
On 32-bit we need to use PRIu64 flags for printfs,
otherwise this segfaults in R600_DEBUG=help otherwise.
Cc: "11.0"
Signed-off-by: Dave Airlie
---
src/gallium/auxiliary/util/u_debug.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/src/gallium/aux
You need to rip out the st_CopyImageSubData I already added in
st_cb_texture.c (I think).
On Thu, Aug 27, 2015 at 7:47 PM, Brian Paul wrote:
> From: Neha Bhende
>
> Basically, get the src/dst pipe_resource surfaces and call the
> pipe_context::resource_copy_region() function.
>
> Reviewed-by: Ma
The ffsll patch seems dodgy... among other things the same logic is in
imports.c and thus would probably best refactored to src/util, but
additionally it's only defined if !_GNU_SOURCE -- is that OK? Would
e.g. a 32-bit build have ffsll?
Perhaps it'd make sense to post the generic patches to the l
In principle, since all gallium drivers implement pipe_context::
resource_copy_region() they should all support GL_ARB_copy_image but until
that's true and verified, drivers must opt-in to support this feature.
Reviewed-by: Marek Olšák
---
src/gallium/drivers/freedreno/freedreno_screen.c | 1 +
We have to disallow the 8 and 16-bit 3-component formats for non-display
target surfaces as we do for softpipe (see earlier commit).
v2: Simplify 3-component format check, per Marek.
Also check that target != PIPE_BUFFER
---
src/gallium/drivers/llvmpipe/lp_screen.c | 17 -
1 file
Reviewed-by: Marek Olšák
---
src/mesa/state_tracker/st_extensions.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/mesa/state_tracker/st_extensions.c
b/src/mesa/state_tracker/st_extensions.c
index 13d636f..dfa8936 100644
--- a/src/mesa/state_tracker/st_extensions.c
+++ b/src/mesa/state_
1. Try to choose R8G8B8A8 unorm/srgb formats before other orders in an
effort to try to match component ordering for UINT/SINT/etc.
2. If we can't get a format such as PIPE_FORMAT_A16_UNORM, try
PIPE_FORMAT_R16G16B16A16_UNORM before shallower formats.
Reviewed-by: Marek Olšák
---
src/mesa/state
Previously, core Mesa's _mesa_CopyImageSubData() created temporary textures
to wrap renderbuffer sources/destinations. This caused a bit of a mess in
the Mesa/gallium state tracker because we had to basically undo that
wrapping.
Instead, change ctx->Driver.CopyImageSubData() to take both gl_rende
From: Neha Bhende
Basically, get the src/dst pipe_resource surfaces and call the
pipe_context::resource_copy_region() function.
Reviewed-by: Marek Olšák
---
src/mesa/Makefile.sources| 2 +
src/mesa/state_tracker/st_cb_copyimage.c | 94
src/mesa
Mesa and gallium don't have a complete set of matching 3-component
texture formats. For example, 8-bit sRGB unorm. To fully support
the GL_ARB_copy_image extension we need to have support for all of
these formats: RGB8_UNORM, RGB8_SNORM, RGB8_SRGB, RGB8_UINT, and
RGB8_SINT using the same componen
This enables GL_ARB_copy_image for softpipe.
---
src/gallium/drivers/softpipe/sp_screen.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/softpipe/sp_screen.c
b/src/gallium/drivers/softpipe/sp_screen.c
index 817572c..4cd8581 100644
--- a/src/gallium/drive
This primarily means added support for copying between compressed
and uncompressed formats.
---
src/gallium/auxiliary/util/u_surface.c | 106 +++--
1 file changed, 88 insertions(+), 18 deletions(-)
diff --git a/src/gallium/auxiliary/util/u_surface.c
b/src/gallium/auxi
If you've seen the announcement of VMware Workstation 12 / Fusion 8, we
now support OpenGL 3.3 in the guest.
The Mesa/gallium patches for this can be seen in my git repo at
cgit.freedesktop.org/~brianp/mesa/log/?h=vmware-gl3
The first 15 patches or so are general gallium util/tgsi/etc chang
https://bugs.freedesktop.org/show_bug.cgi?id=91783
Brian Paul changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
Some of the commit messages, like this one, seem to line-wrapped shorter
than necessary.
For patches 1-4, the changes look OK, AFAICT. Though, it seems there's
some very similar byte-swap code duplicated in several places. Would it
be possible to write a function which does byte swapping for
https://bugs.freedesktop.org/show_bug.cgi?id=91783
--- Comment #3 from Arsen Tufankjian ---
Oops you're right I didn't RTFM. This can be closed for now.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
https://bugs.freedesktop.org/show_bug.cgi?id=91783
--- Comment #2 from Brian Paul ---
I don't recall seeing any patches from you on the mesa-dev list yet.
Generally, we don't give git commit access until the developer has some track
record of producing good patches.
--
You are receiving this ma
https://bugs.freedesktop.org/show_bug.cgi?id=91783
--- Comment #1 from Arsen Tufankjian ---
Created attachment 117956
--> https://bugs.freedesktop.org/attachment.cgi?id=117956&action=edit
GPG Public Key
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the as
https://bugs.freedesktop.org/show_bug.cgi?id=91783
Bug ID: 91783
Summary: Requesting Account
Product: Mesa
Version: git
Hardware: Other
OS: Linux (All)
Status: NEW
Severity: normal
Priority: medi
On Thu, Aug 27, 2015 at 11:50:51AM -0700, Ben Widawsky wrote:
> As far as I can tell, the behavior is preserved from the previous generations.
> Before we set a single bit to tell the FS whether or not we'll be using an
> input
> coverage mask. Now we have some options which are implementing vario
On Thu, Aug 27, 2015 at 3:17 AM, Marta Lofstedt
wrote:
> From: Marta Lofstedt
>
> V2: Conform to new standard for exposing enums for OpenGL ES 3.1.
>
> Signed-off-by: Marta Lofstedt
Reviewed-by: Ilia Mirkin
While this patch is fine, it appears that ES3.1 doesn't have
GL_FRAMEBUFFER_DEFAULT_LA
Hello Glenn,
with your series I got some steps forward on this one:
[r600g] regression: SB-related NI/Turks crash on 'gsraytrace'
https://bugs.freedesktop.org/show_bug.cgi?id=91503
Crash is GONE.
GPU hang is BACK _without_ R600_DEBUG=nosb.
With R600_DEBUG=nosb
vsraytrace, fsraytrace and gsrayt
On 27 August 2015 at 18:35, Ilia Mirkin wrote:
> On Thu, Aug 27, 2015 at 1:11 PM, Emil Velikov
> wrote:
>> HI Julien,
>>
>> On 27 August 2015 at 15:15, Julien Isorce wrote:
>>> Currently nouveau does not support chunk decoding
>>> which is required to support st/va
>>>
>>> The following patches
On 08/27/2015 03:15 PM, Ilia Mirkin wrote:
On Thu, Aug 27, 2015 at 5:15 PM, Brian Paul wrote:
On 08/27/2015 03:05 PM, Ilia Mirkin wrote:
On Thu, Aug 27, 2015 at 4:43 PM, Ilia Mirkin wrote:
On Thu, Aug 27, 2015 at 4:32 PM, Brian Paul wrote:
On 08/27/2015 01:43 PM, Ilia Mirkin wrote:
C
Reviewed-by: Ilia Mirkin
On Thu, Aug 27, 2015 at 5:16 PM, Brian Paul wrote:
> ---
> src/mesa/main/teximage.c | 8
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
> index 8d94903..3647854 100644
> --- a/src/mesa/main
---
src/mesa/main/teximage.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
index 8d94903..3647854 100644
--- a/src/mesa/main/teximage.c
+++ b/src/mesa/main/teximage.c
@@ -3764,7 +3764,7 @@ texturesubimage(struct gl_c
On Thu, Aug 27, 2015 at 5:15 PM, Brian Paul wrote:
> On 08/27/2015 03:05 PM, Ilia Mirkin wrote:
>>
>> On Thu, Aug 27, 2015 at 4:43 PM, Ilia Mirkin wrote:
>>>
>>> On Thu, Aug 27, 2015 at 4:32 PM, Brian Paul wrote:
On 08/27/2015 01:43 PM, Ilia Mirkin wrote:
>
>
> Cube maps ar
On 08/27/2015 03:05 PM, Ilia Mirkin wrote:
On Thu, Aug 27, 2015 at 4:43 PM, Ilia Mirkin wrote:
On Thu, Aug 27, 2015 at 4:32 PM, Brian Paul wrote:
On 08/27/2015 01:43 PM, Ilia Mirkin wrote:
Cube maps are special in that they have separate teximages for each
face. We handled that by copying t
On Thu, Aug 27, 2015 at 4:43 PM, Ilia Mirkin wrote:
> On Thu, Aug 27, 2015 at 4:32 PM, Brian Paul wrote:
>> On 08/27/2015 01:43 PM, Ilia Mirkin wrote:
>>>
>>> Cube maps are special in that they have separate teximages for each
>>> face. We handled that by copying the data to them separately, but
On Thu, Aug 27, 2015 at 4:32 PM, Brian Paul wrote:
> On 08/27/2015 01:43 PM, Ilia Mirkin wrote:
>>
>> Cube maps are special in that they have separate teximages for each
>> face. We handled that by copying the data to them separately, but in
>> case zoffset != 0 or depth != 6 we would read off the
On 08/27/2015 01:43 PM, Ilia Mirkin wrote:
Cube maps are special in that they have separate teximages for each
face. We handled that by copying the data to them separately, but in
case zoffset != 0 or depth != 6 we would read off the end of the client
array or modify the wrong images.
zoffset/de
Yes, I think it's useful too. I'm not familiar with this code, so you
can add
Acked-by: Marek Olšák
and commit if there is no reviewer.
Marek
On Thu, Aug 27, 2015 at 9:48 PM, Dave Airlie wrote:
> On 27 August 2015 at 21:57, Marek Olšák wrote:
>> We could just skip this for radeonsi or any dr
On 27 August 2015 at 21:57, Marek Olšák wrote:
> We could just skip this for radeonsi or any driver that does regalloc,
> because it's useless there.
I did expect this comment from Ilia, though adding a CAP and
piping it through is an option I'd look into later. (or someone who
cares can do it so
Cube maps are special in that they have separate teximages for each
face. We handled that by copying the data to them separately, but in
case zoffset != 0 or depth != 6 we would read off the end of the client
array or modify the wrong images.
zoffset/depth have already been verified by the time th
On 2015-08-05 01:30:10, Iago Toral Quiroga wrote:
> From: Samuel Iglesias Gonsalvez
>
> v2:
> - Reduce the number of lines over 80 character line width
> limit. (Thomas Hellan)
>
> Signed-off-by: Samuel Iglesias Gonsalvez
> ---
> src/glsl/lower_ubo_reference.cpp | 190
>
On 2015-08-05 01:30:10, Iago Toral Quiroga wrote:
> From: Samuel Iglesias Gonsalvez
>
> v2:
> - Reduce the number of lines over 80 character line width
> limit. (Thomas Hellan)
>
> Signed-off-by: Samuel Iglesias Gonsalvez
> ---
> src/glsl/lower_ubo_reference.cpp | 190
>
On Thu, Aug 27, 2015 at 11:50:52AM -0700, Ben Widawsky wrote:
Subject should read gen9. Fixed locally
> Gen9 changes the meaning of this to coarse LOD quality mode. Although that's a
> desirable thing to be setting, it doesn't match the gen8 behavior and this was
> unintentional.
>
> This doesn'
On Wednesday, August 26, 2015 10:24:04 AM Jason Ekstrand wrote:
> These two patches are largely a re-send of other patches in my bigger
> series to push small uniform arrays. The first patch is a really nice
> cleanup that I think we want regardless of pushing uniform arrays.
> Unfortunately, it w
On Thu, Aug 27, 2015 at 11:45 AM, Jason Ekstrand wrote:
> On Tue, Aug 25, 2015 at 1:24 PM, Kenneth Graunke
> wrote:
>> This is a lot more reasonable than returning an offset from NULL.
>>
>> Signed-off-by: Kenneth Graunke
>> ---
>> src/glsl/nir/nir.h | 4 ++--
>> 1 file changed, 2 insertions(+
As far as I can tell, the behavior is preserved from the previous generations.
Before we set a single bit to tell the FS whether or not we'll be using an input
coverage mask. Now we have some options which are implementing various
extensions. These bits are used for the various conservative rasteri
Gen9 changes the meaning of this to coarse LOD quality mode. Although that's a
desirable thing to be setting, it doesn't match the gen8 behavior and this was
unintentional.
This doesn't fix, or regress anything on SKL (AFAICT).
Signed-off-by: Ben Widawsky
---
src/mesa/drivers/dri/i965/brw_sampl
On Tue, Aug 25, 2015 at 1:24 PM, Kenneth Graunke wrote:
> This is a lot more reasonable than returning an offset from NULL.
>
> Signed-off-by: Kenneth Graunke
> ---
> src/glsl/nir/nir.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/glsl/nir/nir.h b/src/glsl/nir
On Thu, Aug 27, 2015 at 1:59 PM, Alex Deucher wrote:
> On Thu, Aug 27, 2015 at 1:55 PM, Hans de Goede wrote:
>> Hi,
>>
>> On 27-08-15 15:46, Marek Olšák wrote:
>>>
>>> On Thu, Aug 27, 2015 at 3:09 PM, Hans de Goede
>>> wrote:
Hi All,
While debugging: https://bugzilla.redhat.c
1 - 100 of 137 matches
Mail list logo
