[PATCH 0/8] Yet more loader cleanups

Alan Coopersmith alan.coopersmith at oracle.com
Tue Sep 21 16:25:07 PDT 2010


Adam Jackson wrote:
> Largely cosmetic cleanups.  Removes a few layers of abstraction that
> resulted in double-tracking reference counts and other such hilarity.
> I think it ends up being approximately equivalent in terms of heap
> usage, since it requires keeping track of the loaded module path in
> case you ever call DuplicateModule on it... and you do.  I have a
> plan for eliminating that too but it's not going to be quite as easy.
> 
> The "unused refcounting from input drivers" commit is technically at
> least an API break and possibly an ABI break for input, but not in
> any way that seems to matter.  I started the server and evdev worked
> without a rebuild, so.
> 
> ---
> 
> The following changes since commit c768cdda92696b636c10bb2df64167d5274b4b99:
> 
>   Merge remote branch 'jamey/reviewed' (2010-09-13 16:48:33 -0700)
> 
> are available in the git repository at:
> 
>   ssh://people.freedesktop.org/~ajax/xserver.git less-loader
> 
> Adam Jackson (8):
>       loader: Merge dlloader directly into the loader
>       loader: Remove unused module serial number
>       loader: Remove unused canonical name field
>       loader: Remove the handle field from LoaderOpen
>       loader: include cleanup
>       loader: Remove a silly layer of reference counting
>       xfree86: Remove unused refcounting from input drivers
>       xfree86: Remove useless module setup functions where appropriate

Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>

-- 
	-Alan Coopersmith-        alan.coopersmith at oracle.com
	 Oracle Solaris Platform Engineering: X Window System



More information about the xorg-devel mailing list