On 06/27/2010 09:17 AM, Justin Clift wrote: > On 06/25/2010 08:32 AM, Eric Blake wrote: >> From: Eduardo Otubo<otubo@xxxxxxxxxxxxxxxxxx> >> >> Add the storage management driver to the Power Hypervisor driver. >> This is a big but simple patch, it's just a new set of functions. >> >> This patch includes: >> * Storage driver: The set of pool-* and vol-* functions. >> * attach-disk function. >> * Support for IVM on the new functions. >> >> Signed-off-by: Eric Blake<eblake@xxxxxxxxxx> > <snip> > > This commit is causing a segfault here when running "pool-list" in > unmodified virsh. > > virConnectNumOfStoragePools() calls phypNumOfStoragePools(), which calls > phypExec(). In this case with session equaling NULL. virConnectNumOfStoragePools should not call successfully phypNumOfStoragePools unless the connection URI is for phyp. Does this patch clean up the mess? https://www.redhat.com/archives/libvir-list/2010-June/msg00707.html -- Eric Blake eblake@xxxxxxxxxx +1-801-349-2682 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list