On 14 September 2016 at 15:04, Emil Velikov wrote:
> Hi Lukasz,
>
> On 5 September 2016 at 17:48, Lukasz Spintzyk
> wrote:
>> This closes filedescriptor owned by kms_dri_sw_winsys struct. It fixes issue
>> where removal of udl or evdi module used by DisplayLink devices was
>> impossible
>> due
Build mesa 2244 failed
Commit e60928f4c4 by Timothy Arceri on 9/7/2016 4:28 AM:
gallium: remove unused PIPE_CC_GCC_VERSION\n\nAcked-by: Edward O'Callaghan
Configure your notification preferences
___
mesa-d
It makes more sense to have this here where we store the other values
from xfb qualifiers. The struct it was previously part of is now only
used to store values to come from the api.
---
src/compiler/glsl/link_varyings.cpp | 4 ++--
src/compiler/glsl/linker.cpp| 8
src/mesa/main/m
https://bugs.freedesktop.org/show_bug.cgi?id=97879
--- Comment #12 from Michel Dänzer ---
(In reply to Andreas Hartmetz from comment #4)
> Assuming the problem was shader compilation, what could be done about it,
> though? Optimizing shader compilation by a factor of 10 seems unrealistic,
Not ne
On Wed, Sep 21, 2016 at 2:15 PM, Anuj Phogat wrote:
> Signed-off-by: Anuj Phogat
>
> ---
> This patch requires below series:
> https://patchwork.freedesktop.org/series/12594/
FYI, this is now pushed. Given that the ext relies on
OES_geometry_shader, the placement of the enable makes sense. Howev
ping
On Tue, Sep 13, 2016 at 8:54 PM, Ilia Mirkin wrote:
> Signed-off-by: Ilia Mirkin
> ---
> src/mesa/state_tracker/st_extensions.c | 20
> 1 file changed, 20 insertions(+)
>
> diff --git a/src/mesa/state_tracker/st_extensions.c
> b/src/mesa/state_tracker/st_extensions.c
ping
On Tue, Sep 13, 2016 at 8:54 PM, Ilia Mirkin wrote:
> For now that's never since advanced blend hasn't been piped through.
>
> Signed-off-by: Ilia Mirkin
> ---
> src/mesa/state_tracker/st_extensions.c | 18 ++
> 1 file changed, 18 insertions(+)
>
> diff --git a/src/mesa/sta
This is needed to be consistent with other drivers.
I used both the context and the buffer as a condition in order to ensure it's an
intentional unbind, and not a programming error, but let me know if you think I
should change it to check only the context.
Also, this is my very first mesa patch,
Signed-off-by: Nanley Chery
---
src/intel/vulkan/anv_formats.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/src/intel/vulkan/anv_formats.c b/src/intel/vulkan/anv_formats.c
index 701add5..accf45c 100644
--- a/src/intel/vulkan/anv_formats.c
+++ b/src/intel/vulkan/anv_formats.c
@
Signed-off-by: Nanley Chery
---
src/intel/vulkan/anv_formats.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/intel/vulkan/anv_formats.c b/src/intel/vulkan/anv_formats.c
index ff59f47..701add5 100644
--- a/src/intel/vulkan/anv_formats.c
+++ b/src/intel/vulkan/anv_formats.c
@@ -408,6
Signed-off-by: Nanley Chery
---
src/intel/isl/isl_gen7.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/intel/isl/isl_gen7.c b/src/intel/isl/isl_gen7.c
index 4f1cc9d..36a3401 100644
--- a/src/intel/isl/isl_gen7.c
+++ b/src/intel/isl/isl_gen7.c
@@ -231,6 +231,12 @@ gen6_filter_tilin
This series prevents invalid compressed textures from being created in
the Vulkan driver. There aren't any Vulkan CTS tests for 1D compressed
textures, so no change has been observed in that test suite.
Nanley Chery (3):
isl: Only allow Y-tiling for ASTC textures
anv/formats: Disallow linear
On Thu, Sep 22, 2016 at 02:12:08PM -0700, Jason Ekstrand wrote:
> On Sep 22, 2016 11:11 PM, "Nanley Chery" wrote:
> >
> > On Wed, Sep 14, 2016 at 01:28:23PM -0700, Jason Ekstrand wrote:
> > > Compressed 1-D textures are a well-defined thing in both GL and Vulkan.
> >
> > Could you provide a refere
Build mesa 2241 failed
Commit e66a2b879b by Dylan Baker on 9/22/2016 6:30 PM:
mapi: export all GLES 3.2 functions in libGLESv2.so\n\nSee commit 5921f372c89a68fac6ddefc009442721d9df4db2 for the rational of\nthis commit.\n\nSigned-off-by: Dylan Baker \nReviewed-b
On Friday, September 23, 2016 1:17:14 AM PDT Lionel Landwerlin wrote:
> Make gen_device_info a mutable structure so we can update the fields that
> can be refined by querying the kernel (like subslices and EU numbers).
>
> This patch does not make any functional change, it just make
> gen_get_devi
Hi,
Kenneth metionned that we might be interested in making gen_device_info
mutable so that its values could be updated (for example after querying the
kernel about the number of EUs and subslices). Here is a first attempt.
I think a follow up series could add the subslices number to gen_device_i
Signed-off-by: Lionel Landwerlin
Cc: Kenneth Graunke
---
src/intel/vulkan/anv_allocator.c | 15 ---
src/intel/vulkan/anv_device.c| 22 +++---
src/intel/vulkan/anv_private.h | 11 ---
src/intel/vulkan/gen7_pipeline.c | 11 ++-
src/intel/vulkan/gen
Make gen_device_info a mutable structure so we can update the fields that
can be refined by querying the kernel (like subslices and EU numbers).
This patch does not make any functional change, it just make
gen_get_device_info() fill a structure rather than returning a const
pointer.
Cc: Kenneth G
Now that we have gen_device_info mutable, we can update its values and drop
all copies we had in brw_context.
Signed-off-by: Lionel Landwerlin
Cc: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_compute.c | 3 ++-
src/mesa/drivers/dri/i965/brw_context.c | 22 +-
src/m
Quoting Ilia Mirkin (2016-09-22 14:37:05)
> On Thu, Sep 22, 2016 at 5:16 PM, Dylan Baker wrote:
> > See commit 5921f372c89a68fac6ddefc009442721d9df4db2 for the rational of
> > this commit.
> >
> > Signed-off-by: Dylan Baker
> > ---
> >
> > Changes since v2:
> >
> > Re-sort the whole list, and
On Thu, Sep 22, 2016 at 5:16 PM, Dylan Baker wrote:
> See commit 5921f372c89a68fac6ddefc009442721d9df4db2 for the rational of
> this commit.
>
> Signed-off-by: Dylan Baker
> ---
>
> Changes since v2:
>
> Re-sort the whole list, and remove duplicates, which there were several.
> I've also
This file currently uses a mixture of 3 and 4 space indent. I have
changed it all to 4 space indent, matching the settings in
$ROOT/.editorconfig.
Signed-off-by: Dylan Baker
---
src/mapi/glapi/gen/static_data.py | 2570 ++---
1 file changed, 1285 insertions(+), 12
See commit 5921f372c89a68fac6ddefc009442721d9df4db2 for the rational of
this commit.
Signed-off-by: Dylan Baker
---
Changes since v2:
Re-sort the whole list, and remove duplicates, which there were several.
I've also checked that there are no duplicates by importing the list in
pyth
Signed-off-by: Dylan Baker
---
src/mapi/glapi/gen/static_data.py | 164 +++---
1 file changed, 82 insertions(+), 82 deletions(-)
diff --git a/src/mapi/glapi/gen/static_data.py
b/src/mapi/glapi/gen/static_data.py
index 396f941..bb11c1d 100644
--- a/src/mapi/glapi/
On Sep 22, 2016 11:11 PM, "Nanley Chery" wrote:
>
> On Wed, Sep 14, 2016 at 01:28:23PM -0700, Jason Ekstrand wrote:
> > Compressed 1-D textures are a well-defined thing in both GL and Vulkan.
>
> Could you provide a reference? I could not find any documentation on 1D
> compressed textures. Instead
Quoting Ilia Mirkin (2016-09-22 13:21:04)
> On Thu, Sep 22, 2016 at 4:17 PM, Dylan Baker wrote:
> > Quoting Ilia Mirkin (2016-09-22 12:59:49)
> >> On Thu, Sep 22, 2016 at 3:56 PM, Dylan Baker wrote:
> >> > @@ -1305,6 +1318,7 @@ functions = [
> >> > "SamplerParameterfv",
> >> > "SamplerP
On Thu, Sep 22, 2016 at 4:17 PM, Dylan Baker wrote:
> Quoting Ilia Mirkin (2016-09-22 12:59:49)
>> On Thu, Sep 22, 2016 at 3:56 PM, Dylan Baker wrote:
>> > @@ -1305,6 +1318,7 @@ functions = [
>> > "SamplerParameterfv",
>> > "SamplerParameteri",
>> > "SamplerParameteriv",
>> > +
Build mesa 2239 completed
Commit 36f0f03182 by Eric Anholt on 9/7/2016 2:45 AM:
nir: Allow opt_peephole_sel to be more aggressive in flattening IFs.\n\nVC4 was running into a major performance regression from enabling control\nflow in the glmark2 conditionals t
Quoting Ilia Mirkin (2016-09-22 12:59:49)
> On Thu, Sep 22, 2016 at 3:56 PM, Dylan Baker wrote:
> > @@ -1305,6 +1318,7 @@ functions = [
> > "SamplerParameterfv",
> > "SamplerParameteri",
> > "SamplerParameteriv",
> > +"SamplerParameterIiv",
> > "Scaled",
> > "Scalef",
On Wed, Sep 14, 2016 at 01:28:23PM -0700, Jason Ekstrand wrote:
> Compressed 1-D textures are a well-defined thing in both GL and Vulkan.
Could you provide a reference? I could not find any documentation on 1D
compressed textures. Instead I found that OpenGL disallows them and
Vulkan does not expl
On Thu, Sep 22, 2016 at 3:56 PM, Dylan Baker wrote:
> @@ -1305,6 +1318,7 @@ functions = [
> "SamplerParameterfv",
> "SamplerParameteri",
> "SamplerParameteriv",
> +"SamplerParameterIiv",
> "Scaled",
> "Scalef",
> "Scalex",
This is the case for a couple of these..
See commit 5921f372c89a68fac6ddefc009442721d9df4db2 for the rational of
this commit.
Signed-off-by: Dylan Baker
---
v2: - fix TexBufferRange typo (Ilia)
- Remove duplicate TexParameterIuiv
src/mapi/glapi/gen/static_data.py | 15 +++
1 file changed, 15 insertions(+)
diff --git
Signed-off-by: Dylan Baker
---
src/mapi/glapi/gen/static_data.py | 164 +++---
1 file changed, 82 insertions(+), 82 deletions(-)
diff --git a/src/mapi/glapi/gen/static_data.py
b/src/mapi/glapi/gen/static_data.py
index 396f941..bb11c1d 100644
--- a/src/mapi/glapi/
This file currently uses a mixture of 3 and 4 space indent. I have
changed it all to 4 space indent, matching the settings in
$ROOT/.editorconfig.
Signed-off-by: Dylan Baker
---
src/mapi/glapi/gen/static_data.py | 2570 ++---
1 file changed, 1285 insertions(+), 12
Quoting Ilia Mirkin (2016-09-22 11:57:14)
> 2016-09-22 14:53 GMT-04:00 Dylan Baker :
> > See commit 5921f372c89a68fac6ddefc009442721d9df4db2 for the rational of
> > this commit.
> >
> > Signed-off-by: Dylan Baker
> > ---
> > src/mapi/glapi/gen/static_data.py | 16
> > 1 file chan
2016-09-22 14:53 GMT-04:00 Dylan Baker :
> See commit 5921f372c89a68fac6ddefc009442721d9df4db2 for the rational of
> this commit.
>
> Signed-off-by: Dylan Baker
> ---
> src/mapi/glapi/gen/static_data.py | 16
> 1 file changed, 16 insertions(+)
>
> diff --git a/src/mapi/glapi/gen/
This file currently uses a mixture of 3 and 4 space indent. I have
changed it all to 4 space indent, matching the settings in
$ROOT/.editorconfig.
Signed-off-by: Dylan Baker
---
src/mapi/glapi/gen/static_data.py | 2570 ++---
1 file changed, 1285 insertions(+), 12
Signed-off-by: Dylan Baker
---
src/mapi/glapi/gen/static_data.py | 164 +++---
1 file changed, 82 insertions(+), 82 deletions(-)
diff --git a/src/mapi/glapi/gen/static_data.py
b/src/mapi/glapi/gen/static_data.py
index 396f941..bb11c1d 100644
--- a/src/mapi/glapi/
See commit 5921f372c89a68fac6ddefc009442721d9df4db2 for the rational of
this commit.
Signed-off-by: Dylan Baker
---
src/mapi/glapi/gen/static_data.py | 16
1 file changed, 16 insertions(+)
diff --git a/src/mapi/glapi/gen/static_data.py
b/src/mapi/glapi/gen/static_data.py
index
This adds the GLES 3.2 symbols to libGLESv2.so. The first two patches (which
have all of the changes in them), do some basic cleanup. The first makes tabs
consistent (this is the giant patch), and the second sorts the list
alphabetically.
Dylan Baker (3):
mapi: retab static_data.py to be consist
Heh... Rb
On Sep 22, 2016 4:46 PM, "Kenneth Graunke" wrote:
> There's an assert right above this.
>
> Signed-off-by: Kenneth Graunke
> ---
> src/mesa/drivers/dri/i965/brw_fs.cpp | 6 +-
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
>
Rb
On Sep 22, 2016 5:28 PM, "Nanley Chery" wrote:
> Change SUFFACE to SURFACE.
>
> Signed-off-by: Nanley Chery
> ---
> src/intel/isl/isl_surface_state.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/intel/isl/isl_surface_state.c
> b/src/intel/isl/isl_surface_stat
On Wed, Sep 21, 2016 at 10:31:05PM -0700, Kenneth Graunke wrote:
> Needed for user clip plane enables. Brkoen since this code was
^^
Broken
-Nanley
> introduced.
>
> Cc: mesa-sta...@lists.freedesktop.org
> Signed-of
https://bugs.freedesktop.org/show_bug.cgi?id=97879
--- Comment #11 from Gregor Münch ---
Experiencing same problem with Radeon HD 7970 Mesa Git radeonsi, LLVM Git,
Linux 4.8 rc6.
The Hangs are always there when:
-start of the match 5-7sec
-multiple times within the 1st minute of the game, up to
Am 22.09.2016 um 17:40 schrieb Emil Velikov:
> Hi Nicholas,
>
> On 26 August 2016 at 00:31, Nicholas Bishop wrote:
>> From: Nicholas Bishop
>>
>> On Intel Pineview M hardware, the i915 gallium driver doesn't output
>> the correct gl_FragCoord. It seems to always have an X coord of 0.0
>> and a Y
Reviewed-by: Bruce Cherniak
> On Sep 20, 2016, at 2:24 PM, Tim Rowley wrote:
>
> Replace old string comparison with a mapping table.
> ---
> src/gallium/drivers/swr/swr_screen.cpp | 344 -
> 1 file changed, 293 insertions(+), 51 deletions(-)
>
> diff --git a/src
On Thu, Sep 22, 2016 at 8:40 AM, Emil Velikov
wrote:
> Hi Nicholas,
>
> On 26 August 2016 at 00:31, Nicholas Bishop wrote:
> > From: Nicholas Bishop
> >
> > On Intel Pineview M hardware, the i915 gallium driver doesn't output
> > the correct gl_FragCoord. It seems to always have an X coord of 0
On Thu, Sep 22, 2016 at 5:45 PM, Emil Velikov wrote:
> On 22 September 2016 at 08:10, Erik Faye-Lund wrote:
>> On Wed, Sep 21, 2016 at 8:57 PM, Kenneth Graunke
>> wrote:
>>> Commit 5921f372c89a68fac6ddefc009442721d9df4db2 exposed GLES 3.1 symbols
>>> in libGLESv2.so. Are we supposed to do the
On Thursday, September 22, 2016 1:54:44 PM PDT Ian Romanick wrote:
> On 09/21/2016 10:20 PM, Kenneth Graunke wrote:
> > In the past, we imported the prototypes of built-in functions, generated
> > calls to those, and waited until link time to resolve the calls and
> > import the actual code for the
On 22 September 2016 at 08:10, Erik Faye-Lund wrote:
> On Wed, Sep 21, 2016 at 8:57 PM, Kenneth Graunke
> wrote:
>> Commit 5921f372c89a68fac6ddefc009442721d9df4db2 exposed GLES 3.1 symbols
>> in libGLESv2.so. Are we supposed to do the same thing for GLES 3.2?
>>
>> I imagine we're supposed to,
Hi Nicholas,
On 26 August 2016 at 00:31, Nicholas Bishop wrote:
> From: Nicholas Bishop
>
> On Intel Pineview M hardware, the i915 gallium driver doesn't output
> the correct gl_FragCoord. It seems to always have an X coord of 0.0
> and a Y coord of the window's height in pixels, e.g. 600.0f or
Hi Nicholas,
On 8 September 2016 at 20:55, Nicholas Bishop wrote:
> Changed dri2_query_image to check the return value of
> resource_get_handle and return GL_FALSE if an error occurs. Similarly
> changed gbm_dri_bo_get_fd to check the return value of queryImage and
> return -1 (an invalid file de
Nayan Deshmukh wrote:
Hi Andy,
On Wed, Sep 21, 2016 at 9:30 PM, Andy Furniss wrote:
Andy Furniss wrote:
Andy Furniss wrote:
Andy Furniss wrote:
Andy Furniss wrote:
bz2 compressed vid showing the issue -
https://drive.google.com/open?id=0BxP5-S1t9VEEUE5sbUFBV20zSms
To be clear th
Hi all,
This series aims to make platform detection/selection easier, explicit and less
fragile.
To do that I've introduced Vulkan inspired EGL_USE_PLATFORM_*_KHR macro
guards. The user _really_ wants to select the on they use, yet
things should continue to compile (albeit there can be ABI iss
On Wed, Sep 21, 2016 at 8:57 PM, Kenneth Graunke wrote:
> Commit 5921f372c89a68fac6ddefc009442721d9df4db2 exposed GLES 3.1 symbols
> in libGLESv2.so. Are we supposed to do the same thing for GLES 3.2?
>
> I imagine we're supposed to, but I'm not certain what spec actually
> defines the ABI or whe
From: Emil Velikov
This commit effectively reverts the following:
7d642442d9339e5b65c30802c44091816cdf18be
400b833592d9aad7b2c4627a897380642d52189f
2c6186390c48e0d1f2b4d4136c11eb0bedfd1aec
eddcecbf74da26716509c047b95e85b00c12bab4
c0f8c9911c82c576cdd82dabad4a2370ac53565c
04c5cc5b8bec1f34f2405b08f
Change SUFFACE to SURFACE.
Signed-off-by: Nanley Chery
---
src/intel/isl/isl_surface_state.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/intel/isl/isl_surface_state.c
b/src/intel/isl/isl_surface_state.c
index e18fa50..4ed177a 100644
--- a/src/intel/isl/isl_surface_st
Christian König wrote:
Am 21.09.2016 um 16:38 schrieb Andy Furniss:
Andy Furniss wrote:
Andy Furniss wrote:
Andy Furniss wrote:
bz2 compressed vid showing the issue -
https://drive.google.com/open?id=0BxP5-S1t9VEEUE5sbUFBV20zSms
To be clear this is just the test rez vid I used so you can
From: Emil Velikov
There is no concept of display on XCB thus the connection is used
instead.
---
XXX: Pure XCB/Xlib-free implementation anyone ?
---
include/EGL/eglplatform.h | 7 +++
1 file changed, 7 insertions(+)
diff --git a/include/EGL/eglplatform.h b/include/EGL/eglplatform.h
index
This is much simpler. This patch is
Reviewed-by: Ian Romanick
On 09/21/2016 10:20 PM, Kenneth Graunke wrote:
> We want to check prior to optimization - otherwise we might fail to
> detect cases where barrier() is in control flow which is always taken
> (and therefore gets optimized away).
>
>
Kenneth Graunke writes:
> Hello,
>
> This series reworks the GLSL compiler to inline built-in functions
> immediately, at compile time, rather than importing prototypes and
> linking against an extra shader later on.
>
> Without this, compile time optimizations are nearly useless. For example,
>
Adam Jackson writes:
> This extension is a prerequisite for EGL 1.5, but the Intel drivers don't
> report support for it, and I'm not really sure why, the driver code certainly
> looks prepared for it. That said, I haven't tested either of these patches,
> and
> I'm not aware of any apps using t
On Wed, Sep 21, 2016 at 3:40 PM, Nicolas Koch wrote:
> Hey Jason, thanks for reviewing.
>
> This is my first mesa contribution, and I'm not really familiar with
> the mailing-list workflow.
>
Thanks for the contribution and welcome to mesa!
> Is there anything else I need to do to get this mer
From: Emil Velikov
In order to avoid the current, somewhat fragile detection in
eglplatform.h introduce explicit platform selection.
The approach is based on the one used in Vulkan and allows one to
explicitly "request" the platform they will be using without the need of
local hacks.
---
XXX: S
From: Emil Velikov
With the previous EGL_USE_PLATFORM_*_KHR in place one no longer needs
any of this fragile and somewhat buggy heuristics.
Users are responsible for requesting their preferred platform, with
adding a default to none/native types.
---
XXX: Similar to previous patch - something w
This reverts commit 1a7275de9a0088cedab17131eec82bd2cada26c3.
No longer required/needed as-of last commit.
Conflicts:
configure.ac
---
configure.ac | 7 ---
1 file changed, 7 deletions(-)
diff --git a/configure.ac b/configure.ac
index 0604ad9..d16447f 100644
--- a/configure.ac
+++ b
Bump, could someone push this?
Thanks,
-Nicholas
On Wed, Sep 14, 2016 at 1:06 PM, Nicholas Bishop wrote:
> Thanks for review. Could someone with commit access push this for me?
>
> On Wed, Sep 14, 2016 at 8:48 AM, Eric Engestrom
> wrote:
>> On Thu, Sep 08, 2016 at 03:55:02PM -0400, Nicholas Bis
Are there tests anywhere for either of these extensions? If not, can
you send some patches to piglit which either add tests or, preferably,
take tests for the existing desktop functionality and make them
ES-friendly (e.g.
https://cgit.freedesktop.org/piglit/commit/?id=d1bb73c483aa93e083ed251e0458d0
From: Emil Velikov
---
XXX: This is the upstream/Khronos definition of it, although I'm not
perfectly sure if/how many users depend on it.
XXX: Worth folding with the original commit which introduces the EGL_USE
PLATFORM... lot ?
---
include/EGL/eglplatform.h | 6 ++
1 file changed, 6 inser
Use the vertex positions described in the PRMs. This has no effect on
rendering but quiets the simulator warnings seen when the vertices
appear out of order.
Signed-off-by: Nanley Chery
Cc: Jason Ekstrand
Cc: Marek Olšák
---
src/intel/blorp/blorp_genX_exec.h | 6 +++---
1 file changed, 3 inser
On Wed, Sep 21, 2016 at 2:38 PM, Nanley Chery wrote:
> Is this to avoid needless computation?
Yes it is.
> If an app copies such an image, that's likely an app bug right?
Yes it is. But I think keeping such simple checks in the driver
saves a lot of app debugging time sometime later.
>
> On Wed,
Half float support already exists for desktop GL. Reuse the
ARB_half_float_vertex enable bit and account for the different enum to
enable the extension for ES2.
Signed-off-by: Kevin Strasser
---
src/mesa/main/extensions_table.h | 1 +
src/mesa/main/varray.c | 7 +--
2 files changed
The extension spec has been revised to include dependencies for OpenGL ES 2
and 3.
Signed-off-by: Kevin Strasser
---
src/mesa/main/extensions_table.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/extensions_table.h b/src/mesa/main/extensions_table.h
index c40d
From: Emil Velikov
---
include/EGL/eglplatform.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/EGL/eglplatform.h b/include/EGL/eglplatform.h
index 16765fb..dc4a3e2 100644
--- a/include/EGL/eglplatform.h
+++ b/include/EGL/eglplatform.h
@@ -133,6 +133,7 @@ typedef Pixmap EGLNativeP
Cc: mesa-sta...@lists.freedesktop.org
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
There's an assert right above this.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs.cpp
b/src/mesa/drivers/dri/i965/brw_fs.cpp
index 1483f41..dc000d9 100644
--- a/sr
Needed for user clip plane enables. Brkoen since this code was
introduced.
Cc: mesa-sta...@lists.freedesktop.org
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen8_ds_state.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/gen8_ds_s
CACHE_NEW_CS_PROG hasn't existed in quite a long time...the old
comment was there, but not the actual bit.
Cc: mesa-sta...@lists.freedesktop.org
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen7_cs_state.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/
3DSTATE_PS doesn't need this. 3DSTATE_PS_EXTRA however does,
for brw_color_buffer_write_enabled().
Cc: mesa-sta...@lists.freedesktop.org
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen8_ps_state.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/driv
Cc: mesa-sta...@lists.freedesktop.org
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_wm_surface_state.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
b/src/mesa/drivers/dri/i965/brw_wm_surface_state.c
ind
commit e07457d0aec96970a39a2dd1cb4e46da6107eb77 removed the last use
of ctx->Transform on Gen6. It looks like Gen7 never needed this, but
I copied it over when I originally ported the driver to Ivybridge.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen6_vs_state.c | 3 +--
src/
Signed-off-by: Nanley Chery
---
src/intel/blorp/blorp.c | 8
src/intel/blorp/blorp.h | 9 +
2 files changed, 9 insertions(+), 8 deletions(-)
diff --git a/src/intel/blorp/blorp.c b/src/intel/blorp/blorp.c
index 1c8fd55..e5dcf0a 100644
--- a/src/intel/blorp/blorp.c
+++ b/src/intel
https://bugs.freedesktop.org/show_bug.cgi?id=97879
--- Comment #10 from Silvan Jegen ---
(In reply to Michel Dänzer from comment #3)
> (In reply to Silvan Jegen from comment #2)
> > Just to clarify: the trace also includes loading of the game itself (which
> > takes a long time too) as well as in
Cc: mesa-sta...@lists.freedesktop.org
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/gen6_clip_state.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/gen6_clip_state.c
b/src/mesa/drivers/dri/i965/gen6_clip_state.c
index 7dc9740..c414a0f 100644
--- a
Is this to avoid needless computation?
If an app copies such an image, that's likely an app bug right?
On Wed, Sep 21, 2016 at 11:17:08AM -0700, Anuj Phogat wrote:
> Signed-off-by: Anuj Phogat
> ---
> src/intel/vulkan/anv_blorp.c | 4
> 1 file changed, 4 insertions(+)
>
> diff --git a/src/
From: Nicolai Hähnle
We will support the waiting option in ARB_query_buffer_object using
WAIT_REG_MEM on an appropriate fence-like dword. Some queries conveniently
write their results with the highest bit set, and we can just use that;
for others, we have to write a fence explicitly.
ZPASS_DONE
From: Nicolai Hähnle
v2: enable only when compute is available
---
docs/features.txt | 2 +-
docs/relnotes/12.1.0.html | 1 +
src/gallium/drivers/radeonsi/si_pipe.c | 21 ++---
3 files changed, 16 insertions(+), 8 deletions(-)
diff --git a/doc
From: Nicolai Hähnle
These functions extract the pipe state structure from the current
descriptors, for state saving.
v2: correctly dereference *buf (Bas)
---
src/gallium/drivers/radeonsi/si_descriptors.c | 46 +++
src/gallium/drivers/radeonsi/si_state.h | 5 +++
From: Nicolai Hähnle
To ensure that fences are properly initialized.
---
src/gallium/drivers/radeon/r600_query.c | 26 ++
src/gallium/drivers/radeon/r600_query.h | 2 +-
2 files changed, 11 insertions(+), 17 deletions(-)
diff --git a/src/gallium/drivers/radeon/r600_quer
From: Nicolai Hähnle
v2: fix a comment (Gustaw Smolarczyk)
---
src/gallium/drivers/radeon/r600_pipe_common.c | 3 +
src/gallium/drivers/radeon/r600_pipe_common.h | 2 +
src/gallium/drivers/radeon/r600_query.c | 391 +-
src/gallium/drivers/radeon/r600_query.h
From: Nicolai Hähnle
There are driver-specific context flags for barriers that are not covered
by the Gallium barrier interfaces.
The R600 settings of these flags may not be optimal, but we're not going
to use them yet anyway.
---
src/gallium/drivers/r600/r600_pipe.c | 6 ++
src/g
From: Nicolai Hähnle
For bottom-of-pipe fences inside the gfx command stream.
---
src/gallium/drivers/radeon/r600_pipe_common.c | 52 +++
src/gallium/drivers/radeon/r600_pipe_common.h | 5 +++
src/gallium/drivers/radeonsi/si_perfcounter.c | 41 ++---
3 fi
On 09/21/2016 10:20 PM, Kenneth Graunke wrote:
> In the past, we imported the prototypes of built-in functions, generated
> calls to those, and waited until link time to resolve the calls and
> import the actual code for the built-in functions.
I thought part of the reason we did this was to accou
On 09/21/2016 10:29 PM, Kenneth Graunke wrote:
> Hi Ian,
>
> I just sent out a four patch series that I think will accomplish a lot
> of what you were trying to do with the "do post-link optimizations, but
> only for built-ins" patches. But it deletes code.
>
> https://lists.freedesktop.org/arch
Am 21.09.2016 um 16:38 schrieb Andy Furniss:
Andy Furniss wrote:
Andy Furniss wrote:
Andy Furniss wrote:
bz2 compressed vid showing the issue -
https://drive.google.com/open?id=0BxP5-S1t9VEEUE5sbUFBV20zSms
To be clear this is just the test rez vid I used so you can
possibly recreate the is
Pending patch 2, this patch is
Reviewed-by: Ian Romanick
On 09/21/2016 10:20 PM, Kenneth Graunke wrote:
> This is now handled directly by ast_function.cpp.
>
> Signed-off-by: Kenneth Graunke
> ---
> src/compiler/glsl/builtin_functions.cpp | 30 --
> 1 file changed,
On 09/21/2016 10:20 PM, Kenneth Graunke wrote:
> Now that we generate built-in functions inline, there's no need to link
> against the built-in shader, and no built-in prototypes to consider.
>
> This lets us delete a bunch of code.
Boy howdy. This simplifies things quite a bit. Pending patch 2
Hi all,
some fixes based on feedback from v1.
In related news, https://reviews.llvm.org/D24822 improves the lowering of
the division-by-constant for timestamp queries in LLVM, but is not required
for this series to work.
Please review!
Nicolai
___
mes
From: Nicolai Hähnle
Save compute shader state that will be used for the ARB_query_buffer_object
implementation.
---
src/gallium/drivers/radeon/r600_pipe_common.h | 3 +++
src/gallium/drivers/radeon/r600_query.h | 7 +++
src/gallium/drivers/radeonsi/si_state.c | 12
On 09/21/2016 10:20 PM, Kenneth Graunke wrote:
> Hello,
>
> This series reworks the GLSL compiler to inline built-in functions
> immediately, at compile time, rather than importing prototypes and
> linking against an extra shader later on.
>
> Without this, compile time optimizations are nearly u
1 - 100 of 104 matches
Mail list logo
