Signed-off-by: Nayan Deshmukh
---
src/gallium/auxiliary/vl/vl_winsys_dri3.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/vl/vl_winsys_dri3.c
b/src/gallium/auxiliary/vl/vl_winsys_dri3.c
index 191a64b..2929928 100644
--- a/src/gallium/auxiliary/vl/vl_wi
This series is,
Reviewed-by: Edward O'Callaghan
On 10/01/2016 08:53 AM, Brian Paul wrote:
> There's no need to cast to st_texture_image. Just use gl_texture_image.
> ---
> src/mesa/state_tracker/st_atom_texture.c | 10 +-
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git
Hi Axel,
On 1 Oct 2016 02:34, "Axel Davy" wrote:
>
> On 16/09/2016 14:51, Nayan Deshmukh wrote:
>>
>>
>> bool flushed;
>> + bool is_different_gpu;
>> };
>>
>
> I get a warning when I build about incompatible pointer type at the line
>
> fd = loader_get_user_preferred_fd(fd, &scrn->is_differ
---
src/compiler/glsl/blob.h | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/compiler/glsl/blob.h b/src/compiler/glsl/blob.h
index ec903ec..0765bf3 100644
--- a/src/compiler/glsl/blob.h
+++ b/src/compiler/glsl/blob.h
@@ -29,6 +29,8 @@
extern "C" {
#endif
+#include
+#include
#inclu
Am 30.09.2016 um 01:49 schrieb Thomas Søndergaard:
> My application window becomes transparent when I run my application with
> llvmpipe by setting LD_LIBRARY_PATH instead of the proprietary NVIDIA
> driver that I otherwise use. I've come to the understanding, after
> digging into the code for a fe
On Sep 30, 2016 3:56 PM, "Jason Ekstrand" wrote:
>
> Flyby comments...
>
> On Sep 30, 2016 3:34 PM, "Anuj Phogat" wrote:
> >
> > Signed-off-by: Anuj Phogat
> > ---
> > src/intel/vulkan/gen7_pipeline.c | 6 ++
> > 1 file changed, 6 insertions(+)
> >
> > diff --git a/src/intel/vulkan/gen7_pip
Patches 1, 2, 3, 5, and 7 are
Reviewed-by: Jason Ekstrand
On Sep 30, 2016 3:33 PM, "Anuj Phogat" wrote:
> Signed-off-by: Anuj Phogat
> ---
> src/intel/genxml/gen75.xml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/intel/genxml/gen75.xml b/src/intel/genxml/gen75
On Sep 30, 2016 3:34 PM, "Anuj Phogat" wrote:
>
> Signed-off-by: Anuj Phogat
> ---
> src/intel/vulkan/gen7_pipeline.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/intel/vulkan/gen7_pipeline.c
b/src/intel/vulkan/gen7_pipeline.c
> index 4bab0bc..e804002 100644
> -
On Sep 30, 2016 3:34 PM, "Anuj Phogat" wrote:
>
> Signed-off-by: Anuj Phogat
> ---
> src/intel/vulkan/gen7_pipeline.c | 4
> 1 file changed, 4 insertions(+)
>
> diff --git a/src/intel/vulkan/gen7_pipeline.c
b/src/intel/vulkan/gen7_pipeline.c
> index 9411631..4bab0bc 100644
> --- a/src/intel
Seems reasonable. I'm generally a fan of explicitly sized types anyway. R-B
On Sep 30, 2016 4:20 PM, "Timothy Arceri"
wrote:
> These structs will be written to disk as part of the shader cache
> so use uint32_t just to be safe.
> ---
> src/mesa/main/mtypes.h | 20 ++--
> 1 file
Signed-off-by: Eric Engestrom
---
src/egl/main/eglsurface.c | 43 ---
1 file changed, 40 insertions(+), 3 deletions(-)
diff --git a/src/egl/main/eglsurface.c b/src/egl/main/eglsurface.c
index 231a5f0..2762503 100644
--- a/src/egl/main/eglsurface.c
+++ b/sr
These structs will be written to disk as part of the shader cache
so use uint32_t just to be safe.
---
src/mesa/main/mtypes.h | 20 ++--
1 file changed, 10 insertions(+), 10 deletions(-)
diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
index d1b0560..efcd50e 100644
---
I remember the first tone you fixed this but when I was going the
ISL_SWIZZLE stuff, I couldn't find it in the git log so I went ahead and
pushed the change. Thanks for fixing it again.
Reviewed-by: Jason Ekstrand
On Sep 30, 2016 1:00 PM, wrote:
> From: Ville Syrjälä
>
> gcc-4 and earlier don
Flyby comments...
On Sep 30, 2016 3:34 PM, "Anuj Phogat" wrote:
>
> Signed-off-by: Anuj Phogat
> ---
> src/intel/vulkan/gen7_pipeline.c | 6 ++
> 1 file changed, 6 insertions(+)
>
> diff --git a/src/intel/vulkan/gen7_pipeline.c
b/src/intel/vulkan/gen7_pipeline.c
> index 8d133b6..f98aef5 100
There's no need to cast to st_texture_image. Just use gl_texture_image.
---
src/mesa/state_tracker/st_atom_texture.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/src/mesa/state_tracker/st_atom_texture.c
b/src/mesa/state_tracker/st_atom_texture.c
index efc8c90..1
Previously, the sampler view code was scattered across several different
files.
Note, the previous REALLOC(), FREE() for st_texture_object::sampler_views
are replaced by realloc(), free() to avoid conflicting macros in Mesa vs.
Gallium.
---
src/mesa/Makefile.sources | 2 +
And minor code reformatting.
---
src/mesa/state_tracker/st_atom_texture.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/mesa/state_tracker/st_atom_texture.c
b/src/mesa/state_tracker/st_atom_texture.c
index 113c0ed..86f1c62 100644
--- a/src/mesa/state_tracker/st_a
Before, we had code to compute the sampler view's format spread across two
different functions: in update_single_texture() and
st_get_texture_sampler_view_from_stobj(). Now it's all in one new function.
Also, use _mesa_texture_base_format() to simplify the code.
---
src/mesa/state_tracker/st_ato
Before, st_get_texture_sampler_view_from_stobj() did a lot of work to
check if the texture parameters matched the sampler view (format,
swizzle, min/max lod, first/last layer, etc). We did this every time
we validated the texture state.
Now, we use a ctx->Driver.TexParameter() callback and a coup
On Fri, Sep 30, 2016 at 3:37 PM, Brian Paul wrote:
>
> Not called from any other file. Also, add a comment.
> ---
> src/mesa/main/teximage.c | 34 +++---
> src/mesa/main/teximage.h | 8
> 2 files changed, 19 insertions(+), 23 deletions(-)
>
> diff --git a/sr
---
src/mesa/main/teximage.c | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
index 3f3d430..1d0a6a2 100644
--- a/src/mesa/main/teximage.c
+++ b/src/mesa/main/teximage.c
@@ -3571,8 +3571,15 @@ formats_differ_in_componen
---
src/mesa/main/texparam.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/main/texparam.c b/src/mesa/main/texparam.c
index bdd3fcb..a814778 100644
--- a/src/mesa/main/texparam.c
+++ b/src/mesa/main/texparam.c
@@ -1486,6 +1486,9 @@ invalid_pname:
}
+/**
+ * Handle a glGetT
Not called from any other file. Also, add a comment.
---
src/mesa/main/teximage.c | 34 +++---
src/mesa/main/teximage.h | 8
2 files changed, 19 insertions(+), 23 deletions(-)
diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
index 1d0a6a2..df
Signed-off-by: Anuj Phogat
---
src/intel/vulkan/gen8_pipeline.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/intel/vulkan/gen8_pipeline.c b/src/intel/vulkan/gen8_pipeline.c
index 1abb97c..7a9a3cf 100644
--- a/src/intel/vulkan/gen8_pipeline.c
+++ b/src/intel/vulkan/gen
Signed-off-by: Anuj Phogat
---
src/intel/vulkan/gen7_pipeline.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/intel/vulkan/gen7_pipeline.c b/src/intel/vulkan/gen7_pipeline.c
index 9411631..4bab0bc 100644
--- a/src/intel/vulkan/gen7_pipeline.c
+++ b/src/intel/vulkan/gen7_pipeline.c
@
Signed-off-by: Anuj Phogat
---
src/intel/vulkan/gen7_pipeline.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/intel/vulkan/gen7_pipeline.c b/src/intel/vulkan/gen7_pipeline.c
index 4bab0bc..e804002 100644
--- a/src/intel/vulkan/gen7_pipeline.c
+++ b/src/intel/vulkan/gen
Signed-off-by: Anuj Phogat
---
src/intel/vulkan/gen7_pipeline.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/src/intel/vulkan/gen7_pipeline.c b/src/intel/vulkan/gen7_pipeline.c
index f98aef5..9411631 100644
--- a/src/intel/vulkan/gen7_pipeline.c
+++ b/src/intel/vulkan/
Signed-off-by: Anuj Phogat
---
src/intel/vulkan/gen7_pipeline.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/intel/vulkan/gen7_pipeline.c b/src/intel/vulkan/gen7_pipeline.c
index 8d133b6..f98aef5 100644
--- a/src/intel/vulkan/gen7_pipeline.c
+++ b/src/intel/vulkan/gen7_pipeline.c
Signed-off-by: Anuj Phogat
---
src/mesa/drivers/dri/i965/gen7_vs_state.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/gen7_vs_state.c
b/src/mesa/drivers/dri/i965/gen7_vs_state.c
index 57e0419..d3bd9fd 100644
--- a/src/mesa/drivers/dri/i965/gen
Signed-off-by: Anuj Phogat
---
src/intel/genxml/gen75.xml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/genxml/gen75.xml b/src/intel/genxml/gen75.xml
index 1debc3a..8e18007 100644
--- a/src/intel/genxml/gen75.xml
+++ b/src/intel/genxml/gen75.xml
@@ -1571,7 +1571,7
Signed-off-by: Anuj Phogat
---
src/intel/vulkan/gen8_pipeline.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/vulkan/gen8_pipeline.c b/src/intel/vulkan/gen8_pipeline.c
index 030f71e..1abb97c 100644
--- a/src/intel/vulkan/gen8_pipeline.c
+++ b/src/intel/vulkan/gen8_
Signed-off-by: Anuj Phogat
---
src/intel/genxml/gen75.xml | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/intel/genxml/gen75.xml b/src/intel/genxml/gen75.xml
index 8e18007..b8f5a14 100644
--- a/src/intel/genxml/gen75.xml
+++ b/src/intel/genxml/gen75.xml
@@ -1365,7 +1
On Fri, Sep 30, 2016 at 11:23 PM, Dave Airlie wrote:
> On 1 October 2016 at 06:11, Marek Olšák wrote:
>> On Fri, Sep 30, 2016 at 4:42 PM, Ilia Mirkin wrote:
>>> So is this getting pushed, or should I push my "don't build vdpau for
>>> nouveau" patch?
>>
>> Please wait.
>>
>> Christian, your comm
On Fri, Sep 30, 2016 at 2:21 PM, Brian Paul wrote:
> Almost all of the other drawing validation code is in api_validate.c
> so put this function there as well.
> ---
> src/mesa/main/api_validate.c | 191 ++
> -
> src/mesa/main/api_validate.h | 3 +
> src
On 1 October 2016 at 06:11, Marek Olšák wrote:
> On Fri, Sep 30, 2016 at 4:42 PM, Ilia Mirkin wrote:
>> So is this getting pushed, or should I push my "don't build vdpau for
>> nouveau" patch?
>
> Please wait.
>
> Christian, your commit says it fixes a radeonsi issue, but Ilia says
> it breaks no
Fixed all Blender issues on r600g/NI/Turks XT.
Apart from that that it's already committed you have my
Tb
Thank you Nicolai!
Dieter.
Am 30.09.2016 11:52, schrieb Nicolai Hähnle:
From: Nicolai Hähnle
This is only needed for r600 which doesn't have ARB_query_buffer_object
and
therefore woul
Almost all of the other drawing validation code is in api_validate.c
so put this function there as well.
---
src/mesa/main/api_validate.c | 191 ++-
src/mesa/main/api_validate.h | 3 +
src/mesa/main/context.c | 188 +---
On 16/09/2016 14:51, Nayan Deshmukh wrote:
bool flushed;
+ bool is_different_gpu;
};
I get a warning when I build about incompatible pointer type at the line
fd = loader_get_user_preferred_fd(fd, &scrn->is_different_gpu);
When compiling on 32 bits.
I get this should be int.
On Fri, Sep 30, 2016 at 4:42 PM, Ilia Mirkin wrote:
> So is this getting pushed, or should I push my "don't build vdpau for
> nouveau" patch?
Please wait.
Christian, your commit says it fixes a radeonsi issue, but Ilia says
it breaks nouveau.
Can we do something to accommodate both drivers? Sho
From: Ville Syrjälä
gcc-4 and earlier don't allow compound literals where a constant
is required in -std=c99/gnu99 mode, so we can't use ISL_SWIZZLE()
when populating the anv_formats[] array. There are a few ways around
it: First one would be -std=c89/gnu89, but the rest of the code
depends on c9
On 09/30/2016 12:17 PM, Marek Olšák wrote:
From: Marek Olšák
This reverts commit dbfeb0ec12d6550e68de1bcd164e422e79bccf2d.
With max_value being rounded to 100, it's often wrong.
Yeah, I noticed that too.
Reviewed-by: Brian Paul
---
src/gallium/auxiliary/hud/hud_context.c | 18 ++---
On 09/30/2016 06:00 AM, Emil Velikov wrote:
> On 30 September 2016 at 07:32, Tapani Pälli wrote:
>> This fixes a crash in egl-create-msaa-pbuffer-surface Piglit test
>> and same crash in many dEQP EGL tests.
>>
>> I also found that some Qt example did a workaround because of this
>> crash: https:/
This fixes a problem where GL headers would only get installed if
glx was enabled. So if osmesa was enabled but not glx, then the
GL headers required by osmesa would be missing from the install.
Signed-off-by: Chuck Atkins
---
configure.ac| 2 ++
src/Makefile.am | 30 ++
From: Marek Olšák
This reverts commit dbfeb0ec12d6550e68de1bcd164e422e79bccf2d.
With max_value being rounded to 100, it's often wrong.
---
src/gallium/auxiliary/hud/hud_context.c | 18 ++
1 file changed, 6 insertions(+), 12 deletions(-)
diff --git a/src/gallium/auxiliary/hud/hu
Reviewed-by: George Kyriazis
mailto:george.kyria...@intel.com>>
From: Bruce Cherniak mailto:bruce.chern...@intel.com>>
Subject: [PATCH] swr: Removed stalling SwrWaitForIdle from queries.
Date: September 27, 2016 at 1:27:08 PM CDT
To: mailto:mesa-dev@lists.freedesktop.org>>
Cc: Bruce Cherniak ma
Hello Emil,
>> +AC_ARG_ENABLE([gallium-extra-hud],
>> +[AS_HELP_STRING([--enable-gallium-extra-hud],
>> +[enable HUD block/NIC I/O HUD stats support
>> @<:@default=disabled@:>@])],
>> +[enable_gallium_extra_hud="$enableval"],
>> +[enable_gallium_extra_hud=no])
> Looking at som
On 09/30/2016 03:01 AM, Emil Velikov wrote:
> From: Emil Velikov
>
> The function deals with fb (style) configs, thus using "visual"
> in the name is misleading. Which in itself had led to the use of
> fbconfig_style_tags argument.
>
> Rename the function to reflect what it does and drop the unn
On Sep 30, 2016 7:57 AM, "Ian Romanick" wrote:
>
> On 09/30/2016 06:23 AM, Brian Paul wrote:
> > On 09/30/2016 04:59 AM, Emil Velikov wrote:
> >> On 30 September 2016 at 03:31, Timothy Arceri
> >> wrote:
> >>> On Thu, 2016-09-29 at 19:17 -0700, Jason Ekstrand wrote:
> >>>
> >>> On Sep 29, 2016 5:
Looks OK to me.
Reviewed-by: Brian Paul
Will push soon.
-Brian
On 09/30/2016 05:58 AM, Steven Toth wrote:
Detect all of the CPUs in the system. Expose metrics
for min, max and current frequency in Hz.
Signed-off-by: Steven Toth
---
src/gallium/auxiliary/Makefile.sources | 1 +
src/ga
On 30 September 2016 at 15:57, Ian Romanick wrote:
> On 09/30/2016 06:23 AM, Brian Paul wrote:
>> On 09/30/2016 04:59 AM, Emil Velikov wrote:
>>> On 30 September 2016 at 03:31, Timothy Arceri
>>> wrote:
On Thu, 2016-09-29 at 19:17 -0700, Jason Ekstrand wrote:
On Sep 29, 2016 5:14 P
On Fri, Sep 30, 2016 at 5:14 PM, Karol Herbst wrote:
> 2016-09-30 16:57 GMT+02:00 Ian Romanick :
>> On 09/30/2016 06:23 AM, Brian Paul wrote:
>>> On 09/30/2016 04:59 AM, Emil Velikov wrote:
On 30 September 2016 at 03:31, Timothy Arceri
wrote:
> On Thu, 2016-09-29 at 19:17 -0700, Jas
On 09/30/2016 07:47 AM, Emil Velikov wrote:
On 30 September 2016 at 14:23, Brian Paul wrote:
---
docs/intro.html | 25 +
1 file changed, 25 insertions(+)
diff --git a/docs/intro.html b/docs/intro.html
index 4b36562..6c91e3f 100644
--- a/docs/intro.html
+++ b/docs/int
2016-09-30 16:57 GMT+02:00 Ian Romanick :
> On 09/30/2016 06:23 AM, Brian Paul wrote:
>> On 09/30/2016 04:59 AM, Emil Velikov wrote:
>>> On 30 September 2016 at 03:31, Timothy Arceri
>>> wrote:
On Thu, 2016-09-29 at 19:17 -0700, Jason Ekstrand wrote:
On Sep 29, 2016 5:14 PM, "Timoth
From: Tapani Pälli
Changes make copy_propagation_elements pass faster, reducing link
time spent in test case of bug 94477. Does not fix the actual issue
but brings down the total time. No regressions seen in CI.
v2 (idr): Formatting / whitespace fixes. Embed the acp_ref in the
acp_entry.
v3 (i
Reviewed-by: Ian Romanick
On 09/29/2016 11:37 PM, Timothy Arceri wrote:
> The warning was also the wrong location, it should have been
> in the else.
> ---
> src/mesa/drivers/dri/i915/intel_fbo.h | 4 +---
> src/mesa/drivers/dri/i965/intel_fbo.h | 4 +---
> 2 files changed, 2 insertions(+), 6 de
From: Tapani Pälli
Changes make copy_propagation_elements pass faster, reducing link
time spent in test case of bug 94477. Does not fix the actual issue
but brings down the total time. No regressions seen in CI.
v2 (idr): Formatting / whitespace fixes. Embed the acp_ref in the
acp_entry.
Signe
On 09/30/2016 06:23 AM, Brian Paul wrote:
> On 09/30/2016 04:59 AM, Emil Velikov wrote:
>> On 30 September 2016 at 03:31, Timothy Arceri
>> wrote:
>>> On Thu, 2016-09-29 at 19:17 -0700, Jason Ekstrand wrote:
>>>
>>> On Sep 29, 2016 5:14 PM, "Timothy Arceri"
>>> wrote:
On Thu, 2016-09-29
So is this getting pushed, or should I push my "don't build vdpau for
nouveau" patch?
On Wed, Sep 14, 2016 at 11:58 PM, Dave Airlie wrote:
> From: Dave Airlie
>
> This reverts commit d180de35320eafa3df3d76f0e82b332656530126.
>
> This is a radeon specific hack that causes problems on nouveau
> wh
Hi Steven,
On 28 September 2016 at 19:58, Steven Toth wrote:
> V8: Feedback based on peer review
> convert if block into a switch
> Constify some func args
>
> V7: Increase precision when measuring lmsensors volts
> Flatten patch series.
>
> V6: Feedback based on peer review
> Sim
On Fri, Sep 30, 2016 at 3:08 PM, Bas Nieuwenhuizen
wrote:
> On Fri, Sep 30, 2016 at 2:13 PM, Marek Olšák wrote:
>> intptr_t reads and writes aren't atomic. p_atomic_set and
>> p_atomic_read functions don't do anything for atomicity. See:
>>
>> #define p_atomic_set(_v, _i) (*(_v) = (_i))
>> #defin
On 30 September 2016 at 14:23, Brian Paul wrote:
> ---
> docs/intro.html | 25 +
> 1 file changed, 25 insertions(+)
>
> diff --git a/docs/intro.html b/docs/intro.html
> index 4b36562..6c91e3f 100644
> --- a/docs/intro.html
> +++ b/docs/intro.html
> @@ -173,6 +173,27 @@ of
I'm not so familiar with the LLVM basic block API, but this series
makes sense to me. For the series:
Reviewed-by: Marek Olšák
Marek
On Thu, Sep 29, 2016 at 3:15 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> This is enabled automatically if shader printing is enabled, or separately
> b
On 30 September 2016 at 06:47, Matt Whitlock
wrote:
> Without this fix, duplicated file descriptors leak into child processes.
> See commit aaac913e901229d11a1894f6aaf646de6b1a542c for one instance
> where the same fix was employed.
>
> Cc:
> Signed-off-by: Matt Whitlock
> ---
> src/egl/drivers
On Fri, Sep 30, 2016 at 3:00 PM, Nicolai Hähnle wrote:
> On 30.09.2016 14:58, Marek Olšák wrote:
>>
>> For patches 2-6:
>>
>> Reviewed-by: Marek Olšák
>
>
> Thanks.
>
>> For patch 1, you'll have to decide whether atomicity is necessary, but
>> at least p_atomic_read and p_atomic_set shouldn't be
On Fri, Sep 30, 2016 at 3:23 PM, Brian Paul wrote:
> On 09/30/2016 04:59 AM, Emil Velikov wrote:
>>
>> On 30 September 2016 at 03:31, Timothy Arceri
>> wrote:
>>>
>>> On Thu, 2016-09-29 at 19:17 -0700, Jason Ekstrand wrote:
>>>
>>> On Sep 29, 2016 5:14 PM, "Timothy Arceri"
>>> wrote:
>
On Fri, Sep 30, 2016 at 3:31 AM, Dave Airlie wrote:
> On 30 September 2016 at 01:07, Jason Ekstrand wrote:
>> On Sep 29, 2016 7:56 AM, "Emil Velikov" wrote:
>>>
>>> On 28 September 2016 at 19:53, Marek Olšák wrote:
>>> > Hi,
>>> >
>>> > It's been almost 4 months since the 12.0 branch was create
On 09/30/2016 04:59 AM, Emil Velikov wrote:
On 30 September 2016 at 03:31, Timothy Arceri
wrote:
On Thu, 2016-09-29 at 19:17 -0700, Jason Ekstrand wrote:
On Sep 29, 2016 5:14 PM, "Timothy Arceri"
wrote:
On Thu, 2016-09-29 at 15:56 +0100, Emil Velikov wrote:
On 28 September 2016 at 19:53, M
---
docs/intro.html | 25 +
1 file changed, 25 insertions(+)
diff --git a/docs/intro.html b/docs/intro.html
index 4b36562..6c91e3f 100644
--- a/docs/intro.html
+++ b/docs/intro.html
@@ -173,6 +173,27 @@ of the OpenGL specification is implemented.
+Version 12.x featur
On Fri, Sep 30, 2016 at 2:13 PM, Marek Olšák wrote:
> intptr_t reads and writes aren't atomic. p_atomic_set and
> p_atomic_read functions don't do anything for atomicity. See:
>
> #define p_atomic_set(_v, _i) (*(_v) = (_i))
> #define p_atomic_read(_v) (*(_v))
That implementation seems bogus to me
https://bugs.freedesktop.org/show_bug.cgi?id=97879
Vedran Rodic changed:
What|Removed |Added
CC||vro...@gmail.com
--
You are receiving th
On 30.09.2016 14:58, Marek Olšák wrote:
For patches 2-6:
Reviewed-by: Marek Olšák
Thanks.
For patch 1, you'll have to decide whether atomicity is necessary, but
at least p_atomic_read and p_atomic_set shouldn't be used on intptr_t.
I do believe atomicity is necessary. Maybe the right solu
On 30 September 2016 at 07:32, Tapani Pälli wrote:
> This fixes a crash in egl-create-msaa-pbuffer-surface Piglit test
> and same crash in many dEQP EGL tests.
>
> I also found that some Qt example did a workaround because of this
> crash: https://bugreports.qt.io/browse/QTBUG-47509
>
> v2: Ian po
For patches 2-6:
Reviewed-by: Marek Olšák
For patch 1, you'll have to decide whether atomicity is necessary, but
at least p_atomic_read and p_atomic_set shouldn't be used on intptr_t.
Marek
On Tue, Sep 27, 2016 at 8:21 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> ---
> src/gallium
From: Emil Velikov
Currently we only saved the id to memcpy the whole _EGLConfig to write
back the exact same id value.
Remove the unneeded and confusing/misleading code.
Signed-off-by: Emil Velikov
---
src/egl/drivers/dri2/egl_dri2.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/src/
intptr_t reads and writes aren't atomic. p_atomic_set and
p_atomic_read functions don't do anything for atomicity. See:
#define p_atomic_set(_v, _i) (*(_v) = (_i))
#define p_atomic_read(_v) (*(_v))
Marek
On Wed, Sep 28, 2016 at 9:47 AM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> This is
>
> Reviewed-by: Brian Paul
>
> and pushed to master. Thanks.
Thank you Brian.
--
Steven Toth - Kernel Labs
http://www.kernellabs.com
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
Detect all of the CPUs in the system. Expose metrics
for min, max and current frequency in Hz.
Signed-off-by: Steven Toth
---
src/gallium/auxiliary/Makefile.sources | 1 +
src/gallium/auxiliary/hud/hud_context.c | 13 ++
src/gallium/auxiliary/hud/hud_cpufreq.c | 266 +
On 09/30/2016 03:18 AM, Ian Romanick wrote:
On 09/29/2016 12:17 AM, Tapani Pälli wrote:
On 09/28/2016 06:14 PM, Ian Romanick wrote:
On 09/16/2016 06:21 PM, Tapani Pälli wrote:
Changes make copy_propagation_elements pass faster, reducing link
time spent in test case of bug 94477. Does not fi
On Wed, 2016-09-28 at 11:42 -0700, Kenneth Graunke wrote:
> On Wednesday, September 28, 2016 9:00:50 AM PDT Eric Anholt wrote:
> >
> > Timothy Arceri writes:
> >
> > >
> > > On Sun, 2016-09-25 at 19:51 -0700, Kenneth Graunke wrote:
> > > >
> > > > On Saturday, September 24, 2016 3:26:09 PM PDT
On 30 September 2016 at 03:31, Timothy Arceri
wrote:
> On Thu, 2016-09-29 at 19:17 -0700, Jason Ekstrand wrote:
>
> On Sep 29, 2016 5:14 PM, "Timothy Arceri"
> wrote:
>>
>> On Thu, 2016-09-29 at 15:56 +0100, Emil Velikov wrote:
>> > On 28 September 2016 at 19:53, Marek Olšák wrote:
>> > >
>> > >
On 30 September 2016 at 02:31, Dave Airlie wrote:
> On 30 September 2016 at 01:07, Jason Ekstrand wrote:
>> On Sep 29, 2016 7:56 AM, "Emil Velikov" wrote:
>>>
>>> On 28 September 2016 at 19:53, Marek Olšák wrote:
>>> > Hi,
>>> >
>>> > It's been almost 4 months since the 12.0 branch was created,
Well, it depends on both the size and the type. Sub-allocation can be
used up to 16KB with the current setting, but the override is clearly
the right solution here :)
Cheers,
Nicolai
On 30.09.2016 12:24, Christian König wrote:
Oh, so you use the allocation type instead of the size to determin
Oh, so you use the allocation type instead of the size to determine if a
buffer can be sub-allocated or not?
I wasn't expecting that, in this case even the DPB would have ended up
in the sub-allocator and that would clearly blow up immediately.
Good that we stumbled over this.
Regards,
Chris
For the series:
Reviewed-by: Marek Olšák
Marek
On Fri, Sep 30, 2016 at 12:01 PM, Emil Velikov wrote:
> From: Emil Velikov
>
> The function deals with fb (style) configs, thus using "visual"
> in the name is misleading. Which in itself had led to the use of
> fbconfig_style_tags argument.
>
>
Patches #1-#3 of this series are Reviewed-by: Christian König
.
Regards,
Christian.
Am 30.09.2016 um 11:52 schrieb Nicolai Hähnle:
From: Nicolai Hähnle
Cc: Christian König
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97976
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=979
Reviewed-by: Marek Olšák
Marek
On Fri, Sep 30, 2016 at 12:04 PM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> Cc: Michel Dänzer
> ---
> src/gallium/drivers/radeon/r600_perfcounter.c | 9 +
> 1 file changed, 9 insertions(+)
>
> diff --git a/src/gallium/drivers/radeon/r600_perfcoun
On 30.09.2016 03:31, Dave Airlie wrote:
On 30 September 2016 at 01:07, Jason Ekstrand wrote:
On Sep 29, 2016 7:56 AM, "Emil Velikov" wrote:
On 28 September 2016 at 19:53, Marek Olšák wrote:
Hi,
It's been almost 4 months since the 12.0 branch was created, and soon
it will have been 3 month
From: Nicolai Hähnle
Cc: Michel Dänzer
---
src/gallium/drivers/radeon/r600_perfcounter.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/src/gallium/drivers/radeon/r600_perfcounter.c
b/src/gallium/drivers/radeon/r600_perfcounter.c
index f8dfda0..0c55fc2 100644
--- a/src/gallium/dr
From: Emil Velikov
The function deals with fb (style) configs, thus using "visual"
in the name is misleading. Which in itself had led to the use of
fbconfig_style_tags argument.
Rename the function to reflect what it does and drop the unneeded
argument.
Signed-off-by: Emil Velikov
---
src/glx
From: Emil Velikov
Return GL_FALSE if we fail to find any fb/visual configs, otherwise we
end up with all sorts of chaos further down the GLX stack.
Signed-off-by: Emil Velikov
---
src/glx/glxext.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/glx/glxext.c b/sr
From: Emil Velikov
The comment/spec says - only for pbuffer drawables, while the code
clears the window/pixmap bit. Practise what you preach and apply the
trivial tweak. In practise this should not cause functional change.
Signed-off-by: Emil Velikov
---
src/glx/glxext.c | 2 +-
1 file changed
Reviewed-by: Marek Olšák
Marek
On Fri, Sep 30, 2016 at 11:52 AM, Nicolai Hähnle wrote:
> From: Nicolai Hähnle
>
> This is only needed for r600 which doesn't have ARB_query_buffer_object and
> therefore wouldn't really need the fences, but let's be optimistic about
> filling in this feature gap
I should add that textures are already never sub-allocated, so with this
patch all the buffers used with UVD and VCE should be covered.
Nicolai
On 30.09.2016 11:52, Nicolai Hähnle wrote:
From: Nicolai Hähnle
Cc: Christian König
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97976
Bu
From: Nicolai Hähnle
This is only needed for r600 which doesn't have ARB_query_buffer_object and
therefore wouldn't really need the fences, but let's be optimistic about
filling in this feature gap eventually.
Cc: Dieter Nützel
---
src/gallium/drivers/radeon/r600_pipe_common.c | 7 ++-
sr
From: Nicolai Hähnle
We don't plan to use sub-allocated buffers with UVD, but just in case one
slips through, this increases the chances of things working out anyway.
---
src/gallium/drivers/radeon/radeon_uvd.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/drivers/radeon/radeon
From: Nicolai Hähnle
We don't plan to use sub-allocated buffers with VCE, but just in case one
slips through, this increases the chances of things working out anyway.
---
src/gallium/drivers/radeon/radeon_vce.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/drivers/radeon/radeon
From: Nicolai Hähnle
Cc: Christian König
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97976
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97969
---
src/gallium/drivers/radeon/r600_buffer_common.c | 3 +++
src/gallium/drivers/radeon/radeon_video.c | 8 +++-
2 files c
From: Nicolai Hähnle
Reviewed-by: Marek Olšák
---
src/gallium/drivers/radeon/radeon_winsys.h | 3 +++
1 file changed, 3 insertions(+)
diff --git a/src/gallium/drivers/radeon/radeon_winsys.h
b/src/gallium/drivers/radeon/radeon_winsys.h
index dcbebe0..d248004 100644
--- a/src/gallium/drivers/ra
On Fri, Sep 30, 2016 at 4:17 AM, Jason Ekstrand wrote:
> On Sep 29, 2016 5:14 PM, "Timothy Arceri"
> wrote:
>>
>> On Thu, 2016-09-29 at 15:56 +0100, Emil Velikov wrote:
>> > On 28 September 2016 at 19:53, Marek Olšák wrote:
>> > >
>> > > Hi,
>> > >
>> > > It's been almost 4 months since the 12.0
Reviewed-by: Marek Olšák
Marek
On Fri, Sep 30, 2016 at 8:32 AM, Tapani Pälli wrote:
> This fixes a crash in egl-create-msaa-pbuffer-surface Piglit test
> and same crash in many dEQP EGL tests.
>
> I also found that some Qt example did a workaround because of this
> crash: https://bugreports.qt.
1 - 100 of 102 matches
Mail list logo