Re: [libvirt] Re: [PATCH 3/3] qemu: fix block stats for virtio and scsi

2008-10-17 Thread Daniel Veillard
On Thu, Oct 16, 2008 at 05:29:58PM +0200, Guido Günther wrote: > On Tue, Oct 14, 2008 at 09:29:38AM -0400, Cole Robinson wrote: > > Guido Günther wrote: > > > On Fri, Oct 10, 2008 at 06:05:37PM +0100, Daniel P. Berrange wrote: > > >> Sorry to mess up your patch, but I just committed the code to tur

Re: [libvirt] Re: [PATCH 3/3] qemu: fix block stats for virtio and scsi

2008-10-16 Thread Guido Günther
On Tue, Oct 14, 2008 at 09:29:38AM -0400, Cole Robinson wrote: > Guido Günther wrote: > > On Fri, Oct 10, 2008 at 06:05:37PM +0100, Daniel P. Berrange wrote: > >> Sorry to mess up your patch, but I just committed the code to turn > >> all linked lists into arrays. So you'll need to tweak this to do

[libvirt] Re: [PATCH 3/3] qemu: fix block stats for virtio and scsi

2008-10-14 Thread Cole Robinson
Guido Günther wrote: > On Fri, Oct 10, 2008 at 06:05:37PM +0100, Daniel P. Berrange wrote: >> Sorry to mess up your patch, but I just committed the code to turn >> all linked lists into arrays. So you'll need to tweak this to do >> >> for (i = 0 ; i < vm->def->ndisks ; i++) >> if (STREQ(v

[libvirt] Re: [PATCH 3/3] qemu: fix block stats for virtio and scsi

2008-10-11 Thread Guido Günther
On Fri, Oct 10, 2008 at 06:05:37PM +0100, Daniel P. Berrange wrote: > Sorry to mess up your patch, but I just committed the code to turn > all linked lists into arrays. So you'll need to tweak this to do > > for (i = 0 ; i < vm->def->ndisks ; i++) > if (STREQ(vm->def->disks[i]->dst)) >

[libvirt] Re: [PATCH 3/3] qemu: fix block stats for virtio and scsi

2008-10-10 Thread Daniel P. Berrange
On Fri, Oct 10, 2008 at 05:28:54PM +0200, Guido G?nther wrote: > Hi, > use qemudDiskDeviceName to determine the block device name (as suggested > by Cole). > -- Guido > +disk = vm->def->disks; > +while (disk) { > +if (STREQ(disk->dst, path)) > +break; > +disk =

[libvirt] Re: [PATCH 3/3] qemu: fix block stats for virtio and scsi

2008-10-10 Thread Cole Robinson
Guido Günther wrote: > Hi, > use qemudDiskDeviceName to determine the block device name (as suggested > by Cole). > -- Guido > > From 6985fee585561b04942036d283632e7cb2bb708f Mon Sep 17 00:00:00 2001 > From: Guido Guenther <[EMAIL PROTECTED]> > Date: Thu, 2 Oct 2008 21:12:20 +0200 > Subject: [PA