Re: [PATCH] virt-manager: 'Add Physical Host Device' wizard

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

 



On Sun, Mar 08, 2009 at 07:34:46PM -0400, Cole Robinson wrote:
> Hi all,
> 
> The attached patch adds an option in the 'Add Hardware' wizard to attach
> a physical host device to a VM. Current explicitly supported options are
> USB, PCI, and Wired NICs (which really just attaches the parent PCI
> device, but this way provides a nicer interface).
> 
> Some screenshots:
> 
> http://fedorapeople.org/~crobinso/virt-manager/vmm-add-hostdev1.png
> http://fedorapeople.org/~crobinso/virt-manager/vmm-add-hostdev2.png
> http://fedorapeople.org/~crobinso/virt-manager/vmm-add-hostdev3.png
> 
> It's all pretty straightforward: most of the work is done at the
> virtinst level to pull the useful info out of the nodedev xml.

I'm not sure I like having 'Wired NICs' as a separate option. How about
listing the logic device type against each PCI / USB devices.

eg, in the drop down for PCI devices, alongside the POCI device
description, also include the nameof the logical device (eth1)
associated with it. This lets you see that assigning PCI device
'X.y.z' means the host will no longer be able to use the device
'eth1'.  I think it would help people to avoid shooting themselvs
in the foot when choosing a PCI/USB device

Regards,
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 :|

_______________________________________________
et-mgmt-tools mailing list
et-mgmt-tools@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/et-mgmt-tools

[Index of Archives]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux