[Libvir] PATCH: Support network interface model in Xen and QEMU driver

Soren Hansen soren at ubuntu.com
Thu May 1 01:23:07 UTC 2008


On Wed, Apr 30, 2008 at 11:36:33PM +0100, Daniel P. Berrange wrote:
> WRT to the network interface type attribute, I advised Soren at the
> virt summit in Austin, that since Rich Jones had already posted the
> patch and we'd all basically agreed on syntax it was reasonably to
> include the patch in Ubuntu. It was only a matter of time before we
> merged it - as I have done today.

Thanks very much. That strikes that bit off of my "Stuff I might need to
worry about" list. :)
 
> Now, the disk model syntax supporting virtio is where I agree with Daniel
> that it should have been posted upstream before inclusion in a product Even
> if the code was just a quick hack, not in a state fit for merging - it is
> always beneficial to post as early as possible just  for the sake of 
> visibility & comment.

This is good advice. Thanks.

> This said I believe the proposed 'bus' atribute for disks is the
> optimal way to handle virtio for disks. 

I agree. A <model type='foo' /> element in the disk definition could
still be used to specify which particular SCSI controller you'd like.

> Just for future enhancements please post ideas to this list asap. 

I'll keep that in mind. I'm truly sorry for the stir I've caused and I
have every intention of making sure it won't happen again.

> I myself have posted ideas more than 1 year before actually getting
> around to implementing them, so there's no requirement to follow
> through with code immediately :-)

:) 

-- 
Soren Hansen               | 
Virtualisation specialist  | Ubuntu Server Team
Canonical Ltd.             | http://www.ubuntu.com/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20080501/82ac54ce/attachment-0001.sig>


More information about the libvir-list mailing list