Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-06 Thread Dietmar Maurer
> Where do you put the button to edit iolimit ? on the hdform ? > > Or do you want to put it on main interface near movedisk resizedisk ? maybe a splitbutton for 'Edit', see Ext.button.Split - what do you think? ___ pve-devel mailing list pve-devel@pve

Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-06 Thread Dietmar Maurer
> Or do you want to put it on main interface near movedisk resizedisk ? yes, between Edit and resizedisk. We can try later to optimize that, maybe usinga a menu for Edit? ___ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bi

Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-06 Thread Alexandre DERUMIER
etmar Maurer" À: "Alexandre DERUMIER" Cc: pve-devel@pve.proxmox.com, "Stefan Priebe" Envoyé: Vendredi 6 Décembre 2013 05:57:26 Objet: RE: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max > Do you want a new window ? (Don't if it

Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-05 Thread Dietmar Maurer
> Do you want a new window ? (Don't if it's a good idea to have hdform + > hdlimitform open at the same time in 2 windows) or maybe a new tab could be > better ? A separate window. I do not thing one need to open hdform + hdlimitform at the same time (why?) _

Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-05 Thread Alexandre DERUMIER
dows) or maybe a new tab could be better ? - Mail original - De: "Dietmar Maurer" À: "Alexandre DERUMIER" Cc: pve-devel@pve.proxmox.com, "Stefan Priebe" Envoyé: Jeudi 5 Décembre 2013 17:44:52 Objet: RE: [pve-devel] [PATCH] qemuserver : iothrottle add sup

Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-05 Thread Dietmar Maurer
> Maybe adding a wizard in pve-manager with to generate predifined > values/worload for newbie users ? > > And keep an advanced gui to manually defined values for expert users. Please can you try to use a separate button/window for the limits, as suggested is a previous post?

Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-05 Thread Alexandre DERUMIER
And keep an advanced gui to manually defined values for expert users. - Mail original - De: "Dietmar Maurer" À: "Alexandre DERUMIER" Cc: pve-devel@pve.proxmox.com, "Stefan Priebe" Envoyé: Jeudi 5 Décembre 2013 10:32:18 Objet: RE: [pve-devel] [PATCH] qe

Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-05 Thread Dietmar Maurer
> >>I would simply like to have (additionally) some simpler setting witch > >>usually fits most user's needs, for example a simple bandwidth: > >> > >>bwlimit = 100MB/s > >> > >>form that you can compute IO_OPS/s (assuming an artificial block size of > >>32k): > >> > >>io_ops = bwlimit/32k > > Ca

Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-05 Thread Alexandre DERUMIER
ea ! - Mail original - De: "Dietmar Maurer" À: "Stefan Priebe" , "Alexandre DERUMIER" Cc: pve-devel@pve.proxmox.com Envoyé: Jeudi 5 Décembre 2013 05:55:51 Objet: RE: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max &g

Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-05 Thread Alexandre DERUMIER
r" Cc: pve-devel@pve.proxmox.com Envoyé: Mercredi 4 Décembre 2013 20:24:32 Objet: Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max Am 04.12.2013 18:25, schrieb Alexandre DERUMIER: >>> or is that too easy? > > I would like to tune value ma

Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-04 Thread Dietmar Maurer
> > (Note that the burst is not in iop/s or MB/s, it's ios or MB. the > > quantity of ios or MB we want to run at full speed before throttling) > > So why do you name those variables 'mbps_max' and 'iops_max'? Oh, ignore me - I just saw qemu use a similar naming. ___

Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-04 Thread Dietmar Maurer
> (Note that the burst is not in iop/s or MB/s, it's ios or MB. the quantity of > ios or > MB we want to run at full speed before throttling) So why do you name those variables 'mbps_max' and 'ioops_max'? ___ pve-devel mailing list pve-devel@pve.proxmo

Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-04 Thread Dietmar Maurer
> > I would like to tune value manually, I have differents vm with same > > throttle, but some of them need more burst, depend of the workload. > > > > Example: I have some databases throttle to 10MB/s which I known then need > 1GB of data write at full speed sometimes. OK > i would like to tune

Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-04 Thread Stefan Priebe
tmar Maurer" À: "Alexandre Derumier" , pve-devel@pve.proxmox.com Envoyé: Mercredi 4 Décembre 2013 17:10:17 Objet: RE: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max That whole thing looks a bit clumsy. Does a user really want to specify all those values

Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-04 Thread Alexandre DERUMIER
proxmox.com Envoyé: Mercredi 4 Décembre 2013 17:10:17 Objet: RE: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max That whole thing looks a bit clumsy. Does a user really want to specify all those values manually? Maybe the user can specify a percentage instead, an

Re: [pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-04 Thread Dietmar Maurer
That whole thing looks a bit clumsy. Does a user really want to specify all those values manually? Maybe the user can specify a percentage instead, and we can automatically compute those burst values? burst=1.3==> ioops_max = ioops * 1.3 or is that too easy?

[pve-devel] [PATCH] qemuserver : iothrottle add support for throttling burst max

2013-12-04 Thread Alexandre Derumier
http://git.qemu.org/?p=qemu.git;a=commit;h=3e9fab690d59ac15956c3733fe0794ce1ae4c4af Signed-off-by: Alexandre Derumier --- PVE/API2/Qemu.pm | 16 ++-- PVE/QemuServer.pm | 14 +++--- 2 files changed, 25 insertions(+), 5 deletions(-) diff --git a/PVE/API2/Qemu.pm b/PVE/API