[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 2/8] block: add basic backup support to block
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v4 2/8] block: add basic backup support to block driver |
Date: |
Wed, 22 May 2013 11:56:45 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 22.05.2013 um 11:54 hat Paolo Bonzini geschrieben:
> Il 22/05/2013 11:38, Kevin Wolf ha scritto:
> >> +
> >> + DPRINTF("brdv_co_backup_cow done C%" PRId64 "\n", start);
> >> + }
> >> +
> >> +out:
> >> + if (bounce_buffer) {
> >> + qemu_vfree(bounce_buffer);
> >> + }
> >> +
> >> + cow_request_end(&cow_request);
> >> +
> >> + qemu_co_rwlock_unlock(&job->flush_rwlock);
> >> +
> >> + return ret;
> >> +}
> >> +
> >> +static void coroutine_fn backup_before_write_notify(Notifier *notifier,
> >> + void *opaque)
> >> +{
> >> + BdrvTrackedRequest *req = opaque;
> >> + backup_do_cow(req->bs, req->sector_num, req->nb_sectors);
> >> +}
> >
> > I don't think you can ignore errors here. Not sure if we can stop the VM
> > and resume later or something like that, but if we can't, the backup
> > will be invalid and we must fail the job.
>
> Yes, there is rerror/werror machinery for jobs that this patch is not using.
This is not enough here. The guest write can't continue before the old
content is saved to the backup image.
Kevin
[Qemu-devel] [PATCH v4 5/8] blockdev: rename BlkTransactionStates to singular, Stefan Hajnoczi, 2013/05/16
[Qemu-devel] [PATCH v4 4/8] qemu-iotests: add 055 drive-backup test case, Stefan Hajnoczi, 2013/05/16
[Qemu-devel] [PATCH v4 1/8] block: add bdrv_add_before_write_notifier(), Stefan Hajnoczi, 2013/05/16
[Qemu-devel] [PATCH v4 6/8] blockdev: add DriveBackup transaction, Stefan Hajnoczi, 2013/05/16