Re: [PATCH 0/2] tools: virsh-domain: display progress with enhanced granularity

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

 



On Wed, Apr 26, 2023 at 11:51:28AM +0000, Shaleen Bathla wrote:
Missed adding V2 in subject title.
Can someone please guide, if I should resend the patch series with the title correction or this can be taken forward ?


That's fine, no need to resend just for this, that can happen to anyone.

Thanks and Regards,
Shaleen Bathla

-----Original Message-----
From: Shaleen Bathla <shaleen.bathla@xxxxxxxxxx>
Sent: Wednesday, April 26, 2023 4:59 PM
To: libvir-list@xxxxxxxxxx
Subject: [PATCH 0/2] tools: virsh-domain: display progress with enhanced granularity

Changes since v1
================================================================================
- Remove superfluous parenthesis
- Update progress as 99.99 instead of 99.00
- Add comment for auto-round-off prevention logic for double

Description
==================================================================================
While working with active blockcommit using virsh/libvirt, we found that in case of heavy IO, the progress may stay stuck at 99% for quite a while.
It will be much better if we show progress upto 2 decimal places more precise which should be sufficient for most scenarios for now.
This will give a user, a better understanding of the progress of blockjob/blockcommit etc
This patch intends to do exactly that by adding more granularity to blockjob/blockcommit etc progress.

Shaleen Bathla (2):
 tools: virsh-domain: refactor variable initialization
 tools: virsh-domain: display progress with enhanced granularity

tools/virsh-domain.c | 19 +++++++++----------
1 file changed, 9 insertions(+), 10 deletions(-)

--
2.31.1

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux