Re: [PATCH] Block device and network stats (version 2)

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

 



On Tue, Aug 21, 2007 at 08:10:33AM -0400, Daniel Veillard wrote:

> > > The Xen impl as coded only works for disks named xvdN, because the code for
> > > calculating device ID assumes xvdN device numbering scheme:
> > > 
> > >     device = 202 * 256 + minor;
> > 
> > What's with device numbering at all? Must we really repeat this crazy
> > Linuxism from xend in libvirt :(
> 
>   Do you have an alternative ? Because this is commited in CVS and I 
> would like to get 0.3.2 rolled out ASAP ! So if you have a practical
> suggestion or a patch, please post it, I may delay the issue but
> just stating 'crazy Linuxism' without specifying why and how this can
> be fixed won't help.

I just checked it out, and it looks like it can be safely encapsulated
in this case. I was worried it would end up like xend but it seems we
can entirely wrap the knowledge of this to be Linux-only.

BTW I am a little concerned that we're re-implementing all this code
when libxenstat can do it. I already wrote this code once :)

apologies,
john

--
Libvir-list mailing list
Libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]