qemu-block
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 1/1] block: use 'unsigned' for in_flight field on driver stat


From: Denis V. Lunev
Subject: Re: [PATCH 1/1] block: use 'unsigned' for in_flight field on driver state
Date: Mon, 13 Jun 2022 13:29:05 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.9.1

On 30.05.2022 12:39, Denis V. Lunev wrote:
This patch makes in_flight field 'unsigned' for BDRVNBDState and
MirrorBlockJob. This matches the definition of this field on BDS
and is generically correct - we should never get negative value here.

Signed-off-by: Denis V. Lunev <den@openvz.org>
CC: John Snow <jsnow@redhat.com>
CC: Vladimir Sementsov-Ogievskiy <vsementsov@yandex-team.ru>
CC: Kevin Wolf <kwolf@redhat.com>
CC: Hanna Reitz <hreitz@redhat.com>
CC: Eric Blake <eblake@redhat.com>
---
  block/mirror.c | 2 +-
  block/nbd.c    | 2 +-
  2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/block/mirror.c b/block/mirror.c
index d8ecb9efa2..3c4ab1159d 100644
--- a/block/mirror.c
+++ b/block/mirror.c
@@ -73,7 +73,7 @@ typedef struct MirrorBlockJob {
uint64_t last_pause_ns;
      unsigned long *in_flight_bitmap;
-    int in_flight;
+    unsigned in_flight;
      int64_t bytes_in_flight;
      QTAILQ_HEAD(, MirrorOp) ops_in_flight;
      int ret;
diff --git a/block/nbd.c b/block/nbd.c
index 6085ab1d2c..512a53bf73 100644
--- a/block/nbd.c
+++ b/block/nbd.c
@@ -77,7 +77,7 @@ typedef struct BDRVNBDState {
      QemuMutex requests_lock;
      NBDClientState state;
      CoQueue free_sema;
-    int in_flight;
+    unsigned in_flight;
      NBDClientRequest requests[MAX_NBD_REQUESTS];
      QEMUTimer *reconnect_delay_timer;
ping v2



reply via email to

[Prev in Thread] Current Thread [Next in Thread]