qemu-devel
[Top][All Lists]
Advanced

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

Re: [PATCH v6 01/10] block: push error reporting into bdrv_all_*_snapsho


From: Eric Blake
Subject: Re: [PATCH v6 01/10] block: push error reporting into bdrv_all_*_snapshot functions
Date: Mon, 19 Oct 2020 14:28:20 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.3.1

On 10/8/20 10:49 AM, Daniel P. Berrangé wrote:
The bdrv_all_*_snapshot functions return a BlockDriverState pointer
for the invalid backend, which the callers then use to report an
error message. In some cases multiple callers are reporting the
same error message, but with slightly different text. In the future
there will be more error scenarios for some of these methods, which
will benefit from fine grained error message reporting. So it is
helpful to push error reporting down a level.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
  block/monitor/block-hmp-cmds.c |  7 ++--
  block/snapshot.c               | 77 +++++++++++++++++-----------------
  include/block/snapshot.h       | 14 +++----
  migration/savevm.c             | 37 +++++-----------
  monitor/hmp-cmds.c             |  7 +---
  replay/replay-debugging.c      |  4 +-
  tests/qemu-iotests/267.out     | 10 ++---
  7 files changed, 67 insertions(+), 89 deletions(-)

and less code doesn't hurt either ;)

Reviewed-by: Eric Blake <eblake@redhat.com>

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org




reply via email to

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