[libvirt] [PATCH] 0/7 host ("node") device enumeration - completed submission

David Lively dlively at virtualiron.com
Fri Nov 7 02:32:17 UTC 2008


Daniel(s) -
  In the attached set of patches, I've merged in DanB's reworking of the
headers, etc. and adjusted my code accordingly.  I've also renamed
"NodeDriver" to "DeviceMonitor", as discussed with DanV, and added a
--cap option to the virsh node-list-devices command.

Dave


On Tue, 2008-11-04 at 14:30 +0100, Daniel Veillard wrote:
> On Mon, Oct 27, 2008 at 02:46:47PM -0400, David Lively wrote:
> > Since the version of this I posted last week still had some outstaning
> > TODO items that I have since finished, I'm re-submitting the whole
> > thing.  I consider this a "complete submission" since I don't think it's
> > necessary to implement any additional functionality to make this
> > acceptable.  (The two remaining unimplemented items are
> > virNodeDeviceCreate / Destroy and a real Devkit impl, neither of which
> > needs to be done immediately, for reasons discussed earlier.)
> > 
> > These patches implement the node device enumeration functionality, as
> > discussed here:
> > https://www.redhat.com/archives/libvir-list/2008-September/msg00398.html
> > 
> > I've broken it into the following pieces:
> >   1-public-api                  additions to the public API
> >   2-internal-api                additions to the internal API
> >   3-local-node-drivers          the HAL & DeviceKit implementations
> >   4-remote-node-driver          the remote driver
> >   5-virsh-support               virsh support
> >   6-python-bindings             python bindings
> >   7-java-bindings		Java bindings (for libvirt-java)
> 
>   Hi Dave,
> 
> Just in case you're wondering why there is no comment on those,
> I don't think there is a problem with the patches, my take is that we
> need to commit first the modularization fron Dan Berrange before pushing
> your patches due to HAL licencing.
> I intend to push them before the 0.5.0 release next week, so unless
> we get more comment on your patches, the current state will be used as
> the base.
> If you could just check that commit done in the meantime don't clash
> or change behaviour that would be nice,
> 
>   thanks in advance !
> 
> Daniel
> 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: host-dev-enum-1-public-api.patch
Type: text/x-patch
Size: 21917 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20081106/8ef68d2f/attachment-0006.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: host-dev-enum-2-internal-api.patch
Type: text/x-patch
Size: 31912 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20081106/8ef68d2f/attachment-0007.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: host-dev-enum-3-local-node-drivers.patch
Type: text/x-patch
Size: 55675 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20081106/8ef68d2f/attachment-0008.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: host-dev-enum-4-remote-node-driver.patch
Type: text/x-patch
Size: 30068 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20081106/8ef68d2f/attachment-0009.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: host-dev-enum-5-virsh-support.patch
Type: text/x-patch
Size: 3553 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20081106/8ef68d2f/attachment-0010.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: host-dev-enum-6-python-bindings.patch
Type: text/x-patch
Size: 12038 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20081106/8ef68d2f/attachment-0011.bin>


More information about the libvir-list mailing list