[PATCH] server: Clean up VGA Arbiter macros.
Matthieu Herrb
matthieu.herrb at laas.fr
Mon Jul 5 06:29:53 PDT 2010
On Mon, Jul 05, 2010 at 02:05:28PM +0300, Tiago Vignatti wrote:
> 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.
>
I would like to see at least the macro to inline functions part
(patches 1-3 of your series) go in.
But I can live without them. I've a bunch of other local changes
anyways that I never manage to send upstreams because I fail to track
HEAD and at the same time manage the stable versions for OpenBSD.
--
Matthieu Herrb
More information about the xorg-devel
mailing list