[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

[libvirt] Re: [PATCH 0/12] Standardized device addressing & SCSI controller/disk hotplug



On Tue, Dec 15, 2009 at 01:06:51PM +0100, Wolfgang Mauerer wrote:
> Hi,
> 
> Daniel P. Berrange wrote:
> > This patch series is a combination of series done by
> > Wolfgang Mauerer to support proper SCSI drive hotplug
> > and new work by myself to introduce generic addressing
> > for all devices.
> >
> > Wolfgang's most recent posting was
> >
> > http://www.redhat.com/archives/libvir-list/2009-November/msg00574.html
> > http://www.redhat.com/archives/libvir-list/2009-November/msg00701.html
> >
> > When testing that series I came across a few minor issues,
> > but more importantly it made me realize how important it is
> > that we introduce explicit device addressing in our XML format.
> > (...)
> 
> essentially, this all looks good to me - thanks for the extensions.
> Unfortunately, I was not yet successful in fully testing the code
> because I have some issues with the underlying qemu that prevent my
> machines from booting correctly with recent qemu-kvms. However, I came
> across a small issue: When certain PCI devices are not present in the
> system, libvirt can crash at startup. The attached patch fixes this.

Hmm, which particular device did you see it crash on? These
functions are only called with 'def' straight out of the
virDomainDefPtr struct, (eg, vm->def->disks[INDEX]) and those
are all supposed to be non-NULL at all times. So if one is NULL
then I've introduced a bug somewhere else ! I guess most likely
in the new controllers code.

Daniel
-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]