On 05.10.2016 20:48, Samuel Pitoiset wrote:
v4: - slightly indent spec quotes (Nicolai)
- drop useless _mesa_has_compute_shaders() check (Nicolai)
- move the fixed local size outside of the loop (Nicolai)
- add missing check for invalid use of work group count
v2: - update formatting
On 05.10.2016 20:48, Samuel Pitoiset wrote:
This is the new layout qualifier introduced by
ARB_compute_variable_group_size which allows to use a variable work
group size.
v4: - add missing '%s' in the monster format string
Signed-off-by: Samuel Pitoiset
Reviewed-by: Ian Romanick
Reviewed-by
On 05.10.2016 22:08, Axel Davy wrote:
Hi there,
here is our serie for Mesa 13.
Basically there are a lot of fixes, from solving app crashes, hangs
and important visual bugs, to lightning issues on old apps.
On the feature side, now the few apps that were missing multisampling
settings now have
On 06.10.2016 02:42, Brian Paul wrote:
Before, we had code to compute the sampler view's format spread across two
different functions: in update_single_texture() and
st_get_texture_sampler_view_from_stobj(). Now it's all in one new function.
Also, use _mesa_texture_base_format() to simplify the
On 06.10.2016 02:42, Brian Paul wrote:
Before, st_get_texture_sampler_view_from_stobj() did a lot of work to
check if the texture parameters matched the sampler view (format,
swizzle, min/max lod, first/last layer, etc). We did this every time
we validated the texture state.
Now, we use a ctx->
On Friday, September 30, 2016 8:12:12 AM PDT Ian Romanick wrote:
> From: Tapani Pälli
>
> Changes make copy_propagation_elements pass faster, reducing link
> time spent in test case of bug 94477. Does not fix the actual issue
> but brings down the total time. No regressions seen in CI.
>
> v2 (i
https://bugs.freedesktop.org/show_bug.cgi?id=98106
Bug ID: 98106
Summary: Primitive restart should be disabled for Draw commands
when GL > 4.5
Product: Mesa
Version: 11.2
Hardware: Other
OS: All
https://bugs.freedesktop.org/show_bug.cgi?id=98106
Markus Wick changed:
What|Removed |Added
CC||wickmar...@web.de
--
You are receiving th
https://bugs.freedesktop.org/show_bug.cgi?id=98106
Martina Kollarova changed:
What|Removed |Added
Summary|Primitive restart should be |Primitive restart should be
On Thursday, October 6, 2016 1:48:04 AM PDT Philipp Zabel wrote:
> According to chapters 16.5. (Timestamp Queries) and 30.2 (Limits) of the
> Vulkan Specification 1.0.29, the .limits.timestampPeriod field returned
> by vkGetPhysicalDeviceProperties is measured in nanoseconds, not in
> seconds.
>
>
On 5 October 2016 at 21:08, Axel Davy wrote:
> Fixes several game bugs with nine.
> Fixes https://github.com/iXit/Mesa-3D/issues/234
>
> Signed-off-by: Axel Davy
> Signed-off-by: Patrick Rudolph
>
You really want s/aliasing/aliasing of dst/ in the commit summary.
Having a description of the bug
On 5 October 2016 at 21:08, Axel Davy wrote:
> Uses the new defines introduced in previous commit.
>
Please describe why the newly introduced flags are used over the
present ones. Worth copying some of the in-source comment or
referencing it ?
-Emil
___
Hi Axel,
On 5 October 2016 at 21:08, Axel Davy wrote:
> And cap to 2 GB on 32 bits.
>
> Fixes https://bugs.freedesktop.org/show_bug.cgi?id=94561
>
> Signed-off-by: Axel Davy
> Signed-off-by: Patrick Rudolph
> ---
> src/gallium/auxiliary/os/os_misc.c | 2 +-
> src/gallium/drivers/llvmpipe
On 5 October 2016 at 21:08, Axel Davy wrote:
> Hi there,
>
> here is our serie for Mesa 13.
>
> Basically there are a lot of fixes, from solving app crashes, hangs
> and important visual bugs, to lightning issues on old apps.
>
Echoing Nicolai and earlier question/suggestion of my, can bugfixes
ap
On 10/06/2016 11:29 AM, Kenneth Graunke wrote:
On Friday, September 30, 2016 8:12:12 AM PDT Ian Romanick wrote:
From: Tapani Pälli
Changes make copy_propagation_elements pass faster, reducing link
time spent in test case of bug 94477. Does not fix the actual issue
but brings down the total t
Ping?
On Oct 4, 2016 11:05 AM, "Chuck Atkins" wrote:
> This fixes a problem where GL headers would only get installed if
> glx was enabled. So if osmesa was enabled but not glx, then the
> GL headers required by osmesa would be missing from the install.
>
> v2: Dropped unneeded mesa_glinterop.h
On 6 October 2016 at 02:27, Dave Airlie wrote:
> If nobody has any remaining major objections I'd rather not spend too
> much time rebasing this stuff, so I'd like to propose merging it
> approx now + 24hr, if anyone wants to explicitly ack or r-b anything
> let me know, but I'm most likely going
From: Emil Velikov
Driver should enumerate only up-to min2(num_available, num_requested)
properties and return VK_INCOMPLETE if the # of requested props is
smaller than the ones available.
Presently we assert out in such cases.
Inspired by a similar fix for RADV.
Should fix: dEQP-VK.api.info.d
https://bugs.freedesktop.org/show_bug.cgi?id=98106
Martina Kollarova changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|mesa-dev
The OpenGL 4.5 spec updated the section on primitive restart, and now it
doesn't have to be performed on drawing commands not taking a parameter,
regardless of whether PRIMITIVE_RESTART_FIXED_INDEX is enabled or not.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98106
Signed-off-by: Marti
Reviewed-by: Pierre Moreau
On 12:36 am - Oct 06 2016, Karol Herbst wrote:
> helped some ue4 demos and divinity OS shaders
>
> total instructions in shared programs : 2818674 -> 2818606 (-0.00%)
> total gprs used in shared programs: 379273 -> 379273 (0.00%)
> total local used in shared progra
https://bugs.freedesktop.org/show_bug.cgi?id=97524
Nicolai Hähnle changed:
What|Removed |Added
QA Contact|dri-devel@lists.freedesktop |mesa-dev@lists.freedesktop.
https://bugs.freedesktop.org/show_bug.cgi?id=97524
Nicolai Hähnle changed:
What|Removed |Added
Summary|Invalid sampler settings|Samplers referring to the
This tries to mirrors the codepath taken by DRI2 in IntelSetTexBuffer2()
and fixes many applications when using DRI3:
- Totem with libva on hw-accelerated decoding
- obs-studio, using Window Capture (Xcomposite) as a Source
- gstreamer with VAAPI
v2:
- introduce get_dri_screen() in the dri3 lo
This allows querying the current active screen from the
loader's common code.
Signed-off-by: Martin Peres
---
src/egl/drivers/dri2/platform_x11_dri3.c | 12
src/glx/dri3_glx.c | 11 +++
src/loader/loader_dri3_helper.h | 1 +
3 files changed, 2
That's funny, because Polaris is our first hardware that supports
primitive restart with DrawArrays (including draw indirect), while
OpenGL removed the support at the same time.
Reviewed-by: Marek Olšák
Marek
On Thu, Oct 6, 2016 at 3:22 PM, Martina Kollarova
wrote:
> The OpenGL 4.5 spec update
On 2016-10-06 15:22, Martina Kollarova wrote:
> The OpenGL 4.5 spec updated the section on primitive restart, and now it
> doesn't have to be performed on drawing commands not taking a parameter,
> regardless of whether PRIMITIVE_RESTART_FIXED_INDEX is enabled or not.
>
> Bugzilla: https://bugs.fr
CLOVER_CL_VERSION_OVERRIDE allows overriding default OpenCL version
supported by Clover, analogous to MESA_GL_VERSION_OVERRIDE for OpenGL.
CLOVER_CL_C_VERSION_OVERRIDE allows overridng default OpenCL C version.
---
docs/envvars.html | 12
src/galliu
Make sure that a struct argument did not get compiled into a pointer
type with the byval attribute. If we try to handle the pointer with
byval, we end up with the pointer size instead of the struct size.
Struct arguments on AMD GPUs will work correctly on Clang versions
containing https://reviews.
On 10/06/2016 04:26 PM, Vedran Miletić wrote:
> CLOVER_CL_VERSION_OVERRIDE allows overriding default OpenCL version
> supported by Clover, analogous to MESA_GL_VERSION_OVERRIDE for OpenGL.
> CLOVER_CL_C_VERSION_OVERRIDE allows overridng default OpenCL C version.
> ---
> docs/envvars.html
On 6 October 2016 at 15:13, Martin Peres wrote:
> This allows querying the current active screen from the
> loader's common code.
>
> Signed-off-by: Martin Peres
> ---
> src/egl/drivers/dri2/platform_x11_dri3.c | 12
> src/glx/dri3_glx.c | 11 +++
> src
On 4 October 2016 at 16:05, Chuck Atkins wrote:
> This fixes a problem where GL headers would only get installed if
> glx was enabled. So if osmesa was enabled but not glx, then the
> GL headers required by osmesa would be missing from the install.
>
> v2: Dropped unneeded mesa_glinterop.h redund
No longer required.
Signed-off-by: Steven Toth
---
src/gallium/auxiliary/hud/hud_cpufreq.c | 9 -
src/gallium/auxiliary/hud/hud_diskstat.c | 8
src/gallium/auxiliary/hud/hud_nic.c | 16
src/gallium/auxiliary/hud/hud_sensors_temp.c | 19 --
Hi Chad,
On 4 October 2016 at 23:37, Chad Versace wrote:
> We locked an unitialized mutex in the callstack
> glClientWaitSync
> intel_gl_client_wait_sync
> brw_fence_client_wait_sync
> because we forgot to initialize it in intel_gl_fence_sync.
> (The EGLSync codepath didn't have this
On 10/06/2016 01:45 AM, Nicolai Hähnle wrote:
On 06.10.2016 02:42, Brian Paul wrote:
Before, st_get_texture_sampler_view_from_stobj() did a lot of work to
check if the texture parameters matched the sampler view (format,
swizzle, min/max lod, first/last layer, etc). We did this every time
we va
On 6 October 2016 at 16:26, Steven Toth wrote:
> No longer required.
>
> Signed-off-by: Steven Toth
> ---
> src/gallium/auxiliary/hud/hud_cpufreq.c | 9 -
> src/gallium/auxiliary/hud/hud_diskstat.c | 8
> src/gallium/auxiliary/hud/hud_nic.c | 16 -
https://bugs.freedesktop.org/show_bug.cgi?id=98048
Emil Velikov changed:
What|Removed |Added
CC||b...@bwidawsk.net,
|
https://bugs.freedesktop.org/show_bug.cgi?id=98048
Emil Velikov changed:
What|Removed |Added
Assignee|mesa-dev@lists.freedesktop. |x...@lists.freedesktop.org
On Thu, Oct 6, 2016 at 1:09 AM, Gustaw Smolarczyk wrote:
> It's supposed to be how much at least we want to grow the cs, not the
> minimum size of the cs after growth.
> ---
> src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/
On Thu, Oct 6, 2016 at 1:09 AM, Gustaw Smolarczyk wrote:
> If the user created a fence with VK_FENCE_CREATE_SIGNALED_BIT set, we
> shouldn't fail to wait for a fence if it was not submitted since that is
> not necessary.
> ---
> src/amd/vulkan/radv_device.c | 6 +++---
> 1 file changed, 3 inserti
On Wed, Oct 5, 2016 at 9:11 PM, Jason Ekstrand wrote:
> On Wed, Oct 5, 2016 at 7:05 PM, Xu, Randy wrote:
>
>> Hi, Jason
>>
>>
>>
>> Do you want to add this assert in the patch? I did some test, no issue
>> found, but I don’t see the case that we need override the texture target in
>> brw_emit_su
Pushed!
Tim, this is going to cause you a bit of rebase trouble on loop unrolling
but it should actually make your code simpler.
On Wed, Oct 5, 2016 at 9:56 PM, Jason Ekstrand wrote:
> On Oct 5, 2016 21:42, "Connor Abbott" wrote:
> >
> > Thanks for doing this! This has always bugged me. For th
On Wed, Oct 5, 2016 at 3:56 PM, Lionel Landwerlin
wrote:
> Embed the xml files into the binary, so aubinator can be used from any
> location.
>
> v2: Split generation packing into another patch (Jason)
> Check for xxd (Jason)
>
> v3: Fix out of tree builds (Jason)
> Generate custom variab
On Thu, 2016-10-06 at 16:26 +0200, Vedran Miletić wrote:
> CLOVER_CL_VERSION_OVERRIDE allows overriding default OpenCL version
> supported by Clover, analogous to MESA_GL_VERSION_OVERRIDE for
> OpenGL.
> CLOVER_CL_C_VERSION_OVERRIDE allows overridng default OpenCL C
> version.
WHat's the use of CL
On Fri, 2014-12-19 at 16:42 +0100, EdB wrote:
> CL_PROGRAM_BINARY_TYPE have been added to clGetProgramBuildInfo in
> CL1.2
> ---
> src/gallium/state_trackers/clover/api/program.cpp | 4 +++
> src/gallium/state_trackers/clover/core/program.cpp | 31
> +-
> src/gallium/state_tr
On Wed, Oct 5, 2016 at 7:25 PM, Timothy Arceri wrote:
> Just
>
> On Wed, 2016-10-05 at 16:23 -0700, Jason Ekstrand wrote:
> >
> >
> > On Thu, Sep 15, 2016 at 12:03 AM, Timothy Arceri > abora.com> wrote:
> > > V2:
> > > - tidy ups suggested by Connor.
> > > - tidy up cloning logic and handle copy
For the series:
Reviewed-by: Marek Olšák
Marek
On Thu, Oct 6, 2016 at 2:42 AM, Brian Paul wrote:
> There's no need to cast to st_texture_image. Just use gl_texture_image.
>
> Reviewed-by: Edward O'Callaghan
> ---
> src/mesa/state_tracker/st_atom_texture.c | 10 +-
> 1 file changed,
On 06/10/2016 11:44, Emil Velikov wrote:
Hi Axel,
You seem to have forgotten/ignored virtually every suggestion for this
patch from last time around.
Did you send the wrong patch or ?
Emil
Well it was sitting on our repo, and I got persuaded I had addressed the
issues.
Looks like not.
I'
https://bugs.freedesktop.org/show_bug.cgi?id=97952
--- Comment #4 from Rob Clark ---
(In reply to Timothy Arceri from comment #3)
> I believe its because the ffs param in string.h is an int while the one in
> bitscan.h is unsigned.
oh, yeah, probably should be int in bitscan.h then
(still not e
On Thu, Oct 6, 2016 at 10:22 AM, Jason Ekstrand
wrote:
>
>
> On Wed, Oct 5, 2016 at 7:25 PM, Timothy Arceri <
> timothy.arc...@collabora.com> wrote:
>
>> Just
>>
>> On Wed, 2016-10-05 at 16:23 -0700, Jason Ekstrand wrote:
>> >
>> >
>> > On Thu, Sep 15, 2016 at 12:03 AM, Timothy Arceri > > abora.co
It's supposed to be how much at least we want to grow the cs, not the
minimum size of the cs after growth.
v2: Unbreak use_ib_bos.
Don't mask the ib_size when !use_ib_bos, since it's not needed.
---
src/amd/vulkan/winsys/amdgpu/radv_amdgpu_cs.c | 11 +++
1 file changed, 7 insertions(+
On 32 bits system, application memory is quite limited.
softpipe uses application memory. To help prevent memory
exhaustion, limit reported memory availability to 2GB.
Some gallium nine apps do check reported memory by allocating
resources until memory is full. Gallium nine refuses allocations
whe
On systems with more than 4GB of ram,
os_get_total_physical_memory was triggering an integer
overflow for the linux and haiku path, when on
32 bits.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94561
Signed-off-by: Axel Davy
---
src/gallium/auxiliary/os/os_misc.c | 4 ++--
1 file chan
On 32 bits system, application memory is quite limited.
llvmpipe uses application memory. To help prevent memory
exhaustion, limit reported memory availability to 2GB.
Some gallium nine apps do check reported memory by allocating
resources until memory is full. Gallium nine refuses allocations
whe
Gallium nine relies on aliasing to work with this function.
Without this patch, dirty region tracking was incorrect, which
could lead to incorrect textures or vertex buffers.
Fixes several game bugs with nine.
Fixes https://github.com/iXit/Mesa-3D/issues/234
Signed-off-by: Axel Davy
Reviewed-by:
Gallium nine relies on aliasing to work with this function.
Without this patch, dirty region tracking was incorrect, which
could lead to incorrect textures or vertex buffers.
Fixes several game bugs with nine.
Fixes https://github.com/iXit/Mesa-3D/issues/234
Signed-off-by: Axel Davy
Reviewed-by:
Sorry, ignore this, I sent a new one with the corrected title.
On 06/10/2016 19:57, Axel Davy wrote:
Gallium nine relies on aliasing to work with this function.
Without this patch, dirty region tracking was incorrect, which
could lead to incorrect textures or vertex buffers.
Fixes several game b
Am 06.10.2016 um 19:51 schrieb Axel Davy:
> On 32 bits system, application memory is quite limited.
> softpipe uses application memory. To help prevent memory
> exhaustion, limit reported memory availability to 2GB.
>
> Some gallium nine apps do check reported memory by allocating
> resources unti
On 10/06/2016 07:15 PM, Jan Vesely wrote:
> On Thu, 2016-10-06 at 16:26 +0200, Vedran Miletić wrote:
>> CLOVER_CL_VERSION_OVERRIDE allows overriding default OpenCL version
>> supported by Clover, analogous to MESA_GL_VERSION_OVERRIDE for
>> OpenGL.
>> CLOVER_CL_C_VERSION_OVERRIDE allows overridng d
On 06/10/2016 11:34, Emil Velikov wrote:
On 5 October 2016 at 21:08, Axel Davy wrote:
Uses the new defines introduced in previous commit.
Please describe why the newly introduced flags are used over the
present ones. Worth copying some of the in-source comment or
referencing it ?
-Emil
Do
If pPhysicalDevices is too small for all physical devices,
the driver must return VK_INCOMPLETE.
Since only a single physical device is supported, this is only the case
when pPhysicalDeviceCount == 0 && pPhysicalDevices != NULL.
---
src/intel/vulkan/anv_device.c | 2 ++
1 file changed, 2 insertion
On Wed, Oct 05, 2016 at 05:36:43PM -0700, Jason Ekstrand wrote:
> Initially, we had intended set_subpass to be an interesting function that
> did whatever (presumably a lot) setup we needed for a subpass. In reality,
> it just sets a pointer and a dirty bit and then emits depth and stencil
> state
On Thu, Oct 6, 2016 at 12:30 PM, Nanley Chery wrote:
> On Wed, Oct 05, 2016 at 05:36:43PM -0700, Jason Ekstrand wrote:
> > Initially, we had intended set_subpass to be an interesting function that
> > did whatever (presumably a lot) setup we needed for a subpass. In
> reality,
> > it just sets a
Signed-off-by: Adam Jackson
---
REVIEWERS | 4
1 file changed, 4 insertions(+)
diff --git a/REVIEWERS b/REVIEWERS
index f7574b3..f822421 100644
--- a/REVIEWERS
+++ b/REVIEWERS
@@ -104,3 +104,7 @@ F: src/egl/drivers/dri2/platform_wayland.c
FREEDRENO
R: Rob Clark
F: src/gallium/dr
On 05/10/2016 22:08, Axel Davy wrote:
HRESULT NINE_WINAPI
NineDevice9_ProcessVertices( struct NineDevice9 *This,
UINT SrcStartIndex,
@@ -3174,33 +3188,69 @@ NineDevice9_ProcessVertices( struct NineDevice9 *This,
IDirect3DVertexDecl
I'd like to have more feedback on the idea of using jemalloc for ralloc.
Right now, I see these options:
1) Use jemalloc for ralloc and make it mandatory for all GL drivers.
- Distributions have shown that they are capable of doing anything
with the Mesa source code, so they don't need --disable-
On Thu, Oct 6, 2016 at 4:24 PM, Axel Davy wrote:
> On 05/10/2016 22:08, Axel Davy wrote:
>>
>> HRESULT NINE_WINAPI
>> NineDevice9_ProcessVertices( struct NineDevice9 *This,
>>UINT SrcStartIndex,
>> @@ -3174,33 +3188,69 @@ NineDevice9_ProcessVertices( struct Nine
On Thu, Oct 06, 2016 at 12:35:53PM -0700, Jason Ekstrand wrote:
> On Thu, Oct 6, 2016 at 12:30 PM, Nanley Chery wrote:
>
> > On Wed, Oct 05, 2016 at 05:36:43PM -0700, Jason Ekstrand wrote:
> > > Initially, we had intended set_subpass to be an interesting function that
> > > did whatever (presumab
Marek Olšák writes:
> I'd like to have more feedback on the idea of using jemalloc for ralloc.
>
> Right now, I see these options:
>
> 1) Use jemalloc for ralloc and make it mandatory for all GL drivers.
> - Distributions have shown that they are capable of doing anything
> with the Mesa source c
On Thu, Oct 6, 2016 at 12:21 PM, Nicolas Koch wrote:
> If pPhysicalDevices is too small for all physical devices,
> the driver must return VK_INCOMPLETE.
> Since only a single physical device is supported, this is only the case
> when pPhysicalDeviceCount == 0 && pPhysicalDevices != NULL.
> ---
>
total instructions in shared programs : 2818606 -> 2818227 (-0.01%)
total gprs used in shared programs: 379273 -> 379238 (-0.01%)
total local used in shared programs : 9505 -> 9505 (0.00%)
total bytes used in shared programs : 25837192 -> 25833736 (-0.01%)
localgpr
https://bugs.freedesktop.org/show_bug.cgi?id=98128
Bug ID: 98128
Summary: nir/tests/control_flow_tests.cpp:79:73: error:
‘nir_loop_first_cf_node’ was not declared in this
scope
Product: Mesa
Version: git
>-Original Message-
>From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf Of
>Lionel Landwerlin
>Sent: Wednesday, October 05, 2016 8:55 AM
>To: mesa-dev@lists.freedesktop.org
>Cc: Landwerlin, Lionel G
>Subject: [Mesa-dev] [PATCH 1/5] intel: aubinator: add missing return
On 10/06/2016 09:25 AM, Nicolai Hähnle wrote:
On 05.10.2016 20:48, Samuel Pitoiset wrote:
v4: - slightly indent spec quotes (Nicolai)
- drop useless _mesa_has_compute_shaders() check (Nicolai)
- move the fixed local size outside of the loop (Nicolai)
- add missing check for invalid
On 10/06/2016 09:27 AM, Nicolai Hähnle wrote:
On 05.10.2016 20:48, Samuel Pitoiset wrote:
This is the new layout qualifier introduced by
ARB_compute_variable_group_size which allows to use a variable work
group size.
v4: - add missing '%s' in the monster format string
Signed-off-by: Samuel P
>-Original Message-
>From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf Of
>Lionel Landwerlin
>Sent: Wednesday, October 05, 2016 8:55 AM
>To: mesa-dev@lists.freedesktop.org
>Cc: Landwerlin, Lionel G
>Subject: [Mesa-dev] [PATCH 2/5] intel: aubinator: use getopt to pars
This work is a revision of this series:
https://lists.freedesktop.org/archives/mesa-dev/2016-September/129845.html
And is dependent on this series:
https://patchwork.freedesktop.org/series/13360/
Cc: Chad Versace
Cc: Jason Ekstrand
Chad Versace (4):
anv: Add anv_image::hiz_surface
anv: Add
From: Jason Ekstrand
Signed-off-by: Nanley Chery
Reviewed-by: Chad Versace
Reviewed-by: Jason Ekstrand
---
src/intel/vulkan/anv_device.c | 20
src/intel/vulkan/anv_image.c | 20
2 files changed, 20 insertions(+), 20 deletions(-)
diff --git a/src/int
From: Chad Versace
Nanley Chery:
(rebase)
- Use isl_surf_get_hiz_surf()
(amend)
- Only add a HiZ surface onto a depth/stencil attachment
- Add comment above HiZ surface addition
- Hide HiZ behind INTEL_VK_HIZ prior to BDW
- Disable HiZ for untested cases
- Remove DISABLE_AUX_BIT instead of
Create a function that performs one of three HiZ operations -
depth/stencil clears, HiZ resolve, and depth resolves.
Signed-off-by: Nanley Chery
Reviewed-by: Jason Ekstrand (v2)
---
v3. Change do_hz to emit_hz (Chad)
Always set XMin YMin explicitly (Chad)
Add anv_finishme for gen7 emit_
Signed-off-by: Nanley Chery
Reviewed-by: Jason Ekstrand
---
src/intel/vulkan/TODO | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/vulkan/TODO b/src/intel/vulkan/TODO
index 8fac370..9ac63eb 100644
--- a/src/intel/vulkan/TODO
+++ b/src/intel/vulkan/TODO
@@ -19,7 +19,7
From: Chad Versace
Unused.
Signed-off-by: Nanley Chery
Reviewed-by: Jason Ekstrand
---
src/intel/vulkan/anv_private.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_private.h
index 4fa403f..dfcedd1 100644
--- a/src/intel/vulkan/anv_
From: Jason Ekstrand
Nanley Chery (amend):
- Change memset value from 0xff to 0 (a defined value for HiZ).
Signed-off-by: Nanley Chery
Reviewed-by: Chad Versace
Reviewed-by: Jason Ekstrand
---
src/intel/vulkan/anv_image.c | 31 ++-
1 file changed, 30 insertions(+
Provides an FPS increase of ~30% on the Sascha triangle and multisampling
demos.
Signed-off-by: Nanley Chery
Reviewed-by: Jason Ekstrand (v2)
---
v3. Emit required clear_params packet (Chad)
Share clear_params code path IVB+ (Jason)
src/intel/vulkan/anv_pass.c| 13 +
s
HiZ is not a color surface, but an auxiliary depth surface.
Signed-off-by: Nanley Chery
Reviewed-by: Chad Versace
Reviewed-by: Jason Ekstrand
---
src/intel/isl/isl.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/isl/isl.h b/src/intel/isl/isl.h
index 29fb3d0..967
From: Chad Versace
Nanley Chery:
(rebase)
- Resolve conflicts with new anv_batch_emit macro
(amend)
- Handle a QPitch TODO
- Emit 3DSTATE_HIER_DEPTH_BUFFER on pre-BDW systems
- Only use HiZ for single-subpass renderpasses
- Emit the HiZ instruction before the stencil instruction to follow th
From: Chad Versace
Signed-off-by: Nanley Chery
Reviewed-by: Jason Ekstrand
---
src/intel/vulkan/anv_private.h | 10 ++
1 file changed, 10 insertions(+)
diff --git a/src/intel/vulkan/anv_private.h b/src/intel/vulkan/anv_private.h
index dfcedd1..fd886bf 100644
--- a/src/intel/vulkan/anv
>-Original Message-
>From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf Of
>Lionel Landwerlin
>Sent: Wednesday, October 05, 2016 8:55 AM
>To: mesa-dev@lists.freedesktop.org
>Cc: Landwerlin, Lionel G
>Subject: [Mesa-dev] [PATCH 4/5] intel: aubinator: enable loading xml
>-Original Message-
>From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf Of
>Lionel Landwerlin
>Sent: Wednesday, October 05, 2016 8:56 AM
>To: mesa-dev@lists.freedesktop.org
>Cc: Landwerlin, Lionel G
>Subject: [Mesa-dev] [PATCH v3 5/5] intel: aubinator: enable loading
On 06/10/2016 22:28, Ilia Mirkin wrote:
On Thu, Oct 6, 2016 at 4:24 PM, Axel Davy wrote:
On 05/10/2016 22:08, Axel Davy wrote:
HRESULT NINE_WINAPI
NineDevice9_ProcessVertices( struct NineDevice9 *This,
UINT SrcStartIndex,
@@ -3174,33 +3188,69 @@ NineDevice
Debian uses dash by default, which doesn't do '+='. Fixes servo's
osmesa-based headless testing system, which was looking for libOSMesa in
the lib/ directory.
---
install-gallium-links.mk | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/install-gallium-links.mk b/install-ga
https://bugs.freedesktop.org/show_bug.cgi?id=98128
Jason Ekstrand changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
In a declaration, when a layout qualifier appears and holds duplicated
layout-qualifier-name, only the last occurrence should be taken into
account.
From page 59 (page 65 of the PDF) of the GLSL 4.40 spec:
" More than one layout qualifier may appear in a single
declaration. Additionally, th
In the case of layout-qualifier-names that can appear multiple times
in different declarations of the same shader or, even, the same
program, we are using the ast_layout_expression class which holds a
list to store all the appearances to be able to check later which is
the overriding value and whet
After the previous modifications in the merging of the
layout-qualifier-name values, we no longer push the final value in a
declaration to the global values.
This regression happens because we don't call for merging on the
right-most layout qualifier of a declaration which is also the
overriding o
From page 46 (page 52 of the PDF) of the GLSL 4.20 spec:
" More than one layout qualifier may appear in a single
declaration. If the same layout-qualifier-name occurs in multiple
layout qualifiers for the same declaration, the last one overrides
the former ones."
Consider this examp
Recently, we added code to check that any appearance of the
"max_vertices" layout-qualifier-name in a program holds the same
value.
Now, we make this the default behavior for any layout-qualifier-name
represented as a ast_layout_expression since, as it happens, the same
constraint applies to all t
Since we modified the way in which multiple repetitions of the same
layout-qualifier-name in a single declaration collapse into the
ast_type_qualifier class, we can simplify the merge_[in|out]_qualifier
APIs through removing the create_node parameter.
Signed-off-by: Andres Gomez
---
src/compiler
On Thu, 2016-10-06 at 10:33 -0700, Jason Ekstrand wrote:
> >
> > On Wed, Oct 5, 2016 at 7:25 PM, Timothy Arceri > abora.com> wrote:
> > > Just
> > >
> > > On Wed, 2016-10-05 at 16:23 -0700, Jason Ekstrand wrote:
> > > >
> > > >
> > > > On Thu, Sep 15, 2016 at 12:03 AM, Timothy Arceri > > i@col
Checking if MAD is supported is definitely wrong, and it's
more likely a typo I introduced few days ago which breaks
NV50 because SHLADD is not supported there.
Signed-off-by: Samuel Pitoiset
---
src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 2 +-
1 file changed, 1 insertion(+), 1 d
1 - 100 of 123 matches
Mail list logo
