[PATCH] server: Clean up VGA Arbiter macros.

Tiago Vignatti tiago.vignatti at nokia.com
Mon Jul 5 04:05:28 PDT 2010


Hi,

On Sun, Jul 04, 2010 at 12:18:21PM +0200, ext Matthieu Herrb wrote:
> On Sun, Jul 04, 2010 at 11:45:46AM +0200, Jamey Sharp wrote:
> > IIRC, Tiago also posted some patch to clean up these macros, and I
> > think I liked that one better. I believe it turned them into inline
> > functions and killed the redundant GC variants. What happened to that
> > patch?
> > 
> 
> I don't know. It even has 2 reviewed-by: tags. 
> 
> I retire my patch if Tiago's one gets committed. But the current 
> code in git doesn't build with gcc 2.95, just because the macros
> are poorly defined.

it's living here:

http://cgit.freedesktop.org/~vignatti/xserver/ [vgaarb-fixes]
git://anongit.freedesktop.org/~vignatti/xserver [vgaarb-fixes]


The reason that Keith didn't pull this branch is due the last patch there,
which he complained that could increase a bit the overhead for being a private
lookup.

Matthieu, let me know if you need this for 1.9 so we can poke Keith,
eventually pulling the remaining patches.


Thanks,
             Tiago


More information about the xorg-devel mailing list