Merging aiglx
David Reveman
davidr at novell.com
Thu Mar 16 04:46:37 PST 2006
On Wed, 2006-03-15 at 12:36 -0500, Kristian Høgsberg wrote:
> David Reveman wrote:
> > On Mon, 2006-03-06 at 14:32 -0500, Kristian Høgsberg wrote:
> >> Hello,
> >>
> >> It's been a couple of weeks since I posted my aiglx patch and there
> >> hasn't been a lot of feedback, so I'm going to go ahead and assume
> >> everybody is happy with the approach. In the meantime, we've fixed a
> >> number of crashers, added better logging and added hooks to allow the
> >> GLXprovider to implement the GLX_EXT_tfp extension.
> >>
> >> Unless there are any objections to this, I'll merge the aiglx branch to
> >> head this friday (yay, one less branch to keep in sync with mesa).
> >
> >
> > OK, I updated the xgl branch earlier today. Seams to be working alright but I
> > haven't done much testing yet.
> >
> > I had to change a few things to make it work with Xgl. I've attached the
> > current diff between the xgl and aiglx branches and you can find a few
> > short comments to this diff below.
>
> Thanks, I've merged the first batch of changes, I'm going through the
> rest of the patch now. Once I get that committed, are you planning to
> merge Xgl to head?
Great.
Yes, once the GLX code is in sync it shouldn't be too hard to merge Xgl
into head.
>
> On a related note, I saw this commit:
>
> http://webcvs.freedesktop.org/xorg/xserver/xorg/GL/mesa/X/xf86glx.c?r1=1.5.8.5&r2=1.5.8.6
>
> and I don't understand why you need the GlxFlushContextCache() calls.
> In the loseCurrent case, StopUsingContext(prevglxc) is called later in
> DoMakeCurrent which clear the context cache. In the destroy context
> case, ctx->destroy() is only called from __glXFreeContext() (except for
> DoCreateContext) where the context cache is also cleared.
Yes, I thought I didn't needed them either. I had them there before I
switched to your new GLX code and it turns out it doesn't work without
them so I put them back for now until I have time to figure out why it
doesn't work.
-David
More information about the xorg
mailing list