2011/6/4 Jose Fonseca :
> I think we need to have a proper review round of the gallium interfaces, so
> that we have an interface everybody feels that we can support going forward,
> which did not happen last round.
>
> That said, I don't think much attention has been given to this branch outside
Le 05/06/2011 03:05, Matt Turner a écrit :
> On Sat, Jun 4, 2011 at 7:14 PM, Benjamin Bellec wrote:
>> Le 03/06/2011 06:09, Matt Turner a écrit :
>>> Also, if you want to check if the value is already a power-of-two,
>>> instead of a case statement for every POT (including 0), just do the
>>> stan
I think we need to have a proper review round of the gallium interfaces, so
that we have an interface everybody feels that we can support going forward,
which did not happen last round.
That said, I don't think much attention has been given to this branch outside
from those working on it. So th
On Sat, Jun 4, 2011 at 7:14 PM, Benjamin Bellec wrote:
> Le 03/06/2011 06:09, Matt Turner a écrit :
>> Also, if you want to check if the value is already a power-of-two,
>> instead of a case statement for every POT (including 0), just do the
>> standard is-power-of-two check:
>>
>> (x & (x - 1)) =
2011/6/4 Christian König :
> Am Samstag, den 04.06.2011, 15:20 -0400 schrieb Matt Turner:
>> Is there a reason that this can't be merged to master?
> Since float buffer support was merged to master I can't see any more
> obstacles. Just gone over the branch one more time and fixed some
> problem cr
When it is sensible to do so,
1) intelCreateBuffer() now attaches separate depth and stencil
buffers
to the framebuffer it creates.
2) intel_update_renderbuffers() requests for the framebuffer
a separate stencil buffer (DRI2BufferStencil).
The criteria for "sensible" i
Assert that the GLX config has an expected depth/stencil bit combination:
one of d24/s8, d16/s0, d0/s0. These are the only depth/stencil
configurations that we advertise.
Remove the check for software stencil, because given the assertions'
constraints the check always fails.
CC: Eric Anholt
CC:
Extract the code that queries DRI2 to obtain the DRIdrawable's buffers
into intel_query_dri2_buffers_no_separate_stencil().
Extract the code that assigns the DRI buffer's DRM region to the
corresponding renderbuffer into
intel_process_dri2_buffer_no_separate_stencil().
Rationale
-
The nex
It's the analog of intel_renderbuffer_set_region(), but for the hiz region
of course.
CC: Eric Anholt
CC: Ian Romanick
CC: Kenneth Graunke
CC: Kristian Høgsberg
Signed-off-by: Chad Versace
---
src/mesa/drivers/dri/intel/intel_fbo.c | 12
src/mesa/drivers/dri/intel/intel_fbo.h
Remove functions intel_override_hiz() and
intel_override_separate_stencil(). They are now located in intel_screen.c.
CC: Eric Anholt
CC: Ian Romanick
CC: Kenneth Graunke
CC: Kristian Høgsberg
Signed-off-by: Chad Versace
---
src/mesa/drivers/dri/intel/intel_context.c | 48 --
Add the fields below to intel_screen. The expression in parens is the
value to which intelInitScreen2() currently sets the field.
GLboolean hw_has_separate_stencil (true iff gen >= 7)
GLboolean hw_must_use_separate_stencil (true iff gen >= 7)
GLboolean hw_has_hiz
... which indicates if the X driver supports DRI2BufferHiz and
DRI2BufferStencil.
I'm placing this in its own commit due to the large comment block.
CC: Eric Anholt
CC: Ian Romanick
CC: Kenneth Graunke
CC: Kristian Høgsberg
Signed-off-by: Chad Versace
---
src/mesa/drivers/dri/intel/intel_sc
CC: Eric Anholt
CC: Ian Romanick
CC: Kenneth Graunke
CC: Kristian Høgsberg
Signed-off-by: Chad Versace
---
include/GL/internal/dri_interface.h |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/include/GL/internal/dri_interface.h
b/include/GL/internal/dri_interface.h
in
Before this commit, if a client were to request a stencil or hiz buffer, then
I830DRI2CreateBuffer() allocated and returned an X-tiled buffer by
accident. (DRI2BufferStencil and DRI2BufferHiz were unintentionally caught
by the default case of a switch statement.)
Now, I830DRI2CreateBuffer() correc
... and bump version to 2.6.
CC: Eric Anholt
CC: Ian Romanick
CC: Kenneth Graunke
CC: Kristian Høgsberg
Signed-off-by: Chad Versace
---
configure.ac |2 +-
dri2proto.txt |6 +-
dri2tokens.h |1 +
3 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/configure.ac b/
Patch 01: dri2proto
Patch 02: xf86-video-intel
Patch 03-10: mesa
I'm aware that this patch series is painful to review. Thank you in advance
for your reviewed-by's.
There is a related patch series on the intel-gfx list that adds the necessary
functionality to xf86-video-intel. See the thread with
Am Samstag, den 04.06.2011, 15:20 -0400 schrieb Matt Turner:
> Is there a reason that this can't be merged to master?
Since float buffer support was merged to master I can't see any more
obstacles. Just gone over the branch one more time and fixed some
problem created by merging.
> It seems like t
On 06/03/2011 03:36 PM, Kenneth Graunke wrote:
> On 06/03/2011 12:47 PM, Chad Versace wrote:
>> Since the stencil buffer is interleaved, the generic Mesa renderbuffer
>> accessors do not suffice. Custom span functions are necessary.
>>
>> Signed-off-by: Chad Versace
>
> I know you had done this wi
On 06/03/2011 03:33 PM, Kenneth Graunke wrote:
> On 06/03/2011 12:47 PM, Chad Versace wrote:
>> When emitting 3DSTATE_DEPTH_BUFFER, also emit 3DSTATE_HIER_DEPTH_BUFFER if
>> there is a hiz buffer. Ditto for 3DSTATE_STENCIL_BUFFER and a separate
>> stencil buffer.
>>
>> Signed-off-by: Chad Versace
>
Le 03/06/2011 01:40, Brian Paul a écrit :
> I'd like to avoid the warning if at all possible. If you replace (val
>>> 32) with (val >> (sizeof(unsigned) * 4)) does that silence it?
Yes it fix, but as Matt Turner said, it's not necessary to check this.
Btw we don't check that in the other function
https://bugs.freedesktop.org/show_bug.cgi?id=37862
--- Comment #5 from Marek Olšák 2011-06-04 12:21:06 PDT ---
Use autogen.sh with --enable-gallium-r600, then type make.
These:
lib/libGL.so*
lib/gallium/r600_dri.so
are the files you should use.
--
Configure bugmail: https://bugs.freedesktop.
2011/4/26 Christian König :
> Hi Andy and everybody on the list,
>
> sorry for the late reply, but i've been on vacation the last couple of
> days.
>
> Am Dienstag, den 12.04.2011, 21:38 +0100 schrieb Andy Furniss:
>> In addition to the quit crash I notice that resizing will also crash.
> Should be
2011/6/4 Brian Paul :
> 2011/6/3 Stéphane Marchesin :
>> ---
>> src/gallium/auxiliary/util/u_blitter.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>>
>> diff --git a/src/gallium/auxiliary/util/u_blitter.c
>> b/src/gallium/auxiliary/util/u_blitter.c
>> index a4c3990..528f344 100
https://bugs.freedesktop.org/show_bug.cgi?id=37862
--- Comment #4 from Pepi 2011-06-04 08:43:09 PDT ---
Thanks Marek,
I downloaded it from:
http://cgit.freedesktop.org/mesa/mesa/commit/?id=6491e9593d5cbc5644eb02593a2f562447efdcbb
Extract it. Make it with: "make linux-x86-64".
But then...what?
2011/6/3 Stéphane Marchesin :
> ---
> src/gallium/auxiliary/util/u_blitter.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/gallium/auxiliary/util/u_blitter.c
> b/src/gallium/auxiliary/util/u_blitter.c
> index a4c3990..528f344 100644
> --- a/src/gallium/auxilia
Looks good.
Marek
2011/6/4 Stéphane Marchesin :
> ---
> src/gallium/auxiliary/util/u_blitter.c | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/src/gallium/auxiliary/util/u_blitter.c
> b/src/gallium/auxiliary/util/u_blitter.c
> index a4c3990..528f344 100644
> --- a
26 matches
Mail list logo
