[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