On Fri, Feb 2, 2018 at 6:55 PM, Nanley Chery wrote:
> On Thu, Feb 01, 2018 at 06:31:18PM -0800, Jason Ekstrand wrote:
> > ---
> > src/intel/vulkan/anv_image.c | 14 +-
>
> We should also update the comment in anv_image that describes 3D as
> having one slice per LOD.
>
Yup. Fi
On Friday, February 2, 2018 5:07:01 PM PST Jordan Justen wrote:
> When the disk shader cache CI testing was enabled, we started noticing
> occasional failures on deqp test runs. (Mainly SNB, rarely HSW)
>
> Before this change, when we cleared the (in memory) program cache we
> reused the same bo.
On Friday, February 2, 2018 5:07:01 PM PST Jordan Justen wrote:
> When the disk shader cache CI testing was enabled, we started noticing
> occasional failures on deqp test runs. (Mainly SNB, rarely HSW)
>
> Before this change, when we cleared the (in memory) program cache we
> reused the same bo.
On Friday, February 2, 2018 5:07:00 PM PST Jordan Justen wrote:
> Signed-off-by: Jordan Justen
> ---
> src/mesa/drivers/dri/i965/brw_program_cache.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_program_cache.c
> b/src/mesa/drivers/
On Fri, Feb 2, 2018 at 6:47 PM, Nanley Chery wrote:
> On Tue, Jan 30, 2018 at 05:20:07PM -0800, Jason Ekstrand wrote:
> > Completely untested.
>
> The message in your fdo branch looks good.
>
> > ---
> > src/intel/blorp/blorp_clear.c | 12 +++-
> > src/intel/blorp/blorp_genX_exec.h |
On Fri, Feb 02, 2018 at 09:02:25PM -0800, Jason Ekstrand wrote:
> On Fri, Feb 2, 2018 at 5:58 PM, Nanley Chery wrote:
>
> > On Fri, Feb 02, 2018 at 04:42:14PM -0800, Jason Ekstrand wrote:
> > > On Fri, Feb 2, 2018 at 2:39 PM, Nanley Chery
> > wrote:
> > >
> > > > On Fri, Jan 19, 2018 at 03:47:38
On Fri, Feb 2, 2018 at 5:58 PM, Nanley Chery wrote:
> On Fri, Feb 02, 2018 at 04:42:14PM -0800, Jason Ekstrand wrote:
> > On Fri, Feb 2, 2018 at 2:39 PM, Nanley Chery
> wrote:
> >
> > > On Fri, Jan 19, 2018 at 03:47:38PM -0800, Jason Ekstrand wrote:
> > > > Now that we're tracking aux properly p
On Thu, Feb 01, 2018 at 06:31:18PM -0800, Jason Ekstrand wrote:
> ---
> src/intel/vulkan/anv_image.c | 14 +-
We should also update the comment in anv_image that describes 3D as
having one slice per LOD.
> src/intel/vulkan/anv_private.h | 9 -
> src/intel/vulkan/ge
Am 03.02.2018 um 03:12 schrieb Marek Olšák:
> On Sat, Feb 3, 2018 at 2:55 AM, Roland Scheidegger wrote:
>> Am 03.02.2018 um 00:31 schrieb Marek Olšák:
>>> On Sat, Feb 3, 2018 at 12:01 AM, Roland Scheidegger
>>> wrote:
Am 02.02.2018 um 23:39 schrieb Marek Olšák:
> On Fri, Feb 2, 2018 at
On Tue, Jan 30, 2018 at 05:20:07PM -0800, Jason Ekstrand wrote:
> Completely untested.
The message in your fdo branch looks good.
> ---
> src/intel/blorp/blorp_clear.c | 12 +++-
> src/intel/blorp/blorp_genX_exec.h | 6 ++
> 2 files changed, 17 insertions(+), 1 deletion(-)
>
>
On Sat, Feb 3, 2018 at 2:55 AM, Roland Scheidegger wrote:
> Am 03.02.2018 um 00:31 schrieb Marek Olšák:
>> On Sat, Feb 3, 2018 at 12:01 AM, Roland Scheidegger
>> wrote:
>>> Am 02.02.2018 um 23:39 schrieb Marek Olšák:
On Fri, Feb 2, 2018 at 10:26 PM, Roland Scheidegger
wrote:
> Am
On Fri, Feb 02, 2018 at 04:42:14PM -0800, Jason Ekstrand wrote:
> On Fri, Feb 2, 2018 at 2:39 PM, Nanley Chery wrote:
>
> > On Fri, Jan 19, 2018 at 03:47:38PM -0800, Jason Ekstrand wrote:
> > > Now that we're tracking aux properly per-slice, we can enable this for
> > > applications which actuall
Am 03.02.2018 um 00:31 schrieb Marek Olšák:
> On Sat, Feb 3, 2018 at 12:01 AM, Roland Scheidegger
> wrote:
>> Am 02.02.2018 um 23:39 schrieb Marek Olšák:
>>> On Fri, Feb 2, 2018 at 10:26 PM, Roland Scheidegger
>>> wrote:
Am 02.02.2018 um 21:48 schrieb Marek Olšák:
> Hi,
>
> Thi
On Fri, Feb 02, 2018 at 02:39:25PM -0800, Jason Ekstrand wrote:
> On Fri, Feb 2, 2018 at 1:47 PM, Nanley Chery wrote:
>
> > On Fri, Jan 19, 2018 at 03:47:37PM -0800, Jason Ekstrand wrote:
> > > This commit completely reworks aux tracking. This includes a number of
> > > somewhat distinct changes
When the disk shader cache CI testing was enabled, we started noticing
occasional failures on deqp test runs. (Mainly SNB, rarely HSW)
Before this change, when we cleared the (in memory) program cache we
reused the same bo. Since the disk shader cache quickly restores
programs, it appears that thi
Signed-off-by: Jordan Justen
---
src/mesa/drivers/dri/i965/brw_program_cache.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_program_cache.c
b/src/mesa/drivers/dri/i965/brw_program_cache.c
index 9266273b5da..f084f94f929 100644
--- a/src/m
Am 02.02.2018 um 06:33 schrieb Dave Airlie:
> From: Dave Airlie
>
> This fixes a few CTS cases in :
> KHR-GL45.texture_view.view_sampling
>
> some multisample cases are still broken, but not sure this is
> the same problem.
>
> v2: fix more cases
>
> Signed-off-by: Dave Airlie
> ---
> src/ga
On Fri, Feb 2, 2018 at 2:39 PM, Nanley Chery wrote:
> On Fri, Jan 19, 2018 at 03:47:38PM -0800, Jason Ekstrand wrote:
> > Now that we're tracking aux properly per-slice, we can enable this for
> > applications which actually care.
> > ---
> > src/intel/vulkan/anv_blorp.c | 22 +
Ported from the radeonsi GL_AMD_pinned_memory implementation.
Signed-off-by: Fredrik Höglund
---
Tested using a version of the cube demo modified to use host memory
allocations for buffers and staging images.
src/amd/vulkan/radv_device.c | 60 ++-
src/a
On Sat, Feb 3, 2018 at 12:01 AM, Roland Scheidegger wrote:
> Am 02.02.2018 um 23:39 schrieb Marek Olšák:
>> On Fri, Feb 2, 2018 at 10:26 PM, Roland Scheidegger
>> wrote:
>>> Am 02.02.2018 um 21:48 schrieb Marek Olšák:
Hi,
This is the second and hopefully final version of 32-bit po
On Fri, Feb 2, 2018 at 11:39 PM, Marek Olšák wrote:
> On Fri, Feb 2, 2018 at 10:26 PM, Roland Scheidegger
> wrote:
>> Am 02.02.2018 um 21:48 schrieb Marek Olšák:
>>> Hi,
>>>
>>> This is the second and hopefully final version of 32-bit pointer
>>> support for radeonsi.
>>>
>>> Constant buffer 0 n
Am 02.02.2018 um 23:39 schrieb Marek Olšák:
> On Fri, Feb 2, 2018 at 10:26 PM, Roland Scheidegger
> wrote:
>> Am 02.02.2018 um 21:48 schrieb Marek Olšák:
>>> Hi,
>>>
>>> This is the second and hopefully final version of 32-bit pointer
>>> support for radeonsi.
>>>
>>> Constant buffer 0 now has re
On Fri, Feb 2, 2018 at 10:44 PM, Axel Davy wrote:
> Hi Marek,
>
> Since the previous patch makes it mandatory to use the flags when required,
> I guess this patch should also add the neccessary changes to gallium nine.
Nine uses user buffers and const_uploader, so it's unaffected.
Marek
On Fri, Feb 2, 2018 at 10:26 PM, Roland Scheidegger wrote:
> Am 02.02.2018 um 21:48 schrieb Marek Olšák:
>> Hi,
>>
>> This is the second and hopefully final version of 32-bit pointer
>> support for radeonsi.
>>
>> Constant buffer 0 now has restrictions on which buffers can be set
>> in that slot.
On Fri, Feb 2, 2018 at 1:47 PM, Nanley Chery wrote:
> On Fri, Jan 19, 2018 at 03:47:37PM -0800, Jason Ekstrand wrote:
> > This commit completely reworks aux tracking. This includes a number of
> > somewhat distinct changes:
> >
> > 1) Since we are no longer fast-clearing multiple slices, we onl
On Fri, Jan 19, 2018 at 03:47:38PM -0800, Jason Ekstrand wrote:
> Now that we're tracking aux properly per-slice, we can enable this for
> applications which actually care.
> ---
> src/intel/vulkan/anv_blorp.c | 22 +++---
> src/intel/vulkan/genX_cmd_buffer.c | 13 +--
Update patch 05/12 with a fix.
From: Boyuan Zhang
Implement encoding of sps, pps, vps, aud, and slice headers for HEVC
based on HEVC specs.
Signed-off-by: Boyuan Zhang
Acked-by: Christian König
---
src/gallium/drivers/radeon/radeon_vcn_enc_1_2.c | 348 +++-
1 file changed
On Fri, Jan 19, 2018 at 03:47:37PM -0800, Jason Ekstrand wrote:
> This commit completely reworks aux tracking. This includes a number of
> somewhat distinct changes:
>
> 1) Since we are no longer fast-clearing multiple slices, we only need
> to track one fast clear color and one fast clear t
Hi Marek,
Since the previous patch makes it mandatory to use the flags when required,
I guess this patch should also add the neccessary changes to gallium nine.
Yours,
Axel Davy
On 02/02/2018 21:48, Marek Olšák wrote:
From: Marek Olšák
---
src/gallium/auxiliary/util/u_inlines.h | 21
On Fri, Feb 2, 2018 at 10:31 PM, Dylan Baker wrote:
> Quoting Bas Nieuwenhuizen (2018-02-02 10:14:04)
>> On Fri, Feb 2, 2018 at 6:59 PM, Emil Velikov
>> wrote:
>> > Hi Bas,
>> >
>> > On 31 January 2018 at 11:31, Bas Nieuwenhuizen wrote:
>> >> deqp does not allow any KHX extensions, and since de
Quoting Bas Nieuwenhuizen (2018-02-02 10:14:04)
> On Fri, Feb 2, 2018 at 6:59 PM, Emil Velikov wrote:
> > Hi Bas,
> >
> > On 31 January 2018 at 11:31, Bas Nieuwenhuizen wrote:
> >> deqp does not allow any KHX extensions, and since deqp is included
> >> in android-cts, android does not allow any k
Quoting Bas Nieuwenhuizen (2018-02-02 11:54:58)
> Reviewed-by: Bas Nieuwenhuizen
>
> On Fri, Feb 2, 2018 at 7:55 PM, Dylan Baker wrote:
> > Currently we always check for 3.9.0, which is pretty safe since
> > everything except radv work with >= 3.9 and 3.9 is pretty old at this
> > point. However
I went ahead and pushed my patch, the only things you should need to do for the
meson side now is move with_gallium_radeonsi, and remove the stray `s` in the
error message.
Dylan
Quoting Bas Nieuwenhuizen (2018-02-02 11:56:26)
> I'd prefer if Dylan's patch goes first, as we are bumping swr too ot
Am 02.02.2018 um 21:48 schrieb Marek Olšák:
> Hi,
>
> This is the second and hopefully final version of 32-bit pointer
> support for radeonsi.
>
> Constant buffer 0 now has restrictions on which buffers can be set
> in that slot.
>
> I plan to push this when my LLVM patch lands in 6.0 (hopefully
This series actually has 10 patches. See 8/7, 9/7, 10/7.
Marek
On Fri, Feb 2, 2018 at 9:48 PM, Marek Olšák wrote:
> Hi,
>
> This is the second and hopefully final version of 32-bit pointer
> support for radeonsi.
>
> Constant buffer 0 now has restrictions on which buffers can be set
> in that s
From: Marek Olšák
State trackers must use a user buffer or const_uploader,
or set pipe_resource::flags same as const_uploader->flags.
Reviewed-by: Samuel Pitoiset
---
src/gallium/drivers/radeonsi/si_descriptors.c | 6 ++
src/gallium/drivers/radeonsi/si_get.c | 4 +++-
2 files chang
From: Marek Olšák
Reviewed-by: Samuel Pitoiset
---
Continuation of the 32-bit pointer series.
src/gallium/drivers/radeon/r600_buffer_common.c | 3 +++
src/gallium/drivers/radeon/r600_pipe_common.c | 5 +++--
src/gallium/drivers/radeon/r600_pipe_common.h | 1 +
3 files changed, 7 insertion
From: Marek Olšák
SGPRS: 2170102 -> 2158430 (-0.54 %)
VGPRS: 1645656 -> 1641516 (-0.25 %)
Spilled SGPRs: 9078 -> 8810 (-2.95 %)
Spilled VGPRs: 130 -> 114 (-12.31 %)
Scratch size: 1508 -> 1492 (-1.06 %) dwords per thread
Code Size: 52094872 -> 52692540 (1.15 %) bytes
Max Waves: 371848 -> 372723 (0
Hi,
This is the second and hopefully final version of 32-bit pointer
support for radeonsi.
Constant buffer 0 now has restrictions on which buffers can be set
in that slot.
I plan to push this when my LLVM patch lands in 6.0 (hopefully it
will be accepted there).
There will also be a dependency
From: Marek Olšák
Required by radeonsi for optimal behavior.
---
src/gallium/docs/source/screen.rst | 4
src/gallium/drivers/etnaviv/etnaviv_screen.c | 1 +
src/gallium/drivers/freedreno/freedreno_screen.c | 1 +
src/gallium/drivers/i915/i915_screen.c | 1 +
src/
From: Marek Olšák
Reviewed-by: Samuel Pitoiset
---
src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 63 ---
src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 9 ++--
src/gallium/winsys/radeon/drm/radeon_drm_winsys.h | 11 ++--
3 files changed, 47 insertions(+), 36 dele
From: Marek Olšák
Reviewed-by: Samuel Pitoiset
---
src/gallium/winsys/amdgpu/drm/amdgpu_bo.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
b/src/gallium/winsys/amdgpu/drm/amdgpu_bo.c
index 5d565ff..8ce131c 100644
--- a/src/galli
From: Marek Olšák
---
src/amd/common/ac_gpu_info.c | 7 +++
src/amd/common/ac_gpu_info.h | 1 +
2 files changed, 8 insertions(+)
diff --git a/src/amd/common/ac_gpu_info.c b/src/amd/common/ac_gpu_info.c
index 6d9dcb5..b5b059e 100644
--- a/src/amd/common/ac_gpu_info.c
+++ b/src/amd/common/ac_
From: Marek Olšák
Reviewed-by: Samuel Pitoiset
---
src/gallium/winsys/radeon/drm/radeon_drm_bo.c | 42 +++
src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 28 ++-
src/gallium/winsys/radeon/drm/radeon_drm_winsys.h | 2 ++
3 files changed, 64 insertions(+)
From: Marek Olšák
---
src/gallium/auxiliary/util/u_inlines.h | 21 +
src/gallium/auxiliary/vl/vl_compositor.c| 2 +-
src/gallium/drivers/radeonsi/si_pipe.c | 2 +-
src/gallium/state_trackers/xa/xa_renderer.c | 7 ---
4 files changed, 27 insertions(+), 5 d
From: Marek Olšák
Reviewed-by: Samuel Pitoiset
---
src/gallium/drivers/radeon/radeon_winsys.h | 47 --
1 file changed, 44 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/radeon/radeon_winsys.h
b/src/gallium/drivers/radeon/radeon_winsys.h
index 238c92
Thanks, my automake knowledge is a bit rusty so that I didn't know
about this option to limit the build to "make check".
Reviewed-By: Gert Wollny
Am Freitag, den 02.02.2018, 16:59 +0100 schrieb Bas Nieuwenhuizen:
> We don't need the library if we don't build tests, and building
> it adds a depen
I'd prefer if Dylan's patch goes first, as we are bumping swr too otherwise.
Though either way:
Reviewed-by: Bas Nieuwenhuizen
On Fri, Feb 2, 2018 at 8:15 PM, Dylan Baker wrote:
> FYI: I sent out a patch a few minutes ago to set the minimum version more like
> autotools does, but I'm fine with
Reviewed-by: Bas Nieuwenhuizen
On Fri, Feb 2, 2018 at 7:55 PM, Dylan Baker wrote:
> Currently we always check for 3.9.0, which is pretty safe since
> everything except radv work with >= 3.9 and 3.9 is pretty old at this
> point. However, radv actually requires 4.0, and there is a patch for
> rad
This needs to link the state tracker with --whole-archive to expose the
right symbols.
Fixes: 22a817af8a89eb3c7 ("meson: build gallium xvmc state tracker")
Signed-off-by: Dylan Baker
---
src/gallium/targets/xvmc/meson.build | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
di
This needs to use --whole-archive (link_whole in meson) to properly
expose symbols.
Fixes: 0ba909f0f111824 ("meson: build gallium xa state tracker")
Signed-off-by: Dylan Baker
---
src/gallium/targets/xa/meson.build | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
diff --git
This needs to use --whole-archive (link_whole in meson) to properly
expose symbols.
Fixes: 1d36dc674d528b93b ("meson: build gallium omx state tracker")
Signed-off-by: Dylan Baker
---
src/gallium/targets/omx-bellagio/meson.build | 13 +++--
1 file changed, 11 insertions(+), 2 deletions(-)
The VDPAU state tracker needs to be linked with whole-archive (autotools
does this). Because we are linking the whole archive we alos need to
link with libswdri and libswkmsdri if those have been enabled.
Fixes: 68076b87474e7959 ("meson: build gallium vdpau state tracker")
Signed-off-by: Dylan Bak
The state tracker needs to be linked with whole-archive (like
autotools). As a result there are symbols from libswdri and libswkmsdri
that are needed, so link those as well.
Fixes: 5a785d51a6d6 ("meson: build gallium va state tracker")
Signed-off-by: Dylan Baker
---
src/gallium/targets/va/meson.
The linkage of all of the gallium media targets is broken in various
ways in the meson build. This series should correct that by doing more
what the autotools build does.
You may note in the last 5 patches the addition of libswdri and
libswkmsdri, these are needed because we're now linking the who
Fixes: 5a785d51a6d6 ("meson: build gallium va state tracker")
Signed-off-by: Dylan Baker
---
meson.build | 2 +-
src/gallium/state_trackers/va/meson.build | 6 +++---
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/meson.build b/meson.build
index 323f7
This state tracker actually needs to link, unlike vdpau.
Fixes: 1d36dc674d528b93b ("meson: build gallium omx state tracker")
Signed-off-by: Dylan Baker
---
meson.build | 5 -
1 file changed, 5 deletions(-)
diff --git a/meson.build b/meson.build
index 4c6283c..2df2497 100644
--- a/meson.buil
Unlike vdpau this is required.
Fixes: 22a817af8a89eb3c7 ("meson: build gallium xvmc state tracker")
Signed-off-by: Dylan Baker
---
meson.build | 3 ---
1 file changed, 3 deletions(-)
diff --git a/meson.build b/meson.build
index 2df2497..4f280d9 100644
--- a/meson.build
+++ b/meson.build
@@ -439
This makes the dependencies easier to manage, since each media target
doesn't need to worry about linking to half a dozen libraries.
Fixes: b1b65397d0c4978e3 ("meson: Build gallium auxiliary")
Signed-off-by: Dylan Baker
---
src/gallium/auxiliary/meson.build| 7 ++-
src/gallium/ta
FYI: I sent out a patch a few minutes ago to set the minimum version more like
autotools does, but I'm fine with this landing first.
Reviewed-by: Dylan Baker
Quoting Marek Olšák (2018-02-02 11:07:57)
> Adding this:
>
> diff --git a/meson.build b/meson.build
> index 80ea60f..22eb702 100644
> ---
Currently we always check for 3.9.0, which is pretty safe since
everything except radv work with >= 3.9 and 3.9 is pretty old at this
point. However, radv actually requires 4.0, and there is a patch for
radeonsi to do the same.
Fixes: 673dda833076 ("meson: build "radv" vulkan driver for radeon har
Adding this:
diff --git a/meson.build b/meson.build
index 80ea60f..22eb702 100644
--- a/meson.build
+++ b/meson.build
@@ -991,12 +991,12 @@ endif
_llvm = get_option('llvm')
if _llvm == 'auto'
dep_llvm = dependency(
-'llvm', version : '>= 3.9.0', modules : llvm_modules,
+'llvm', versio
Also change meson.build?
On Fri, Feb 2, 2018 at 7:34 PM, Marek Olšák wrote:
> From: Marek Olšák
>
> Only these are supported:
> - LLVM 4.0
> - LLVM 5.0
> - LLVM 6.0
> - master (7.0)
> ---
> configure.ac | 4 +-
> src/amd/common/ac_llvm_build.c
From: Marek Olšák
Only these are supported:
- LLVM 4.0
- LLVM 5.0
- LLVM 6.0
- master (7.0)
---
configure.ac | 4 +-
src/amd/common/ac_llvm_build.c | 187 ++---
src/amd/common/ac_llvm_helper.cpp | 10 --
https://bugs.freedesktop.org/show_bug.cgi?id=104915
Bug ID: 104915
Summary: Indexed SHADING_LANGUAGE_VERSION query not supported
Product: Mesa
Version: git
Hardware: All
OS: All
Status: NEW
Severity: minor
On Fri, Feb 2, 2018 at 6:59 PM, Emil Velikov wrote:
> Hi Bas,
>
> On 31 January 2018 at 11:31, Bas Nieuwenhuizen wrote:
>> deqp does not allow any KHX extensions, and since deqp is included
>> in android-cts, android does not allow any khx extensions.
>>
>> So disable VK_KHX_multiview on android.
Reviewed-by: Bas Nieuwenhuizen
On Fri, Feb 2, 2018 at 6:56 PM, Samuel Pitoiset
wrote:
> Fixes: df1d5174fcc ("ac/nir: replace SI.buffer.load.dword with
> amdgcn.buffer.load")
> Signed-off-by: Samuel Pitoiset
> ---
> src/amd/common/ac_nir_to_llvm.c | 3 +++
> 1 file changed, 3 insertions(+)
>
>
There is no fpow in hardware, so it's always lowered somewhere,
but it appears that lowering at NIR level is better. Figured while
comparing compute shaders between RadeonSI and RADV.
Polaris10:
Totals from affected shaders:
SGPRS: 18936 -> 18904 (-0.17 %)
VGPRS: 12240 -> 12220 (-0.16 %)
Spilled S
Hi Bas,
On 31 January 2018 at 11:31, Bas Nieuwenhuizen wrote:
> deqp does not allow any KHX extensions, and since deqp is included
> in android-cts, android does not allow any khx extensions.
>
> So disable VK_KHX_multiview on android.
> ---
> src/amd/vulkan/radv_extensions.py | 2 +-
> 1 file c
Fixes: df1d5174fcc ("ac/nir: replace SI.buffer.load.dword with
amdgcn.buffer.load")
Signed-off-by: Samuel Pitoiset
---
src/amd/common/ac_nir_to_llvm.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/amd/common/ac_nir_to_llvm.c b/src/amd/common/ac_nir_to_llvm.c
index 0f7d6258acd..e0386
Am 02.02.2018 um 05:56 schrieb Dave Airlie:
> From: Dave Airlie
>
> Doing a straight uint/int->fp32->fp64 conversion causes
> some precision issues, Roland suggested splitting the
> integer into two portions and doing two separate
> int->fp32->fp64 conversions then adding the results.
>
> This p
Am 02.02.2018 um 08:29 schrieb Dave Airlie:
> From: Dave Airlie
>
> If this is an image buffer, we need to calculate the correct resource
> id.
>
> Fixes:
> KHR-GL45.shader_image_size.*
>
> Signed-off-by: Dave Airlie
> ---
> src/gallium/drivers/r600/r600_shader.c | 5 -
> 1 file changed,
The whole series are the updated version. Changes are made mainly based on the
comments from previous code review, plus fixing a few typos.
-Original Message-
From: Zhang, Boyuan
Sent: February-02-18 11:11 AM
To: mesa-dev@lists.freedesktop.org
Cc: Zhang, Boyuan
Subject: [PATCH 01/12] vl:
From: Boyuan Zhang
Enable vcn encode for HEVC main profile on Raven.
Signed-off-by: Boyuan Zhang
Acked-by: Christian König
---
src/gallium/drivers/radeonsi/si_get.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeonsi/si_get.c
b/src/gallium/driv
From: Boyuan Zhang
Logics that related to dual instances encode should only be done for
H264, not other codecs.
Signed-off-by: Boyuan Zhang
Acked-by: Christian König
---
src/gallium/state_trackers/va/picture.c | 3 ++-
src/gallium/state_trackers/va/surface.c | 23 +--
2 f
From: Boyuan Zhang
Add entrypoint check for HEVC to differentiate decode and encode jobs.
Signed-off-by: Boyuan Zhang
Acked-by: Christian König
---
src/gallium/state_trackers/va/context.c | 22 --
1 file changed, 12 insertions(+), 10 deletions(-)
diff --git a/src/gallium/
From: Boyuan Zhang
Add a separate file for HEVC encode functions.
Signed-off-by: Boyuan Zhang
Acked-by: Christian König
---
src/gallium/state_trackers/va/Makefile.sources | 1 +
src/gallium/state_trackers/va/meson.build| 2 +-
src/gallium/state_trackers/va/picture.c | 31
From: Boyuan Zhang
Implement HEVC encode functions based on VAAPI HEVC encode interface.
Signed-off-by: Boyuan Zhang
Acked-by: Christian König
---
src/gallium/state_trackers/va/picture_hevc_enc.c | 150 ++-
1 file changed, 144 insertions(+), 6 deletions(-)
diff --git a/sr
From: Boyuan Zhang
Implement required ibs for vcn HEVC encode.
Signed-off-by: Boyuan Zhang
Acked-by: Christian König
---
src/gallium/drivers/radeon/radeon_vcn_enc_1_2.c | 267
1 file changed, 222 insertions(+), 45 deletions(-)
diff --git a/src/gallium/drivers/radeon/
From: Boyuan Zhang
Implement encoding of sps, pps, vps, aud, and slice headers for HEVC
based on HEVC specs.
Signed-off-by: Boyuan Zhang
Acked-by: Christian König
---
src/gallium/drivers/radeon/radeon_vcn_enc_1_2.c | 348 +++-
1 file changed, 347 insertions(+), 1 deletion(
From: Boyuan Zhang
Move all H264 encode related functions into separate file. Similar to
VAAPI decode side, there will be separate file for each codec on encode
side as well.
Signed-off-by: Boyuan Zhang
Acked-by: Christian König
---
src/gallium/state_trackers/va/Makefile.sources | 1 +
sr
From: Boyuan Zhang
Add vcn encode interface for HEVC, and rename radeon_enc_h264_enc_pic
to radeon_enc_pic since radeon_enc_pic is used by both H264 and HEVC.
Signed-off-by: Boyuan Zhang
Acked-by: Christian König
---
src/gallium/drivers/radeon/radeon_vcn_enc.h | 81 +++
From: Boyuan Zhang
Add HEVC picture desc, and add codec check when creating and destroying
context.
Signed-off-by: Boyuan Zhang
Acked-by: Christian König
---
src/gallium/state_trackers/va/context.c| 26 ++
src/gallium/state_trackers/va/va_private.h | 1 +
2 files
From: Boyuan Zhang
Pass pipe_picture_desc instead of pipe_h264_enc_picture_desc so that
it can be used for different codecs. Add functions to handle picture
parameters that will be used for HEVC encode.
Signed-off-by: Boyuan Zhang
Acked-by: Christian König
---
src/gallium/drivers/radeon/radeo
From: Boyuan Zhang
Add HEVC encode interface
Signed-off-by: Boyuan Zhang
Acked-by: Christian König
---
src/gallium/include/pipe/p_video_state.h | 99
1 file changed, 99 insertions(+)
diff --git a/src/gallium/include/pipe/p_video_state.h
b/src/gallium/include
Am 02.02.2018 um 12:16 schrieb Emil Velikov:
> Hi Roland,
>
> On 30 January 2018 at 05:07, wrote:
>> From: Roland Scheidegger
>>
>> By the looks of it it seems hemlock is treated separately to cypress, but
>> certainly it won't need the stack workarounds cedar/redwood (and
>> seemingly every ot
We don't need the library if we don't build tests, and building
it adds a dependency on gtest which adds a dependency on cxxabi.h.
Fixes: 6569b33b6e "mesa/st/tests: unify MockCodeLine* classes"
---
src/mesa/state_tracker/tests/Makefile.am | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
dif
https://bugs.freedesktop.org/show_bug.cgi?id=104374
Michel Dänzer changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Fri, Feb 2, 2018 at 11:00 PM, Rob Herring wrote:
> On Fri, Feb 2, 2018 at 2:01 AM, Tomasz Figa wrote:
>> Hi Rob,
>>
>> On Tue, Jan 30, 2018 at 9:36 PM, Robert Foss
>> wrote:
uint32_t (*get_fd)(buffer_handle_t handle, uint32_t plane);
uint64_t (*get_modifier)(buf
On Fri, Feb 2, 2018 at 2:01 AM, Tomasz Figa wrote:
> Hi Rob,
>
> On Tue, Jan 30, 2018 at 9:36 PM, Robert Foss
> wrote:
>>> uint32_t (*get_fd)(buffer_handle_t handle, uint32_t plane);
>>> uint64_t (*get_modifier)(buffer_handle_t handle, uint32_t
>>> plane);
>>> u
On Fri, Feb 2, 2018 at 7:55 AM, Gert Wollny wrote:
> Am Freitag, den 02.02.2018, 06:56 -0500 schrieb Ilia Mirkin:
>> On Fri, Feb 2, 2018 at 4:07 AM, Gert Wollny
>> wrote:
>> > Am Freitag, den 02.02.2018, 09:04 +0100 schrieb Roland Scheidegger:
>> > >
>> > >
>> > > Yes, the _GL spec_ says it is an
Am Freitag, den 02.02.2018, 06:56 -0500 schrieb Ilia Mirkin:
> On Fri, Feb 2, 2018 at 4:07 AM, Gert Wollny
> wrote:
> > Am Freitag, den 02.02.2018, 09:04 +0100 schrieb Roland Scheidegger:
> > >
> > >
> > > Yes, the _GL spec_ says it is an array.
> > > But in gallium it can't be. Therefore I thin
On 1 February 2018 at 17:48, Jon Turney wrote:
> An additional stub for applegl_create_context() is needed
> Cannot test indirect API as it's not built on osx, currently
>
> Signed-off-by: Jon Turney
> ---
> src/glx/tests/fake_glx_screen.cpp | 11 +++
> src/glx/tests/indirect_api.cpp
On 2 February 2018 at 10:16, Eric Engestrom wrote:
> On Sunday, 2018-01-28 14:18:27 +, Jon Turney wrote:
>> mesa/src/glx/glxcmds.c:1295:21: error: implicit declaration of function
>> 'env_var_as_boolean' is invalid in C99
>> [-Werror,-Wimplicit-function-declaration]
>> mesa/src/glx/apple/app
On 2 February 2018 at 10:16, Eric Engestrom wrote:
> On Sunday, 2018-01-28 14:18:27 +, Jon Turney wrote:
>> mesa/src/glx/glxcmds.c:1295:21: error: implicit declaration of function
>> 'env_var_as_boolean' is invalid in C99
>> [-Werror,-Wimplicit-function-declaration]
>> mesa/src/glx/apple/app
On Fri, Feb 2, 2018 at 4:07 AM, Gert Wollny wrote:
> Am Freitag, den 02.02.2018, 09:04 +0100 schrieb Roland Scheidegger:
>>
>>
>> Yes, the _GL spec_ says it is an array.
>> But in gallium it can't be. Therefore I think it's incorrect if we
>> end up with array accesses there (albeit I was too lazy
Hi Roland,
On 30 January 2018 at 05:07, wrote:
> From: Roland Scheidegger
>
> By the looks of it it seems hemlock is treated separately to cypress, but
> certainly it won't need the stack workarounds cedar/redwood (and
> seemingly every other eg chip except cypress/juniper) need.
> (Discovered
On 1 February 2018 at 07:32, wrote:
> From: Mathias Fröhlich
>
> Since the first material attribute no longer aliases with
> the generic0 attribute, only aliasing between generic0 and
> position is left and entirely dependent on the enabled
> state of the VAO. So introduce a gl_attribute_map_mod
Hi Mathias,
I've noticed you pushed this already. Just sharing some fly-by idea.
On 1 February 2018 at 07:32, wrote:
> From: Mathias Fröhlich
>
> The materials are now moved to the end of the
> generic attributes block to the range 4-15.
>
> Before, the way the position and generic 0 attribute
Reviewed-by: Alejandro Piñeiro
On 01/02/18 16:03, Andres Gomez wrote:
> downsize_format_if_needed takes an integer as number of uploads
> parameter. Hence, let's do an integer comparation instead of a boolean
> check, since that is confusing.
>
> Since we are at it, fix a couple of wrongly tabbed
1 - 100 of 113 matches
Mail list logo
