Hi,
The patch series I have submitted handles the case of needing to resolve
texture surfaces when a draw (or compute) accesses a texture which is astc5x5.
As it is quite clear you understand the issue and know the code of i965 the
patch centers on, you are an excellent person to review the co
On 04/12/17 20:21, Eduardo Lima Mitev wrote:
v2: Bail out if we see more that one shader for the same stage, and add
a corresponding comment. (Timothy Arceri)
v3: Adds also a linker error log to the condition above. (Timothy Arceri)
---
src/mesa/main/glspirv.c | 64 ++
A couple of notes:
1) I *think* I gave you enough reviews to land the UBO/SSBO part and the
optimizations in 26-28. If reviews are still missing anywhere, please let
me know. If not, let's try and get that part landed.
2) I send out a patch to rewrite assign_constant_locations which I think
s
Reviewed-by: Timothy Arceri
On 04/12/17 20:21, Eduardo Lima Mitev wrote:
This is the main fork of the shader compilation code-path, where a NIR
shader is obtained by calling spirv_to_nir() or glsl_to_nir(),
depending on its nature..
v2: Use 'spirv_data' member from gl_linked_shader to know whi
On Mon, Dec 4, 2017 at 9:07 PM, Jason Ekstrand wrote:
> On Mon, Dec 4, 2017 at 8:53 PM, Jordan Justen
> wrote:
>
>> On 2017-12-01 17:20:09, Jason Ekstrand wrote:
>> > ---
>> > src/intel/vulkan/anv_device.c | 1 +
>> > src/intel/vulkan/anv_pipeline.c | 6 ++
>> > 2 files changed, 7 inserti
On Mon, Dec 4, 2017 at 8:53 PM, Jordan Justen
wrote:
> On 2017-12-01 17:20:09, Jason Ekstrand wrote:
> > ---
> > src/intel/vulkan/anv_device.c | 1 +
> > src/intel/vulkan/anv_pipeline.c | 6 ++
> > 2 files changed, 7 insertions(+)
> >
> > diff --git a/src/intel/vulkan/anv_device.c
> b/src/
Ping! The outstanding patches for review are:
4, 12, 22, 23
Gallium specific patches:
17-21
The following have a v1 r-b Nicolai but have changed since:
13, 14, 15
Branch available here:
https://github.com/tarceri/Mesa.git gallium-program-binary
On 29/11/17 12:24, Timothy Arceri wrote:
V3:
On 2017-12-01 17:20:09, Jason Ekstrand wrote:
> ---
> src/intel/vulkan/anv_device.c | 1 +
> src/intel/vulkan/anv_pipeline.c | 6 ++
> 2 files changed, 7 insertions(+)
>
> diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c
> index 937efb9..43781bd 100644
> --- a/src/
https://bugs.freedesktop.org/show_bug.cgi?id=104084
Timothy Arceri changed:
What|Removed |Added
Resolution|--- |NOTABUG
Status|NEEDINFO
GLSL IR operation arguments can sometimes have an implicit swizzle as a
result of a vector arg and a scalar arg, where the scalar argument is
implicitly expanded to the size of the vector argument.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=103955
Signed-off-by: Ilia Mirkin
---
v1 ->
https://bugs.freedesktop.org/show_bug.cgi?id=104084
--- Comment #4 from gamedev1...@gmail.com ---
Thanks it worked but the entire thing was a bit silly just for vulkan maybe
Mesa can add a better way
Sent from my iPhone
> On Dec 4, 2017, at 7:37 PM, bugzilla-dae...@freedesktop.org wrote:
>
> C
Thanks for your reply.
The attachment is the Xorg log.
If you need other information, pls let me know.
Thanks.
-邮件原件-
发件人: Michel Dänzer [mailto:mic...@daenzer.net]
发送时间: 2017年12月5日 1:02
收件人: Lvzhihong (ReJohn)
抄送: mesa-dev@lists.freedesktop.org; Liuchen (Greentea)
; sunnanyong
主题:
https://bugs.freedesktop.org/show_bug.cgi?id=103955
--- Comment #9 from Younghun Jang ---
(In reply to Ilia Mirkin from comment #8)
> This patch fixes the sample case for me:
>
> https://patchwork.freedesktop.org/patch/191532/
I concur. The patch fixes the bug.
And I believe this was a typo:
Reviewed-by: Jason Ekstrand
On Mon, Dec 4, 2017 at 6:21 PM, Rob Herring wrote:
> Commit 94ca8e04adf6 ("spirv: Add vtn_fail and vtn_assert helpers") broke
> Android builds which have -Werror enabled with the following errors:
>
> external/mesa3d/src/compiler/spirv/spirv_to_nir.c:272:1: error: co
Commit 94ca8e04adf6 ("spirv: Add vtn_fail and vtn_assert helpers") broke
Android builds which have -Werror enabled with the following errors:
external/mesa3d/src/compiler/spirv/spirv_to_nir.c:272:1: error: control may
reach end of non-void function [-Werror,-Wreturn-type]
external/mesa3d/src/comp
On 04/12/17 16:33, Leo Liu wrote:
> On 12/04/2017 04:32 AM, Emil Velikov wrote:
>> On 2 December 2017 at 15:26, Julian Lai wrote:
>>> Since the crappy vaapi-glx was dropped off by mpv-git player, there is no
>>> way to play HEVC 10bit video with zero-copy hardware acceleration, vdpau
>>> can't do
On Mon, Dec 4, 2017 at 4:50 PM, Chema Casanova
wrote:
> On 30/11/17 21:45, Jason Ekstrand wrote:
> > On Wed, Nov 29, 2017 at 6:50 PM, Jose Maria Casanova Crespo
> > mailto:jmcasan...@igalia.com>> wrote:
> >
> > v2: Fix alignment style (Topi Pohjolainen)
> > (Jason Ekstrand)
> >
On Fri, Dec 1, 2017 at 5:20 PM, Jason Ekstrand wrote:
> We've had this optimization in the GL driver since July but never got
> around to hooking it up in Vulkan. This lets us turn UBOs into push
> constants which are significantly faster since they read the UBO data
> once and the shader dispat
On 30/11/17 21:45, Jason Ekstrand wrote:
> On Wed, Nov 29, 2017 at 6:50 PM, Jose Maria Casanova Crespo
> mailto:jmcasan...@igalia.com>> wrote:
>
> v2: Fix alignment style (Topi Pohjolainen)
> (Jason Ekstrand)
> - Enable bit_size parameter to scattered messages to enable
> d
On Mon, Nov 27, 2017 at 07:05:51PM -0800, Jason Ekstrand wrote:
> Right now, we have different entrypoints and enums in blorp for these
> different operations. This provides us a central enum which we can
> begin to transition to.
> ---
> src/intel/isl/isl.h | 74
> ++
On 04/12/17 20:52, Leo Liu wrote:
> Signed-off-by: Leo Liu
> ---
> src/gallium/state_trackers/va/context.c | 99
> -
> 1 file changed, 49 insertions(+), 50 deletions(-)
>
> diff --git a/src/gallium/state_trackers/va/context.c
> b/src/gallium/state_trackers/va/co
Quoting Emil Velikov (2017-11-23 11:04:34)
> On 20 November 2017 at 23:12, Dylan Baker wrote:
> > This doesn't actually accomplish what it's meant to do, as extern C
> > doesn't undefine __cplusplus, so the included headers define a template
> > (because __cplusplus is defined), but then that code
On Mon, Dec 4, 2017 at 3:12 PM, Rob Herring wrote:
> Looks like Jason is the actual culprit here. Unfortunately, the build
> can't keep up with every commit. Looks to me like these should be
> fixed.
>
The problem is that you need to define HAVE_FUNC_ATTRIBUTE_NORETURN
> I wonder how long unti
Those operations do not map to actual hardware instructions, therefore
those should always be lowered to 32-bit instructions.
Fixes: 009c54aa7af "nv50/ir: Split 64-bit integer MAD/MUL operations"
Signed-off-by: Pierre Moreau
---
src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 2 +-
1
https://bugs.freedesktop.org/show_bug.cgi?id=104084
--- Comment #3 from Dylan Baker ---
The archwiki has good documentation on this here:
https://wiki.archlinux.org/index.php/AMDGPU#Enable_Southern_Islands_.28SI.29_and_Sea_Islands_.28CIK.29_support
Basically the problem is that by default the
Looks like Jason is the actual culprit here. Unfortunately, the build
can't keep up with every commit. Looks to me like these should be
fixed.
I wonder how long until I give up and just set Android back to -Wno-error...
Rob
-- Forwarded message --
From:
Date: Mon, Dec 4, 2017
drm.h header includes 'sys/ioccom.h' for ioctl
definitions on non-linux Unix platforms, but QNX
seems to define those in 'sys/ioctl.h' instead.
Signed-off-by: Miguel A Vico Moya
---
include/drm/drm.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/include/drm/drm.h b/include/drm/drm.h
i
Quoting Eric Engestrom (2017-12-04 07:38:04)
> On Thursday, 2017-11-30 10:34:06 -0800, Dylan Baker wrote:
> > Quoting Eric Engestrom (2017-11-29 07:44:08)
> > > On Tuesday, 2017-11-28 15:30:21 -0800, Dylan Baker wrote:
> > > > This series is the gallium media state trackers, the "nine" Direct3D
>
On Fri, Dec 01, 2017 at 04:42:22PM -0800, Jason Ekstrand wrote:
> On Fri, Dec 1, 2017 at 2:44 PM, Nanley Chery wrote:
>
> > Gen10 doesn't automatically decode the clear color of sRGB buffers. To
> > get correct rendering, avoid fast-clearing such buffers for now.
> >
> > The driver now passes the
https://bugs.freedesktop.org/show_bug.cgi?id=104084
--- Comment #2 from gamedev1...@gmail.com ---
I’m very confused since when do we need a special kernel loaded to use vulkan
that seems a bit silly doesn’t it anyway to get this kernel loaded what
should I do on arch
Sent from my iPhone
>
Fixes: 5e71efef44b992b5d70b ("meson: Add lmsensors support")
Signed-off-by: Dylan Baker
---
src/gallium/targets/libgl-xlib/meson.build | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/targets/libgl-xlib/meson.build
b/src/gallium/targets/libgl-xlib/meson.build
index
https://bugs.freedesktop.org/show_bug.cgi?id=104084
Bas Nieuwenhuizen changed:
What|Removed |Added
Component|Other |Drivers/Vulkan/radeon
S
Signed-off-by: Leo Liu
---
src/gallium/state_trackers/va/context.c | 99 -
1 file changed, 49 insertions(+), 50 deletions(-)
diff --git a/src/gallium/state_trackers/va/context.c
b/src/gallium/state_trackers/va/context.c
index 0ad4309568..8c624d05c1 100644
--- a/s
From: Mark Thompson
It will be present from libva 2.1 (VAAPI 1.1.0 or higher).
v2: rebase to previous patches(Leo)
Signed-off-by: Mark Thompson
Signed-off-by: Leo Liu
---
src/gallium/state_trackers/va/context.c | 2 +-
src/gallium/state_trackers/va/surface.c | 2 +-
2 files changed, 2 insert
Following VA spec
Signed-off-by: Leo Liu
---
src/gallium/state_trackers/va/context.c| 2 +-
src/gallium/state_trackers/va/subpicture.c | 2 +-
src/gallium/state_trackers/va/va_private.h | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/va/conte
Hi,
On 30/11/17 21:43, Neil Roberts wrote:
Kenneth Graunke writes:
We have a number of similar names now:
SYSTEM_VALUE_BASE_VERTEX
SYSTEM_VALUE_BASE_VERTEX_ID
SYSTEM_VALUE_VERTEX_ID
SYSTEM_VALUE_VERTEX_ID_ZERO_BASE
BASE_VERTEX and BASE_VERTEX_ID are really similar names, an
This VS system value will contain the value passed as
for indexed draw calls or the value passed as for non-indexed
draw calls. It will be used to calculate the gl_VertexID as
SYSTEM_VALUE_VERTEX_ID_ZERO_BASE plus SYSTEM_VALUE_FIRST_VERTEX.
Note that the current calculation which uses SYSTEM_VALU
From: Neil Roberts
---
src/mesa/drivers/dri/i965/brw_context.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_context.c
b/src/mesa/drivers/dri/i965/brw_context.c
index b62852d90c8..249f62847b9 100644
--- a/src/mesa/drivers/dri/i965/brw_contex
From: Neil Roberts
base_vertex will be zero for non-indexed calls, but we need it to
include the ‘first’ parameter. This is true for both GL and Vulkan.
I think this patch will also affect freedreno and radeonsi. I believe
if they are relying on this lowering then they are currently already
brok
- From the OpenGL 4.6 (11.1.3.9 Shader Inputs) specification:
"gl_BaseVertex holds the integer value passed to the baseVertex parameter to the
command that resulted in the current shader invocation. In the case where the
command has no baseVertex parameter, the value of gl_BaseVertex is zero."
-
From: Neil Roberts
The base vertex in Vulkan is different from GL in that for non-indexed
primitives the value is taken from the firstVertex parameter instead
of being set to zero. This coincides with the new SYSTEM_VALUE_FIRST_VERTEX
instead of BASE_VERTEX.
---
src/compiler/spirv/vtn_variables.
The new order is:
* VE 1:
* VE 2:
Previously it was:
* VE 1:
* VE 2:
The gl_BaseVertex is in a new location now, and firstvertex occupies
the old gl_BaseVertex place. This way we can keep pointing to the
indirect buffer for indirect draw calls.
Reviewed-by: Neil Roberts
---
src/intel/compi
From: Neil Roberts
Reviewed-by: Kenneth Graunke
---
src/intel/compiler/brw_compiler.h | 1 +
src/intel/compiler/brw_vec4.cpp | 4
2 files changed, 5 insertions(+)
diff --git a/src/intel/compiler/brw_compiler.h
b/src/intel/compiler/brw_compiler.h
index 28aed833245..6661427e982 100644
--
This patch series is a v2 of the one we sent some weeks ago to fix
gl_BaseVertex in i965. The original patch series can be found on here:
https://patchwork.freedesktop.org/series/33623/
* Rationale:
gl_BaseVertex must be zero for non-indexed draw comands. However,
for this kind of draw comands,
> Am 30.11.2017 um 22:18 schrieb Leo Liu:
>>
>>
>> On 11/30/2017 04:12 PM, boyuan.zh...@amd.com wrote:
>>> From: Boyuan Zhang
>>>
>>> Vaapi encode interface provides idr frame flags, where omx interface
>>> doesn't.
>>> Therefore, change to use picture type to determine idr frame, which
>>> will
Yeah, the typo was a last second edit, as always. Thanks for making a
sample failing case, will investigate. I hate all these interpolateat tests
:( so many edge cases...
On Dec 4, 2017 2:51 PM, "Marek Olšák" wrote:
> There is a typo, so it doesn't build.
>
> It breaks these CTS tests:
> gl45-ct
There is a typo, so it doesn't build.
It breaks these CTS tests:
gl45-cts@shader_multisample_interpolation@render@interpolate_at_offset*
Error:
state_tracker/st_glsl_to_tgsi.cpp:1347: void
glsl_to_tgsi_visitor::visit_expression(ir_expression*, st_src_reg*):
Assertion `ir->operands[1]->type->vecto
https://bugs.freedesktop.org/show_bug.cgi?id=104084
Bug ID: 104084
Summary: Vulkan doesnt work at all
Product: Mesa
Version: 17.2
Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: NEW
Severity: major
On 04.12.2017 20:45, Marek Olšák wrote:
Do we need to handle B10G10R10X2 as well?
Good catch, that is missing indeed. Also noticed that dEQP wide color
test does not test BGRX.
Marek
On Wed, Nov 29, 2017 at 5:20 AM, Mario Kleiner
wrote:
This format + type combo is good for BGRA1010102 f
For patches 4-5, 7, 9-19, 21-22:
Reviewed-by: Marek Olšák
Marek
On Wed, Nov 29, 2017 at 5:21 AM, Mario Kleiner
wrote:
> Some clients may not like rgb10 fbconfigs and visuals.
> Support driconf option 'allow_rgb10_configs' on gallium
> to allow per application enable/disable.
>
> The option def
Do we need to handle B10G10R10X2 as well?
Marek
On Wed, Nov 29, 2017 at 5:20 AM, Mario Kleiner
wrote:
> This format + type combo is good for BGRA1010102 framebuffers
> for use with glReadPixels() under GLES, so add it for the
> GL_IMPLEMENTATION_COLOR_READ_TYPE_OES query.
>
> Allows successful t
Quoting Emil Velikov (2017-12-04 09:28:19)
> On 1 December 2017 at 23:06, Dylan Baker wrote:
> > There are some case where the dri3 loader is covering for underlinkage
> > for GLX and EGL, provide the linkage that they actually need.
> >
> > Signed-off-by: Dylan Baker
> > ---
> > src/egl/meson.b
On 4 December 2017 at 18:25, Dylan Baker wrote:
> Quoting Emil Velikov (2017-12-04 10:14:15)
>> On 4 December 2017 at 18:06, Dylan Baker wrote:
>> > Quoting Emil Velikov (2017-12-04 05:20:03)
>> >> On 29 November 2017 at 00:56, Dylan Baker wrote:
>> >> > Signed-off-by: Dylan Baker
>> >> > ---
>
Reviewed-by: Bas Nieuwenhuizen
On Mon, Dec 4, 2017 at 3:32 PM, Samuel Pitoiset
wrote:
> module can be NULL, oops.
>
> v2: really check that module is not NULL
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/amd/vulkan/radv_shader.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>
Quoting Emil Velikov (2017-12-04 10:14:15)
> On 4 December 2017 at 18:06, Dylan Baker wrote:
> > Quoting Emil Velikov (2017-12-04 05:20:03)
> >> On 29 November 2017 at 00:56, Dylan Baker wrote:
> >> > Signed-off-by: Dylan Baker
> >> > ---
> >> > meson.build | 11 ++
From: Tomasz Figa
There is no API available to properly query the IMPLEMENTATION_DEFINED
format. As a workaround we rely here on gralloc allocating either
an arbitrary YCbCr 4:2:0 or RGBX_, with the latter being recognized
by lock_ycbcr failing.
Reviewed-on: https://chromium-review.googlesou
On 12/04/2017 12:47 PM, Emil Velikov wrote:
On 4 December 2017 at 16:33, Leo Liu wrote:
On 12/04/2017 04:32 AM, Emil Velikov wrote:
On 2 December 2017 at 15:26, Julian Lai wrote:
Since the crappy vaapi-glx was dropped off by mpv-git player, there is no
way to play HEVC 10bit video with ze
On 4 December 2017 at 18:06, Dylan Baker wrote:
> Quoting Emil Velikov (2017-12-04 05:20:03)
>> On 29 November 2017 at 00:56, Dylan Baker wrote:
>> > Signed-off-by: Dylan Baker
>> > ---
>> > meson.build | 11 +++
>> > meson_options.txt
Quoting Eric Engestrom (2017-12-04 07:08:01)
> src/gallium/winsys/pl111/drm/libpl111winsys.a(pl111_drm_winsys.c.o): In
> function `pl111_drm_screen_create':
> pl111_drm_winsys.c:(.text+0x33): undefined reference to
> `vc4_drm_screen_create_renderonly'
>
> Signed-off-by: Eric Engestrom
> ---
>
Reviewed-by: Dylan Baker
Quoting Rob Clark (2017-12-04 05:44:59)
> Otherwise huge amount of spam from instr-a2xx.h.. gcc has no way to know
> that freedreno was never built with such an old gcc version to care
> about the bugs in old gcc ;-)
>
> Patch is really Eric Engestrom's, he showed me how
Quoting Emil Velikov (2017-12-04 05:20:03)
> On 29 November 2017 at 00:56, Dylan Baker wrote:
> > Signed-off-by: Dylan Baker
> > ---
> > meson.build | 11 +++
> > meson_options.txt | 7 +++
> > src/gallium/drivers/etnaviv/meson.b
On 12/04/2017 04:32 AM, Emil Velikov wrote:
On 2 December 2017 at 15:26, Julian Lai wrote:
Since the crappy vaapi-glx was dropped off by mpv-git player, there is no
way to play HEVC 10bit video with zero-copy hardware acceleration, vdpau
can't do it either, please implement it to make it work
On 4 December 2017 at 16:33, Leo Liu wrote:
>
>
> On 12/04/2017 04:32 AM, Emil Velikov wrote:
>>
>> On 2 December 2017 at 15:26, Julian Lai wrote:
>>>
>>> Since the crappy vaapi-glx was dropped off by mpv-git player, there is no
>>> way to play HEVC 10bit video with zero-copy hardware acceleratio
On 1 December 2017 at 22:03, Dylan Baker wrote:
> Currently this pkg-config file is only installed if a classic dri driver
> is built. This is wrong, it should be installed if any dri driver is
> installed, which includes the gallium dri target.
>
> Reported-by: Marc Dietrich
> Signed-off-by: Dyl
This series is
Reviewed-by: Rafael Antognolli
On Mon, Dec 04, 2017 at 04:21:26PM +, Lionel Landwerlin wrote:
> Hi all,
>
> We've had some timestamp related issues on Gen10. Those are mostly
> related to the fact that the frequency of the timestamps is variable
> depending on the part you ge
On 1 December 2017 at 23:06, Dylan Baker wrote:
> There are some case where the dri3 loader is covering for underlinkage
> for GLX and EGL, provide the linkage that they actually need.
>
> Signed-off-by: Dylan Baker
> ---
> src/egl/meson.build | 2 +-
> src/glx/meson.build | 3 ++-
> 2 files cha
On 1 December 2017 at 23:06, Dylan Baker wrote:
> This was covering for underinkage elsewhere. With that fixed these can
> be removed.
>
> Signed-off-by: Dylan Baker
> ---
> src/loader/meson.build | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/src/loader/meson.build b
Add a new helper that drivers can use to emulate various things that
need special handling in particular in transfer_map:
1) z32_s8x24.. gl/gallium treats this as a single buffer with depth
and stencil interleaved but hardware frequently treats this as
separate z32 and s8 buffers. Specia
Signed-off-by: Rob Clark
---
src/gallium/drivers/freedreno/freedreno_resource.c | 271 -
src/gallium/drivers/freedreno/freedreno_resource.h | 2 +-
2 files changed, 44 insertions(+), 229 deletions(-)
diff --git a/src/gallium/drivers/freedreno/freedreno_resource.c
b/src/gal
On 2017-12-04 03:49 AM, Lvzhihong (ReJohn) wrote:
> Hi,all,
>
> We met a problem on ubuntu17.10 for arm server with amdgpu(AMD
> RADEON PRO WX7100), we use open source driver which are integrated in
> ubuntu17.10. And the architecture is AArch64-linux-gnu.
>
>
>
> we install
El 01/12/17 a las 09:06, Pohjolainen, Topi escribió:
> On Thu, Nov 30, 2017 at 03:07:59AM +0100, Jose Maria Casanova Crespo wrote:
>> From: Alejandro Piñeiro
>>
>> Although it is possible to emit them directly as AND/OR on brw_fs_nir,
>> having a specific opcode makes it easier to remove duplicate
On Monday, 2017-12-04 12:48:55 +0200, Vadim Shovkoplias wrote:
> Hi Eric,
Hey, sorry, I forgot to hit "send" on the reply I wrote on friday :]
>
> Mostly by a static analysis tool. It found at least 7 issues with useless
> free() calls and other problems that probably should be fixed.
What tool
El 01/12/17 a las 09:03, Pohjolainen, Topi escribió:
> On Thu, Nov 30, 2017 at 03:07:55AM +0100, Jose Maria Casanova Crespo wrote:
>> v2: Fixed calculation of scalar size for 16-bit types. (Jason Ekstrand)
>>
>> Signed-off-by: Jose Maria Casanova Crespo
>> Signed-off-by: Eduardo Lima
>> Reviewed-
According with OpenGL GLSL 4.20 spec, section 4.3.9, page 57:
"It is a link-time error if any particular shader interface
contains:
- two different blocks, each having no instance name, and each
having a member of the same name, or
- a variable outside a block, and a blo
Hi all,
We've had some timestamp related issues on Gen10. Those are mostly
related to the fact that the frequency of the timestamps is variable
depending on the part you get. There is no way to tell from the
PCI-id, you can only get that information by reading a couple of
registers.
We introduced
We cannot figure this value out of the PCI-id anymore. Let's read it
from the kernel (which computes this from a few registers).
When running on a (upcoming) 4.16-rc1+ kernel, this will fixes piglit
tests on CNL :
spec@arb_timer_query@query gl_timestamp
spec@arb_timer_query@timestamp-get
The reference value in gen_device_info isn't going to be acurate on
Gen10+. We should query it from the kernel, which reads a couple of
register to compute the actual value.
Signed-off-by: Lionel Landwerlin
---
src/intel/vulkan/anv_device.c | 13 +
1 file changed, 13 insertions(+)
d
Taken from drm-next ca797d29cd63e7b71b4eea29aff3b1cefd1ecb59
Signed-off-by: Lionel Landwerlin
---
include/drm-uapi/README | 8 +++---
include/drm-uapi/drm.h | 41 ++
include/drm-uapi/drm_mode.h | 70 +++--
include/drm-uapi
On Saturday, 2017-12-02 13:28:04 -0500, Ilia Mirkin wrote:
> On Sat, Dec 2, 2017 at 1:22 PM, Rhys Kidd wrote:
> > Signed-off-by: Rhys Kidd
> > ---
> > src/gallium/drivers/nouveau/codegen/nv50_ir_emit_nv50.cpp | 2 +-
> > src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 2 +-
> > 2 fil
El 30/11/17 a las 10:25, Pohjolainen, Topi escribió:
> On Thu, Nov 30, 2017 at 03:07:45AM +0100, Jose Maria Casanova Crespo wrote:
>> From: Eduardo Lima Mitev
> Just a few style nits, see below.
>
>> Adds new INT16, UINT16 and FLOAT16 base types.
>>
>> The corresponding GL types for half floats w
On Friday, 2017-12-01 15:06:17 -0800, Dylan Baker wrote:
> As we discussed elsewhere, this should fix the linkage of the dri3 loader and
> glx and egl.
>
> Dylan Baker (3):
> meson: Reformat meson code to match more common style
> meson: fix underlinkage without dri3
> meson: Fix overlinkage
On Friday, 2017-12-01 14:03:16 -0800, Dylan Baker wrote:
> Currently this pkg-config file is only installed if a classic dri driver
> is built. This is wrong, it should be installed if any dri driver is
> installed, which includes the gallium dri target.
>
> Reported-by: Marc Dietrich
> Signed-of
On Thursday, 2017-11-30 14:20:08 +, Emil Velikov wrote:
> On 29 November 2017 at 18:12, Rob Herring wrote:
> > On Wed, Nov 29, 2017 at 11:01 AM, Emil Velikov
> > wrote:
> >> On 27 November 2017 at 19:36, Rob Herring wrote:
> >>> AOSP master has changed the build default to -Werror making al
On Thursday, 2017-11-30 10:34:06 -0800, Dylan Baker wrote:
> Quoting Eric Engestrom (2017-11-29 07:44:08)
> > On Tuesday, 2017-11-28 15:30:21 -0800, Dylan Baker wrote:
> > > This series is the gallium media state trackers, the "nine" Direct3D state
> > > tracker, and an architectural change in the
Reviewed-by: Marek Olšák
Marek
On Sat, Dec 2, 2017 at 1:36 PM, Mauro Rossi wrote:
> libmesa_amd_common static dependency is added in Android build
> to avoid the following building errors:
>
> In file included from
> external/mesa/src/gallium/drivers/radeon/r600_buffer_common.c:24:
> In file i
src/gallium/winsys/pl111/drm/libpl111winsys.a(pl111_drm_winsys.c.o): In
function `pl111_drm_screen_create':
pl111_drm_winsys.c:(.text+0x33): undefined reference to
`vc4_drm_screen_create_renderonly'
Signed-off-by: Eric Engestrom
---
meson.build | 4
src/galliu
Thanks to Emil's -Wundef, t_dd_dmatmp.h now complains that intel_render.c
is missing a couple `#define`s.
Assigning them to 0 keeps the existing behaviour; I'll let someone else
turn them on if this is the behaviour that was intended.
Signed-off-by: Eric Engestrom
---
src/mesa/drivers/dri/i915/
module can be NULL, oops.
v2: really check that module is not NULL
Signed-off-by: Samuel Pitoiset
---
src/amd/vulkan/radv_shader.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/amd/vulkan/radv_shader.h b/src/amd/vulkan/radv_shader.h
index 91f2e7f2a1..f6486863f8 100
On Monday, 2017-12-04 08:44:59 -0500, Rob Clark wrote:
> Otherwise huge amount of spam from instr-a2xx.h.. gcc has no way to know
> that freedreno was never built with such an old gcc version to care
> about the bugs in old gcc ;-)
>
> Patch is really Eric Engestrom's, he showed me how to do this
On 12/03/2017 10:04 AM, Gurkirpal Singh wrote:
I sent the modified patches in another thread a while ago.
Please review in case got missed.
Please be patient for a few days to see if any other comments.
After then please rebase, add rb/ab to your patches, and send them to me,
I will commit the
On 27 November 2017 at 18:40, Eric Engestrom wrote:
> On Sunday, 2017-11-26 10:12:46 +1100, Timothy Arceri wrote:
>> On 25/11/17 08:02, Timothy Arceri wrote:
>> > On 25/11/17 05:07, Eric Engestrom wrote:
>> > > Signed-off-by: Eric Engestrom
>> > > ---
>> > > src/intel/vulkan/anv_private.h | 2 +
On Mon, Nov 27, 2017 at 07:06:06PM -0800, Jason Ekstrand wrote:
> This is a bit less awkward than passing in the subpass because it means
> we don't have to extract the subpass id from the subpass.
> ---
> src/intel/vulkan/genX_cmd_buffer.c | 12 +---
> 1 file changed, 5 insertions(+), 7 d
On Mon, Nov 27, 2017 at 07:06:05PM -0800, Jason Ekstrand wrote:
> Having begin/end_subpass is a bit nicer than the begin/next/end hooks
> that Vulkan gives us.
> ---
> src/intel/vulkan/genX_cmd_buffer.c | 55
> +-
> 1 file changed, 31 insertions(+), 24 deletion
Otherwise huge amount of spam from instr-a2xx.h.. gcc has no way to know
that freedreno was never built with such an old gcc version to care
about the bugs in old gcc ;-)
Patch is really Eric Engestrom's, he showed me how to do this in meson.
Cc: Eric Engestrom
Signed-off-by: Rob Clark
---
src
On 29 November 2017 at 00:56, Dylan Baker wrote:
> Signed-off-by: Dylan Baker
> ---
> meson.build | 11 +++
> meson_options.txt | 7 +++
> src/gallium/drivers/etnaviv/meson.build | 2 +-
> src/gallium/drivers/freedreno/meson.b
Patches 1 and 3-7:
Reviewed-by: Nicolai Hähnle
On 01.12.2017 21:19, Marek Olšák wrote:
From: Marek Olšák
---
src/gallium/auxiliary/util/u_upload_mgr.c| 9 ++---
src/gallium/auxiliary/util/u_upload_mgr.h| 2 +-
src/gallium/drivers/freedreno/a3xx/fd3_context.c | 2 +-
On Dec 4, 2017 12:58 PM, "Nicolai Hähnle" wrote:
On 01.12.2017 21:19, Marek Olšák wrote:
> From: Marek Olšák
>
> Cc: 17.3
>
What's the use-case for this?
OpenCL interop.
What if somebody exports as DMABUF, then re-imports in a different API and
exports as FD for inter-process sharing from
On 01.12.2017 21:19, Marek Olšák wrote:
From: Marek Olšák
and anything that clones these uploaders, like u_threaded_context.
Bottom/top-of-pipe fences will have to be moved away from using the
stream_uploader.
---
src/gallium/drivers/radeon/r600_pipe_common.c | 6 --
1 file change
On 01.12.2017 21:19, Marek Olšák wrote:
From: Marek Olšák
Cc: 17.3
What's the use-case for this?
What if somebody exports as DMABUF, then re-imports in a different API
and exports as FD for inter-process sharing from there?
---
src/gallium/drivers/radeon/r600_texture.c | 4 +++-
1 f
Reviewed-by: Nicolai Hähnle
On 04.12.2017 05:17, Timothy Arceri wrote:
---
src/gallium/drivers/radeonsi/si_shader.c | 15 +++
1 file changed, 7 insertions(+), 8 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_shader.c
b/src/gallium/drivers/radeonsi/si_shader.c
index
1 - 100 of 114 matches
Mail list logo
