[PATCH 20/42] Move DGA from extmod to built-in
Daniel Stone
daniel at fooishbar.org
Wed Jan 4 10:44:24 PST 2012
Hi,
On Fri, Dec 02, 2011 at 07:17:24PM -0200, Paulo Zanoni wrote:
> 2011/12/2 Daniel Stone <daniel at fooishbar.org>:
> > @@ -90,6 +104,9 @@ load_extension_config(void)
> > }
> > #endif
> > }
> > +
> > + for (i = 0; i < ARRAY_SIZE(extensionModules); i++)
> > + LoadExtension(&extensionModules[i], TRUE);
> > }
> >
> > void
>
> With this statement, Xorg will try to load its modules twice. Maybe it
> was added by accident? Don't forget to remove the "int i" declaration
> too.
>
> Look: http://cgit.freedesktop.org/~daniels/xserver/tree/hw/xfree86/common/xf86Extensions.c?h=extension-cleanup&id=2c4cb57dfa2e37ef910be298c2bcbb9731e5f137#n108
>
> xf86ExtensionInit will call load_extension_config, which will do that
> loop quoted above (line 108). Then, at line 119 xf86ExtensionInit will
> run the same loop again.
Looks like I only responded to this on IRC, oops. I've fixed this a
while ago now.
> I cloned your full git tree, and Xorg was not starting because of
> that. After I removed exactly the lines I quoted, Xorg starts and I
> can login and play OpenArena (but I didn't test too much). I also
> checked that simple Xorg startup/shutdown without clients doesn't
> introduce any new memory leaks, and it also accidentaly fixes a memory
> leak because you removed LoaderSortExtensions.
> As a side note, don't we get a faster server startup after all this?
> Did anyone measure?
>
> I'll keep running your tree to see if I find anything else.
Thanks. :) I haven't measured startup times, no; I'll try to get
something concrete later.
Cheers,
Daniel
More information about the xorg-devel
mailing list