<div dir="ltr">It seems I can make the vbox driver work by changing the registration order in daemonInitialze. So, I can do test with it for now.<div>But I still don't understand why libvirt keep the vbox network and storage driver. I don't find anyway else than libvirtd that uses vbox driver.</div>
<div>Meanwhile, the daemonInitialize registers general driver prior to vbox driver, which makes the vbox driver actually unused.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2014-08-15 17:23 GMT+08:00 Daniel P. Berrange <span dir="ltr"><<a href="mailto:berrange@redhat.com" target="_blank">berrange@redhat.com</a>></span>:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Fri, Aug 15, 2014 at 11:06:18AM +0200, Michal Privoznik wrote:<br>
> Dear list,<br>
><br>
> Virtualbox driver has its own implementation of storage and network<br>
> sub-drivers. But as of commit ba5f3c7c8ecc1037e44904916989a1c65777a9d5<br>
> (contained in the 1.0.6 release) when the VBox moved from client to daemon,<br>
> the storage and network sub-drivers are indeed registered but in fact never<br>
> called. It's due to our virConnectOpen function where the general network<br>
> and storage drivers take precedence. So I guess my question is: should we<br>
> drop the VBox sub-drivers or perhaps fix the virConnectOpen function?<br>
<br>
</div></div>It should simply be a matter of changing the order of registration. ie<br>
we want to call vboxRegister before storageRegister, so that the vbox<br>
storage driver is first in the list. The vbox storage driver is written<br>
so it is a no-op unless the primary virt driver is vbox.<br>
<br>
<br>
Regards,<br>
Daniel<br>
<span class="HOEnZb"><font color="#888888">--<br>
|: <a href="http://berrange.com" target="_blank">http://berrange.com</a>      -o-    <a href="http://www.flickr.com/photos/dberrange/" target="_blank">http://www.flickr.com/photos/dberrange/</a> :|<br>
|: <a href="http://libvirt.org" target="_blank">http://libvirt.org</a>              -o-             <a href="http://virt-manager.org" target="_blank">http://virt-manager.org</a> :|<br>
|: <a href="http://autobuild.org" target="_blank">http://autobuild.org</a>       -o-         <a href="http://search.cpan.org/~danberr/" target="_blank">http://search.cpan.org/~danberr/</a> :|<br>
|: <a href="http://entangle-photo.org" target="_blank">http://entangle-photo.org</a>       -o-       <a href="http://live.gnome.org/gtk-vnc" target="_blank">http://live.gnome.org/gtk-vnc</a> :|<br>
</font></span></blockquote></div><br></div>