Re: New style CDROM config in Xen 3.0.3

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

 



On Fri, Aug 25, 2006 at 05:56:58PM +0100, Daniel P. Berrange wrote:
> So in a recent change to xen-unstable.hg the way CDROM devices are configured
> for HVM guests changed. They are now treated as regular IDE devices, and a
> CDROM is signified by presence of ':cdrom' suffix on the device name
> 
>   http://lists.xensource.com/archives/html/xen-devel/2006-08/msg00369.html
> 
> libvirt obviously needs to support both this new style config, and the old
> (<= 3.0.2) style configs. Thus I requested addition of 'xend_config_format'
> attribute in the /xend/node SEXPR data from XenD. Christian recently
> committed this patch, so we can now support both CDROM style configs in
> libvirt. I am attaching a patch to implement this support - basically it
> touches xend_internal.c to fix XML->SEXPR conversion, and xml.c to fix
> SEXPR->XML conversion. Then this is corresponding additions to the unit
> tests to validate the new code, and also validate that I didn't break
> existing code.
> 
> I am attaching the patches, but I don't intend to commit them to libvirt
> just yet because I don't actually have the new 3.0.3 xend running on my
> test systems fully yet. If anyone is using 3.0.3 any testing would be
> appreciated, otherwise I aim to test this towards end of next week.

Jeremy recently tested this after upgrading Fedora to Xen 3.0.3 codebase
pre-release & verified that HVM cdrom config is handled correctly. Thus
I've committed the patches for new style cdrom config

Regards,
Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=| 


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]