On 10/03/2020 17:50, Max Reitz wrote:
On 06.03.20 08:38, Vladimir Sementsov-Ogievskiy wrote:
offset/bytes pair is more usual naming in block layer, let's use it.
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsement...@virtuozzo.com>
Reviewed-by: Andrey Shinkevich <andrey.shinkev...@virtuozzo.com>
Reviewed-by: Max Reitz <mre...@redhat.com>
---
include/block/block-copy.h | 4 +-
block/block-copy.c | 84 +++++++++++++++++++-------------------
2 files changed, 44 insertions(+), 44 deletions(-)
[...]
diff --git a/block/block-copy.c b/block/block-copy.c
index 4c947e548b..2b29131653 100644
--- a/block/block-copy.c
+++ b/block/block-copy.c
@@ -25,13 +25,13 @@
#define BLOCK_COPY_MAX_MEM (128 * MiB)
static BlockCopyInFlightReq *find_conflicting_inflight_req(BlockCopyState *s,
- int64_t start,
- int64_t bytes)
+ int64_t offset,
+ int64_t bytes)
The alignment’s off now.
Max
After applying the patch, it looks aligned in my vim editor.
--
With the best regards,
Andrey Shinkevich
