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);
>> uint32_t (*get_offsets)(buffer_handle_t handle, uint32_t plane);
Are you sure of that? You only get 11 stride bits to program, and they
are in bytes. Therefore I can't see how you could program 2048 (unless
the hw would interpet 0 as 2048 but I think stride 0 is valid there?).
Roland
Am 02.02.2018 um 07:23 schrieb Dave Airlie:
> From: Dave Airlie
>
> This f
Am 02.02.2018 um 07:53 schrieb Gert Wollny:
> Am Freitag, den 02.02.2018, 02:13 +0100 schrieb Roland Scheidegger:
>> Am 30.01.2018 um 09:21 schrieb Gert Wollny:
>>> Although gl_SampleMaskIn is declared as an array in GLSL, it is
>>> effectively a 32 bit mask on all hardware supported by mesa, so th
Oops my bad...
Reviewed-by: Roland Scheidegger
Am 02.02.2018 um 07:57 schrieb Dave Airlie:
> From: Dave Airlie
>
> This fixes a crash in:
> KHR-GL45.texture_cube_map_array.texture_size_compute_sh.
>
> Signed-off-by: Dave Airlie
> ---
> src/gallium/drivers/r600/r600_state_common.c | 2 +-
> 1
Hello Tim,
_this_ version brake UH, UV, mpv, blender 2.79 (some test files not
all).
Must be something with the cache file(s).
export R600_DEBUG sisched,nir
UH and UV:
Start menu OK and then both:
Loading "core/properties/unigine.prop" 2 properties 0ms
Received signal SIGSEGV, invalid memory
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 to actually
> look at the tgsi, but I'm pretty sure it i
On 2018-02-02 — 18:07, Timothy Arceri wrote:
>
>
> On 02/02/18 17:21, Timothy Arceri wrote:
> > On 02/02/18 16:38, Jan Vesely wrote:
> > > On Fri, 2018-02-02 at 15:03 +1100, Timothy Arceri wrote:
> > > > When PIPE_SHADER_IR_LLVM existed this query made sense but now it
> > > > always returns PIPE
On 02/02/18 19:26, Dieter Nützel wrote:
Hello Tim,
_this_ version brake UH, UV, mpv, blender 2.79 (some test files not all).
Must be something with the cache file(s).
The cache currently needs to be deleted when switching between nir and
tgsi. I'm not sure it I should try to avoid this or not
https://bugs.freedesktop.org/show_bug.cgi?id=104302
--- Comment #1 from Matt ---
Have attempted using a different wine solution to run the game -
https://github.com/roderickc/wine-vulkan . Users of this have reported it
working on the RX480, but is still incorrectly rendering (as per original
vid
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/apple_visual.c:85:28: error: implicit declaration of
> funct
On Wed, 2018-01-31 at 01:38 +0100, Matteo Bruni wrote:
> 2018-01-30 16:11 GMT+01:00 Juan A. Suarez Romero :
> > According with OpenGL GLSL 3.20 spec, section 4.3.9:
> >
> > "It is a link-time error if any particular shader interface
> >contains:
> > - two different blocks, each having n
If extensions GL_KHR_texture_compression_astc_hdr or
GL_KHR_texture_compression_astc_sliced_3d are implemented then ASTC
format are supported in CompressedTex*Îmage3D.
Fixes KHR-GLES2.texture_3d.* with this format.
CC: Eric Anholt
Signed-off-by: Juan A. Suarez Romero
---
src/mesa/main/teximage
https://bugs.freedesktop.org/show_bug.cgi?id=104302
--- Comment #2 from Matt ---
Further to this - I have the VulkanSDK installed under my wine instance - would
a VKtrace dump help bisect the issue or is this useless for driver debug?
--
You are receiving this mail because:
You are the QA Conta
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
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 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 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
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 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
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 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
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 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
https://bugs.freedesktop.org/show_bug.cgi?id=104374
Michel Dänzer changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
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
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
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
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 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
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
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
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
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 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
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
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
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
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:
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,
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
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
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
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
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(+)
>
>
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.
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
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 --
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
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
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
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
> ---
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
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
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
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
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(-)
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 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
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
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
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
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
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
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
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
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
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
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
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
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
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
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
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, 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
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, 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 +--
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 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
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 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
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
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 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 +
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
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
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
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
1 - 100 of 113 matches
Mail list logo
