Hi Andy,
On Mon, Sep 19, 2016 at 3:27 PM, Andy Furniss wrote:
> Nayan Deshmukh wrote:
>
>> Hi Andy,
>>
>> Thanks for testing the patches!!
>>
>> On Mon, Sep 19, 2016 at 5:24 AM, Andy Furniss
>> wrote:
>>
>> Andy Furniss wrote:
>>>
>>> Nayan Deshmukh
On 16.09.2016 19:19, Ian Romanick wrote:
On 09/16/2016 06:48 AM, Nicolai Hähnle wrote:
Hi all,
this is really Dave's work, with a few touch-ups from me that I think make
sense. I've kept those separate with the intention to squash. I'd like to
land these in master even before the main
Seems they haven't got lost, because I see these patches in the mailing-list
webpage:
https://lists.freedesktop.org/archives/mesa-dev/2016-September/128847.html
https://lists.freedesktop.org/archives/mesa-dev/2016-September/128845.html
From: Dave Airlie
v2:
- no PIPE_CAP_INT64 yet
- emit DIV/MOD without the divide-by-zero workaround
Reviewed-by: Marek Olšák (v1)
Signed-off-by: Dave Airlie
---
.../drivers/radeon/radeon_setup_tgsi_llvm.c| 69
From: Dave Airlie
This enables 64-bit integer support in gallivm and
llvmpipe.
v2: add conversion opcodes.
v3:
- PIPE_CAP_INT64 is not there yet
- restrict DIV/MOD defaults to the CPU, as for 32 bits
- TGSI_OPCODE_I2U64 becomes TGSI_OPCODE_U2I64
Signed-off-by: Dave Airlie
From: Nicolai Hähnle
This should be analogous to 32-bit integers.
---
src/gallium/auxiliary/gallivm/lp_bld_tgsi.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/auxiliary/gallivm/lp_bld_tgsi.c
b/src/gallium/auxiliary/gallivm/lp_bld_tgsi.c
index
From: Dave Airlie
This adds all the opcodes to tgsi_exec for softpipe to use.
v2: add conversion opcodes.
v3:
- no PIPE_CAP_INT64 yet
- change TGSI_OPCODE_I2U64 to TGSI_OPCODE_U2I64
Signed-off-by: Dave Airlie
---
src/gallium/auxiliary/tgsi/tgsi_exec.c
This series is,
Reviewed-by: Edward O'Callaghan
On 09/19/2016 11:08 PM, Nicolai Hähnle wrote:
> Hi everybody,
>
> here's a v2 of the series. Compared to previously, I have now squashed my
> changes in. I have also included Roland's comments on the tgsi.rst docs,
>
Andy Furniss wrote:
Andy Furniss wrote:
bz2 compressed vid showing the issue -
https://drive.google.com/open?id=0BxP5-S1t9VEEUE5sbUFBV20zSms
To be clear this is just the test rez vid I used so you can
possibly recreate the issue your self.
Hmm, interesting, with bicubic, hqscaling=1 this
From: Dave Airlie
This just adds the basic support for 64-bit opcodes,
and the new types.
v2: add conversion opcodes.
add documentation.
v3:
- make docs more consistent
- change TGSI_OPCODE_I2U64 to TGSI_OPCODE_U2I64
Reviewed-by: Marek Olšák (v2)
Hi everybody,
here's a v2 of the series. Compared to previously, I have now squashed my
changes in. I have also included Roland's comments on the tgsi.rst docs,
and, following his comment, I have changed the 32-bit to 64-bit conversion
so that there is now a TGSI_OPCODE_I2I64 and a
From: Dave Airlie
This adds support to TGSI for 64-bit integer immediates.
Reviewed-by: Marek Olšák
Reviewed-by: Nicolai Hähnle
Signed-off-by: Dave Airlie
---
src/gallium/auxiliary/tgsi/tgsi_dump.c |
On 16 September 2016 at 18:02, Emil Velikov wrote:
> Introduce a helper and use it throughout the platform code. This allows
> us to reduce the amount of ifdef(s) and (potentially) use
> kms_swrast_dri.so for !drm platforms (namely wayland and x11).
Reviewed-by: Daniel
https://bugs.freedesktop.org/show_bug.cgi?id=97863
--- Comment #1 from Elio ---
Created attachment 126627
--> https://bugs.freedesktop.org/attachment.cgi?id=126627=edit
Dmesg
--
You are receiving this mail because:
You are the assignee for the bug.
You are the
https://bugs.freedesktop.org/show_bug.cgi?id=97863
Bug ID: 97863
Summary: [BXT] Webglc is failing a lot of tests related to
extensions textures uniforms and more
Product: Mesa
Version: unspecified
Hardware: x86-64 (AMD64)
It all looks fine to me. Feel free to add a
Reviewed-by: Jason Ekstrand
That said, my knowledge of the details of the DRI vfuncs is very limited so
I'd like to see Emil or Axel sign off on it too, especially since they were
the ones who had all the comments.
--Jason
On
Am 19.09.2016 um 15:08 schrieb Nicolai Hähnle:
> From: Dave Airlie
>
> This just adds the basic support for 64-bit opcodes,
> and the new types.
>
> v2: add conversion opcodes.
> add documentation.
> v3:
> - make docs more consistent
> - change TGSI_OPCODE_I2U64 to
https://bugs.freedesktop.org/show_bug.cgi?id=97863
Elio changed:
What|Removed |Added
Summary|[BXT] Webglc is failing a |[BXT] Webglc is
Am 19.09.2016 um 15:08 schrieb Nicolai Hähnle:
> From: Dave Airlie
>
> This enables 64-bit integer support in gallivm and
> llvmpipe.
>
> v2: add conversion opcodes.
> v3:
> - PIPE_CAP_INT64 is not there yet
> - restrict DIV/MOD defaults to the CPU, as for 32 bits
> -
I sent this reply on Saturday however is seems something went wrong and
it didn't make it out so here it is again.
On Fri, 2016-09-16 at 15:25 -0700, Jason Ekstrand wrote:
> On Thu, Sep 15, 2016 at 12:03 AM, Timothy Arceri abora.com> wrote:
> > From: Thomas Helland
On Fri, 2016-09-16 at 15:25 -0700, Jason Ekstrand wrote:
> > On Thu, Sep 15, 2016 at 12:03 AM, Timothy Arceri wrote:
> > > > From: Thomas Helland
> >
> >
> > This pass detects induction variables and calculates the
> >
> > trip count
On 16/09/16 07:33 PM, Nayan Deshmukh wrote:
> so that the texture is rendered to back buffer before calling
> flush_frontbuffer and can be copied to a different buffer in
> the function
>
> v2: change comment style
>
> Signed-off-by: Nayan Deshmukh
> ---
>
Hi Leo,
On Tue, Sep 20, 2016 at 1:56 AM, Leo Liu wrote:
>
>
> On 09/17/2016 07:33 AM, Nayan Deshmukh wrote:
>
> Hi Leo,
>
> Could you push the patches? I don't have the push access.
>
>
>
> Can you rebase all your reviewed patches, and add RB to it, and then you
> can send
so that the texture is rendered to back buffer before calling
flush_frontbuffer and can be copied to a different buffer in
the function
v2: change comment style
Signed-off-by: Nayan Deshmukh
Reviewed-by: Michel Dänzer
---
so that the texture is rendered to back buffer before calling
flush_frontbuffer and can be copied to a different buffer in
the function
Signed-off-by: Nayan Deshmukh
Reviewed-by: Michel Dänzer
---
https://bugs.freedesktop.org/show_bug.cgi?id=97863
Tapani Pälli changed:
What|Removed |Added
CC||lem...@gmail.com
--
In case of prime when rendering is done on GPU other then the
server GPU, use a seprate linear buffer for each back buffer
which will be displayed using present extension.
v2: Use a seprate linear buffer for each back buffer (Michel)
v3: Change variable names and fix coding style (Leo and Emil)
https://bugs.freedesktop.org/show_bug.cgi?id=88354
Eero Tamminen changed:
What|Removed |Added
Status|NEW |NEEDINFO
---
OpAtomicLoad/Store should have pointer to images just like the rest of the
atomic operators. These couple of lines were poorly copied from the
ssbo/shared_vars cases (the only ones currently tests by the CTS).
Fixes 2afb950161f847d9b0a7 "spirv/nir: Add support for OpAtomicLoad/Store"
Cc: Timothy
On 07/28/2016 07:52 AM, Francisco Jerez wrote:
> Emil Velikov writes:
>
>> On 6 June 2016 at 00:02, Vedran Miletić wrote:
>>> On 06/04/2016 04:18 AM, Francisco Jerez wrote:
Serge Martin writes:
> From: Vedran
This looks good to me.
Reviewed-by: Jason Ekstrand
Do we have tests for this? If not, we should write some. I know there are
other image atomic tests in the CTS. They shouldn't be hard to extend.
On Sep 19, 2016 9:36 AM, "Lionel Landwerlin" wrote:
On 09/19/2016 07:08 PM, Vedran Miletić wrote:
> On 07/28/2016 07:52 AM, Francisco Jerez wrote:
>> Emil Velikov writes:
>>
>>> On 6 June 2016 at 00:02, Vedran Miletić wrote:
On 06/04/2016 04:18 AM, Francisco Jerez wrote:
>
> Serge
On 19/09/16 18:02, Jason Ekstrand wrote:
This looks good to me.
Reviewed-by: Jason Ekstrand >
Do we have tests for this? If not, we should write some. I know there
are other image atomic tests in the CTS. They shouldn't be hard to extend.
https://bugs.freedesktop.org/show_bug.cgi?id=97863
--- Comment #2 from Elio ---
Forgot to include the website:
www.khronos.org/registry/webgl/conformance-suites/1.0.3/webgl-conformance-tests.html
--
You are receiving this mail because:
You are the assignee for
2016-09-15 16:27 GMT-04:00 Marek Olšák :
> Update your gcc I guess? Sorry, I don't know much about LLVM build
> requirements. It works with gcc 5.4.0.
I'm using a pretty recent gcc, and the 64-bit compilation works like a
charm, it's only the 32-bit compilation that has trouble.
On 09/07/2016 06:52 PM, Vedran Miletić wrote:
> LLVM and Mesa both define the DEBUG macro in incompatible ways. As a
> general practice, we should avoid using such generic names when it is
> possible to do so.
>
> This patch renames all occurrences of the DEBUG macro to MESA_DEBUG,
> and removes
OpenCL apps can quote arguments they pass to the OpenCL compiler, most
commonly include paths containing spaces.
If the Clang OpenCL compiler was called via a shell, the shell would
split the arguments with respect to to quotes and then remove quotes
before passing the arguments to the compiler.
On Fri, Sep 02, 2016 at 05:01:28PM -0700, Jason Ekstrand wrote:
> On Wed, Aug 31, 2016 at 8:29 PM, Nanley Chery wrote:
>
> > From: Jason Ekstrand
> >
>
> First off, this is your patch not mine. The patch of mine you based this
> on was little
The options specified in the CLOVER_EXTRA_COMPILER_OPTIONS shell
variable are appended to the compiler options specified by the OpenCL
program, if any.
Analogously, the options specified in the CLOVER_EXTRA_LINKER_OPTIONS
variable are appended to the linker options and the options spoecified
in
On Fri, Sep 02, 2016 at 05:27:11PM -0700, Jason Ekstrand wrote:
> On Wed, Aug 31, 2016 at 8:29 PM, Nanley Chery wrote:
>
> > This series enables Hierarchical depth buffer rendering and fast depth
> > clears
> > for render passes with a single subpass running on platforms
Not intended for upstream. Should cause a GPU hang if some thread is
executed with a non-contiguous dispatch mask breaking assumptions of
brw_stage_has_packed_dispatch(). Doesn't cause any CTS, DEQP or
Piglit regressions, while replacing brw_stage_has_packed_dispatch()
with a dummy
Seems OK here.
Tested-by: Brian Paul
On 09/19/2016 02:43 AM, Nicolai Hähnle wrote:
Reviewed-by: Nicolai Hähnle
However, you might want to check with the VMWare guys. I seem to recall
that MSVC is a bit peculiar with some of these library
This instruction is available since SM20 (Fermi) and allow to do
(a << b) + c in one shot. In some situations, IMAD should be
replaced by SHLADD when b is a power of 2, and ADD+SHL should be
replaced by SHLADD as well.
Signed-off-by: Samuel Pitoiset
---
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 8
1 file changed, 8 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
On 19 September 2016 at 16:38, Jason Ekstrand wrote:
> It all looks fine to me. Feel free to add a
>
> Reviewed-by: Jason Ekstrand
>
> That said, my knowledge of the details of the DRI vfuncs is very limited so
> I'd like to see Emil or Axel sign off
Unfortunately, we can't use the emit helpers for GF100/GK110
because src1 and src2 are swapped.
Signed-off-by: Samuel Pitoiset
---
.../drivers/nouveau/codegen/nv50_ir_emit_gk110.cpp | 53 ++
.../drivers/nouveau/codegen/nv50_ir_emit_gm107.cpp | 32
On 09/19/2016 11:59 PM, Brian Paul wrote:
Seems OK here.
Tested-by: Brian Paul
Thanks for testing and reviewing guys.
On 09/19/2016 02:43 AM, Nicolai Hähnle wrote:
Reviewed-by: Nicolai Hähnle
However, you might want to check with the VMWare
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp
index
Only and only if src1 is a power of 2 we can replace IMAD by SHLADD.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 13 +
1 file changed, 13 insertions(+)
diff --git
Commutativity is not allowed with SHLADD, but src2 can accept
loads. To allow the load propagation pass to do its job, add a
special case like for SUCLAMP because src1 is always an immediate.
This IMAD to SHLADD optimization helps a bunch of shaders from Tomb
Raider, Victor Vran, UE4 demos (+15%
https://bugs.freedesktop.org/show_bug.cgi?id=97230
Eero Tamminen changed:
What|Removed |Added
CC|
Those patches got lost somehow? Maybe they weren't sent out as replies
to your first email, check the git configuration for sendemail.thread or
the git send-email --thread flag.
Cheers,
Nicolai
On 18.09.2016 09:04, Weng, Chuanbo wrote:
Ping for review. Thanks.
-Original Message-
Reviewed-by: Nicolai Hähnle
However, you might want to check with the VMWare guys. I seem to recall
that MSVC is a bit peculiar with some of these library functions.
Cheers,
Nicolai
On 14.09.2016 20:37, Samuel Pitoiset wrote:
This allows to use hexadecimal numbers
Vedran Miletić writes:
> The options specified in the CLOVER_EXTRA_COMPILER_OPTIONS shell
> variable are appended to the compiler options specified by the OpenCL
> program, if any.
> Analogously, the options specified in the CLOVER_EXTRA_LINKER_OPTIONS
> variable are appended
On Fri, Sep 02, 2016 at 05:12:58PM -0700, Jason Ekstrand wrote:
> On Wed, Aug 31, 2016 at 8:29 PM, Nanley Chery wrote:
>
> > Provides an FPS increase of ~30% on the Sascha triangle and multisampling
> > demos.
> >
> > Clears that happen within a render pass via
2016-09-19 18:40 GMT-04:00 Marek Olšák :
> Do you mean the PC is frozen for 2 minutes or just occasional hiccups
> for 2 minutes?
Occasional hiccups for 2 minutes. The game works well, I wander around
and, from time to time, it freezes for a random duration between 2
seconds and
On Fri, Sep 02, 2016 at 03:16:21PM -0700, Chad Versace wrote:
> On Wed 31 Aug 2016, Nanley Chery wrote:
> > From: Chad Versace
> >
> > Nanley Chery:
> > (rebase)
> > - Resolve conflicts with new anv_batch_emit macro
> > (amend)
> > - Remove wip! tag and handle a QPitch
On 09/17/2016 07:33 AM, Nayan Deshmukh wrote:
Hi Leo,
Could you push the patches? I don't have the push access.
Can you rebase all your reviewed patches, and add RB to it, and then you
can send them to me ?
Sorry for too busy to do this for you.
Regards,
Leo
Regards,
Nayan.
On Fri,
Hi Nayan,
On 16 September 2016 at 13:51, Nayan Deshmukh wrote:
> + scrn->pipe = scrn->base.pscreen->context_create(scrn->base.pscreen,
> + >base, 0);
> +
I think you're forgetting to destroy the context in
Nayan Deshmukh wrote:
Hi Andy,
Thanks for testing the patches!!
On Mon, Sep 19, 2016 at 5:24 AM, Andy Furniss
wrote:
Andy Furniss wrote:
Nayan Deshmukh wrote:
This series implements lanczos interpolation filter.
Andy, I have made some changes to the code. Can you
Andy Furniss wrote:
bz2 compressed vid showing the issue -
https://drive.google.com/open?id=0BxP5-S1t9VEEUE5sbUFBV20zSms
To be clear this is just the test rez vid I used so you can
possibly recreate the issue your self.
___
mesa-dev mailing list
On Friday, September 9, 2016 4:14:55 PM PDT Dylan Baker wrote:
> This extension is a combination of AMD_vertex_shader_viewport_index and
> AMD_vertex_shader_layer, making it rather trivial to implement.
>
> For gallium I *think* this needs a new cap because of the addition of
> support in
On Mon, Sep 19, 2016 at 6:11 PM, Samuel Pitoiset
wrote:
> This instruction is available since SM20 (Fermi) and allow to do
> (a << b) + c in one shot. In some situations, IMAD should be
> replaced by SHLADD when b is a power of 2, and ADD+SHL should be
> replaced by
On Mon, Sep 19, 2016 at 6:11 PM, Samuel Pitoiset
wrote:
> Only and only if src1 is a power of 2 we can replace IMAD by SHLADD.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 13
Vedran Miletić writes:
> OpenCL apps can quote arguments they pass to the OpenCL compiler, most
> commonly include paths containing spaces.
>
> If the Clang OpenCL compiler was called via a shell, the shell would
> split the arguments with respect to to quotes and then remove
On Mon, Sep 19, 2016 at 7:51 PM, Romain Failliot
wrote:
> 2016-09-15 16:27 GMT-04:00 Marek Olšák :
>> Update your gcc I guess? Sorry, I don't know much about LLVM build
>> requirements. It works with gcc 5.4.0.
>
> I'm using a pretty recent gcc, and
On 09/20/2016 12:16 AM, Ilia Mirkin wrote:
On Mon, Sep 19, 2016 at 6:11 PM, Samuel Pitoiset
wrote:
This instruction is available since SM20 (Fermi) and allow to do
(a << b) + c in one shot. In some situations, IMAD should be
replaced by SHLADD when b is a power of
On Mon, Sep 19, 2016 at 6:11 PM, Samuel Pitoiset
wrote:
> Unfortunately, we can't use the emit helpers for GF100/GK110
> because src1 and src2 are swapped.
>
> Signed-off-by: Samuel Pitoiset
> ---
>
This extension is a combination of AMD_vertex_shader_viewport_index and
AMD_vertex_shader_layer, making it rather trivial to implement.
For gallium I *think* this needs a new cap because of the addition of
support in tessellation evaluation shaders, and since I don't have any
hardware to test it
69 matches
Mail list logo