9:02 a.m.
... or at least I can't see it being called anywhere.
Rich.
--
Emerging Technologies, Red Hat http://et.redhat.com/~rjones/
64 Baker Street, London, W1U 7DF Mobile: +44 7866 314 421
"[Negative numbers] darken the very whole doctrines of the equations
and make dark of the things which are in their nature excessively
obvious and simple" (Francis Maseres FRS, mathematician, 1759)
9:12 a.m.
On Fri, Jan 26, 2007 at 03:02:01PM +0000, Richard W.M. Jones wrote:
... or at least I can't see it being called anywhere.
It should probably, currently only xenHypervisorInit is hooked,
and we call it as part of the Open() ... that's not very clean though
I'm not sure it should be called directly from virInitialize() as we
may need only a subset of the available drivers in a given session.
Daniel
--
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard | virtualization library http://libvirt.org/
veillard(a)redhat.com | libxml GNOME XML XSLT toolkit http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/
10:06 a.m.
On Fri, Jan 26, 2007 at 10:12:09AM -0500, Daniel Veillard wrote:
On Fri, Jan 26, 2007 at 03:02:01PM +0000, Richard W.M. Jones wrote:
>
> ... or at least I can't see it being called anywhere.
It should probably, currently only xenHypervisorInit is hooked,
and we call it as part of the Open() ... that's not very clean though
I'm not sure it should be called directly from virInitialize() as we
may need only a subset of the available drivers in a given session.
I don't see any particular need for a special 'init' method in the driver
API - as you say the only impl is for xenHypervisorInit and that's called
directly by the Open() as needed. So I vote for killing the 'init' driver
method.
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 -=|
2:22 a.m.
Daniel P. Berrange wrote:
On Fri, Jan 26, 2007 at 10:12:09AM -0500, Daniel Veillard wrote:
> On Fri, Jan 26, 2007 at 03:02:01PM +0000, Richard W.M. Jones wrote:
>> ... or at least I can't see it being called anywhere.
> It should probably, currently only xenHypervisorInit is hooked,
> and we call it as part of the Open() ... that's not very clean though
> I'm not sure it should be called directly from virInitialize() as we
> may need only a subset of the available drivers in a given session.
I don't see any particular need for a special 'init' method in the driver
API - as you say the only impl is for xenHypervisorInit and that's called
directly by the Open() as needed. So I vote for killing the 'init' driver
method.
I'm deep in the remote backend at the moment, but I'll try & get a patch
out to fix this this morning.
Rich.
--
Emerging Technologies, Red Hat http://et.redhat.com/~rjones/
64 Baker Street, London, W1U 7DF Mobile: +44 7866 314 421
"[Negative numbers] darken the very whole doctrines of the equations
and make dark of the things which are in their nature excessively
obvious and simple" (Francis Maseres FRS, mathematician, 1759)
4:27 a.m.
Daniel P. Berrange wrote:
On Fri, Jan 26, 2007 at 10:12:09AM -0500, Daniel Veillard wrote:
> On Fri, Jan 26, 2007 at 03:02:01PM +0000, Richard W.M. Jones wrote:
>> ... or at least I can't see it being called anywhere.
> It should probably, currently only xenHypervisorInit is hooked,
> and we call it as part of the Open() ... that's not very clean though
> I'm not sure it should be called directly from virInitialize() as we
> may need only a subset of the available drivers in a given session.
I don't see any particular need for a special 'init' method in the driver
API - as you say the only impl is for xenHypervisorInit and that's called
directly by the Open() as needed. So I vote for killing the 'init' driver
method.
If we just delete this method, are we breaking our commitment to
ABI-compatibility?
Rich.
--
Emerging Technologies, Red Hat http://et.redhat.com/~rjones/
64 Baker Street, London, W1U 7DF Mobile: +44 7866 314 421
"[Negative numbers] darken the very whole doctrines of the equations
and make dark of the things which are in their nature excessively
obvious and simple" (Francis Maseres FRS, mathematician, 1759)
4:31 a.m.
On Wed, Jan 31, 2007 at 10:27:10AM +0000, Richard W.M. Jones wrote:
Daniel P. Berrange wrote:
>On Fri, Jan 26, 2007 at 10:12:09AM -0500, Daniel Veillard wrote:
>>On Fri, Jan 26, 2007 at 03:02:01PM +0000, Richard W.M. Jones wrote:
>>>... or at least I can't see it being called anywhere.
>> It should probably, currently only xenHypervisorInit is hooked,
>>and we call it as part of the Open() ... that's not very clean though
>>I'm not sure it should be called directly from virInitialize() as we
>>may need only a subset of the available drivers in a given session.
>
>I don't see any particular need for a special 'init' method in the
driver
>API - as you say the only impl is for xenHypervisorInit and that's called
>directly by the Open() as needed. So I vote for killing the 'init' driver
>method.
If we just delete this method, are we breaking our commitment to
ABI-compatibility?
A priori no, it should be completely internal, it's not an exported symbol.
Daniel
--
Red Hat Virtualization group http://redhat.com/virtualization/
Daniel Veillard | virtualization library http://libvirt.org/
veillard(a)redhat.com | libxml GNOME XML XSLT toolkit http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine http://rpmfind.net/
6535
days inactive
6540
days old
5 comments
3 participants
participants (3)
-
Daniel P. Berrange -
Daniel Veillard -
Richard W.M. Jones