[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver |
Date: |
Tue, 26 Feb 2013 17:52:39 +0100 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Feb 25, 2013 at 03:49:11PM +0000, Dietmar Maurer wrote:
> > > We previously used LVM and run backup with 'idle' IO priority (CFQ) to
> > > avoid
> > such behavior.
> > >
> > > But qemu does not provide an IO queue where we can set scheduling
> > priorities?
> >
> > QEMU block jobs support rate-limiting. Set it to 10-20% of the disk's
> > throughput
> > and the slowness should go away but the backup takes longer.
>
> Sorry, but I don't want to set a rate limit. I want to set an IO priority -
> this is something different.
> My solution does not delay the backup.
I just reread the code and noticed the delay is 1 microsecond, not 1
millisecond.
The problem is that this is a magic value. It worked on your machine
with your workload, but there's no guarantee it works on anyone else's
machine or workload. We can't depend on magic values like this.
If you want to implement an alternative to rate-limiting, please do it
in a separate patch series and make it work for all block job types.
Stefan
- [Qemu-devel] [PATCH v4 0/6] Efficient VM backup for qemu, Dietmar Maurer, 2013/02/20
- [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Dietmar Maurer, 2013/02/20
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Stefan Hajnoczi, 2013/02/20
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Dietmar Maurer, 2013/02/21
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Stefan Hajnoczi, 2013/02/21
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Dietmar Maurer, 2013/02/25
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Stefan Hajnoczi, 2013/02/25
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Dietmar Maurer, 2013/02/25
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver,
Stefan Hajnoczi <=
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Dietmar Maurer, 2013/02/26
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Stefan Hajnoczi, 2013/02/27
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Dietmar Maurer, 2013/02/27
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Kevin Wolf, 2013/02/28
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Stefan Hajnoczi, 2013/02/26
- Re: [Qemu-devel] [PATCH v4 2/6] add basic backup support to block driver, Dietmar Maurer, 2013/02/26
[Qemu-devel] [PATCH v4 6/6] add vm state to backups, Dietmar Maurer, 2013/02/20
[Qemu-devel] [PATCH v4 1/6] add documenation for new backup framework, Dietmar Maurer, 2013/02/20