On 07/25/2017 05:37 PM, Eric Blake wrote:
On 07/25/2017 04:28 PM, John Snow wrote:
On 07/21/2017 02:32 PM, Eric Blake wrote:
We've been documenting the value in bytes since its introduction
in commit b9a9b3a4 (v1.3), where it was actually reported in bytes.
Commit e4654d2 (v2.0) then removed things from block/qapi.c, in
preparation for a rewrite to a list of dirty sectors in the next
commit 21b5683 in block.c, but the new code mistakenly started
reporting in sectors.
Fixes: https://bugzilla.redhat.com/1441460
CC: qemu-sta...@nongnu.org
Signed-off-by: Eric Blake <ebl...@redhat.com>
Reviewed-by: John Snow <js...@redhat.com>
I must have forgotten about this -- Didn't this get reviewed as part of
your byte series? Is this a resend for a stable branch?
Yes, it was reviewed during my part 2-of-4 series on byte-base block
status (the dirty bitmap series). Series 1 made it into 2.10, but
series 2-4 did not; ergo, I filtered out the true bug-fixes that were
still worthy post-freeze.
Thanks for the clarification. Short term memory is a little strapped
right now due to my move.
ACK