[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/3] quorum: Return an error if the blkverify mo
From: |
Alberto Garcia |
Subject: |
Re: [Qemu-devel] [PATCH 2/3] quorum: Return an error if the blkverify mode has invalid settings |
Date: |
Wed, 17 Oct 2018 17:34:09 +0200 |
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Wed 17 Oct 2018 05:31:32 PM CEST, Kevin Wolf wrote:
> Am 17.10.2018 um 16:33 hat Alberto Garcia geschrieben:
>> The blkverify mode of Quorum can only be enabled if the number of
>> children is exactly two and the value of vote-threshold is also two.
>>
>> If the user tries to enable it but the other settings are incorrect
>> then QEMU simply prints an error message to stderr and carries on
>> disabling the blkverify setting.
>>
>> This patch makes quorum_open() fail and return an error in this case.
>>
>> Signed-off-by: Alberto Garcia <address@hidden>
>> Reported-by: Markus Armbruster <address@hidden>
>
> While reviewing this, I think I found another problem: Shouldn't
> .bdrv_add_child refuse to add a child if s->is_blkverify == true? (And
> probably assert in .bdrv_del_child that s->is_blkverify == false after
> checking s->num_children <= s->threshold)
I think you're right. Do you want me to send this in a later patch or do
you prefer that I make it part of this series?
Berto