https://bugs.freedesktop.org/show_bug.cgi?id=101334
--- Comment #23 from John ---
Some apps worked, others froze the system.
I'm still hopeful to find a fix here :)
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
https://bugs.freedesktop.org/show_bug.cgi?id=101334
--- Comment #22 from Marko ---
(In reply to John from comment #21)
> I believe that's a same generation card, so it would make sense to behave
> similarly.
Yeah. Did it ever work for you? RADV was a no-go on my card from day one.
--
You are r
When using a read-only CPU mapping, we may encounter stale buffer
contents. For example, the Piglit primitive-restart test offers the
following scenario:
1. Read data via a CPU map.
2. Destroy that buffer.
3. Create a new buffer - obtaining the same one via the BO cache.
4. Call Buffe
Just map the buffer and memcpy. This will do a CPU mmap, which should
be reasonably efficient, and doing this gives us full control over the
domains and caching instead of leaving it to the kernel.
This prevents regressions on Braswell in the next commit. Specifically
GL45-CTS.shader_atomic_coun
Hello,
Mesa master has been hitting assert failures when running "XCOM: Enemy
Unknown" since commit f8d69beed49c64f883bb8ffb28d4960306baf575, where we
started asserting that the SAMPLER_STATE LOD Bias value actually fits in
the correct number of bits.
Apparently, XCOM calls
glTexEnv(GL_TEXTUR
On Wed, Jul 5, 2017 at 1:19 PM, Eric Anholt wrote:
> For the automake build, -Isrc/ is implied from the gallium cflags, while
> Android gallium driver builds don't get that by default. I think it'll be
> better for vc4 to have broadcom includes appear as "#include
> " to make it more clear where
Add Gao, Shuo
-Original Message-
From: Wu, Zhongmin
Sent: Tuesday, July 11, 2017 10:03
To: 'Emil Velikov' ; Marathe, Yogesh
Cc: Widawsky, Benjamin ; Liu, Zhiquan
; 'Eric Engestrom' ; 'Rob Clark'
; 'Tomasz Figa' ; 'Kenneth
Graunke' ; Kondapally, Kalyan
; 'ML mesa-dev' ;
'Timothy Ar
These two patches allow us to change how we build and link the swr
driver; details are in the second patch commit message.
Change in disk space:
libGL.so6464 Kb -> 7000 Kb
libswrAVX.so 10068 Kb -> 5432 Kb
libswrAVX2.so 9828 Kb -> 5200 Kb
Total 26360 Kb -> 17632 Kb
v2:
Use the SWR rasterizer API through the table returned from
SwrGetInterface rather than referencing the functions directly.
This will allow us to move to a model of having the driver dynamically
load the appropriate swr architecture library.
---
src/gallium/drivers/swr/swr_clear.cpp | 6 ++---
s
swr used to build and link the rasterizer to the driver, and to support
multiple architectures we needed to have multiple versions of the
driver/rasterizer combination, which needed to link in much of mesa.
Changing to having one instance of the driver and just building
architecture specific versi
From: Dave Airlie
Some of the Sascha Willems demos pick a D32/S8 format for the depth
buffer, then do a LOAD_OP_CLEAR/LOAD_OP_DONT_CARE on it, which means
we don't get to merge the undefined->depth and clear htile transitions.
This add the stencil aspect to the pending clears if there is a depth
By the way,
For cancelBuffer, sorry I forget such function, thanks for notice. It should
also pass the same fence fd as the queuebuffer.
And Yogesh, you mentioned the gallium, is it another platform supported by
mesa ? I am sorry I have no idea about this, could you please help to check
t
We were using the "cp" union fields, which are only valid for compute
shaders. The threads calculation affects the availabel GPRs, so just
pick a small number for other shader types to avoid limiting available
registers.
Signed-off-by: Ilia Mirkin
Cc: mesa-sta...@lists.freedesktop.org
---
src/ga
Hi Emil and Yogesh
Thank you for your comments, and thanks Yogesh for giving the detailed
explanations
And according to the document of Android below
(https://source.android.com/devices/graphics/arch-bq-gralloc):
Recent Android devices support the sync framework, which enables the system to
On Mon, Jul 10, 2017 at 1:34 PM, Andres Gomez wrote:
> Jason, which is the status of this patch? Has it been superseded or
> discarded?
>
Still awaiting review.
> On Mon, 2017-06-26 at 09:01 -0700, Jason Ekstrand wrote:
> > This fixes the Piglit ARB_texture_views rendering-formats test.
> >
>
For the series,
Reviewed-by: Neha Bhende
Regards,
Neha
From: Brian Paul
Sent: Monday, July 10, 2017 2:50:25 PM
To: mesa-dev@lists.freedesktop.org
Cc: Charmaine Lee; Neha Bhende
Subject: [PATCH 5/5] svga: s/unsigned/enum tgsi_texture_type/
---
src/gallium/d
On Mon, Jul 10, 2017 at 09:35:25AM -0700, Jason Ekstrand wrote:
> On Wed, Jun 28, 2017 at 2:14 PM, Nanley Chery wrote:
>
> > We'll be performing a GPU memcpy in more places to copy small amounts of
> > data. Add an alternate function that thrashes less state.
> >
> > v2:
> > - Make a new function
On Mon, Jul 10, 2017 at 3:50 PM, Matt Turner wrote:
> On Mon, Jul 10, 2017 at 1:10 PM, Connor Abbott wrote:
>> On Thu, Jul 6, 2017 at 4:48 PM, Matt Turner wrote:
>>> We already had a channel_num system value, which I'm renaming to
>>> subgroup_invocation to match the rest of the new system value
On Wed, Jun 28, 2017 at 2:14 PM, Nanley Chery wrote:
> Image layouts only let us know that an image *may* be fast-cleared. For
> this reason we can end up with redundant resolves. Testing has shown
> that such resolves can measurably hurt performance and that predicating
> them can avoid the pena
On Mon, Jul 10, 2017 at 1:10 PM, Connor Abbott wrote:
> On Thu, Jul 6, 2017 at 4:48 PM, Matt Turner wrote:
>> We already had a channel_num system value, which I'm renaming to
>> subgroup_invocation to match the rest of the new system values.
>>
>> Note that while ballotARB(true) will return zeros
Series looks good.
Reviewed-by: Charmaine Lee
From: Brian Paul
Sent: Monday, July 10, 2017 2:50:25 PM
To: mesa-dev@lists.freedesktop.org
Cc: Charmaine Lee; Neha Bhende
Subject: [PATCH 5/5] svga: s/unsigned/enum tgsi_texture_type/
---
src/gallium/driver
https://bugs.freedesktop.org/show_bug.cgi?id=101668
Ernst Sjöstrand changed:
What|Removed |Added
CC||ern...@gmail.com
--
You are receivin
https://bugs.freedesktop.org/show_bug.cgi?id=101668
--- Comment #4 from Ernst Sjöstrand ---
I saw this too. The next time Padoka updates LLVM that fix should come in,
let's see if that fixes it.
--
You are receiving this mail because:
You are the assignee for the bug.
You are the QA Contact for
Ian Romanick writes:
> From: Ian Romanick
>
> On NV20 (and probably also on earlier NV GPUs that lack
> GL_ARB_texture_border_clamp) fixes the following piglit tests:
>
> gl-1.0-beginend-coverage gltexparameter[if]{v,}
> push-pop-texture-state
> texwrap 1d
> texwrap 1d proj
>
On Mon, Jul 10, 2017 at 09:28:05AM -0700, Jason Ekstrand wrote:
> On Wed, Jun 28, 2017 at 2:14 PM, Nanley Chery wrote:
>
> > v2: Remove ::first_subpass_layout assertion (Jason Ekstrand).
> > v3: Allow some fast clears in the GENERAL layout.
> >
> > Signed-off-by: Nanley Chery
> > ---
> > src/in
---
src/gallium/drivers/svga/svga_tgsi_vgpu10.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_tgsi_vgpu10.c
b/src/gallium/drivers/svga/svga_tgsi_vgpu10.c
index bbaad20..d29ac28 100644
--- a/src/gallium/drivers/svga/svga_tgsi_vgpu10.c
+++
And s/unsigned/enum tgsi_interpolate_loc/
---
src/gallium/drivers/svga/svga_tgsi_vgpu10.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/gallium/drivers/svga/svga_tgsi_vgpu10.c
b/src/gallium/drivers/svga/svga_tgsi_vgpu10.c
index cd4cab4..d02dbb6 100644
--- a/src/gallium
---
src/gallium/drivers/svga/svga_tgsi_vgpu10.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_tgsi_vgpu10.c
b/src/gallium/drivers/svga/svga_tgsi_vgpu10.c
index d02dbb6..bbaad20 100644
--- a/src/gallium/drivers/svga/svga_tgsi_vgpu10.c
+++
Makes gdb debugging a little nicer.
---
src/gallium/drivers/svga/svga_link.c | 2 +-
src/gallium/drivers/svga/svga_pipe_streamout.c | 3 ++-
src/gallium/drivers/svga/svga_swtnl_state.c| 2 +-
src/gallium/drivers/svga/svga_tgsi_vgpu10.c| 11 ++-
4 files changed, 10 inse
---
src/gallium/drivers/svga/svga_tgsi_vgpu10.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/src/gallium/drivers/svga/svga_tgsi_vgpu10.c
b/src/gallium/drivers/svga/svga_tgsi_vgpu10.c
index 070d67f..cd4cab4 100644
--- a/src/gallium/drivers/svga/svga_tgsi_vgpu1
On Mon, Jul 10, 2017 at 09:18:56AM -0700, Jason Ekstrand wrote:
> On Wed, Jun 28, 2017 at 2:14 PM, Nanley Chery wrote:
>
> > v2: Rewrite functions.
> >
> > Signed-off-by: Nanley Chery
> > ---
> > src/intel/vulkan/genX_cmd_buffer.c | 93 ++
> >
> > 1 file cha
On Wed, Jun 28, 2017 at 2:14 PM, Nanley Chery wrote:
> For readability, bring the assignment of CCS closer to the assignment of
> NONE and MCS.
>
> Signed-off-by: Nanley Chery
> ---
> src/intel/vulkan/genX_cmd_buffer.c | 62 ++
>
> 1 file changed, 30 inserti
On Mon 10 Jul 2017, Chad Versace wrote:
> On Fri 07 Jul 2017, Rob Herring wrote:
> > On Wed, Jul 5, 2017 at 5:14 PM, Chad Versace
> > wrote:
> > > On Fri 30 Jun 2017, Rob Herring wrote:
> > >> Add support for 32-bit RGBX/RGBA formats which are required for Android.
> > >>
> > >> The original patc
Nicolai Hähnle writes:
> From: Nicolai Hähnle
>
> This is a further lowering of default-block uniform loads that transforms
> load_uniform intrinsics into load_ubo intrinsics. This simplies the rest
"simplifies"
> of the backend.
I don't think I'll be able to use it, but this seems like a rea
Hi,
On Tue, 2017-06-20 at 15:19 +0100, Emil Velikov wrote:
> > +for (i = 0; i < count; i++) {
> > + modifiers[i] = (uint64_t) mod_parts[i * 2] << 32;
> > + modifiers[i] |= (uint64_t) mod_parts[i * 2 + 1] &
> > 0xff;
> > +}
> > + }
> >
Nicolai Hähnle writes:
> From: Nicolai Hähnle
>
> In that case, prog->nir must be assigned at the end.
Reviewed-by: Eric Anholt
signature.asc
Description: PGP signature
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesk
On Fri 07 Jul 2017, Rob Herring wrote:
> On Wed, Jul 5, 2017 at 5:14 PM, Chad Versace wrote:
> > On Fri 30 Jun 2017, Rob Herring wrote:
> >> Add support for 32-bit RGBX/RGBA formats which are required for Android.
> >>
> >> The original patch (commit ccdcf91104a5) was reverted (commit
> >> c0c6ca4
From: Marek Olšák
clock_crystal_freq is always non-zero now.
---
src/gallium/drivers/radeonsi/si_pipe.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_pipe.c
b/src/gallium/drivers/radeonsi/si_pipe.c
index 371d337..e2ec377 100644
--- a/
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_debug.c | 29 +++--
1 file changed, 23 insertions(+), 6 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_debug.c
b/src/gallium/drivers/radeonsi/si_debug.c
index 0d26ce5..06dea61 100644
--- a/src/gallium/driv
From: Marek Olšák
If the queue is full, util_queue_add_job will wait while bo_fence_lock is
held.
It pb_slab wants to reuse a buffer, it will lock the pb_slab mutex and
try to check BO fence busyness, but it has to wait for bo_fence_lock to get
released. Both bo_fence_lock and pb_slab mutex are
From: Marek Olšák
by setting PIPE_CONTEXT_DEBUG in the caller
---
src/gallium/drivers/radeonsi/si_pipe.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_pipe.c
b/src/gallium/drivers/radeonsi/si_pipe.c
index 8a4bc41..371d337 100644
-
From: Marek Olšák
---
src/gallium/drivers/radeonsi/si_pipe.c | 12
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_pipe.c
b/src/gallium/drivers/radeonsi/si_pipe.c
index e2ec377..4df60b6 100644
--- a/src/gallium/drivers/radeonsi/si_pipe.
From: Marek Olšák
During bring-up, this is often 0. Prevent automatic disablement of
ARB_timer_query and demotion of the OpenGL version to 3.2 by setting
a non-zero frequency. Print an error message instead.
---
src/amd/common/ac_gpu_info.c | 4
1 file changed, 4 insertions(+)
diff --git a
From: Marek Olšák
Consider the following situation:
mtx_lock(mutex);
do_something();
util_queue_add_job(...);
mtx_unlock(mutex);
If the queue is full, util_queue_add_job will wait for a free slot.
If the job which is currently being executed tries to lock the mutex,
it will be stuck fore
From: Marek Olšák
This should lead to better MSAA performance on GFX9.
---
src/amd/common/ac_surface.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/amd/common/ac_surface.c b/src/amd/common/ac_surface.c
index a4df595..5f38205 100644
--- a/src/amd/common/ac_surface.c
From: Marek Olšák
It looks like there is no way to monitor SDMA busyness on GFX9.
---
src/gallium/drivers/radeon/r600_gpu_load.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeon/r600_gpu_load.c
b/src/gallium/drivers/radeon/r600_gpu_load.c
index 3b45
From: Marek Olšák
This is overly cautious, but better safe than sorry.
---
src/gallium/drivers/radeonsi/si_state_shaders.c | 17 ++---
1 file changed, 2 insertions(+), 15 deletions(-)
diff --git a/src/gallium/drivers/radeonsi/si_state_shaders.c
b/src/gallium/drivers/radeonsi/si_sta
Nicolai Hähnle writes:
> From: Nicolai Hähnle
>
> It's a duplicate of glsl_type::count_attribute_slots.
Reviewed-by: Eric Anholt
It's a bit unfortunate to duplicate the little wrapper function
everywhere, but I know st_glsl_type_size() has been a pain for linking,
and deleting this much code
On Monday, July 10, 2017 11:59:45 AM PDT Matt Turner wrote:
> On Thu, Jul 6, 2017 at 5:22 PM, Kenneth Graunke wrote:
> > This actually takes advantage of the newly pushed UBO data, avoiding
> > pull loads.
> >
> > XXX: quote performance numbers
> > ---
> > src/intel/compiler/brw_fs.cpp | 35 +
On Mon, Jul 10, 2017 at 1:46 PM, Andres Gomez wrote:
> James, it doesn't seem like this patch has landed in master. Are you in
> need of review or is it that this has been superseded?
>
Sorry. My fault.
Reviewed-by: Jason Ekstrand
and pushed. Thanks!
> Thanks!
>
> On Mon, 2017-06-26 at 10
James, it doesn't seem like this patch has landed in master. Are you in
need of review or is it that this has been superseded?
Thanks!
On Mon, 2017-06-26 at 10:46 +0100, James Legg wrote:
> We were hitting the
> unreachable("Invalid image opcode")
> near the end of vtn_handle_image when par
Ville, has this patch fallen through the cracks ?
On Fri, 2017-06-23 at 14:58 +0300, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä
>
> Pass the correct src coordinates to CopyTexSubImage()
> when creating the temporary texture, and also take care to adjust
> flipX/Y if the original
Jason, which is the status of this patch? Has it been superseded or
discarded?
On Mon, 2017-06-26 at 09:01 -0700, Jason Ekstrand wrote:
> This fixes the Piglit ARB_texture_views rendering-formats test.
>
> Cc: "17.1"
> ---
> src/mesa/drivers/dri/i965/brw_blorp.c | 10 +-
> 1 file chang
Am Montag, 10. Juli 2017, 16:28:28 CEST schrieb Grigori Goronzy:
> The semantics are similar to glBufferData. Fixes a crash with VMWare
> Player.
>
> Signed-off-by: Grigori Goronzy
Tested-by: Marc Dietrich
> ---
> src/mesa/main/marshal.c | 17 +
> 1 file changed, 13 insertions
On Thu, Jul 6, 2017 at 4:48 PM, Matt Turner wrote:
> We already had a channel_num system value, which I'm renaming to
> subgroup_invocation to match the rest of the new system values.
>
> Note that while ballotARB(true) will return zeros in the high 32-bits on
> systems where gl_SubGroupSizeARB <=
Reviewed-by: Bruce Cherniak
> On Jul 7, 2017, at 4:25 PM, Tim Rowley wrote:
>
> Use the SWR rasterizer API through the table returned from
> SwrGetInterface rather than referencing the functions directly.
> This will allow us to move to a model of having the driver dynamically
> load the appro
On Thu, Jul 6, 2017 at 5:22 PM, Kenneth Graunke wrote:
> Hello,
>
> This series begins pushing UBOs (rather than resorting to pull loads)
> for scalar shaders on Gen7.5+, for the OpenGL driver. Future work is
> to hook it up for Vulkan (haven't started), for the vec4 shader stages
> (I have about
Performance delta on AMD Phenom II X3 720 / RX 470
The Witcher 2: +18%
---
src/mesa/drivers/dri/common/drirc | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/mesa/drivers/dri/common/drirc
b/src/mesa/drivers/dri/common/drirc
index 69b735ce70..3108451090 100644
--- a/src/mesa/drivers
---
src/intel/compiler/brw_nir_analyze_ubo_ranges.c | 28 -
1 file changed, 27 insertions(+), 1 deletion(-)
diff --git a/src/intel/compiler/brw_nir_analyze_ubo_ranges.c
b/src/intel/compiler/brw_nir_analyze_ubo_ranges.c
index 3535e67758c..b365728e77b 100644
--- a/src/intel
On Thu, Jul 6, 2017 at 5:22 PM, Kenneth Graunke wrote:
> This actually takes advantage of the newly pushed UBO data, avoiding
> pull loads.
>
> XXX: quote performance numbers
> ---
> src/intel/compiler/brw_fs.cpp | 35 ++-
> src/intel/compiler/brw_fs.h |
Reviewed-by :Charmaine Lee
From: Brian Paul
Sent: Monday, July 10, 2017 7:40 AM
To: mesa-dev@lists.freedesktop.org
Cc: Charmaine Lee; mesa-sta...@lists.freedesktop.org
Subject: [PATCH] svga: fix PIPE_CAP_MAX_TEXTURE_BUFFER_SIZE value
This query is suppos
On Thu, Jul 6, 2017 at 5:22 PM, Kenneth Graunke wrote:
> This adds a NIR pass that decides which portions of UBOS we should
> upload as push constants, rather than pull constants.
> ---
> src/intel/Makefile.sources | 1 +
> src/intel/compiler/brw_compiler.h |
On Friday, July 7, 2017 5:12:54 AM PDT Chris Wilson wrote:
> Currently, we use set_domain() to cause a stall on rendering. But the
> set-domain ioctl has the side-effect of changing the kernel's cache
> domain underneath the struct_mutex, which may perturb state if there was
> no rendering to wait
I'll leave it out, then.
Thanks for the feedback! ☺
On Mon, 2017-07-10 at 13:35 -0400, Ilia Mirkin wrote:
> I wouldn't object to it being in stable, but it's also not
> super-important. It does fix some piglits for freedreno though. (I
> don't think vc4 exposes GL 3.0, so the problematic conditio
Great!
Thanks for the feedback! ☺
On Mon, 2017-07-10 at 12:18 +0200, Olivier Lauffenburger wrote:
> Technically, a correctly written application should not rely on
> ChoosePixelFormat but should enumerate by itself through all the
> pixel formats and select the best one with its own algorithm.
>
I wouldn't object to it being in stable, but it's also not
super-important. It does fix some piglits for freedreno though. (I
don't think vc4 exposes GL 3.0, so the problematic condition can't
happen there.)
On Mon, Jul 10, 2017 at 1:30 PM, Andres Gomez wrote:
> Ilia, would we want this patch in
Ilia, would we want this patch in -stable ?
On Fri, 2017-07-07 at 20:34 -0400, Ilia Mirkin wrote:
> Fixes a bunch of gl_BackColor interpolation tests that had explicit
> interpolation specified on the fragment shader gl_Color.
>
> Signed-off-by: Ilia Mirkin
> ---
> src/compiler/nir/nir_lower_tw
OK, thanks for the feedback! ☺
On Mon, 2017-07-10 at 09:46 -0700, Anuj Phogat wrote:
> These patches are not fixing any known bug. But they are adding the
> previously missing surface size limits for the hardware. It's really hard
> to hit these limits. But, let's pick them to stable for the sake
No use in taking a 64-bit value when we know the high 32-bits are zero.
---
v2: Update for v2 of 16/20 (Connor)
src/intel/compiler/brw_compiler.c | 2 +-
src/intel/compiler/brw_fs_nir.cpp | 6 +-
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/intel/compiler/brw_compiler.c
Some hardware, like i965, doesn't support group sizes greater than 32.
In that case, we can reduce the destination size of the ballot
intrinsic, which will simplify our code generation.
---
v2: Just change the intrinsic size, and don't add a new intrinsic (Connor)
src/compiler/nir/nir.h
On Thu, Jul 6, 2017 at 7:39 PM, Connor Abbott wrote:
> I've thought about this a little bit, and I think we'd rather just
> decrease the bitsize of the intrinsic rather than add a whole new one.
> The separate intrinsic isn't really buying you anything, I don't think
> it's going to make anything
On Thu, Jul 6, 2017 at 8:04 PM, Connor Abbott wrote:
> On Thu, Jul 6, 2017 at 4:48 PM, Matt Turner wrote:
>> ... trivially (as allowed by the spec!) by reusing the existing
>> nir_opt_intrinsics code.
>> ---
>> src/compiler/nir/nir.h| 4
>> src/compiler/nir/nir_opt_intrinsic
On 10 July 2017 at 15:26, Marathe, Yogesh wrote:
> Hello Emil, My two cents since I too spent some time on this.
>
>> -Original Message-
>> From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf
>> Of Emil Velikov
>> Sent: Monday, July 10, 2017 4:41 PM
>> To: Wu, Zhongmin
Reviewed-by: Charmaine Lee
From: Brian Paul
Sent: Sunday, July 9, 2017 1:00 PM
To: mesa-dev@lists.freedesktop.org
Cc: Charmaine Lee
Subject: [PATCH] svga: fix breakage in create_backed_surface_view()
This fixes a regression in some piglit tests since com
On Wed, Jun 28, 2017 at 2:14 PM, Nanley Chery wrote:
> v2: Expound on comment for the pipe controls (Jason Ekstrand).
>
> Signed-off-by: Nanley Chery
> ---
> src/intel/vulkan/anv_blorp.c | 4 +-
> src/intel/vulkan/genX_cmd_buffer.c | 183 ++
> +++
> 2 fil
On Jul 10, 2017, at 8:24 AM, Emil Velikov
mailto:emil.l.veli...@gmail.com>> wrote:
Hi Tim,
On 7 July 2017 at 22:25, Tim Rowley
mailto:timothy.o.row...@intel.com>> wrote:
swr used to build and link the rasterizer to the driver, and to support
multiple architectures we needed to have multiple ve
These patches are not fixing any known bug. But they are adding the
previously missing surface size limits for the hardware. It's really hard
to hit these limits. But, let's pick them to stable for the sake of completion.
Thanks for marking them for mesa-stable.
On Fri, Jul 7, 2017 at 7:25 AM, An
On Wed, Jun 28, 2017 at 2:14 PM, Nanley Chery wrote:
> v2: Rewrite functions, change location of synchronization.
>
> Signed-off-by: Nanley Chery
> ---
> src/intel/vulkan/genX_cmd_buffer.c | 114 ++
> +++
> 1 file changed, 114 insertions(+)
>
> diff --git a/src/i
https://bugs.freedesktop.org/show_bug.cgi?id=101614
--- Comment #14 from Emil Velikov ---
George, I'm only build-testing those, I have no setup to actually test those.
No idea on the on the symbols part. In general scons or mingw (the w64 of
course), Brian and Jose are the people to check with.
https://bugs.freedesktop.org/show_bug.cgi?id=101614
--- Comment #13 from Emil Velikov ---
(In reply to Trevor SANDY from comment #10)
> Ok, Thanks to George's patch (0001-mingw-fixes.patch), the behaviour
> reported in this ticket is fixed.
>
> I applied the set below (excluding the conflict) to
On Wed, Jun 28, 2017 at 2:14 PM, Nanley Chery wrote:
> We'll be performing a GPU memcpy in more places to copy small amounts of
> data. Add an alternate function that thrashes less state.
>
> v2:
> - Make a new function (Jason Ekstrand).
> - Move the #define into the function.
> v3:
> - Update th
On Wed, Jun 28, 2017 at 2:14 PM, Nanley Chery wrote:
> v2: Remove ::first_subpass_layout assertion (Jason Ekstrand).
> v3: Allow some fast clears in the GENERAL layout.
>
> Signed-off-by: Nanley Chery
> ---
> src/intel/vulkan/anv_pass.c| 22 ++
> src/intel/vulkan/anv
Technically, a correctly written application should not rely on
ChoosePixelFormat but should enumerate by itself through all the pixel formats
and select the best one with its own algorithm.
However, for the (numerous) applications that use ChoosePixelFormat, color
depth is more a question of q
On Wed, Jun 28, 2017 at 2:14 PM, Nanley Chery wrote:
> v2: Rewrite functions.
>
> Signed-off-by: Nanley Chery
> ---
> src/intel/vulkan/genX_cmd_buffer.c | 93 ++
>
> 1 file changed, 84 insertions(+), 9 deletions(-)
>
> diff --git a/src/intel/vulkan/genX_cmd_
On Monday, 2017-07-10 17:02:48 +0100, Emil Velikov wrote:
> On 10 July 2017 at 16:08, Eric Engestrom wrote:
> > No implementation of unreachable() should allow code execution to
> > keep going past it.
> >
> > We can discuss whether we should have a dead loop, abort(), or do
> > something else, bu
https://bugs.freedesktop.org/show_bug.cgi?id=101614
--- Comment #12 from George Kyriazis ---
Emil,
you said that cross compiling software drivers on arch works for you.
Have you tried running the generated binaries? I've created a patch for
Trevor, but for some reason the swr driver is not loa
On 10 July 2017 at 16:08, Eric Engestrom wrote:
> No implementation of unreachable() should allow code execution to
> keep going past it.
>
> We can discuss whether we should have a dead loop, abort(), or do
> something else, but the current "meh, let's just keep going" is
> just wrong.
>
> Cc: me
https://bugs.freedesktop.org/show_bug.cgi?id=101614
George Kyriazis changed:
What|Removed |Added
Status|RESOLVED|REOPENED
Resolution|FIXED
https://bugs.freedesktop.org/show_bug.cgi?id=101614
Trevor SANDY changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
On Sat, Jul 8, 2017 at 12:18 PM, Matt Turner wrote:
> On Sat, Jul 8, 2017 at 11:05 AM, Jason Ekstrand
> wrote:
> > On July 7, 2017 1:52:54 PM Chris Wilson
> wrote:
> >
> >> Quoting Jason Ekstrand (2017-07-07 21:37:29)
> >>>
> >>> The reason we were doing this was to ensure that the kernel did t
No implementation of unreachable() should allow code execution to
keep going past it.
We can discuss whether we should have a dead loop, abort(), or do
something else, but the current "meh, let's just keep going" is
just wrong.
Cc: mesa-sta...@lists.freedesktop.org
Signed-off-by: Eric Engestrom
Signed-off-by: Eric Engestrom
---
scons/gallium.py | 15 ---
1 file changed, 12 insertions(+), 3 deletions(-)
diff --git a/scons/gallium.py b/scons/gallium.py
index 61643a6d4f..c8e47a39db 100755
--- a/scons/gallium.py
+++ b/scons/gallium.py
@@ -145,6 +145,17 @@ def check_cc(env, cc,
This query is supposed to return the max texture buffer size/width in
texels, not size in bytes. Divide by 16 (the largest format size) to
return texels.
Fixes Piglit arb_texture_buffer_object-max-size test.
Cc: mesa-sta...@lists.freedesktop.org
---
src/gallium/drivers/svga/svga_screen.c | 5 ++
The semantics are similar to glBufferData. Fixes a crash with VMWare
Player.
Signed-off-by: Grigori Goronzy
---
src/mesa/main/marshal.c | 17 +
1 file changed, 13 insertions(+), 4 deletions(-)
diff --git a/src/mesa/main/marshal.c b/src/mesa/main/marshal.c
index 8db4531..b801bdc
Hello Emil, My two cents since I too spent some time on this.
> -Original Message-
> From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf
> Of Emil Velikov
> Sent: Monday, July 10, 2017 4:41 PM
> To: Wu, Zhongmin
> Cc: Widawsky, Benjamin ; Liu, Zhiquan
> ; Eric Engestr
Hi Tim,
On 7 July 2017 at 22:25, Tim Rowley wrote:
> swr used to build and link the rasterizer to the driver, and to support
> multiple architectures we needed to have multiple versions of the
> driver/rasterizer combination, which needed to link in much of mesa.
>
> Changing to having one instan
Reviewed-by: Erik Faye-Lund
On Fri, Jul 7, 2017 at 4:11 PM, Brian Paul wrote:
> This function always returned GL_TRUE. Just make it a void function.
> Remove unreachable code following the call to vbo_save_NotifyBegin()
> in save_Begin() in dlist.c
>
> There were some stale comments that no lon
https://bugs.freedesktop.org/show_bug.cgi?id=101614
--- Comment #9 from Emil Velikov ---
Trevor, please try and get [ideally] all of the patches upstreamed. Be that in
mesa[1] or the respective project. Frédéric Devernay may be able to lend a
hand?
Any of your local patches may be changing Mesa
On Sun, Jul 9, 2017 at 10:11 PM, Zhongmin Wu wrote:
> Before we queued the buffer with a invalid fence (-1), it will
> make some benchmarks failed to test such as flatland.
>
> Now we get the out fence during the flushing buffer and then pass
> it to SurfaceFlinger in eglSwapbuffer function.
>
> C
On 8 July 2017 at 20:59, Andres Gomez wrote:
> This reflects closer what we are actually doing.
>
> Signed-off-by: Andres Gomez
> ---
> docs/releasing.html | 15 ---
> 1 file changed, 4 insertions(+), 11 deletions(-)
>
> diff --git a/docs/releasing.html b/docs/releasing.html
> index
1 - 100 of 110 matches
Mail list logo
