On 20.04.2016 01:32, Bas Nieuwenhuizen wrote:
> URL:
> http://cgit.freedesktop.org/mesa/mesa/commit/?id=464cef5b06e65aa740704e4adac68b7f5fee1b88
> Author: Bas Nieuwenhuizen
> Date: Sat Mar 19 15:16:50 2016 +0100
>
> radeonsi: enable TGSI support cap for compute shaders
This change brok
On Tue, 2016-04-19 at 15:32 -0700, Jason Ekstrand wrote:
>
>
> On Tue, Apr 12, 2016 at 1:05 AM, Samuel Iglesias Gonsálvez
> wrote:
> From: Iago Toral Quiroga
>
> At least i965 hardware does not have native support for
> truncating doubles.
> ---
>
On 2016-04-20 11:46, Nicolai Hähnle wrote:
On 19.04.2016 17:50, Bas Nieuwenhuizen wrote:
Signed-off-by: Bas Nieuwenhuizen
---
src/gallium/drivers/radeonsi/si_compute.c | 5 -
src/gallium/drivers/radeonsi/si_state_shaders.c | 5 -
2 files changed, 8 insertions(+), 2 deletions(-
On 19/04/16 23:52, Jason Ekstrand wrote:
> On Tue, Apr 12, 2016 at 1:05 AM, Samuel Iglesias Gonsálvez <
> sigles...@igalia.com> wrote:
>
>> From: Connor Abbott
>>
>> v2: Move to compiler/nir (Iago)
>>
>> Signed-off-by: Iago Toral Quiroga
>> ---
>> src/compiler/Makefile.sources | 1 +
Now Windows users have the same doxygen files than *nix users
Not tested (I don't have a windows)
Signed-off-by: Elie TOURNIER
---
doxygen/doxy.bat | 7 +++
1 file changed, 7 insertions(+)
diff --git a/doxygen/doxy.bat b/doxygen/doxy.bat
index e566ca3..408964e 100644
--- a/doxygen/doxy.bat
On Tue, Apr 19, 2016 at 6:45 PM, Connor Abbott wrote:
> On Tue, Apr 12, 2016 at 4:05 AM, Samuel Iglesias Gonsálvez
> wrote:
> > From: Connor Abbott
> >
> > v2: Move to compiler/nir (Iago)
> >
> > Signed-off-by: Iago Toral Quiroga
> > ---
> > src/compiler/Makefile.sources | 1 +
> >
https://bugs.freedesktop.org/show_bug.cgi?id=94957
--- Comment #9 from Roland Scheidegger ---
So, the r32i/ui failures are actually due to an overflow.
One example I've seen samples a rgb8 unorm texture, scales to int range,
converts to int and outputs this. The problem is that rescaling to 2^31
On Tue, Apr 19, 2016 at 8:12 PM, Jason Ekstrand
wrote:
>
> On Apr 19, 2016 6:10 PM, "Emil Velikov" wrote:
> >
> > On 19 April 2016 at 23:18, Chad Versace wrote:
> > > On Tue 19 Apr 2016, Emil Velikov wrote:
> > >> On 16 April 2016 at 20:45, Jason Ekstrand
> wrote:
> > >> > C++ doesn't support
On Apr 19, 2016 6:10 PM, "Emil Velikov" wrote:
>
> On 19 April 2016 at 23:18, Chad Versace wrote:
> > On Tue 19 Apr 2016, Emil Velikov wrote:
> >> On 16 April 2016 at 20:45, Jason Ekstrand wrote:
> >> > C++ doesn't support designated initializers and g++ in particular
doesn't
> >> > handle them
On Tue, Apr 19, 2016 at 8:43 PM, Rob Clark wrote:
> On Tue, Apr 19, 2016 at 9:03 PM, Emil Velikov
> wrote:
>> Hi Rob,
>>
>> Please bear in mind that there's a fair bit of comments, but before
>> all don't mix refactoring and new code. Please ?
>>
>> On 15 April 2016 at 17:03, Rob Herring wrote
On Tue, Apr 19, 2016 at 8:03 PM, Emil Velikov wrote:
> Hi Rob,
>
> Please bear in mind that there's a fair bit of comments, but before
> all don't mix refactoring and new code. Please ?
Okay.
> On 15 April 2016 at 17:03, Rob Herring wrote:
>> Add support for creating images from Android native
On Tue, Apr 19, 2016 at 07:00:18PM +0100, Emil Velikov wrote:
> On 19 April 2016 at 04:03, Jonathan Gray wrote:
> > Any objections to this?
> >
> Absolutely none. I simply missed it.
>
> > On Mon, Dec 21, 2015 at 04:39:55PM +1100, Jonathan Gray wrote:
> >> Use dev_node_from_fd() with HAVE_LIBDRM
LOCAL_C_INCLUDES needs an additional path to fix the following build error:
external/mesa/src/compiler/glsl/glcpp/glcpp-lex.l:30:25: fatal error:
glcpp-parse.h: No such file or directory
#include "glcpp-parse.h"
^
compilation terminated.
build/core/binary.mk:821: recipe
On Tue, Apr 12, 2016 at 4:05 AM, Samuel Iglesias Gonsálvez
wrote:
> From: Connor Abbott
>
> v2: Move to compiler/nir (Iago)
>
> Signed-off-by: Iago Toral Quiroga
> ---
> src/compiler/Makefile.sources | 1 +
> src/compiler/nir/nir.h | 7 +
> src/compiler/nir/nir_lo
On 19.04.2016 17:50, Bas Nieuwenhuizen wrote:
Signed-off-by: Bas Nieuwenhuizen
---
src/gallium/drivers/radeonsi/si_compute.c | 5 -
src/gallium/drivers/radeonsi/si_state_shaders.c | 5 -
2 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi
On Tue, Apr 19, 2016 at 9:03 PM, Emil Velikov wrote:
> Hi Rob,
>
> Please bear in mind that there's a fair bit of comments, but before
> all don't mix refactoring and new code. Please ?
>
> On 15 April 2016 at 17:03, Rob Herring wrote:
>> Add support for creating images from Android native buffe
On 19 April 2016 at 23:18, Chad Versace wrote:
> On Tue 19 Apr 2016, Emil Velikov wrote:
>> On 16 April 2016 at 20:45, Jason Ekstrand wrote:
>> > C++ doesn't support designated initializers and g++ in particular doesn't
>> > handle them when the struct gets complicated, i.e. has a union.
>> > ---
On Tue, Apr 19, 2016 at 08:12:08PM +0200, Michael Schellenberger Costa wrote:
> Hi Tom,
>
> Am 19.04.2016 um 19:52 schrieb Tom Stellard:
> > The range metadata tells LLVM the range of expected values for this
> > intrinsic,
> > so it can do some additional optimizations on the result.
> > ---
> >
https://bugs.freedesktop.org/show_bug.cgi?id=92850
--- Comment #63 from Michel Dänzer ---
(In reply to Iaroslav Andrusyak from comment #61)
> Works for me with aces -safe and with any settings but without shadows
> enabled, with shadows i got
>
> state_tracker/st_cb_fbo.c:431:st_update_renderbu
On 2016-04-20 09:29, Bas Nieuwenhuizen wrote:
I retract patch 1 and 2. Large scratch buffers are nice, but the
hardware only supports a 32-bit offset into it.
- Bas
On Wed, Apr 20, 2016 at 12:50 AM, Bas Nieuwenhuizen
wrote:
Use the CE suballocator instead of the normal one as the usage
is mos
Hi Rob,
Please bear in mind that there's a fair bit of comments, but before
all don't mix refactoring and new code. Please ?
On 15 April 2016 at 17:03, Rob Herring wrote:
> Add support for creating images from Android native buffers with dma-buf
> fd. As dma-buf support also requires DRI image
Am 2016-04-19 01:12, schrieb Tim Rowley:
Code was using an incorrect address for the base pointer.
Tested-by: Markus Wick
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94979
---
src/gallium/drivers/swr/swr_state.cpp | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff
Hi Jason,
stupid optional nitpick , but could you go for anv_batch_emit_block?
Normally abbreviations below 3 characters aren't really worth it.
Michael
Am 19/04/2016 um 02:10 schrieb Jason Ekstrand:
> This new macro uses a for loop to create an actual code block in which to
> place the macro se
Hi Tom,
Am 19.04.2016 um 19:52 schrieb Tom Stellard:
> The range metadata tells LLVM the range of expected values for this intrinsic,
> so it can do some additional optimizations on the result.
> ---
> src/gallium/drivers/radeonsi/si_shader.c | 29 ++---
> 1 file changed,
Hi Tom
Am 19.04.2016 um 19:52 schrieb Tom Stellard:
> We're trying to move to more of the new style intrinsics with include
> the correct target name, and map directly to ISA instructions.
>
> v2:
> - Only do this with LLVM 3.8 and newer.
> ---
> src/gallium/drivers/radeonsi/si_shader.c | 17 ++
On Wed, Apr 20, 2016 at 2:13 AM, Nicolai Hähnle wrote:
> On 19.04.2016 18:29, Bas Nieuwenhuizen wrote:
>>
>> I retract patch 1 and 2. Large scratch buffers are nice, but the
>> hardware only supports a 32-bit offset into it.
>
>
> Do you mean patch 2 and 3? Do you plan alternative patches to error
This is just a slight massaging of the patch you sent previously. What
happened to the discussion we had about how to do this properly?
Nicolai
On 19.04.2016 18:13, Grigori Goronzy wrote:
Small IBs help to reduce stalls for workloads that require a lot of
synchronization. On the other hand, if
On 19.04.2016 17:50, Bas Nieuwenhuizen wrote:
Use the CE suballocator instead of the normal one as the usage
is most similar to the CE, i.e. only read and written on GPU
and not mapped to CPU.
The scratch buffer is also only read and written by the GPU...
Signed-off-by: Bas Nieuwenhuizen
--
On 19.04.2016 18:29, Bas Nieuwenhuizen wrote:
I retract patch 1 and 2. Large scratch buffers are nice, but the
hardware only supports a 32-bit offset into it.
Do you mean patch 2 and 3? Do you plan alternative patches to error out
when there is an integer overflow? That's still kind of importa
On Tue, Apr 19, 2016 at 5:07 PM, Kenneth Graunke wrote:
> So...what would you like me to change? Just update the comment to match
> the code? Or change the code somehow as well?
I think just update the comment.
___
mesa-dev mailing list
mesa-dev@lists
On Tuesday, April 19, 2016 4:52:44 PM PDT Matt Turner wrote:
> On Tue, Apr 19, 2016 at 4:51 PM, Kenneth Graunke
wrote:
> > On Tuesday, April 19, 2016 11:19:54 AM PDT Matt Turner wrote:
> >> On Mon, Apr 18, 2016 at 11:48 PM, Kenneth Graunke
> > wrote:
> >> > On Gen7+, the thread payload provides
https://bugs.freedesktop.org/show_bug.cgi?id=92850
--- Comment #62 from Nicolai Hähnle ---
Hi Iaroslav, do you have an apitrace with shadows enabled that triggers the
assertion?
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the bug.
On Tue, Apr 19, 2016 at 4:51 PM, Kenneth Graunke wrote:
> On Tuesday, April 19, 2016 11:19:54 AM PDT Matt Turner wrote:
>> On Mon, Apr 18, 2016 at 11:48 PM, Kenneth Graunke
> wrote:
>> > On Gen7+, the thread payload provides the sample ID - we can read it
>> > in two instructions, without any ela
On Tuesday, April 19, 2016 11:19:54 AM PDT Matt Turner wrote:
> On Mon, Apr 18, 2016 at 11:48 PM, Kenneth Graunke
wrote:
> > On Gen7+, the thread payload provides the sample ID - we can read it
> > in two instructions, without any elaborate calculations. We don't even
> > need a state dependency
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> Rather than using a loop, and augmenting a variable use a generator and
> sum to achieve the same result.
>
> Signed-off-by: Dylan Baker
> ---
> src/mapi/glapi/gen/gl_x86_asm.py | 10 ++
> 1 file chan
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> Don't assign variables that aren't actually used, just put them in _,
> the python unused variable.
>
> Signed-off-by: Dylan Baker
> ---
> src/mapi/glapi/gen/gl_x86-64_asm.py | 2 +-
> 1 file changed, 1 inser
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> This just saves a bit of typing.
>
> Signed-off-by: Dylan Baker
> ---
> src/mapi/glapi/gen/glX_server_table.py | 12 ++--
> 1 file changed, 2 insertions(+), 10 deletions(-)
>
> diff --git a/src/mapi/
Did you try commenting the 'return 0' to see how that affected the
binary size? My old experiment was stupid... I just looked at 'ls -l'
on the .o file. A better experiment is to look at the output of 'size
libglapi.so.0.0.0'.
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> This code has for a long
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> This makes the function a little simpler by using a special case for the
> initial "ARB", and just walking the str as a sequence rather than
> indexing into it.
>
> It also uses a list to join rather than overw
That's weird. It looks like I added this in 66a5548f, but I have no
idea why. Even then I knew what pass was for. *shrug*
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> The pass keyword is the solution for python's whitespace significant
> code. It's used
On Wed, Apr 20, 2016 at 1:13 AM, Grigori Goronzy wrote:
> Add missing break, add default case. Additionally initialize variables
> to avoid compiler warnings.
> ---
> src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 6 +-
> 1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/src/galliu
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> str is a builtin in python, although python doesn't prevent builtins
> from being shadowed, its still a bad idea.
>
> Signed-off-by: Dylan Baker
> ---
> src/mapi/glapi/gen/glX_proto_send.py | 14 +++--
On Tue, Apr 19, 2016 at 7:04 PM, Matt Turner wrote:
> On Tue, Apr 19, 2016 at 3:06 PM, Emil Velikov
> wrote:
>> Hi Rob,
>>
>> Speaking of maintainers... I have a patch somewhere that adds you as
>> maintainer for $linux/drivers/drm/gpu/msm/.
>> Guess I should send it already ;-)
>>
>> On 19 Apri
Heh... I don't think _ was a thing when I first wrote this. I like it.
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> Rather than using a variable called junk, use _ for unused unpacked
> values. This is much more idiomatic.
>
> Signed-off-by: Dylan Baker
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> There are two ways to handle this. One is to use the _ placeholder value
> when multiple values are needed. The second is to use list indexing to
> get only the values actually needed when only one value is desi
Quoting Ian Romanick (2016-04-19 16:13:51)
> On 04/01/2016 11:47 AM, Ilia Mirkin wrote:
> > On Fri, Apr 1, 2016 at 2:41 PM, Dylan Baker wrote:
> >> Quoting Ilia Mirkin (2016-04-01 08:46:19)
> >>> IMHO this is still quite fancy and unnecessarily complicated.
> >>>
> >>> How about
> >>>
> >>> temp
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> This can easily be restored with a revert if it's ever actually needed.
>
> Signed-off-by: Dylan Baker
> ---
> src/mapi/glapi/gen/glX_proto_common.py | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/s
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> The variable has the same value as the explicit None return, but makes
> it clearer what's going on.
If we're going to do that, I think it makes sense to make the other
returns be breaks instead.
>
> Signed-off-by: Dylan Baker
> ---
> src/mapi/glapi
I never really was clear on when to use which kind of thing. This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> rather than a list.
>
> In python tuples are mainly for heterogenous collections, especially
> when position has relevance. Which makes them the rig
I retract patch 1 and 2. Large scratch buffers are nice, but the
hardware only supports a 32-bit offset into it.
- Bas
On Wed, Apr 20, 2016 at 12:50 AM, Bas Nieuwenhuizen
wrote:
> Use the CE suballocator instead of the normal one as the usage
> is most similar to the CE, i.e. only read and writt
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> This method is a no-op, but it shuts up tools that get upset that Print
> calls a non-existent method.
>
> Signed-off-by: Dylan Baker
> ---
> src/mapi/glapi/gen/gl_XML.py | 3 +++
> 1 file changed, 3 insertio
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> This is a pretty minor refactor, but it avoids having an else statement
> with a nested if. It instead makes it one big if statement.
>
> Signed-off-by: Dylan Baker
> ---
> src/mapi/glapi/gen/gl_XML.py | 13 +
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> This avoids recompiles of the re each time the function is called.
>
> Signed-off-by: Dylan Baker
> ---
> src/mapi/glapi/gen/gl_XML.py | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --gi
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> Don't call and store a value that might not be used unless it's know
known
With that fixed, this patch is
Reviewed-by: Ian Romanick
> it's needed.
>
> Signed-off-by: Dylan Baker
> --
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> Use a boolean instead of 0 or 1, and use an assert rather than a runtime
> error.
>
> Signed-off-by: Dylan Baker
> ---
> src/mapi/glapi/gen/gl_XML.py | 12 +---
> 1 file changed, 5 insertions(+), 7 de
On Tue, Apr 19, 2016 at 3:38 PM, Chad Versace
wrote:
> On Sat 16 Apr 2016, Jason Ekstrand wrote:
> > ---
> > src/intel/isl/isl_format.c | 34 --
> > 1 file changed, 16 insertions(+), 18 deletions(-)
> >
> > diff --git a/src/intel/isl/isl_format.c b/src/intel/isl/i
Quoting Ian Romanick (2016-04-19 15:14:59)
> On 03/31/2016 05:04 PM, Dylan Baker wrote:
> > This does basically the same thing as the previous implementation,
> > although it uses sorted() to act on the dictionary without first copying
> > the keys out into a list, and then indexing back into the d
On Tue, Apr 19, 2016 at 3:29 PM, Chad Versace
wrote:
> On Sat 16 Apr 2016, Jason Ekstrand wrote:
> > ---
> > src/intel/isl/isl.h | 7 +++
> > src/intel/isl/isl_storage_image.c | 10 ++
> > 2 files changed, 17 insertions(+)
>
> > +bool
> > +isl_has_matching_typed_storag
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> This reworks this method to use the sorted() built-in rather than taking
> the dictionary keys (which involves creating a brand new copy of the
> list in memory), sorting it, then iterating the new sorted list a
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> This uses the collections.defaultdict to remove the need to check and
> add an empty dict, since defaultdict will call the default factory for
> us whenever a KeyError would be raised (by a missing key value).
>
Add missing break, add default case. Additionally initialize variables
to avoid compiler warnings.
---
src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_cs.c
b/src/gallium/winsys/amdgpu/drm/amdgp
Small IBs help to reduce stalls for workloads that require a lot of
synchronization. On the other hand, if there is no notable
synchronization, we can use a large IB size to slightly improve
performance in some cases.
This introduces tuning of the IB size based on feedback on the average
buffer wa
On 04/01/2016 11:47 AM, Ilia Mirkin wrote:
> On Fri, Apr 1, 2016 at 2:41 PM, Dylan Baker wrote:
>> Quoting Ilia Mirkin (2016-04-01 08:46:19)
>>> IMHO this is still quite fancy and unnecessarily complicated.
>>>
>>> How about
>>>
>>> temp = dict((f.offset, f) for f in self.functions_by_name.iterv
On Tue, Apr 19, 2016 at 3:18 PM, Chad Versace
wrote:
> On Tue 19 Apr 2016, Emil Velikov wrote:
> > On 16 April 2016 at 20:45, Jason Ekstrand wrote:
> > > C++ doesn't support designated initializers and g++ in particular
> doesn't
> > > handle them when the struct gets complicated, i.e. has a uni
Quoting Ian Romanick (2016-04-19 14:55:19)
> On 03/31/2016 05:04 PM, Dylan Baker wrote:
> > Rather than using the string module, use the same method for the str
> > instance (join, split, replace, etc).
> >
> > It's more forward looking, since it will make smoothing over the
> > str/bytes/unicode
Quoting Ian Romanick (2016-04-19 15:07:09)
> On 03/31/2016 05:04 PM, Dylan Baker wrote:
> > RuntimeError is a very specific type of error. In almost very case it's
> > being raised where an assert is the right choice, and in the rest using
> > a plain Exception is better, at least there it's obviou
Quoting Ian Romanick (2016-04-19 14:56:11)
> On 03/31/2016 05:04 PM, Dylan Baker wrote:
> > Remove parens that don't do anything. Mostly these are extra grouping
> > parens in if statements, that aren't needed. Unlike C python doesn't
> > require parens around the conditions of if statements, excep
Quoting Ian Romanick (2016-04-19 14:49:19)
> I think I did that because it made emacs do-the-right-thing with the
> indentation. The return told it to move up a level. Either way, this
> patch is
It doesn't surprise me that emacs would want the return (or did, they
might have fixed it in the int
On Tue, Apr 19, 2016 at 3:06 PM, Emil Velikov wrote:
> Hi Rob,
>
> Speaking of maintainers... I have a patch somewhere that adds you as
> maintainer for $linux/drivers/drm/gpu/msm/.
> Guess I should send it already ;-)
>
> On 19 April 2016 at 22:11, Rob Clark wrote:
>> From: Rob Clark
>>
>> Copi
Quoting Ian Romanick (2016-04-19 13:00:16)
> This patch is
>
> Reviewed-by: Ian Romanick
>
> I guess the spurious whitespace changes in the previous patch should
> have been in this patch.
Probably, I'll see if I can fix that when I spin v2.
>
> On 03/31/2016 05:04 PM, Dylan Baker wrote:
> >
Quoting Ian Romanick (2016-04-19 12:53:00)
> On 03/31/2016 05:04 PM, Dylan Baker wrote:
> > It's a bit odd to call IBM out in particular, especially since Brian
> > Paul is listed as a copyright holder on chronologically before IBM.
> > Either way this should probably have the standard "authors and
On Tue 19 Apr 2016, Jason Ekstrand wrote:
> On Tue, Apr 19, 2016 at 3:14 PM, Chad Versace
> wrote:
>
> > On Tue 19 Apr 2016, Matt Turner wrote:
> > > On Sat, Apr 16, 2016 at 12:45 PM, Jason Ekstrand
> > wrote:
> > > > C++ doesn't define the "restrict" keyword so g++ barfs when it sees
> > isl.h.
https://bugs.freedesktop.org/show_bug.cgi?id=95003
--- Comment #2 from ros...@gmail.com ---
hrm so is there a way with clover to find out the core count of the GPU(320 in
my case)?
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.__
Quoting Ian Romanick (2016-04-19 12:51:04)
> On 03/31/2016 05:04 PM, Dylan Baker wrote:
> > This allows, among other things, the proper use of the Copyright symbol
> > in the copyright header (which this patch also does).
> >
> > It cannot be used in the printed templates however, because they are
2 and 2a are
Reviewed-by: Jason Ekstrand
On Tue, Apr 19, 2016 at 1:52 PM, Eric Anholt wrote:
> Rob Clark writes:
>
> > From: Rob Clark
> >
> > Signed-off-by: Rob Clark
>
> Both are:
>
> Reviewed-by: Eric Anholt
>
___
mesa-dev mailing list
mesa-de
On Tue, Apr 19, 2016 at 3:14 PM, Chad Versace
wrote:
> On Tue 19 Apr 2016, Matt Turner wrote:
> > On Sat, Apr 16, 2016 at 12:45 PM, Jason Ekstrand
> wrote:
> > > C++ doesn't define the "restrict" keyword so g++ barfs when it sees
> isl.h.
> > > Having our own define lets us define it to be a no-
On Sat 16 Apr 2016, Jason Ekstrand wrote:
> ---
> src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 8 +++-
> src/mesa/drivers/dri/i965/brw_fs_surface_builder.cpp | 7 +--
> src/mesa/drivers/dri/i965/brw_fs_surface_builder.h | 4 ++--
> 3 files changed, 10 insertions(+), 9 deletions
Allows allocation of >= 4 GiB.
Signed-off-by: Bas Nieuwenhuizen
---
src/gallium/drivers/radeonsi/si_compute.c | 23 +++
src/gallium/drivers/radeonsi/si_pipe.c | 4 ++--
src/gallium/drivers/radeonsi/si_pipe.h | 4 ++--
src/gallium/drivers/radeonsi/si_
Signed-off-by: Bas Nieuwenhuizen
---
src/gallium/drivers/radeonsi/si_compute.c | 5 -
src/gallium/drivers/radeonsi/si_state_shaders.c | 5 -
2 files changed, 8 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_compute.c
b/src/gallium/drivers/radeonsi/si_co
Signed-off-by: Bas Nieuwenhuizen
---
src/gallium/drivers/radeonsi/si_compute.c | 5 -
src/gallium/drivers/radeonsi/si_state_shaders.c | 2 +-
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_compute.c
b/src/gallium/drivers/radeonsi/si_compu
Use the CE suballocator instead of the normal one as the usage
is most similar to the CE, i.e. only read and written on GPU
and not mapped to CPU.
Signed-off-by: Bas Nieuwenhuizen
---
src/gallium/drivers/radeonsi/si_cp_dma.c | 27 ++-
1 file changed, 10 insertions(+), 17
Reviewed-by: Jason Ekstrand
On Tue, Apr 12, 2016 at 1:05 AM, Samuel Iglesias Gonsálvez <
sigles...@igalia.com> wrote:
> From: Iago Toral Quiroga
>
> At least i965 hardware does not have native support for ceil on doubles.
> ---
> src/compiler/nir/nir.h | 1 +
> src/compiler/n
Reviewed-by: Jason Ekstrand
On Tue, Apr 12, 2016 at 1:05 AM, Samuel Iglesias Gonsálvez <
sigles...@igalia.com> wrote:
> From: Iago Toral Quiroga
>
> At least i965 hardware does not have native support for floor on doubles.
> ---
> src/compiler/nir/nir.h | 1 +
> src/compiler/
On Sat 16 Apr 2016, Jason Ekstrand wrote:
> ---
> src/intel/isl/isl.h| 23 +++
> src/intel/isl/isl_format.c | 24
> 2 files changed, 43 insertions(+), 4 deletions(-)
Patch 8 is
Reviewed-by: Chad Versace
___
On Sat 16 Apr 2016, Jason Ekstrand wrote:
> ---
> src/intel/isl/isl_format.c | 34 --
> 1 file changed, 16 insertions(+), 18 deletions(-)
>
> diff --git a/src/intel/isl/isl_format.c b/src/intel/isl/isl_format.c
> index 32bd701..b0450d9 100644
> --- a/src/intel/isl/
Reviewed-by: Jason Ekstrand
On Tue, Apr 12, 2016 at 1:05 AM, Samuel Iglesias Gonsálvez <
sigles...@igalia.com> wrote:
> From: Iago Toral Quiroga
>
> At least i965 hardware does not have native support for fract() on doubles.
> ---
> src/compiler/nir/nir.h | 1 +
> src/compile
On Tue, Apr 12, 2016 at 1:05 AM, Samuel Iglesias Gonsálvez <
sigles...@igalia.com> wrote:
> From: Iago Toral Quiroga
>
> At least i965 hardware does not have native support for truncating doubles.
> ---
> src/compiler/nir/nir.h | 1 +
> src/compiler/nir/nir_lower_double_ops.c |
On Sat 16 Apr 2016, Jason Ekstrand wrote:
> ---
> src/intel/vulkan/anv_image.c | 15 +++
> 1 file changed, 3 insertions(+), 12 deletions(-)
Patch 6 is
Reviewed-by: Chad Versace
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https:
On Sat 16 Apr 2016, Jason Ekstrand wrote:
> ---
> src/intel/isl/isl.h | 7 +++
> src/intel/isl/isl_storage_image.c | 10 ++
> 2 files changed, 17 insertions(+)
> +bool
> +isl_has_matching_typed_storage_image_format(const struct brw_device_info
> *devinfo,
> +
On Tue, Apr 19, 2016 at 6:06 PM, Emil Velikov wrote:
> Hi Rob,
>
> Speaking of maintainers... I have a patch somewhere that adds you as
> maintainer for $linux/drivers/drm/gpu/msm/.
> Guess I should send it already ;-)
>
> On 19 April 2016 at 22:11, Rob Clark wrote:
>> From: Rob Clark
>>
>> Copi
Patch 4 is
Reviewed-by: Chad Versace
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Tue 19 Apr 2016, Emil Velikov wrote:
> On 16 April 2016 at 20:45, Jason Ekstrand wrote:
> > C++ doesn't support designated initializers and g++ in particular doesn't
> > handle them when the struct gets complicated, i.e. has a union.
> > ---
> > src/intel/isl/isl.h | 32 ---
That's much better. This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> Use the max function and a generator expression instead of a loop and
> if. This is certainly no less efficient, and saves several lines of
> code.
>
> Signed-off-by: Dylan Baker
> ---
>
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> This does basically the same thing as the previous implementation,
> although it uses sorted() to act on the dictionary without first copying
> the keys out into a list, and then indexing back into the dict; and uses
> yield to make the method a generato
On Tue 19 Apr 2016, Matt Turner wrote:
> On Sat, Apr 16, 2016 at 12:45 PM, Jason Ekstrand wrote:
> > C++ doesn't define the "restrict" keyword so g++ barfs when it sees isl.h.
> > Having our own define lets us define it to be a no-op for C++.
> > ---
> > src/intel/isl/isl.h | 18 ++---
It looks like this hasn't been used since cbabf5f9. Should add that to
the commit message for posterity. This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> Originally I planned to make it more efficient, but then realized that
> it wasn't actually used anywher
On 19 April 2016 at 20:51, Rob Herring wrote:
> Commit bfd17c76c126 ("i965: Port INTEL_PRECISE_TRIG=1 to NIR.") added a
> generated file brw_nir_trig_workarounds.c which broke the Android build.
> Add the necessary makefiles to the Android build.
>
> Cc: Kenneth Graunke
> Cc: Emil Velikov
Thanks
This patch is
Reviewed-by: Ian Romanick
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> Python has what are often referred to as "magic methods". __iter__,
> __str__, __add__, and countless more. All of these methods exist to be
> called by another function or by an operator. The __add__ method
> i
On 03/31/2016 05:04 PM, Dylan Baker wrote:
> RuntimeError is a very specific type of error. In almost very case it's
> being raised where an assert is the right choice, and in the rest using
> a plain Exception is better, at least there it's obvious it's not that
> the python runtime hit an interna
Hi Rob,
Speaking of maintainers... I have a patch somewhere that adds you as
maintainer for $linux/drivers/drm/gpu/msm/.
Guess I should send it already ;-)
On 19 April 2016 at 22:11, Rob Clark wrote:
> From: Rob Clark
>
> Copied from linux kernel, with minimal changes to script (just to
> recog
1 - 100 of 268 matches
Mail list logo
