On Wed, May 30, 2012 at 03:50:37PM -0300, Luiz Capitulino wrote: > On Mon, 21 May 2012 17:59:53 +0100 > "Daniel P. Berrange" <berra...@redhat.com> wrote: > > @@ -4564,6 +4709,11 @@ static void sortcmdlist(void) > > * End: > > */ > > > > +void monitor_global_init(void) > > +{ > > It's better to call it monitor_early_init() (or monitor_init_early()).
This was a bit of stupidity on my part. The existing 'monitor_init' function (which is called per monitor instance), already has a code block where one-time global initialization is done. So I'm removing this function in the next patch posting. Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|