[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-block] [PATCH 2/7] dirty-bitmaps: fix comment about dirty_bitmap_m
From: |
Vladimir Sementsov-Ogievskiy |
Subject: |
[Qemu-block] [PATCH 2/7] dirty-bitmaps: fix comment about dirty_bitmap_mutex |
Date: |
Mon, 26 Mar 2018 14:20:51 +0300 |
Clarify first two cases and fix Modify -> Any access in third case.
Also, drop 'only' from third case, as it a bit confuses, when thinking
about case where we modify BdrvDirtyBitmap and access HBitmap.
Signed-off-by: Vladimir Sementsov-Ogievskiy <address@hidden>
---
include/block/block_int.h | 12 ++++++++----
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/include/block/block_int.h b/include/block/block_int.h
index 27e17addba..e5af7820ff 100644
--- a/include/block/block_int.h
+++ b/include/block/block_int.h
@@ -701,10 +701,14 @@ struct BlockDriverState {
uint64_t write_threshold_offset;
NotifierWithReturn write_threshold_notifier;
- /* Writing to the list requires the BQL _and_ the dirty_bitmap_mutex.
- * Reading from the list can be done with either the BQL or the
- * dirty_bitmap_mutex. Modifying a bitmap only requires
- * dirty_bitmap_mutex. */
+ /* Writing to the list (i.e. to any field of BdrvDirtyBitmap or to the
+ * list-head) requires both the BQL _and_ the dirty_bitmap_mutex.
+ *
+ * Reading from the list (from any field of BdrvDirtyBitmap or from the
+ * list-head) can be done with either the BQL or the dirty_bitmap_mutex.
+ *
+ * Any access to underlying HBitmap requires dirty_bitmap_mutex.
+ */
QemuMutex dirty_bitmap_mutex;
QLIST_HEAD(, BdrvDirtyBitmap) dirty_bitmaps;
--
2.11.1
- [Qemu-block] [PATCH 0/7] Dirty bitmaps fixing and refactoring, Vladimir Sementsov-Ogievskiy, 2018/03/26
- [Qemu-block] [PATCH 7/7] blockdev: unify block-dirty-bitmap-clear command and transaction action, Vladimir Sementsov-Ogievskiy, 2018/03/26
- [Qemu-block] [PATCH 1/7] block/dirty-bitmap: add lock to bdrv_enable/disable_dirty_bitmap, Vladimir Sementsov-Ogievskiy, 2018/03/26
- [Qemu-block] [PATCH 3/7] dirty-bitmap: remove missed bdrv_dirty_bitmap_get_autoload header, Vladimir Sementsov-Ogievskiy, 2018/03/26
- [Qemu-block] [PATCH 4/7] dirty-bitmap: separate unused meta-bitmap related functions, Vladimir Sementsov-Ogievskiy, 2018/03/26
- [Qemu-block] [PATCH 6/7] block/dirty-bitmap: bdrv_clear_dirty_bitmap: drop unused parameter, Vladimir Sementsov-Ogievskiy, 2018/03/26
- [Qemu-block] [PATCH 2/7] dirty-bitmaps: fix comment about dirty_bitmap_mutex,
Vladimir Sementsov-Ogievskiy <=
- Re: [Qemu-block] [Qemu-devel] [PATCH 0/7] Dirty bitmaps fixing and refactoring, Paolo Bonzini, 2018/03/26
- [Qemu-block] [PATCH 5.5/7] dirty-bitmap: drop unused bdrv_undo_clear_dirty_bitmap, Vladimir Sementsov-Ogievskiy, 2018/03/26