From: Marta Lofstedt
V2: Conform to new standard for exposing enums for OpenGL ES 3.1.
Signed-off-by: Marta Lofstedt
---
src/mapi/glapi/gen/ARB_framebuffer_no_attachments.xml | 4 ++--
src/mapi/glapi/gen/apiexec.py | 4 ++--
src/mesa/main/get_hash_params.py
On Wed, Aug 26, 2015 at 03:46:05PM -0700, Ben Widawsky wrote:
> This reverts commit 1bba29ed403e735ba0bf04ed8aa2e571884fcaaf
> Author: Topi Pohjolainen
> Date: Thu Jun 25 14:00:41 2015 +0300
>
> i965: Stop aux data compare preventing program binary re-use
>
> This fixes an intermittent fai
On 07/30/2015 05:05 PM, Predut, Marius wrote:
> Was sent 2 month ago, still not review/upstream
>
> -Original Message-
> From: Predut, Marius
> Sent: Thursday, July 30, 2015 7:04 PM
> To: mesa-dev@lists.freedesktop.org
> Cc: Predut, Marius
> Subject: [Mesa-dev][PATCH] i915/aa: fixing anti
On Wed, Aug 26, 2015 at 06:19:52PM +0100, Ian Romanick wrote:
> From: Ian Romanick
>
> All of the other state upload functions are static because the only use
> is in the brw_tracked_state structure.
If and when I get my "i965-meta-ops reusing blorp launch"-series to be
available again, I need t
On Mon, Aug 24, 2015 at 05:04:22PM -0600, Brian Paul wrote:
> 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.
>
https://bugs.freedesktop.org/show_bug.cgi?id=91711
--- Comment #1 from YuGiOhJCJ ---
OK I found how to solve this problem.
Instead of using the libopengl32.a file provided by mesa I need to use the one
provided by my cross-compiler.
The problem is that my current cross-compiler [1] does not provi
Reviewed-by: Marek Olšák
Marek
On Wed, Aug 26, 2015 at 10:55 PM, Brian Paul wrote:
> Previously, we used PROGRAM_ARRAY only for variables which were
> arrays or matrices. But if the variable is a structure containing
> an array or matrix, we need to use PROGRAM_ARRAY for that too.
>
> Before,
On 08/26/2015 08:19 PM, Ian Romanick wrote:
> From: Ian Romanick
>
> All of the other state upload functions are static because the only use
> is in the brw_tracked_state structure.
>
> Signed-off-by: Ian Romanick
This patch is
Reviewed-by: Abdiel Janulgue
> ---
> src/mesa/drivers/dri/i9
On 08/27/2015 09:19 AM, Pohjolainen, Topi wrote:
> On Wed, Aug 26, 2015 at 06:19:52PM +0100, Ian Romanick wrote:
>> From: Ian Romanick
>>
>> All of the other state upload functions are static because the only use
>> is in the brw_tracked_state structure.
>
> If and when I get my "i965-meta-ops re
On 08/26/2015 10:11 PM, Jason Ekstrand wrote:
> On Wed, Aug 26, 2015 at 10:20 AM, Ian Romanick wrote:
>> From: Ian Romanick
>>
>> Signed-off-by: Ian Romanick
>> ---
>> src/glsl/nir/nir_constant_expressions.py | 13 +
>> 1 file changed, 5 insertions(+), 8 deletions(-)
>>
>> diff --gi
Reviewed-by: Marek Olšák
Marek
On Thu, Aug 27, 2015 at 2:02 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> Instead of looking this up lots, lets just cache it in the instruction
> translation up front. I just noticed this function what high in a profile
> of shader-db on radeonsi.
>
> Signed-o
On 08/26/2015 06:44 PM, Jason Ekstrand wrote:
> I think I'd rather just run it through indent like we do for some of
> the other auto-generated code. That way the mako file (the one you'll
> actually be editing) can remain readable.
I had thought about doing that. We use indent in a few places w
We could just skip this for radeonsi or any driver that does regalloc,
because it's useless there.
Marek
On Thu, Aug 27, 2015 at 5:30 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> The glsl->tgsi convertor does some temporary register reduction
> however in profiling shader-db this shows up qui
Hi All,
While debugging: https://bugzilla.redhat.com/show_bug.cgi?id=1008089
I made a apitrace recording of the a single slide transition
animation, and since I suspected memory corruption replayed
it using ElectrFence + glretrace, this finds a 0 sized array
allocation at src/mesa/state_tracker/
On Aug 27, 2015 4:08 AM, "Ian Romanick" wrote:
>
> On 08/26/2015 10:11 PM, Jason Ekstrand wrote:
> > On Wed, Aug 26, 2015 at 10:20 AM, Ian Romanick
wrote:
> >> From: Ian Romanick
> >>
> >> Signed-off-by: Ian Romanick
> >> ---
> >> src/glsl/nir/nir_constant_expressions.py | 13 +
> >
On Thu, Aug 27, 2015 at 3:09 PM, Hans de Goede wrote:
> Hi All,
>
> While debugging: https://bugzilla.redhat.com/show_bug.cgi?id=1008089
>
> I made a apitrace recording of the a single slide transition
> animation, and since I suspected memory corruption replayed
> it using ElectrFence + glretrace
https://bugs.freedesktop.org/show_bug.cgi?id=89969
Signed-off-by: Julien Isorce
---
src/gallium/drivers/nouveau/nouveau_vp3_video.h| 6 --
.../drivers/nouveau/nouveau_vp3_video_bsp.c| 79 --
src/gallium/drivers/nouveau/nv50/nv98_video_bsp.c | 5 +-
src/gallium/
https://bugs.freedesktop.org/show_bug.cgi?id=89969
Signed-off-by: Julien Isorce
---
src/gallium/drivers/nouveau/nouveau_vp3_video_bsp.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/drivers/nouveau/nouveau_vp3_video_bsp.c
b/src/gallium/drivers/nouveau/nouveau_vp3_video_bsp
This patch add 3 functions to split nouveau_vp3_bsp:
nouveau_vp3_bsp_begin
nouveau_vp3_bsp_next
nouveau_vp3_bsp_end
https://bugs.freedesktop.org/show_bug.cgi?id=89969
Signed-off-by: Julien Isorce
---
src/gallium/drivers/nouveau/nouveau_vp3_video.h| 15 +++
.../drivers/nouveau/nouveau_vp3_
https://bugs.freedesktop.org/show_bug.cgi?id=89969
Signed-off-by: Julien Isorce
---
src/gallium/drivers/nouveau/nvc0/nvc0_video_bsp.c | 34 +++
1 file changed, 28 insertions(+), 6 deletions(-)
diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_video_bsp.c
b/src/gallium/drive
Currently nouveau does not support chunk decoding
which is required to support st/va
The following patches refactor nouveau_vp3_bsp
and nvc0_decoder_bsp in order to implement
pipe_video_codec::begin_frame/decode_bitstream/end_frame.
So that decode_bitstream can be call multiple times
between each
It will allow to call nvc0_decoder_bsp_next's content
mutlitple times between one begin/end.
https://bugs.freedesktop.org/show_bug.cgi?id=89969
Signed-off-by: Julien Isorce
---
src/gallium/drivers/nouveau/nvc0/nvc0_video.h | 14 ++
src/gallium/drivers/nouveau/nvc0/nvc0_video_bsp.c | 197 ++
vainfo fails in vaDriverInit because "dd_create_screen"
does not reach strcmp(driver_name, "nouveau") code.
Indeed when compiling the va target.c, the macro GALLIUM_NOUVEAU
is not defined.
This patch define the macro the same it is done for dri and
vdpau targets.
Tested with:
./autogen.sh --enable
In order to add support for chunck decoding.
Required to support st/va.
https://bugs.freedesktop.org/show_bug.cgi?id=89969
Signed-off-by: Julien Isorce
---
src/gallium/drivers/nouveau/nvc0/nvc0_video.c | 48 +--
1 file changed, 37 insertions(+), 11 deletions(-)
diff --gi
https://bugs.freedesktop.org/show_bug.cgi?id=89969
Signed-off-by: Julien Isorce
---
src/gallium/drivers/nouveau/nvc0/nvc0_video.c | 12 +-
src/gallium/drivers/nouveau/nvc0/nvc0_video.h | 8 --
src/gallium/drivers/nouveau/nvc0/nvc0_video_bsp.c | 167 --
3 files chan
On 08/27/2015 02:28 AM, Pohjolainen, Topi wrote:
On Mon, Aug 24, 2015 at 05:04:22PM -0600, Brian Paul wrote:
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 becau
instead of call prog->getTarget(), since the target never change in prog.
Signed-off-by: Marcos Paulo de Souza
---
src/gallium/drivers/nouveau/codegen/nv50_ir.cpp | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir.cpp
b/src/gall
https://bugs.freedesktop.org/show_bug.cgi?id=91747
--- Comment #2 from Sinclair Yeh ---
Can you provide output from glxinfo?
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
___
mesa-dev mai
Shaders that contain instruction data after an instruction with EOP could end
up parsing that as an instruction, leading to various crashes and asserts in
SB as it gets very confused if it sees for instance a loop start instruction
jumping off to some random point.
Add a couple of asserts, and pri
e8e443 missed adding check for undef values also in
unreserve function, leading to an assert triggering.
Signed-off-by: Glenn Kennard
---
src/gallium/drivers/r600/sb/sb_sched.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/r600/sb/sb_sched.cpp
b/src/g
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
> > Date: Thu Jun 25 14:00:41 2015 +0300
> >
> > i965: Stop aux da
Signed-off-by: Glenn Kennard
---
src/gallium/drivers/r600/sb/sb_bc_parser.cpp | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/r600/sb/sb_bc_parser.cpp
b/src/gallium/drivers/r600/sb/sb_bc_parser.cpp
index 748aae2..c479927 100644
--- a/src/gallium/driver
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 refactor nouveau_vp3_bsp
> and nvc0_decoder_bsp in order to implement
> pipe_video_codec::begin_frame/decode_bitstream/end
Hi Glenn,
On 27 August 2015 at 18:04, Glenn Kennard wrote:
> e8e443 missed adding check for undef values also in
> unreserve function, leading to an assert triggering.
>
Can you please add the mesa-stable tag before pushing this series.
Please also add 10.6 if applicable.
Thanks
Emil
___
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 refactor nouveau_vp3_bsp
>> and nvc0_decoder_bsp in order t
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.com/show_bug.cgi?id=1008089
I made a apitrace recording of the a single slide transition
animation, and since I suspected memory corruption replaye
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.com/show_bug.cgi?id=1008089
>>>
>>> I made a apitrace recording of
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
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
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
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
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(+
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: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 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
>
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 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
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 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
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 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 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 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
---
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
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
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
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
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 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
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
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
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
--- 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 #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.
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
Brian Paul changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
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
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
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
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
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
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
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_
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
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 +
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
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
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
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
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
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 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 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 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 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 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 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?
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
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
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
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 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
>
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_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/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
1 - 100 of 137 matches
Mail list logo
