I forgot to cc the stable mailing list for this patch.
2014-04-19 21:05 GMT+02:00 Benjamin Bellec :
>
> Signed-off-by: Benjamin Bellec
> ---
> src/mesa/main/getstring.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/mesa/main/getstr
It's also broken on RV770, tested with Valley 1.0 (64-bit) Basic preset.
2014-09-02 10:42 GMT+02:00 Glenn Kennard :
> On Mon, 01 Sep 2014 00:02:18 +0200, Glenn Kennard
> wrote:
>
> Signed-off-by: Glenn Kennard
>> ---
>> Tested on radeon 6670, all sample shading piglits pass, no
>> regressions
Tested-by: Benjamin Bellec
2014-09-02 21:14 GMT+02:00 Marek Olšák :
> From: Marek Olšák
>
> *_update_db_shader_control depends on the alpha test state. The problem was
> it was in a block which is only entered if the pixel shader is changed.
>
> Bugzilla: https://bu
2011/6/8 Chris Bandy
> On 06/08/2011 06:38 AM, Benjamin Bellec wrote:
> > Here is the v4 patch.
> >
> > Benjamin
>
> As an uninformed bystander, I have some nitpicks that may just be coding
> style.
>
> From the patch:
>
> + if (x <= 1)
> +
And what about removing the "RadeonProgram" page [1] ?
>From my point of view, this page depicts a wrong picture of the Radeon Mesa
driver. While I'm pretty sure most of the current and past games runs fine
today with r300g/r600g/radeonsi, this is not what one could think at a
first look when read
RadeonProgram page just removed.
2015-02-05 20:52 GMT+01:00 Alex Deucher :
> On Thu, Feb 5, 2015 at 2:47 PM, Benjamin Bellec
> wrote:
> > And what about removing the "RadeonProgram" page [1] ?
> >
> > From my point of view, this page depicts a wrong pictu
Hello all,
Do you know a game or app using doubles ?
2015-02-20 21:08 GMT+01:00 Ilia Mirkin :
> On Fri, Feb 20, 2015 at 3:07 PM, Ilia Mirkin wrote:
> > On Thu, Feb 19, 2015 at 9:59 PM, Glenn Kennard
> wrote:
> >>> + if ((write_mask & 0x3) != 0x3 &&
> >>> + (write_mask & 0xc) !=
Tested-by: Benjamin Bellec
On Sat, Aug 8, 2015, at 10:05 PM, Marek Olšák wrote:
>* From: Marek Olšák <http://lists.freedesktop.org/mailman/listinfo/mesa-dev>>
*> >* MESA_LLVM_VERSION_PATCH is undefined.
*>* ---
*>* src/gallium/drivers/radeon/r600_pipe_common.c | 9 +
Hello,
Where does this mapping list come from ? Is this an official map (Vendor or
Windows drivers) ? Is that used just for information ?
Because many of them are not really precise, if not completely wrong at all
(eg. the Radeon R400/R500 series names).
Regards.
--
Benjamin
2015-08-16 17:27
> of your vendor, or advertise another vendor (some games do disable
> some features for some cards).
>
> Yours,
>
> Axel Davy
>
> On 17/08/2015 19:30, Benjamin Bellec write :
>
> Hello,
>>
>> Where does this mapping list come from ? Is this an official m
Hello,
Does your branch should works on R700 ?
I tested it on a RV770 but get many corruptions on Counter Strike Source
(native). I haven't tested anything else.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailm
nd drm protocol available like this ?
Please note those patches are for wayland/weston 1.1.0
Regards,
Benjamin
--
Benjamin Gaignard
Graphic Working Group
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
From b185fe89a1ea8ddbe19d943cb19272f369b7b438 Mon Sep 1
-core make it more easily accessible for all
software even for those who don't use Mesa.
Benjamin
PS: I have an updated version of my patch for latest wayland
2013/11/27 Jasper St. Pierre :
> Wasn't EGLStreams supposed to solve the use case of passing hardware buffers
> around in a
On Mon, 2013-12-16 at 13:30 +0100, Marek Olšák wrote:
> Wouldn't it be easier to just revert the gallium endianness rework? If
> it breaks all hw drivers on big endian machines, it's apparently not
> done right.
It also makes llvmpipe work on P7 and P8 and enables r600 ... which some
people rely o
t/?id=de48199693484fc903627ce53b007901af2e37a8
And current code is still:
/* Supported except the original R600. */
case PIPE_CAP_INDEP_BLEND_ENABLE:
case PIPE_CAP_INDEP_BLEND_FUNC:
/* R600 doesn't support per-MRT blends */
return family == CHIP_R600 ? 0 : 1;
Regards.
this extension is mandatory do be "OpenGL 3.0 compliant"?
(I don't know how really works the OpenGL specification)
Does the R600 cards can expose GL3.0 in glxinfo without this extension?
Benjamin
Le 31/03/2014 20:43, Ilia Mirkin a écrit :
> On Mon, Mar 31, 2014 at 2:39 PM, Benj
e at least).
And is your tool strictly from a Mesa perspective? I mean, for instance,
GL3.3 is only supported for R600/R700 with kernel 3.14. Before, only
GL3.1 is exposed. This information is - indirectly - noted on
RadeonFeature web page though.
Benjamin
Le 31/03/2014 20:43, Ilia Mirkin a éc
Signed-off-by: Benjamin Bellec
diff --git a/docs/relnotes/10.2.html b/docs/relnotes/10.2.html
index da79e7d..05fd502 100644
--- a/docs/relnotes/10.2.html
+++ b/docs/relnotes/10.2.html
@@ -51,7 +51,11 @@ Note: some of the new features are only available with certain drivers.
Bug fixes
-TBD
Oh OK, I wasn't aware of this process and script.
That said, the commit that fixed the bug was not intended to. And it
will not be caught by the script.
Noted for git send-email.
Benjamin
Le 11/04/2014 20:24, Emil Velikov a écrit :
> On 11/04/14 19:07, Benjamin Bellec wrote:
>>
Test pts/tremulous
I have a Radeon HD5850 (with 1 GB VRAM).
Any idea to try to spot your issue? (decrease VRAM size?)
Regards.
Benjamin
Le 15/04/2014 09:35, Lauri Kasanen a écrit :
> Hi,
>
> Tremulous and Smoking Guns regressed in Mesa master, ok in 020c43f,
> broken in 4ddf51db.
>
pr 2014 10:16:53 +0300
> Lauri Kasanen wrote:
>
>> On Fri, 18 Apr 2014 01:41:46 +0200
>> Benjamin Bellec wrote:
>>
>>> Hi Lauri,
>>>
>>> I tested with both commit but cannot seeing something relevant, I got 60
>>> FPS in both case.
>
Signed-off-by: Benjamin Bellec
---
src/mesa/main/getstring.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/getstring.c b/src/mesa/main/getstring.c
index 3ac62d4..1ac92e2 100644
--- a/src/mesa/main/getstring.c
+++ b/src/mesa/main/getstring.c
@@ -166,7 +166,7
lanned ?
Regards.
Benjamin
Le 06/05/2014 13:40, Jonas Kulla a écrit :
> Hello list,
>
> after about 3 days of debugging, I was able to isolate a rather weird
> behavior in Mesa GL.
> The gist of it is the following: When I create a buffer object and
> allocate uninitilaized
> memo
steam)/version(1401381906_client)
libGL: Can't open configuration file /home/benjamin/.drirc: Aucun
fichier ou dossier de ce type.
libGL: Can't open configuration file /home/benjamin/.drirc: Aucun
fichier ou dossier de ce type.
steam: ../../src/glsl/ralloc.c :84 : get_header: l'asse
Hello,
Just to report that libdrm_radeon version (2.4.44) has not been updated
in scons/gallium.py too (still at version 2.4.42).
Regards.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
ki (for instance
http://dri.freedesktop.org/wiki/R600ToDo/) or elsewhere. Tasks that you
(as experimented) consider very easy, "useless" or with very low
priority... for beginners these kind of tasks could already be a hard
work to begin with.
Regards.
Benjamin
>
> -Brian
&
Hi,
Could you bisect?
Or provide an easy way to build ogl-samples!
Regards.
Le 17/03/2014 15:34, Dieter Nützel a écrit :
> Trying to run ogl-samples GL 3.2+ tests on my RV730 AGP.
>
> /opt/ogl-samples> ./build/release/gl-320-primitive-sprite
> libGL: screen 0 does not appear to be DRI3 capable
>
Hello,
Maybe this change should be mentioned in the relnotes ?
2014-12-12 18:04 GMT+01:00 Matt Turner :
>
> Thanks Timothy. The whole series looks good to me:
>
> Reviewed-by: Matt Turner
>
> I'd wait a couple of days for objections before pushing the series.
> ___
Signed-off-by: Benjamin Bellec
---
src/gallium/auxiliary/hud/hud_context.c | 5 +
src/gallium/drivers/radeon/r600_pipe_common.c | 2 +-
src/gallium/include/pipe/p_defines.h | 1 +
src/mesa/state_tracker/st_cb_perfmon.c| 5 +
4 files changed, 12 insertions(+), 1
Yes I meant "unit", "sign" is a bad translation.
In gallium/hud, the temperature is displayed as a percentage, which is
quite confusing. But maybe there is better and simpler approach to fix
that. I do not really understand the code so I tried to copy other part of
the code.
___
Signed-off-by: Benjamin Bellec
---
src/gallium/drivers/radeon/r600_pipe_common.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c
b/src/gallium/drivers/radeon/r600_pipe_common.c
index ed5d1da..2a8d906 100644
--- a/src/gallium
Forgot this one.
I will resend correctly, just trying to understand how git-format-patch and
git-send-mail works...
2015-09-04 20:18 GMT+02:00 Benjamin Bellec :
> Signed-off-by: Benjamin Bellec
> ---
> src/gallium/drivers/radeon/r600_pipe_common.c | 2 +-
> 1 file changed, 1 ins
Signed-off-by: Benjamin Bellec
---
src/gallium/drivers/radeon/r600_pipe_common.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c
b/src/gallium/drivers/radeon/r600_pipe_common.c
index ed5d1da..2a8d906 100644
--- a/src/gallium
Signed-off-by: Benjamin Bellec
---
src/gallium/auxiliary/hud/hud_context.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/auxiliary/hud/hud_context.c
b/src/gallium/auxiliary/hud/hud_context.c
index 95eed26..71529bc 100644
--- a/src/gallium/auxiliary/hud/hud_context.c
+++ b
ay a "C" instead of nothing.
Ref. your suggestion on PIPE_DRIVER_QUERY_TYPE_TEMPERATURE, I'm not skilled
enough to answer.
2015-09-04 20:42 GMT+02:00 Brian Paul :
> On 09/04/2015 12:30 PM, Benjamin Bellec wrote:
>
>> Signed-off-by: Benjamin Bellec
>> ---
>
backs.
Regards.
--
Benjamin
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Le 05/09/2015 10:19, Samuel Pitoiset a écrit :
>
> On 09/04/2015 08:57 PM, Benjamin Bellec wrote:
>> Currently, the temperature is displayed with a "%" symbol in
>> gallium/hud, which is quite odd.
>> Marek suggested to only change the value "100" to ano
Hi Glenn,
The series doesn't apply on current master.
Regard.
--
Benjamin
2015-10-07 19:00 GMT+02:00 Ilia Mirkin :
> On Wed, Oct 7, 2015 at 12:53 PM, Marek Olšák wrote:
> > On Wed, Oct 7, 2015 at 5:17 PM, Glenn Kennard
> wrote:
> >> Signed-off-by: Glenn Kennard
Le 07/10/2015 19:13, Glenn Kennard a écrit :
> On Wed, 07 Oct 2015 19:04:15 +0200, Benjamin Bellec
> wrote:
>
>> Hi Glenn,
>>
>> The series doesn't apply on current master.
>>
>> Regard.
>>
>
> It's not meant to apply directly on
e_drawing);
There is 5 instances like that.
Regards.
- Benjamin
2014-11-08 23:52 GMT+01:00 Glenn Kennard :
> Requires evergreen/cayman, and updated radeon kernel module.
>
> Signed-off-by: Glenn Kennard
> ---
> See also kernel side patch sent to d
Hello,
Did you tried with R600_DEBUG=nosb ?
Regards.
Le 19 nov. 2014 07:18, "Dave Airlie" a écrit :
> Hey,
>
> I got piglit to run on my laptop rv635,
>
> two things seem to hang it
>
> geometry shader texel fetches (wierd incorrect values for 2D, hangs for
> 2DMS)
> and
> the clip scissor msaa b
Hello,
This error is due to the recent removal of "--disable-64-bit" and
"--enable-32-bit" options.
You should now replace them by "--build=i686-pc-linux-gnu" and
"--host=i686-pc-linux-gnu"
See here :
http://lists.freedesktop.org/archives/mesa-dev/2014-A
Hello,
I would like to have an access to the RadeonProgram web page
(http://www.x.org/wiki/RadeonProgram/) in order to update and improve it.
I requested this access 9 months ago to jrayh...@freedesktop.org and
tfh...@freedesktop.org and still have no access.
- Benjamin
No problem with Valley on Evergeen (CYPRESS).
Unigine Valley 1.0 (64-bit) Basic preset without "forcedma" :
Benchmark results:
Time:188.792
Frames:9194
FPS:48.6992
Min FPS:20.5505
Max FPS:78.946
Score:2037.5
Unigine Valley 1.0 (64-bit) Basic preset with "forcedma" :
Benchm
and stream !
2014-09-07 22:47 GMT+02:00 Andy Furniss :
> Benjamin Bellec wrote:
>
>> No problem with Valley on Evergeen (CYPRESS).
>>
>> Unigine Valley 1.0 (64-bit) Basic preset without "forcedma" :
>>
>
> It doesn't produce them on basic for m
I'm working on radeonsi support in the Chrome OS Android container
(ARC++). Mesa in ARC++ uses autotools instead of Android.mk, but all
the necessary EGL bits are there, so the existing check is too strict.
Signed-off-by: Benjamin Gordon
---
configure.ac | 3 ++-
1 file changed, 2 inser
On Mon, Oct 23, 2017 at 11:10 AM, Eric Engestrom
wrote:
> On Friday, 2017-10-20 15:34:57 -0600, Benjamin Gordon wrote:
> > I'm working on radeonsi support in the Chrome OS Android container
> > (ARC++). Mesa in ARC++ uses autotools instead of Android.mk, but all
> >
On Wed, Jun 13, 2018 at 9:46 AM Dylan Baker wrote:
> Quoting Eric Engestrom (2018-06-13 03:03:25)
> > On Tuesday, 2018-06-12 11:19:40 -0600, bmgor...@chromium.org wrote:
> > > From: Benjamin Gordon
> > >
> > > When building the Chrome OS Android container, w
A current incomplete framebuffer was incorrectly used as a
st_framebuffer. When accessing st_framebuffer childs bad things happen:
e.g. st_framebuffer::iface was used to check whether its an incomplete
fb, instead we need to compare st_framebuffer::Base against
mesa_get_incomplete_framebuffer.
Bug
So we can use the wayland scanner makro, which is way
better than our previous runtime-pkgconfig hack.
---
configs/autoconf.in |7 ---
configs/default |7 ---
configure.ac | 24 +++--
src/egl/dri
u got better ideas?
2012/1/26 Stéphane Marchesin :
> Fails here too (fedora 15).
>
> Stéphane
>
>
> On Thu, Jan 26, 2012 at 01:35, Jose Fonseca wrote:
>> Benjamin,
>>
>> I'm getting build failures with your commit:
>>
>> + ./autogen.sh --prefix=/usr
Thanks, commited.
2012/1/29 :
> From: Juan Zhao
>
> to fix the header file missing when building wayland-egl
> ---
> src/egl/wayland/wayland-egl/Makefile.am | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/src/egl/wayland/wayland-egl/Makefile.am
> b/src/egl/wayla
Prior commit 576161289df68eedade591fbca4013329c9e5ded,
the parameter format was bpp, thus both 24bit and 32bit formats were
requested with format set to 32. Handle 24bit seperately now.
Fixes RGBX formats in wayland platform for egl_dri2 (EGL_ALPHA_SIZE=0).
Note: This is a candidate for the 8.0 b
The patch has autogenerated and manual changes mixed, that should
always be avoided.
Changes done by "make de.po" should be a first commit, then as a
second your changes.
Additionally options.h needs to be regenerated, use make
PYTHON2=python options.h
(of course as another commit again, but that
It looks like this should be converted to a new dri_drawable hook
(like allocate_textures),
so the the drisw specific bits would be in sw/drisw.c (the hook in
drm/dri2.c would just be a noop).
This is the way the other stuff in st/dri is implemented and
dri_drawable.c wouldnt need to be cloned.
20
In preparation for wayland ALPHA_FORMAT usage,
see commit 7b1d94e5d1f53ac5f59000176aea1d02fc9a1181.
Changes:
- New native_config surface_type bit: alpha_format_pre_bit
- Introduction of native_surface_attribs
(as new parameter in create_{window,pixmap}_surface)
- New attrib alpha_format in n
2011/9/2 Chia-I Wu :
> On Thu, Sep 1, 2011 at 5:53 PM, Benjamin Franzke
> wrote:
>> In preparation for wayland ALPHA_FORMAT usage,
>> see commit 7b1d94e5d1f53ac5f59000176aea1d02fc9a1181.
>>
>> Changes:
>> - New native_config surface_type bit: alpha_
configs = MALLOC(display->num_configs * sizeof(*configs));
> if (configs) {
> - configs[0] = &display->config[0].base;
> - configs[1] = &display->config[1].base;
> + for (i = 0; i < display->num_configs; i++) {
> + configs[i] = &displ
2011/9/8 Chia-I Wu :
> From: Chia-I Wu
>
> Return true for NATIVE_PARAM_PREMULTIPLIED_ALPHA when all formats with
> alpha support premultiplied alpha. Currently, it means when argb32 and
> argb32_pre are both supported.
> ---
> .../state_trackers/egl/wayland/native_drm.c | 8 ++--
>
2011/9/8 Chia-I Wu :
> On Thu, Sep 8, 2011 at 3:11 PM, Benjamin Franzke
> wrote:
>> First thanks for taking this on.
>>
>> There are some things I'd like to have addtionally/differently:
>>
>> Supported shm formats are exposed via a "format" ev
2011/9/8 Chia-I Wu :
> On Thu, Sep 8, 2011 at 3:13 PM, Benjamin Franzke
> wrote:
>> 2011/9/8 Chia-I Wu :
>>> From: Chia-I Wu
>>>
>>> Return true for NATIVE_PARAM_PREMULTIPLIED_ALPHA when all formats with
>>> alpha support premultiplied alpha. Cu
This reverts commit 02f1b50987c0d24da3dcc36dbb44821c20d0660c.
CC: Stuart Abercrombie
CC: Stéphane Marchesin
---
src/gallium/state_trackers/dri/sw/dri_drawable.c | 50 +++---
1 files changed, 7 insertions(+), 43 deletions(-)
diff --git a/src/gallium/state_trackers/dri/sw/dri_d
This reverts commit 569bde1fa7d03fb7688d0d391b32e61e857ad44e.
CC: Stuart Abercrombie
CC: Stéphane Marchesin
---
src/gallium/state_trackers/dri/sw/dri_drawable.c | 270 +-
1 files changed, 1 insertions(+), 269 deletions(-)
mode change 100644 => 12 src/gallium/state_trac
This is a cleanup of commit 02f1b50987c0d24da3dcc36dbb44821c20d0660c.
Update tex buffer using a dri_drawable hook from implemented in sw/drisw.c.
This saves us the duplication of dri_drawable.c.
CC: Stuart Abercrombie
CC: Stéphane Marchesin
---
.../state_trackers/dri/common/dri_drawable.c
is a maximum of 4 enabled render targets for 8-sample
surfaces.
Is this related ?
I don't know what is 8-sample surfaces... so I don't know if the
attached fix is as simple as this.
Regards.
Benjamin Bellec
diff --git a/src/gallium/drivers/r600/r600_pipe.c b/src/gallium/drivers/r600/r
wrote some words on this (in french) here:
http://forums.fedora-fr.org/viewtopic.php?id=51681
Its still updated despite the title of the topic.
> But the ultimate first question would be: where is the bug tracker? :)
https://bugs.freedesktop.org/
Benjamin
iling mesa thanks to the site Benjamin Bellec gave to
> me http://forums.fedora-fr.org/viewtopic.php?id=51681
> And with this simple command line:
>
> $ ./autogen.sh --with-gallium-drivers=nouveau --with-dri-drivers=
>
> Everything went well. (I'm not sure about the empty param
2011/10/18 chris :
> Dear
>
> Could anyone tell me where is the "wl_drm_create" function defined? I can't
> find it anywhere, no definition can be found. No google information can be
> searched! Why?
Seems like you're compiling mesa 7.11 again wayland master, but they
are incompatible.
The wl_drm_
2011/11/2 George Sapountzis :
> Hi,
>
> [please cc: me in any replies]
>
> I rebased (with minor cleanups) the patches submitted by Benjamin
> Franzke and put them at:
Thanks for reminding. I wanted to wait for feedback, but there was no
answer (same as for my comments on the o
I think in i915g the CURSOR flag should be used in i9x5_scanout_layout
for the "special case for cursors" as well, instead of only checking
only pt->width0 == 64 && pt->height0 == 64.
2011/5/2 Daniel Vetter :
> On Sun, May 1, 2011 at 11:59 PM, Marcin Slusarz
> wrote:
>> We need to distinguish sur
*wayland_bufmgr;
};
/**
diff --git a/src/gallium/state_trackers/egl/common/native_wayland_bufmgr.h
b/src/gallium/state_trackers/egl/common/native_wayland_bufmgr.h
new file mode 100644
index 000..b29fd15
--- /dev/null
+++ b/src/gallium/state_trackers/egl/common/native_wayland_bufmgr.h
---
src/gallium/state_trackers/dri/drm/dri2.c |9 +
1 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/src/gallium/state_trackers/dri/drm/dri2.c
b/src/gallium/state_trackers/dri/drm/dri2.c
index 30326a2..e471e8e 100644
--- a/src/gallium/state_trackers/dri/drm/dri2.c
+++ b/
---
.../state_trackers/egl/common/egl_g3d_image.c |5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/src/gallium/state_trackers/egl/common/egl_g3d_image.c
b/src/gallium/state_trackers/egl/common/egl_g3d_image.c
index e1c8316..34bb92c 100644
--- a/src/gallium/state_t
---
If these checks are not sufficent see this as feature request. :)
---
src/gallium/drivers/r600/r600_texture.c | 17 +++--
1 files changed, 11 insertions(+), 6 deletions(-)
diff --git a/src/gallium/drivers/r600/r600_texture.c
b/src/gallium/drivers/r600/r600_texture.c
index 7ffe
uot;, the result is stored in a GLuint. But not in
"via_tex.c"
Here is a patch to merge it, at least in "src/mesa/main/teximage.c". I
will understand any comments, especially on the relevance of this patch.
(The patch still uses a integer parameter).
Regards.
Benjamin Bellec
diff --g
ference is
even bigger.
There is no piglit regressions.
But there is an issue, during compilation there is an error (-Wall)
"warning: right shift count >= width of type [enabled by default]".
Regards.
Benjamin Bellec
diff --git a/src/gallium/auxiliary/util/u_math.h b/src/gallium/a
== 0
My own tests (on a Core2) shows that it's less efficient to do that, at
least with -O2 optimization enabled. With -O0, it's equal.
So here is a v2 patch with a builtin GCC optimization which is the
fastest (thx Matt to point me to this solution).
Regards.
Benjamin Bellec
diff --git a/sr
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
SET definition.
Hope this suits the need for driver loaders, but also driver implementations.
Benjamin Franzke (6):
Add i915 & i965 pci id lists
Add radeon pci id lists
egl_dri2: Use external driver pci list
targets/egl: Support driver name lookup using pci lists
radeon: Use pciid
---
src/mesa/drivers/dri/intel/i915_pci_ids.h | 19 +++
src/mesa/drivers/dri/intel/i965_pci_ids.h | 27 +++
2 files changed, 46 insertions(+), 0 deletions(-)
create mode 100644 src/mesa/drivers/dri/intel/i915_pci_ids.h
create mode 100644 src/mesa/drive
---
src/mesa/drivers/dri/radeon/r200_pci_ids.h | 22 +++
src/mesa/drivers/dri/radeon/r300_pci_ids.h | 218 +
src/mesa/drivers/dri/radeon/r600_pci_ids.h | 261 ++
src/mesa/drivers/dri/radeon/radeon_pci_ids.h | 23 +++
4 files changed, 524 inse
Make use of this in drm and wayland st/egl backends.
---
src/gallium/state_trackers/egl/drm/native_drm.c| 33 +
.../state_trackers/egl/wayland/native_drm.c| 40 +--
src/gallium/targets/egl/egl.c | 80 +++-
src/gallium/targets/e
---
src/mesa/drivers/dri/radeon/radeon_chipset.h | 499 +-
1 files changed, 8 insertions(+), 491 deletions(-)
diff --git a/src/mesa/drivers/dri/radeon/radeon_chipset.h
b/src/mesa/drivers/dri/radeon/radeon_chipset.h
index bd23662..0e51325 100644
--- a/src/mesa/drivers/dri
---
src/gallium/winsys/r600/drm/Makefile |1 +
src/gallium/winsys/r600/drm/radeon_pciid.c | 486 +---
2 files changed, 6 insertions(+), 481 deletions(-)
diff --git a/src/gallium/winsys/r600/drm/Makefile
b/src/gallium/winsys/r600/drm/Makefile
index 7310734..45fd
---
src/egl/drivers/dri2/Makefile |2 +-
src/egl/drivers/dri2/common.c | 110 ++
src/egl/drivers/dri2/egl_dri2.h |2 +
src/egl/drivers/dri2/pci_ids.h | 62
src/egl/drivers/dri2/platform_drm.c | 663 +--
5 files changed, 176
Well radeon_drm_public.h declares radeon_drm_winsys_create(),
but yea is_r3xx should be replaced.
Patch attached.
2011/6/6 Alex Deucher :
> On Mon, Jun 6, 2011 at 11:49 AM, Benjamin Franzke
> wrote:
>> We need pci id to driver-name mapping for drm and
>> wayland platfor
Le 06/06/2011 17:34, Roland Scheidegger a écrit :
> Am 05.06.2011 03:55, schrieb Benjamin Bellec:
>> 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 :
>>>>
2011/6/6 Alex Deucher :
> Looks good. Thanks. Overall the patch set looks fine to me. I agree
> with Marek that it might be better to put the pci ids together
> somewhere. For the series:
>
> Reviewed-by: Alex Deucher
Ok, moved the lists into include/pci_ids/, hope thats ok.
Updated patch se
2011/6/6 Alex Deucher :
> Sorry, I just thought of one tricky situation. Only r600g supports
> CAYMAN asics, so r600c shouldn't have the CAYMAN pci ids. Maybe just
> split the CAYMAN ids out into a new header, cayman_pci_ids.h, and
> include both r600_pci_ids.h and cayman_pci_ids.h in r600g and o
2011/6/6 Eric Anholt :
> These 4 chipsets aren't part of the i915 driver.
>
> Other than that, thanks for taking this on! Looks like much more
> sanity, and we've talked about using something like the third argument
> to avoid some of the if trees we've got around.
>
Ok, thanks updated.
If there
x86_64_entry_start needs to be bound global, in order to have the
correct address in entry_get_public (seems not to be needed on x86).
Otherwise addresses needed for _glapi_proc_address will be computed
from some random offset (0x6400229a61058b48 in my case).
---
src/mapi/mapi/entry_x86-64_tls.h
d by
Jeff Chua I actually forgot the conditional for it.
As I dont have commit access, would someone push these fixes, available at:
git://anongit.freedesktop.org/~bnf/mesa egl_dri2-fixes
Benjamin Franzke (3):
egl_dri2: Use libudev only if available
egl_dri2: Add missing license
Le 06/06/2011 23:54, Roland Scheidegger a écrit :
> Am 06.06.2011 23:18, schrieb Tormod Volden:
>> On Sun, Jun 5, 2011 at 1:14 AM, Benjamin Bellec wrote:
>>> So here is a v2 patch with a builtin GCC optimization which is the
>>> fastest (thx Matt to point me to this solu
Le 06/06/2011 23:54, Roland Scheidegger a écrit :
> Also I believe this builtin requires gcc 3.4 - not sure though if the
> rest of the code compiles on older gcc.
>
> Roland
Yes I checked, __builtin_clz() appeared first time in GCC 3.4.0
documentatio
clared as extern, the compiler will generate code
to lookup a global object, not a local.
By declaring x86_64_entry_start as static the correct local address is
loaded here.
Patch attached.
From 7ca323bf472004066f35dbec51791c90a671a42e Mon Sep 17 00:00:00 2001
From: Benjamin Franzke
Date: Wed,
On a Roland Scheidegger idea!
Benjamin
diff --git a/src/mesa/main/teximage.c b/src/mesa/main/teximage.c
index 3e42911..da2e72f 100644
--- a/src/mesa/main/teximage.c
+++ b/src/mesa/main/teximage.c
@@ -86,6 +86,12 @@ _mesa_free_texmemory(void *m)
static GLuint
logbase2(GLuint n)
{
+#if defined
ring gcc 3.4 we should fix that otherwise could just make gcc
> 3.4 a requirement.
Ok I did'nt know that.
Here is my 3 patchs fixed. I hope this are the last of the series :-)
Benjamin
diff --git a/src/gallium/auxiliary/util/u_math.h b/src/gallium/auxiliary/util/u_math.h
index 65a99f
2011/6/11 Chia-I Wu :
> I got this warning after applying the patch
>
> ../../../src/mapi/mapi/entry_x86-64_tls.h:69:1: warning:
> ‘x86_64_current_tls’ used but never defined
>
> Maybe the function declaration should be decorated with "extern"?
Yeah, should work at least, just checked that caling
Removing this flag seems right to me, but always building the egl
state tracker when gallium and egl is enabled not.
So when --with-state-trackers is also removed, we'd need a new
--with-egl-drivers=auto|gallium,dri2,glx or so.
The usecase is that you could choose to use st/dri together with
egl_d
---
.../state_trackers/dri/common/dri_context.c|5 +
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/src/gallium/state_trackers/dri/common/dri_context.c
b/src/gallium/state_trackers/dri/common/dri_context.c
index e23c1bc..0f9e4e5 100644
--- a/src/gallium/state_track
1 - 100 of 202 matches
Mail list logo
