[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH v2 2/4] monitor: Separate QUORUM_REPORT_BAD even
From: |
Eric Blake |
Subject: |
Re: [Qemu-block] [PATCH v2 2/4] monitor: Separate QUORUM_REPORT_BAD events according to the node name |
Date: |
Thu, 10 Mar 2016 13:35:32 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 |
On 03/10/2016 04:55 AM, Alberto Garcia wrote:
> The QUORUM_REPORT_BAD event is emitted whenever there's an I/O error
> in a child of a Quorum device. This event is emitted at a maximum rate
> of 1 per second. This means that an error in one of the children will
> mask errors in the other children if they happen within the same 1
> second interval.
>
> This patch modifies qapi_event_throttle_equal() so QUORUM_REPORT_BAD
> events are kept separately if they come from different children.
>
> Signed-off-by: Alberto Garcia <address@hidden>
> Reviewed-by: Max Reitz <address@hidden>
> ---
> monitor.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/monitor.c b/monitor.c
> index e99ca8c..c9fe862 100644
> --- a/monitor.c
> +++ b/monitor.c
> @@ -572,6 +572,10 @@ static unsigned int qapi_event_throttle_hash(const void
> *key)
> hash += g_str_hash(qdict_get_str(evstate->data, "id"));
> }
>
> + if (evstate->event == QAPI_EVENT_QUORUM_REPORT_BAD) {
> + hash += g_str_hash(qdict_get_str(evstate->data, "node-name"));
Is ^= any better than += when computing hashes, so that carry bits
aren't weakening the distribution of bits? But as long as the
computation is consistent, I'm not too worried, since you were copying
the line above.
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- [Qemu-block] [PATCH v2 0/4] Separate QUORUM_REPORT_BAD events according to their node name, Alberto Garcia, 2016/03/10
- [Qemu-block] [PATCH v2 1/4] quorum: Fix crash in quorum_aio_cb(), Alberto Garcia, 2016/03/10
- [Qemu-block] [PATCH v2 2/4] monitor: Separate QUORUM_REPORT_BAD events according to the node name, Alberto Garcia, 2016/03/10
- Re: [Qemu-block] [PATCH v2 2/4] monitor: Separate QUORUM_REPORT_BAD events according to the node name,
Eric Blake <=
- [Qemu-block] [PATCH v2 3/4] monitor: Use QEMU_CLOCK_VIRTUAL for the event queue in qtest mode, Alberto Garcia, 2016/03/10
- [Qemu-block] [PATCH v2 4/4] iotests: Add test for QMP event rates, Alberto Garcia, 2016/03/10
- Re: [Qemu-block] [PATCH v2 0/4] Separate QUORUM_REPORT_BAD events according to their node name, Max Reitz, 2016/03/14