From: Christopher James Halse Rogers
A first pass at a Mir platform for EGL.
This is RFC mainly because I hope to shortly make Mir's API less painfully
pointless-callback-based. There's no point in committing this as-is, but
it's hopefully still useful to review.
---
configure.ac
Jose Fonseca writes:
> I see. But one can use apitrace through LD_LIBRARY_PATH instead of
> LD_PRELOAD. Steam is not the first app to have issues with LD_PRELOAD.
I spend some time looking at this issue today.
First, things aren't as bad as I had first thought. I had
misunderstood the original
https://bugs.freedesktop.org/show_bug.cgi?id=61828
--- Comment #1 from Roland Scheidegger ---
Whoa I thought noone ever defined TXP for array textures. But it turns out I
was wrong since apparently GL_MESA_texture_array supports array textures even
with fixed function hence TXP. Sick. (The spec d
https://bugs.freedesktop.org/show_bug.cgi?id=61828
Priority: medium
Bug ID: 61828
Keywords: have-backtrace, regression
CC: srol...@vmware.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: [softpipe] tgsi/tgsi_exec.c:1936:exec
https://bugs.freedesktop.org/show_bug.cgi?id=61827
Priority: medium
Bug ID: 61827
Keywords: regression
CC: anuj.pho...@gmail.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: [softpipe] piglit fbo-clear-formats
From: Roland Scheidegger
Otherwise the state tracker will crash if the texture instructions
have offsets.
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp | 20 ++--
1 file changed, 18 insertions(+), 2 deletions(-)
diff --git a/src/mesa/state_tracker/st_glsl_to_tgsi.cpp
b/src/m
https://bugs.freedesktop.org/show_bug.cgi?id=61821
Priority: medium
Bug ID: 61821
Keywords: regression
CC: consume.no...@gmail.com
Assignee: mesa-dev@lists.freedesktop.org
Summary: src/mesa/drivers/dri/common/xmlpool.h:96:29: fat
Kenneth Graunke writes:
> If we've written N pairs of values to the buffer, then last_index = N,
> but the values are 0 .. N-1. Thus, we need to use <, not <=.
>
> This worked anyway because we fill the buffer with zeroes, so we just
> added an extra (0 - 0) to our results.
This series is:
Rev
Am 04.03.2013 23:04, schrieb Brian Paul:
> Note: This is a candidate for the stable branches.
> ---
> src/gallium/drivers/llvmpipe/lp_scene.c | 20
> 1 files changed, 20 insertions(+), 0 deletions(-)
>
> diff --git a/src/gallium/drivers/llvmpipe/lp_scene.c
> b/src/gallium/
On Mon, Mar 4, 2013 at 11:47 PM, Marek Olšák wrote:
> On Mon, Mar 4, 2013 at 4:54 AM, Dave Airlie wrote:
>> I've been playing with softpipe msaa on and off, but I hit a problem
>> with the clears and am just wondering if the state tracker should be
>> doing something like this.
>>
>> Or maybe onl
On Mon, Mar 4, 2013 at 11:41 AM, Ian Romanick wrote:
> On 03/04/2013 11:38 AM, Matt Turner wrote:
>>
>> Reported-by: Marek Olšák
>
>
> Do our ETC2 piglit tests run in this mode? It would be good to see this
> change actually fix a bug by making a failing test pass. :)
>
ETC2 tests currently runs
This was only necessary because our bounds checking was off by one, and
thus we read an extra pair of values.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/dri/i965/brw_queryobj.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_queryobj.c
b/src/mesa/
If we've written N pairs of values to the buffer, then last_index = N,
but the values are 0 .. N-1. Thus, we need to use <, not <=.
This worked anyway because we fill the buffer with zeroes, so we just
added an extra (0 - 0) to our results.
Signed-off-by: Kenneth Graunke
---
src/mesa/drivers/d
---
src/gallium/drivers/llvmpipe/lp_scene.h |4
1 files changed, 0 insertions(+), 4 deletions(-)
diff --git a/src/gallium/drivers/llvmpipe/lp_scene.h
b/src/gallium/drivers/llvmpipe/lp_scene.h
index 801829d..1d0cd0e 100644
--- a/src/gallium/drivers/llvmpipe/lp_scene.h
+++ b/src/gallium/d
We advertise a max texture/surfaces size of 8K x 8K but the old values
for these limits didn't actually allow us to handle that surface size.
For 8K x 8K we'll have 16384 bins. Each bin needs at least one cmd_block
object which was 2192 bytes in size. Since 16384 * 2192 exceeded
LP_SCENE_MAX_SIZ
Note: This is a candidate for the stable branches.
---
src/gallium/drivers/llvmpipe/lp_scene.c | 20
1 files changed, 20 insertions(+), 0 deletions(-)
diff --git a/src/gallium/drivers/llvmpipe/lp_scene.c
b/src/gallium/drivers/llvmpipe/lp_scene.c
index fec2f74..8056906 1006
On Mon, Mar 4, 2013 at 11:38 AM, Matt Turner wrote:
> Reported-by: Marek Olšák
> ---
> src/mesa/main/teximage.c |2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
> index 0dcf88a..e5260d1 100644
> --- a/src/mesa/main
---
src/mesa/main/teximage.c | 22 --
1 file changed, 12 insertions(+), 10 deletions(-)
diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
index 0dcf88a..a7b88d1 100644
--- a/src/mesa/main/teximage.c
+++ b/src/mesa/main/teximage.c
@@ -3494,19 +3494,21 @@ copyte
On 03/04/2013 11:38 AM, Matt Turner wrote:
Reported-by: Marek Olšák
Do our ETC2 piglit tests run in this mode? It would be good to see this
change actually fix a bug by making a failing test pass. :)
---
src/mesa/main/teximage.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(
Reported-by: Marek Olšák
---
src/mesa/main/teximage.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
index 0dcf88a..e5260d1 100644
--- a/src/mesa/main/teximage.c
+++ b/src/mesa/main/teximage.c
@@ -520,7 +520,7 @@ _mesa
On Mon, Mar 4, 2013 at 11:14 AM, Brian Paul wrote:
> On 03/04/2013 11:31 AM, Matt Turner wrote:
>>
>> X11 is already checked conditionally below.
>>
>> Fixes OSMesa-only configurations to not require X11.
>> ---
>> configure.ac |8
>> 1 files changed, 0 insertions(+), 8 deletions(-
On 03/04/2013 11:31 AM, Matt Turner wrote:
X11 is already checked conditionally below.
Fixes OSMesa-only configurations to not require X11.
---
configure.ac |8
1 files changed, 0 insertions(+), 8 deletions(-)
diff --git a/configure.ac b/configure.ac
index 7852595..1b13d06 10064
On Mon, Mar 4, 2013 at 2:05 PM, Michel Dänzer wrote:
> On Mon, 2013-03-04 at 13:17 -0500, j.gli...@gmail.com wrote:
>> From: Jerome Glisse
>>
>> Some code calling the flush function gave a fence pointer that point
>> to an old fence and should be unreference to avoid leaking fence.
>>
>> Candidat
On Mon, 2013-03-04 at 13:17 -0500, j.gli...@gmail.com wrote:
> From: Jerome Glisse
>
> Some code calling the flush function gave a fence pointer that point
> to an old fence and should be unreference to avoid leaking fence.
>
> Candidate for 9.1
>
> Signed-off-by: Jerome Glisse
> ---
> src/g
On Thu, Feb 28, 2013 at 10:39 AM, Daniel Martin wrote:
>
> Signed-off-by: Daniel Martin
> ---
> There's a small logic error preventing mesa to be build with swrast only
> and not having libdrm.
>
> v2: "|| test" replaced by "-a"
>
> configure.ac | 2 +-
> 1 file changed, 1 insertion(+), 1 deleti
Should have been removed with e273ed37.
---
configure.ac |6 --
1 files changed, 0 insertions(+), 6 deletions(-)
diff --git a/configure.ac b/configure.ac
index 1b13d06..ea56a04 100644
--- a/configure.ac
+++ b/configure.ac
@@ -832,12 +832,6 @@ if test "x$enable_dri" = xyes; then
fi
f
X11 is already checked conditionally below.
Fixes OSMesa-only configurations to not require X11.
---
configure.ac |8
1 files changed, 0 insertions(+), 8 deletions(-)
diff --git a/configure.ac b/configure.ac
index 7852595..1b13d06 100644
--- a/configure.ac
+++ b/configure.ac
@@ -832
From: Jerome Glisse
Some code calling the flush function gave a fence pointer that point
to an old fence and should be unreference to avoid leaking fence.
Candidate for 9.1
Signed-off-by: Jerome Glisse
---
src/gallium/drivers/r600/r600_pipe.c | 8 +---
src/gallium/drivers/radeonsi
https://bugs.freedesktop.org/show_bug.cgi?id=61761
--- Comment #4 from Blaž Hrastnik ---
Yeah, it seems to be some obscurely ancient extension. The only reason I "used"
it is because I took over the work on OpenGL bindings for ruby, and the library
is over 6 years old, so some tests test really o
https://bugs.freedesktop.org/show_bug.cgi?id=61761
--- Comment #3 from Roland Scheidegger ---
I guess if nvidia doesn't advertize it (and by the looks of it they never did)
it should be safe to remove (and amd doesn't seem to advertize it neither in
their closed-source drivers). After all Polygon
https://bugs.freedesktop.org/show_bug.cgi?id=61761
--- Comment #2 from Brian Paul ---
I wonder if we should just drop the GL_EXT_polygon_offset extension. I see
that NVIDIA doesn't advertise it.
--
You are receiving this mail because:
You are the assignee for the bug.
_
https://bugs.freedesktop.org/show_bug.cgi?id=61764
--- Comment #4 from Blaž Hrastnik ---
Ah yes, I have found that bit later in the documentation. Thank you for
clarification!
--
You are receiving this mail because:
You are the assignee for the bug.
_
On 03/03/2013 01:27 PM, Dave Airlie wrote:
From: Dave Airlie
This is to be used to implement glGet GL_SAMPLE_POSITION.
Signed-off-by: Dave Airlie
---
src/gallium/include/pipe/p_context.h | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/include/pipe/p_context.h
b/src/galli
Just minor formatting nits below.
On 03/03/2013 01:27 PM, Dave Airlie wrote:
From: Dave Airlie
This just calls into the gallium interface.
Signed-off-by: Dave Airlie
---
src/mesa/sources.mak| 1 +
src/mesa/state_tracker/st_cb_msaa.c | 52
https://bugs.freedesktop.org/show_bug.cgi?id=61764
--- Comment #3 from Roland Scheidegger ---
(In reply to comment #2)
> I'd still argue that this is a bug. If the extensions were removed, their
> implementation should be fully removed (as in, these functions, especially
> the EXT/ARB versions of
On Mon, Mar 4, 2013 at 10:11 AM, Pohjolainen, Topi
wrote:
> On Mon, Mar 04, 2013 at 09:56:34AM -0500, Kristian H?gsberg wrote:
>> On Mon, Mar 4, 2013 at 4:55 AM, Pohjolainen, Topi
>> wrote:
>> > On Fri, Mar 01, 2013 at 10:03:45AM -0500, Kristian H?gsberg wrote:
>> >> On Fri, Mar 1, 2013 at 3:51 A
https://bugs.freedesktop.org/show_bug.cgi?id=61395
Brian Paul changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Wed, Feb 27, 2013 at 6:11 PM, Marek Olšák wrote:
> The states were split because we thought it caused a hardlock. Now we know
> the hardlock was caused by something else and has since been fixed.
For the serie:
Reviewed-by: Jerome Glisse
> ---
> src/gallium/drivers/r600/evergreen_state.c
On Sun, Mar 3, 2013 at 8:39 AM, Marek Olšák wrote:
> also change names of other functions, so that they make sense
For the serie:
Reviewed-by: Jerome Glisse
> ---
> src/gallium/drivers/r600/evergreen_state.c |4 +-
> src/gallium/drivers/r600/r600_pipe.h |8 +--
> src/gallium/
On Sun, Mar 3, 2013 at 9:13 AM, Marek Olšák wrote:
> This avoids the kernel CS checker errors with MSAA textures.
Reviewed-by: Jerome Glisse
> ---
> src/gallium/drivers/r600/r600_texture.c |2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/r600/r600
On Mon, Mar 04, 2013 at 09:56:34AM -0500, Kristian H?gsberg wrote:
> On Mon, Mar 4, 2013 at 4:55 AM, Pohjolainen, Topi
> wrote:
> > On Fri, Mar 01, 2013 at 10:03:45AM -0500, Kristian H?gsberg wrote:
> >> On Fri, Mar 1, 2013 at 3:51 AM, Pohjolainen, Topi
> >> wrote:
> >> > On Tue, Feb 26, 2013 at
On Mon, Mar 4, 2013 at 4:55 AM, Pohjolainen, Topi
wrote:
> On Fri, Mar 01, 2013 at 10:03:45AM -0500, Kristian H?gsberg wrote:
>> On Fri, Mar 1, 2013 at 3:51 AM, Pohjolainen, Topi
>> wrote:
>> > On Tue, Feb 26, 2013 at 04:05:25PM +, Tom Cooksey wrote:
>> >> Hi Topi,
>> >>
>> >> > The second mo
On Fri, Mar 01, 2013 at 05:55:31PM +0100, Marek Olšák wrote:
Patches 2-4 are:
Reviewed-by: Tom Stellard
Thanks for working on this, it is a nice improvement.
-Tom
> ---
> src/gallium/drivers/r600/compute_memory_pool.c|1 +
> src/gallium/drivers/r600/evergreen_compute.h |
https://bugs.freedesktop.org/show_bug.cgi?id=61764
--- Comment #2 from Blaž Hrastnik ---
I'd still argue that this is a bug. If the extensions were removed, their
implementation should be fully removed (as in, these functions, especially the
EXT/ARB versions of them should be removed), instead of
https://bugs.freedesktop.org/show_bug.cgi?id=60121
Andy Furniss changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Sun, Mar 3, 2013 at 9:27 PM, Dave Airlie wrote:
> From: Dave Airlie
>
> This adds support to the mesa state tracker for ARB_texture_multisample.
>
> hardware doesn't seem to use a different texture instructions, so
> I don't think we need to create one for TGSI at this time.
>
> Thanks to Mare
Note: This is a candidate for the stable branches.
Signed-off-by: Chris Wilson
---
src/mesa/drivers/dri/intel/intel_screen.c | 14 ++
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/src/mesa/drivers/dri/intel/intel_screen.c
b/src/mesa/drivers/dri/intel/intel_screen.c
Note: This is a candidate for the stable branches.
Signed-off-by: Chris Wilson
---
src/mesa/drivers/dri/common/utils.c |5 +
1 file changed, 5 insertions(+)
diff --git a/src/mesa/drivers/dri/common/utils.c
b/src/mesa/drivers/dri/common/utils.c
index ac0773b..41cab13 100644
--- a/src/mes
On Mon, Mar 4, 2013 at 4:54 AM, Dave Airlie wrote:
> I've been playing with softpipe msaa on and off, but I hit a problem
> with the clears and am just wondering if the state tracker should be
> doing something like this.
>
> Or maybe only if any bound buffer has nr_samples > 1, or fallback to
> t
The sample positions can be found in r600_emit_msaa_state,
evergreen_emit_msaa_state, and cayman_emit_msaa_state, though
extracting them from the arrays might not be so straightforward.
Marek
On Sun, Mar 3, 2013 at 9:27 PM, Dave Airlie wrote:
> From: Dave Airlie
>
> get sample position is most
The conditional should be "if (i > 1 && .."; with that fixed, this is:
Reviewed-by: Marek Olšák
Marek
On Mon, Mar 4, 2013 at 9:02 AM, Dave Airlie wrote:
> From: Dave Airlie
>
> I hit an assert playing with softpipe and texture MSAA today, makes me think
> we might need something like this to
On Mon, Mar 4, 2013 at 11:53 AM, Roland Scheidegger wrote:
> Am 02.03.2013 04:26, schrieb Dave Airlie:
>> On Sat, Mar 2, 2013 at 12:02 PM, Roland Scheidegger
>> wrote:
>>> Am 02.03.2013 01:36, schrieb Brian Paul:
---
src/mesa/state_tracker/st_glsl_to_tgsi.cpp |3 +++
1 files
https://bugs.freedesktop.org/show_bug.cgi?id=61761
--- Comment #1 from Roland Scheidegger ---
Looks like when using PolygonOffsetEXT() mesa will multiply the bias value by
_DepthMaxF to set offset units (hence the same as calling PolygonOffset(). But
the get will just return offset units instead
https://bugs.freedesktop.org/show_bug.cgi?id=61764
Roland Scheidegger changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
Am 03.03.2013 21:27, schrieb Dave Airlie:
> From: Dave Airlie
>
> This is to be used to implement glGet GL_SAMPLE_POSITION.
>
> Signed-off-by: Dave Airlie
> ---
> src/gallium/include/pipe/p_context.h | 4
> 1 file changed, 4 insertions(+)
>
> diff --git a/src/gallium/include/pipe/p_conte
Am 04.03.2013 04:54, schrieb Dave Airlie:
> I've been playing with softpipe msaa on and off, but I hit a problem
> with the clears and am just wondering if the state tracker should be
> doing something like this.
>
> Or maybe only if any bound buffer has nr_samples > 1, or fallback to
> the non-qu
Am 02.03.2013 04:26, schrieb Dave Airlie:
> On Sat, Mar 2, 2013 at 12:02 PM, Roland Scheidegger
> wrote:
>> Am 02.03.2013 01:36, schrieb Brian Paul:
>>> ---
>>> src/mesa/state_tracker/st_glsl_to_tgsi.cpp |3 +++
>>> 1 files changed, 3 insertions(+), 0 deletions(-)
>>>
>>> diff --git a/src/me
On Fri, Mar 01, 2013 at 10:03:45AM -0500, Kristian H?gsberg wrote:
> On Fri, Mar 1, 2013 at 3:51 AM, Pohjolainen, Topi
> wrote:
> > On Tue, Feb 26, 2013 at 04:05:25PM +, Tom Cooksey wrote:
> >> Hi Topi,
> >>
> >> > The second more or less questionable part is the support for creating YUV
> >>
From: Dave Airlie
I hit an assert playing with softpipe and texture MSAA today, makes me think
we might need something like this to stop creating illegal shaders.
Signed-off-by: Dave Airlie
---
src/gallium/auxiliary/util/u_blitter.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/
59 matches
Mail list logo
