For regular ast_add, we can implicitly change either a or b's type.
However in an assignment situation, the type of the lvalue is fixed. So
if the implicit conversion logic decides to change it, it means that the
rhs's type could not be converted to the lhs type.
Emit a specific error for this
Reviewed-by: Edward O'Callaghan
On 07/09/2016 03:42 AM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> The blitter sets mask == 1, which is fine since it doesn't use smoothing.
> Fixes a regression introduced in commit 5bcfbf91.
> ---
>
Signed-off-by: Ilia Mirkin
Cc: "11.2 12.0"
---
v1 -> v2: also include a mesa_is_etc2_format function which takes a GLenum.
src/mesa/main/glformats.c | 23 +++
src/mesa/main/glformats.h | 3 +++
On Fri, Jul 8, 2016 at 8:16 PM, Anuj Phogat wrote:
> On Fri, Jul 8, 2016 at 3:42 PM, Ilia Mirkin wrote:
>> Signed-off-by: Ilia Mirkin
>> Cc: "11.2 12.0"
>> ---
>>
>> This is sort of pending a
On Fri, Jul 8, 2016 at 3:42 PM, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
> Cc: "11.2 12.0"
> ---
>
> This is sort of pending a question to KHR at
>
> https://www.khronos.org/bugzilla/show_bug.cgi?id=1511
>
>
https://bugs.freedesktop.org/show_bug.cgi?id=79706
--- Comment #3 from Kenneth Graunke ---
Closing this out as having an "all regressions ever" tracker is useless...it's
just a meta-bugzilla. Per-release trackers are more reasonable.
At the time of closing, these were
https://bugs.freedesktop.org/show_bug.cgi?id=80835
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=66346
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=86939
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=89199
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=86837
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=89773
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=88806
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=90539
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=86944
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=89960
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=85189
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=86980
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=80069
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=72326
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=90346
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=82471
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=89112
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=70359
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=87276
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=90081
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=78318
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=45348
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=75661
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=61153
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=79098
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=79706
Kenneth Graunke changed:
What|Removed |Added
Depends on|44519, 66346, 70359, 71199, |
https://bugs.freedesktop.org/show_bug.cgi?id=77288
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=82477
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=59777
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=88467
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=89330
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=77161
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=71199
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=61326
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=44519
Kenneth Graunke changed:
What|Removed |Added
Blocks|79706 |
Referenced
https://bugs.freedesktop.org/show_bug.cgi?id=79706
Kenneth Graunke changed:
What|Removed |Added
Status|NEW |RESOLVED
This extension is needed by Android (via ARC++) on Chrome OS.
Essentially, this extension provides a translation between EGLSync and
explicit cross-process synchronization points, represented as "sync
fds". The relevant documentation is:
[1]:
TODO: The i915 kernel interface nor the libdrm interface is upstream
yet. So small details may change between now and the final patch.
TODO: Agree on fd ownership rules between EGL and driver with Rob Clark.
TODO: Handle errors from ppoll().
TODO: Test it!
---
From: Rob Clark
Required to implement EGL_ANDROID_native_fence_sync.
Signed-off-by: Rob Clark
---
include/GL/internal/dri_interface.h | 44 -
1 file changed, 43 insertions(+), 1 deletion(-)
diff --git
A variant of intel_batchbuffer_flush() with parameters for in and out
fence fds.
TODO: The i915 kernel interface is not yet upstream.
TODO: The fence variants of libdrm functions drm_intel_*_exec() are not
upstream.
---
src/mesa/drivers/dri/i965/intel_batchbuffer.c | 30
Don't set SyncCondition twice.
---
src/egl/main/eglsync.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/egl/main/eglsync.c b/src/egl/main/eglsync.c
index 375f223..d65fbdd 100644
--- a/src/egl/main/eglsync.c
+++ b/src/egl/main/eglsync.c
@@ -119,7 +119,6 @@ _eglInitSync(_EGLSync *sync,
See the comments.
---
src/egl/drivers/dri2/egl_dri2.c | 12
1 file changed, 12 insertions(+)
diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c
index c7b81ce..03ed4b9 100644
--- a/src/egl/drivers/dri2/egl_dri2.c
+++ b/src/egl/drivers/dri2/egl_dri2.c
@@
The function is tiny and called exactly once. There's no need for it to
exist.
---
src/mesa/drivers/dri/i965/intel_syncobj.c | 12
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/intel_syncobj.c
b/src/mesa/drivers/dri/i965/intel_syncobj.c
From: Rob Clark
Reduce the noise in the next patch. For EGL_SYNC_NATIVE_FENCE_ANDROID
the sync condition is conditional on EGL_SYNC_NATIVE_FENCE_FD_ANDROID
attribute.
Signed-off-by: Rob Clark
---
src/egl/main/eglsync.c | 16
Mesa was incrementing and decrementing EGL refcounts with ++ and --.
---
src/egl/main/egldisplay.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/src/egl/main/egldisplay.c b/src/egl/main/egldisplay.c
index bbc3063..78b5d20 100644
--- a/src/egl/main/egldisplay.c
Close the sync fd *after* the sync's refcount drops to 0.
---
src/egl/drivers/dri2/egl_dri2.c | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c
index a9d351e..c7b81ce 100644
---
See the comments.
---
include/GL/internal/dri_interface.h | 9 +
1 file changed, 9 insertions(+)
diff --git a/include/GL/internal/dri_interface.h
b/include/GL/internal/dri_interface.h
index 99c83ec..051ddb4 100644
--- a/include/GL/internal/dri_interface.h
+++
This bool maps to I915_PARAM_HAS_EXEC_FENCE_FD.
TODO: The i915 param is not yet upstream. Wait for the kernel interface
before committing.
---
src/mesa/drivers/dri/i965/intel_screen.c | 3 +++
src/mesa/drivers/dri/i965/intel_screen.h | 2 +-
2 files changed, 4 insertions(+), 1 deletion(-)
From: Rob Clark
[chadv]: Fix bugs in attribute parsing.
Signed-off-by: Rob Clark
---
src/egl/drivers/dri2/egl_dri2.c | 49 +
src/egl/main/eglapi.c | 36 +++---
Cacheing the signal status complicates the code for questionable
performance benefit. I added the cacheing long ago, and I now think it
was the wrong decision.
---
src/mesa/drivers/dri/i965/intel_syncobj.c | 28 +++-
1 file changed, 3 insertions(+), 25 deletions(-)
diff
https://bugs.freedesktop.org/show_bug.cgi?id=89599
Vinson Lee changed:
What|Removed |Added
CC||v...@freedesktop.org
https://bugs.freedesktop.org/show_bug.cgi?id=92630
Vinson Lee changed:
What|Removed |Added
Resolution|--- |DUPLICATE
https://bugs.freedesktop.org/show_bug.cgi?id=75661
Vinson Lee changed:
What|Removed |Added
Status|NEW |RESOLVED
https://bugs.freedesktop.org/show_bug.cgi?id=79706
Bug 79706 depends on bug 75661, which changed state.
Bug 75661 Summary: st_glsl_to_tgsi.cpp:637:get_opcode: Assertion `src0.type !=
GLSL_TYPE_STRUCT' failed.
https://bugs.freedesktop.org/show_bug.cgi?id=75661
What|Removed
https://bugs.freedesktop.org/show_bug.cgi?id=79471
Vinson Lee changed:
What|Removed |Added
Resolution|--- |FIXED
Hi all,
Mesa 12.0.1 is available.
It resolves r600/radeonsi build issues introduced in the 12.0.0 release.
Emil Velikov (4):
docs: add sha256 checksums for 12.0.0
radeon: reference the correct cdw/max_dw
Update version to 12.0.1
docs: add release notes for 12.0.1
git
https://bugs.freedesktop.org/show_bug.cgi?id=96853
--- Comment #4 from Brian Paul ---
I'll try to take a look at this when I get a little time...
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for the
Move the error handling into a little helper function. This will keep
the code clean when later adding error handling for attribute
EGL_SYNC_NATIVE_FENCE_FD_ANDROID.
And fix the log message to work when EGLAttrib is 32-bit or 64-bit.
---
src/egl/main/eglsync.c | 31
I found these fixes and cleanups while trying to implement
EGL_ANDROID_native_fence_sync for i965.
No changes in test results when running `deqp-egl --deqp-case='*sync*'`
under X11.
Branch lives at
git://git.kiwitree.net/~chadv/mesa#review/egl-sync-cleanups-v01
This function converts an attribute list from EGLint[] to EGLAttrib[].
Will be use in following patches to cleanup EGLSync attribute parsing.
---
src/egl/main/eglapi.c | 41 +
src/egl/main/eglapi.h | 2 ++
2 files changed, 43 insertions(+)
diff --git
eglCreateSyncKHR takes an attribute list of type EGLint[]. eglCreateSync
takes one of type EGLAttrib[]. Implement eglCreateSyncKHR's attribute
parsing in terms of eglCreateSync's by converting the EGLint[] into
EGLAttrib[].
This cleanup will simplify the code when later implementing
- The list elements have type EGLAttrib, not EGLint.
- The array index iterator should be size_t, not EGLint.
---
src/egl/main/eglsync.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/src/egl/main/eglsync.c b/src/egl/main/eglsync.c
index 33625e9..48714d1 100644
Signed-off-by: Ilia Mirkin
Cc: "11.2 12.0"
---
This is sort of pending a question to KHR at
https://www.khronos.org/bugzilla/show_bug.cgi?id=1511
but OTOH, no matter their answer, mesa doesn't have online compression for
etc2. So we
I do appreciate the cleverness, but unfortunately it prevents a lot more
cleverness in the form of additional compiler optimizations brought on
by -fstrict-aliasing.
No difference in OglBatch7 (n=20).
Co-authored-by: Davin McCall
---
I took Ian's suggestion to add
On Fri, 2016-07-08 at 14:39 -0700, Francisco Jerez wrote:
> Jan Vesely writes:
>
> > On Sun, 2016-07-03 at 17:51 -0700, Francisco Jerez wrote:
> > > This gets rid of most ifdef's from the invocation.cpp code --
> > > Only a
> > > couple of them are left which will be
Jan Vesely writes:
> On Sun, 2016-07-03 at 17:51 -0700, Francisco Jerez wrote:
>> This gets rid of most ifdef's from the invocation.cpp code -- Only a
>> couple of them are left which will be removed differently in the
>> following commits.
>>
>> Reviewed-by: Serge
On Sun, 2016-07-03 at 17:51 -0700, Francisco Jerez wrote:
> This gets rid of most ifdef's from the invocation.cpp code -- Only a
> couple of them are left which will be removed differently in the
> following commits.
>
> Reviewed-by: Serge Martin
> ---
>
Only keep track of a single current context, instead of separate contexts for
GL and GLES.
In EGL 1.4 (and 1.5), EGL_OPENGL_API and EGL_OPENGL_ES_API are supposed to be
interchangeable for all purposes except for eglCreateContext.
The _EGLThreadInfo::CurrentContexts array is now a single pointer
https://bugs.freedesktop.org/show_bug.cgi?id=61907
Matt Turner changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Iago Toral writes:
> On Thu, 2016-07-07 at 19:36 -0700, Francisco Jerez wrote:
>> Samuel Iglesias Gonsálvez writes:
>>
>> >
>> > From: Iago Toral Quiroga
>> >
>> > In fp64 we can produce code like this:
>> >
>> > mov(16)
https://bugs.freedesktop.org/show_bug.cgi?id=96865
Bug ID: 96865
Summary: [swrast] egl-create-pbuffer-surface regression
Product: Mesa
Version: git
Hardware: Other
OS: All
Status: NEW
Keywords: bisected,
https://bugs.freedesktop.org/show_bug.cgi?id=96765
Corentin Wallez changed:
What|Removed |Added
Status|NEW |RESOLVED
Forget the previous email. It was an error on the test. The texture
barrier works fine.
Sorry for the noise.
BR
On 08/07/16 15:58, Alejandro Piñeiro wrote:
> Hi, today I was updating the piglit test, as I got a detailed review.
> While testing the changes, I found that there are some parameters
ping.
is there any other way to get dynamic indices working?
Jan
On Sun, 2016-06-26 at 20:40 -0400, Jan Vesely wrote:
> Both explicit and implicit.
> Using vtx 0 (as existing llvm code implies) does not work for dynamic
> offsets.
>
> Signed-off-by: Jan Vesely
> ---
>
When a glTexImage call updates the parameters of a currently bound
framebuffer, we might miss out on revalidating whether it is complete.
Make sure to set _NEW_BUFFERS which will trigger the revalidation in
that case.
Also while we're at it, fix the fb parameter passed in to the eventual
From: Nicolai Hähnle
The blitter sets mask == 1, which is fine since it doesn't use smoothing.
Fixes a regression introduced in commit 5bcfbf91.
---
src/gallium/drivers/radeonsi/si_state.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
On Fri, Jul 08, 2016 at 12:35:11PM -0400, Ilia Mirkin wrote:
> Probably a copy-paste from mesa_meta_pbo_GetTexSubImage where tex_image
> may apparently be null.
>
> Signed-off-by: Ilia Mirkin
Indeed, tex_image is dereferenced 30 lines before, so at this point it
can't be
Probably a copy-paste from mesa_meta_pbo_GetTexSubImage where tex_image
may apparently be null.
Signed-off-by: Ilia Mirkin
---
Happened upon this when trying to figure out the "incomplete fbo" issue.
src/mesa/drivers/common/meta_tex_subimage.c | 2 +-
1 file changed, 1
On Sat, Jul 2, 2016 at 12:52 PM, Rob Clark wrote:
> So, games/apps that are aware of how a tiler gpu works will make an
> effort to avoid mid-batch (tile pass) updates to textures, UBOs, etc,
> since this will force a flush, and extra resolve (tile->mem) and
> restore
Series is
Reviewed-by: Ilia Mirkin
On Tue, Jul 5, 2016 at 8:01 AM, Samuel Pitoiset
wrote:
> Signed-off-by: Samuel Pitoiset
> ---
> .../nouveau/codegen/nv50_ir_lowering_nvc0.cpp | 24
> --
>
Hi all,
It's a real honour to announce Mesa 12.0.0.
This release has massive amount of features, but without a doubt the biggest
ones are:
- Vulkan driver for Intel hardware from Ivy Bridge onward.
- OpenGL 4.3 for nvc0, radeonsi and i965 (Gen8+)
- OpenGL ES 3.1 on nvc0 and radeonsi
- GLVND
Hi, today I was updating the piglit test, as I got a detailed review.
While testing the changes, I found that there are some parameters
combinations that are still not passing. They are related to the number
of draw passes (or in other words, the number of glDrawElements calls
for the same
On Fri, Jul 8, 2016 at 6:22 AM, Eric Engestrom
wrote:
> On Fri, Jul 08, 2016 at 02:48:04AM -0400, Ilia Mirkin wrote:
>> Signed-off-by: Ilia Mirkin
>> ---
>> src/mesa/state_tracker/st_cb_texture.c | 15 +--
>> 1 file changed, 9
https://bugs.freedesktop.org/show_bug.cgi?id=96853
--- Comment #3 from Roland Scheidegger ---
(In reply to denis.fisseler from comment #2)
> This sounds plausible, thank you for shedding some light on this! The faking
> of large points through triangles in a GS would also
https://bugs.freedesktop.org/show_bug.cgi?id=96853
--- Comment #2 from denis.fisse...@tu-dortmund.de ---
This sounds plausible, thank you for shedding some light on this! The faking of
large points through triangles in a GS would also explain, why some of my GS
code, I use with large points and
On 08/07/16 00:27, Francisco Jerez wrote:
> Samuel Iglesias Gonsálvez writes:
>
>> From: Iago Toral Quiroga
>>
>> Gen7 hardware does not support double immediates so these need
>> to be moved in 32-bit chunks to a regular vgrf instead. Instead
>> of
On Fri, Jul 08, 2016 at 02:48:04AM -0400, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
> ---
> src/mesa/state_tracker/st_cb_texture.c | 15 +--
> 1 file changed, 9 insertions(+), 6 deletions(-)
>
> diff --git a/src/mesa/state_tracker/st_cb_texture.c
>
Hi Chuck,
this commit breaks 32-bit builds at least of radeonsi and probably
others because malloc()ed structures are only aligned to 8 bytes, see
https://bugs.freedesktop.org/show_bug.cgi?id=96835
I presume there are two possible fixes:
1. Drop the alignment on 32-bit.
2. Align affected
https://bugs.freedesktop.org/show_bug.cgi?id=96835
Nicolai Hähnle changed:
What|Removed |Added
QA Contact|dri-devel@lists.freedesktop
On 07.07.2016 23:49, Marek Olšák wrote:
On Thu, Jul 7, 2016 at 9:39 AM, Nicolai Hähnle wrote:
From: Nicolai Hähnle
We only have to stay single-threaded when debug output must be synchronous.
This yields better parallelism in shader-db runs for me.
On 07.07.2016 16:40, Jan Vesely wrote:
On Thu, 2016-07-07 at 09:39 +0200, Nicolai Hähnle wrote:
From: Nicolai Hähnle
---
src/gallium/include/pipe/p_state.h | 6 ++
src/gallium/state_trackers/clover/core/queue.cpp | 5 -
On Fri, 2016-07-08 at 09:21 +0200, Iago Toral wrote:
> On Thu, 2016-07-07 at 19:36 -0700, Francisco Jerez wrote:
> >
> > Samuel Iglesias Gonsálvez writes:
> >
> > >
> > >
> > > From: Iago Toral Quiroga
> > >
> > > In fp64 we can produce code like
On 07/07/16 23:38, Francisco Jerez wrote:
> Samuel Iglesias Gonsálvez writes:
>
>> So far we only used instructions with this size in situations where we
>> did not operate per-channel and we wanted to ignore the execution mask,
>> but gen7 fp64 will need to emit code
On Fri, 2016-07-08 at 09:21 +0200, Iago Toral wrote:
> On Thu, 2016-07-07 at 19:36 -0700, Francisco Jerez wrote:
> >
> > Samuel Iglesias Gonsálvez writes:
> >
> > >
> > >
> > > From: Iago Toral Quiroga
> > >
> > > In fp64 we can produce code like
1 - 100 of 106 matches
Mail list logo