On Aug 5, 2016 11:37 PM, "Jason Ekstrand" wrote:
>
> I am going to try and give this a proper look on Monday. Given how
sketchy the multisampling support is on gen7 at the moment, I am a but
concerned that it doesn't fix more tests. What's left to fix for MSAA?
Also, I think I've mentioned this
From: Mathias Fröhlich
v2: Split into two patches.
Signed-off-by: Mathias Fröhlich
---
src/util/bitscan.h | 12
1 file changed, 12 insertions(+)
diff --git a/src/util/bitscan.h b/src/util/bitscan.h
index 0743fe7..a5bb34e 100644
--- a/src/util/bitscan.h
+++ b/src/util/bitscan.h
@@
From: Mathias Fröhlich
Following the split up patch as a series and on top
the include fix to u_bitscan.h.
Please review!
Thanks!
Mathias
As requested with the initial creation of util/bitscan.h
now move other bitscan related functions into util.
v2: Split into two patches.
Signed-off-by:
From: Mathias Fröhlich
We need this for 'inline'.
Signed-off-by: Mathias Fröhlich
---
src/gallium/auxiliary/util/u_bitcast.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/auxiliary/util/u_bitcast.h
b/src/gallium/auxiliary/util/u_bitcast.h
index b1f9938..e8fb0fe 100644
---
I am going to try and give this a proper look on Monday. Given how sketchy
the multisampling support is on gen7 at the moment, I am a but concerned
that it doesn't fix more tests. What's left to fix for MSAA?
On Aug 5, 2016 4:55 PM, "Lionel Landwerlin" wrote:
Fixes the following failures :
dE
Hi Brian,
On Friday, August 05, 2016 09:27:34 Brian Paul wrote:
> > As requested with the initial creation of util/bitscan.h
> > now move other bitscan related functions into util.
> > Make use of win32 intrinsics for util_last_bit/fls if present.
> >
> > Signed-off-by: Mathias Fröhlich
> >
> >
>
On Friday, August 05, 2016 07:14:07 Jason Ekstrand wrote:
> On Aug 4, 2016 10:36 PM, wrote:
> >
> > From: Mathias Fröhlich
> >
> > Hi Brian,
> >
> > As requested with the initial creation of util/bitscan.h
> > now move other bitscan related functions into util.
> > Make use of win32 intrinsics fo
On Fri, Aug 5, 2016 at 7:05 PM, ⚛ <0xe2.0x9a.0...@gmail.com> wrote:
> Question 4: Is it planned for *_dri.so belonging to Gallium/DRI _not_
> to be mapped to the same inode on the filesystem in the future? If
> there is no such plan, what was the original point of having multiple
> _dri.so files ma
On Fri, Aug 5, 2016 at 8:42 PM, Jan Ziak <0xe2.0x9a.0...@gmail.com> wrote:
> Mesa source code prior to this patch uses both RTLD_NOW and RTLD_LAZY.
> This patch removes all RTLD_NOW in favor of RTLD_LAZY.
>
> In comparison to early binding, lazy binding reduces CPU instruction count
> of small GL a
On Sat, Aug 6, 2016 at 3:37 AM, Jan Vesely wrote:
> On Sat, 2016-08-06 at 02:42 +0200, Jan Ziak wrote:
>> Mesa source code prior to this patch uses both RTLD_NOW and
>> RTLD_LAZY.
>> This patch removes all RTLD_NOW in favor of RTLD_LAZY.
>>
>> In comparison to early binding, lazy binding reduces C
On Sat, 2016-08-06 at 02:42 +0200, Jan Ziak wrote:
> Mesa source code prior to this patch uses both RTLD_NOW and
> RTLD_LAZY.
> This patch removes all RTLD_NOW in favor of RTLD_LAZY.
>
> In comparison to early binding, lazy binding reduces CPU instruction
> count
> of small GL apps (e.g: glxinfo)
Mesa source code prior to this patch uses both RTLD_NOW and RTLD_LAZY.
This patch removes all RTLD_NOW in favor of RTLD_LAZY.
In comparison to early binding, lazy binding reduces CPU instruction count
of small GL apps (e.g: glxinfo) by 6 million instructions.
Larger apps won't notice the differenc
On Fri, 2016-08-05 at 16:27 -0700, Eric Anholt wrote:
> vc4 wants to have per-scalar IO load/stores so that dead code
> elimination
> can happen on a more granular basis, which it has been doing in the
> backend using a multiplication by 4 of the intrinsic's
> driver_location.
> We can represent it
Fixes the following failures :
dEQP-VK.api.copy_and_blit.resolve_image.whole_4_bit
dEQP-VK.api.copy_and_blit.resolve_image.whole_8_bit
dEQP-VK.api.copy_and_blit.resolve_image.partial_4_bit
dEQP-VK.api.copy_and_blit.resolve_image.partial_8_bit
dEQP-VK.api.copy_and_blit.resolve_image.with_regions_4_
On Fri, Aug 5, 2016 at 4:38 PM, Jason Ekstrand wrote:
>
>
> On Fri, Aug 5, 2016 at 4:29 PM, Anuj Phogat wrote:
>
>>
>>
>> On Fri, Aug 5, 2016 at 3:57 PM, Lionel Landwerlin
>> wrote:
>>
>>> Fixes the following failures :
>>>
>>> dEQP-VK.api.copy_and_blit.resolve_image.whole_4_bit
>>> dEQP-VK.api
On Fri, Aug 5, 2016 at 4:29 PM, Anuj Phogat wrote:
>
>
> On Fri, Aug 5, 2016 at 3:57 PM, Lionel Landwerlin
> wrote:
>
>> Fixes the following failures :
>>
>> dEQP-VK.api.copy_and_blit.resolve_image.whole_4_bit
>> dEQP-VK.api.copy_and_blit.resolve_image.whole_8_bit
>> dEQP-VK.api.copy_and_blit.re
https://bugs.freedesktop.org/show_bug.cgi?id=93103
Jan Ziak <0xe2.0x9a.0...@gmail.com> changed:
What|Removed |Added
CC||0xe2.0x9a.0...@gmail
On Fri, Aug 5, 2016 at 3:57 PM, Lionel Landwerlin
wrote:
> Fixes the following failures :
>
> dEQP-VK.api.copy_and_blit.resolve_image.whole_4_bit
> dEQP-VK.api.copy_and_blit.resolve_image.whole_8_bit
> dEQP-VK.api.copy_and_blit.resolve_image.partial_4_bit
> dEQP-VK.api.copy_and_blit.resolve_image
vc4 wants to have per-scalar IO load/stores so that dead code elimination
can happen on a more granular basis, which it has been doing in the
backend using a multiplication by 4 of the intrinsic's driver_location.
We can represent it properly in the NIR using the first_component field,
though.
---
This reduces the diff between GLSL-to-NIR and TGSI-to-NIR, and gives NIR
more optimization to work on.
---
src/gallium/auxiliary/nir/tgsi_to_nir.c| 30 --
.../drivers/freedreno/ir3/ir3_compiler_nir.c | 46 --
src/gallium/drivers/vc4/vc4_nir_lower_i
This gives a chance for things like "gl_FrontFace != 0" to get optimized
out.
---
src/compiler/nir/nir_search.c | 60 ---
1 file changed, 45 insertions(+), 15 deletions(-)
diff --git a/src/compiler/nir/nir_search.c b/src/compiler/nir/nir_search.c
index b21f
Here's a little miniseries from my trying to convert from using TTN to
usually using GTN in vc4. No change on shader-db, but seems like a
good idea.
Eric Anholt (3):
nir: Tell opt_algebraic that load_front_face is a boolean.
nir: Use the system-value front face for twoside lowering.
ttn: Us
GLSL-to-NIR generates system value usage, and vc4/freedreno would both
like the system value instead of the varying, so switch this pass over to
it.
---
src/compiler/nir/nir_lower_two_sided_color.c | 23 +++
1 file changed, 7 insertions(+), 16 deletions(-)
diff --git a/src/com
Fixes the following failures :
dEQP-VK.api.copy_and_blit.resolve_image.whole_4_bit
dEQP-VK.api.copy_and_blit.resolve_image.whole_8_bit
dEQP-VK.api.copy_and_blit.resolve_image.partial_4_bit
dEQP-VK.api.copy_and_blit.resolve_image.partial_8_bit
dEQP-VK.api.copy_and_blit.resolve_image.with_regions_4_
Fixes the following failures :
dEQP-VK.api.copy_and_blit.resolve_image.whole_4_bit
dEQP-VK.api.copy_and_blit.resolve_image.whole_8_bit
dEQP-VK.api.copy_and_blit.resolve_image.partial_4_bit
dEQP-VK.api.copy_and_blit.resolve_image.partial_8_bit
dEQP-VK.api.copy_and_blit.resolve_image.with_regions_4_
Hi,
This updates a previous patch, based on the review feedback from Anuj.
Thanks a lot for your time!
Lionel
Lionel Landwerlin (2):
anv: gen7: rename rasterization state variable
anv: gen7/75: enable multisampling
src/intel/vulkan/gen7_pipeline.c | 39 -
Signed-off-by: Lionel Landwerlin
Cc: anuj.pho...@gmail.com
---
src/intel/vulkan/gen7_pipeline.c | 16
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/src/intel/vulkan/gen7_pipeline.c b/src/intel/vulkan/gen7_pipeline.c
index 6acdd85..c2a38ac 100644
--- a/src/intel/vu
On Fri, Aug 5, 2016 at 1:23 PM, Anuj Phogat wrote:
>
> On Fri, Aug 5, 2016 at 10:05 AM, Lionel Landwerlin <
> lionel.g.landwer...@intel.com> wrote:
>
>> Fixes 6 failures from dEQP-VK.api.copy_and_blit.resolve_image.*
>>
> No tests are run with --deqp-case=dEQP-VK.api.copy_a
> nd_blit.resolve_ima
On Sun, Jul 31, 2016 at 12:43 AM, Pohjolainen, Topi <
topi.pohjolai...@intel.com> wrote:
> On Sun, Jul 31, 2016 at 10:32:33AM +0300, Pohjolainen, Topi wrote:
> > On Tue, Jul 26, 2016 at 03:11:06PM -0700, Jason Ekstrand wrote:
> > > ---
> > > src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 3 ++-
>
On Fri, Aug 5, 2016 at 10:05 AM, Lionel Landwerlin <
lionel.g.landwer...@intel.com> wrote:
> Fixes 6 failures from dEQP-VK.api.copy_and_blit.resolve_image.*
>
No tests are run with --deqp-case=dEQP-VK.api.copy_and_blit.resolve_image.*
This is what i'm seeing on hsw without this patch:
./deqp-vk
On Friday, August 5, 2016 6:23:47 PM PDT Lionel Landwerlin wrote:
> On 05/08/16 18:08, Jason Ekstrand wrote:
> >
> > On Aug 5, 2016 9:49 AM, "Lionel Landwerlin"
> > mailto:lionel.g.landwer...@intel.com>>
> > wrote:
> > >
> > > It doesn't seem like we have any support for 32bits of something +
>
On Fri, Aug 5, 2016 at 9:51 AM, Nanley Chery wrote:
> On Thu, Aug 04, 2016 at 08:07:08PM -0700, Jason Ekstrand wrote:
> > Drp... There's more work to do here to get the behavior 100% correct on
> > gen7 but this fixes a bug and is a step in the right direction.
> >
>
> You're right. To be more ac
On Fri, Aug 5, 2016 at 10:23 AM, Lionel Landwerlin <
lionel.g.landwer...@intel.com> wrote:
> On 05/08/16 18:08, Jason Ekstrand wrote:
>
> On Aug 5, 2016 9:49 AM, "Lionel Landwerlin"
> wrote:
> >
> > It doesn't seem like we have any support for 32bits of something + 8bits
> of
> > something else.
> Most people just consume the emails/patches directly via their email client.
> You can make the subject be generated properly by passing -vN to git
> format-patch.
I think I will create a tool for me.
___
mesa-dev mailing list
mesa-dev@lists.freedesk
On 05/08/16 18:08, Jason Ekstrand wrote:
On Aug 5, 2016 9:49 AM, "Lionel Landwerlin"
mailto:lionel.g.landwer...@intel.com>>
wrote:
>
> It doesn't seem like we have any support for 32bits of something +
8bits of
> something else.
Not quite. Depth-stencil formats are weird because they descr
On Aug 5, 2016 9:49 AM, "Lionel Landwerlin"
wrote:
>
> It doesn't seem like we have any support for 32bits of something + 8bits
of
> something else.
Not quite. Depth-stencil formats are weird because they describe two
different things at the same time. In our hardware (post Sandy Bridge) the
dept
Fixes 6 failures from dEQP-VK.api.copy_and_blit.resolve_image.*
Tested on IVB/HSW
v2: Program multisample dispatch mode correctly
Signed-off-by: Lionel Landwerlin
---
src/intel/vulkan/gen7_pipeline.c | 46 ++--
1 file changed, 30 insertions(+), 16 deletions(
On Thu, Aug 04, 2016 at 08:07:08PM -0700, Jason Ekstrand wrote:
> Drp... There's more work to do here to get the behavior 100% correct on
> gen7 but this fixes a bug and is a step in the right direction.
>
You're right. To be more accurate, I'll rename the commit title to:
"anv/gen7_pipeline: Set
It doesn't seem like we have any support for 32bits of something + 8bits of
something else. This fixes
dEQP-VK.api.copy_and_blit.blit_image.all_formats.scaling_partial_d32_sfloat_s8_uint_d32_sfloat_s8_uint_nearest
Signed-off-by: Lionel Landwerlin
---
src/intel/vulkan/anv_formats.c | 2 +-
1 file
https://bugs.freedesktop.org/show_bug.cgi?id=97214
Mark Janes changed:
What|Removed |Added
CC||emil.l.veli...@gmail.com
--
You are receiv
On Aug 5, 2016 07:56, "⚛" <0xe2.0x9a.0...@gmail.com> wrote:
>
> On Fri, Aug 5, 2016 at 2:34 PM, Eric Engestrom
> wrote:
> > On Wed, Aug 03, 2016 at 01:22:35PM +0200, Jan Ziak wrote:
> > It is customary to put [PATCH vN] in the subject to keep track of which
> > version is current, as well as to ad
The idea is that gallium is an unstable unversioned API, and if you want
something else, then you make a state tracker which exposes a stable API.
This can be an API for which there is a preexisting standard, like va-api
or vdpau, or it can be one of your own creation which is consumed by some
othe
On 08/04/2016 11:35 PM, mathias.froehl...@gmx.net wrote:
From: Mathias Fröhlich
Hi Brian,
As requested with the initial creation of util/bitscan.h
now move other bitscan related functions into util.
Make use of win32 intrinsics for util_last_bit/fls if present.
Signed-off-by: Mathias Fröhlich
On 08/04/2016 11:31 PM, mathias.froehl...@gmx.net wrote:
From: Mathias Fröhlich
Hi all,
The below was found by inspection.
Please review.
Thanks
Mathias
On gl{Push,Pop}Attrib(GL_CLIENT_VERTEX_ARRAY_BIT) take
care that gl_vertex_array_object::VertexAttribBufferMask
matches the bound buffer
On Friday 05 August 2016, mathias.froehl...@gmx.net wrote:
> From: Mathias Fröhlich
>
> Hi all,
>
> The below was found by inspection.
> Please review.
>
> Thanks
>
> Mathias
>
>
>
> On gl{Push,Pop}Attrib(GL_CLIENT_VERTEX_ARRAY_BIT) take
> care that gl_vertex_array_object::VertexAttribBuffe
On Aug 4, 2016 10:36 PM, wrote:
>
> From: Mathias Fröhlich
>
> Hi Brian,
>
> As requested with the initial creation of util/bitscan.h
> now move other bitscan related functions into util.
> Make use of win32 intrinsics for util_last_bit/fls if present.
Of you made material changes during the mov
On Aug 5, 2016 2:30 AM, "Lionel Landwerlin"
wrote:
>
> On 03/08/16 16:01, Jason Ekstrand wrote:
>>
>> On Aug 3, 2016 5:53 AM, "Lionel Landwerlin" <
lionel.g.landwer...@intel.com> wrote:
>> >
>> > We let the user believe we support some transfer destination formats
>> > which we don't. This can lea
On Fri, Aug 5, 2016 at 2:34 PM, Eric Engestrom
wrote:
> On Wed, Aug 03, 2016 at 01:22:35PM +0200, Jan Ziak wrote:
> It is customary to put [PATCH vN] in the subject to keep track of which
> version is current, as well as to add a description of what changed
> since the previous version, here, afte
On Wed, Aug 03, 2016 at 01:25:08PM +0200, ⚛ wrote:
> Your patch created rev4 at https://patchwork.freedesktop.org/series/10429/
>
> I just created rev5 which no longer includes field hasPresent.
Oh, OK, I read this mail after sending my previous mail (a couple
minutes ago), but my point there sti
Fixes 6 failures from dEQP-VK.api.copy_and_blit.resolve_image.*
Tested on IVB/HSW
Signed-off-by: Lionel Landwerlin
---
src/intel/vulkan/gen7_pipeline.c | 46 ++--
1 file changed, 30 insertions(+), 16 deletions(-)
diff --git a/src/intel/vulkan/gen7_pipeline.c
On Wed, Aug 03, 2016 at 01:22:35PM +0200, Jan Ziak wrote:
> Signed-off-by: Jan Ziak (http://atom-symbol.net) <0xe2.0x9a.0...@gmail.com>
> ---
It is customary to put [PATCH vN] in the subject to keep track of which
version is current, as well as to add a description of what changed
since the previo
On Aug 5, 2016 10:54 AM, "Nicolai Hähnle" wrote:
>
>
>
> On 04.08.2016 21:42, Marek Olšák wrote:
>>
>> From: Marek Olšák
>>
>> Some ideas copied from Jakob Sinclair's implementation, but the color
>> clearing is completely different.
>>
>> v2: remove leftover code, disable conditional rendering
>
On Fri, 2016-08-05 at 08:35 +1000, Timothy Arceri wrote:
> Thanks for you patience with this series :) The parameter check above
The other way around, thanks to you for reviewing! ☺
> is a nice cleanup too thanks. Patches 2 and 3 are:
>
> Reviewed-by: Timothy Arceri
... and it has already lan
https://bugs.freedesktop.org/show_bug.cgi?id=97214
Laurent carlier changed:
What|Removed |Added
CC||drink...@chromium.org
--
You are rece
https://bugs.freedesktop.org/show_bug.cgi?id=97214
Alexandr Zelinskii changed:
What|Removed |Added
CC||mexahota...@w1l.ru
--
You are rece
https://bugs.freedesktop.org/show_bug.cgi?id=97214
Bug ID: 97214
Summary: X not running with error "Failed to make EGL context
current"
Product: Mesa
Version: git
Hardware: x86-64 (AMD64)
OS: Linux (All)
On 05/08/16 01:53, Timothy Arceri wrote:
> On Thu, 2016-08-04 at 18:36 +0200, Alejandro Piñeiro wrote:
>> Hi,
>>
>> these days I have been trying to fix a test that uses transform
>> feedback
>> on the out varying of a tessellation shader. The relevant part on
>> that
>> shader is like this:
>>
>>
Hi!
I have taken a look to the GL44-CTS.compute_shader.api-indirect test
and I agree with Piñeiro that this test is not correct. A call to
glUseProgram to set the active program is missing in the test.
The test passes invalid values as parameters to
glDispatchComputeIndirect
like,
glDispa
On 03/08/16 16:01, Jason Ekstrand wrote:
On Aug 3, 2016 5:53 AM, "Lionel Landwerlin"
mailto:lionel.g.landwer...@intel.com>>
wrote:
>
> We let the user believe we support some transfer destination formats
> which we don't. This can lead to crashes when actually trying to use
> those formats fo
On 04.08.2016 21:42, Marek Olšák wrote:
From: Marek Olšák
Some ideas copied from Jakob Sinclair's implementation, but the color
clearing is completely different.
v2: remove leftover code, disable conditional rendering
---
docs/GL3.txt | 2 +-
docs/relnotes/12.1
On Friday, August 5, 2016 11:08:20 AM PDT Timothy Arceri wrote:
> ---
> src/compiler/nir/nir_opt_dead_cf.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/src/compiler/nir/nir_opt_dead_cf.c
> b/src/compiler/nir/nir_opt_dead_cf.c
> index 81c1b65..3551124 100644
> --- a/
On 05/08/16 03:39, Chad Versace wrote:
On 08/04/2016 02:28 PM, Martin Peres wrote:
On 05/08/16 00:24, Mark Janes wrote:
I saw no regressions from these tests. However, I wasn't able to run
the dEQP-EGL tests the last time I tried. What branch of dEQP are you
running?
InternalError (Runt
On 04.08.2016 22:30, Marek Olšák wrote:
On Tue, Aug 2, 2016 at 4:38 PM, Nicolai Hähnle wrote:
On 29.07.2016 23:42, Marek Olšák wrote:
From: Marek Olšák
---
src/gallium/winsys/amdgpu/drm/amdgpu_cs.c | 2 +-
src/gallium/winsys/radeon/drm/radeon_drm_cs.c | 2 +-
2 files changed, 2 inserti
63 matches
Mail list logo
