qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: "Benoît Canet" <benoit.canet@irqsave.net>
To: qemu-devel@nongnu.org
Cc: kwolf@redhat.com, "Benoît Canet" <benoit.canet@irqsave.net>,
	stefanha@redhat.com, mreitz@redhat.com
Subject: [Qemu-devel] [PATCH V2 for 2.0 0/2] Allow to repair broken quorum files
Date: Thu,  6 Mar 2014 15:06:20 +0100	[thread overview]
Message-ID: <1394114782-22228-1-git-send-email-benoit.canet@irqsave.net> (raw)

I would like to make this series merge in 2.0 before hard freeze because it
would allow a production usage of quorum so it would let me move on the rest of
the block filter infrastructure.(throttling as filter, filter insertion and
removal etc..)

in v2:
    Fix typos and spelling mistakes [Eric]
    Fix alignement [Eric]
    Remove comments about Fam's new API [Eric]
    Add a new test to check that the to-replace-node-name is locked as in use
       during mirroring [Benoît]

This patch allow to build a mirror of a block device and replace an arbitrary
named node of the graph with the mirror.

The obvious first usage is Quorum: with this patch one quand drive-mirror the
block device say quorum0 and then bdrv_swap the result in place of a broken
quorum file (dead filer for example).

The second patch implement drive-mirror tests for this mode.

Benoît Canet (2):
  block: Add node-name and to-replace-node-name arguments to
    drive-mirror.
  qemu-iotests: Add TestRepairQuorum to 041 to test drive-mirror
    node-name mode.

 block/mirror.c             |  51 ++++++++---
 blockdev.c                 |  61 ++++++++++++-
 hmp.c                      |   3 +-
 include/block/block_int.h  |   3 +
 qapi-schema.json           |  15 ++-
 qmp-commands.hx            |   9 +-
 tests/qemu-iotests/041     | 221 ++++++++++++++++++++++++++++++++++++++++++++-
 tests/qemu-iotests/041.out |   4 +-
 8 files changed, 337 insertions(+), 30 deletions(-)

-- 
1.8.3.2

             reply	other threads:[~2014-03-06 14:06 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-06 14:06 Benoît Canet [this message]
2014-03-06 14:06 ` [Qemu-devel] [PATCH V2 for 2.0 1/2] block: Add node-name and to-replace-node-name arguments to drive-mirror Benoît Canet
2014-03-06 14:06 ` [Qemu-devel] [PATCH V2 for 2.0 2/2] qemu-iotests: Add TestRepairQuorum to 041 to test drive-mirror node-name mode Benoît Canet
2014-03-06 16:09   ` Benoît Canet

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1394114782-22228-1-git-send-email-benoit.canet@irqsave.net \
    --to=benoit.canet@irqsave.net \
    --cc=kwolf@redhat.com \
    --cc=mreitz@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).