[v3] Change virFileIsCDROM to check cdrom status

[libvirt] [PATCH v3 0/2] Change virFileIsCDROM to check cdrom status

Han Han posted 2 patches 5 years, 9 months ago

Diff against v1 v2
Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20180711094442.18303-1-hhan@redhat.com

Test syntax-check passed

src/libvirt_private.syms   |  2 +-
src/qemu/qemu_domain.c     |  4 ++--
src/storage/storage_util.c |  8 ++++++++
src/util/virfile.c         | 41 ++++++++++++++++++++++++++++++++------
src/util/virfile.h         | 13 +++++++++++-
5 files changed, 58 insertions(+), 10 deletions(-)

Expand all Fold all

[libvirt] [PATCH v3 0/2] Change virFileIsCDROM to check cdrom status

Posted by Han Han 5 years, 9 months ago

v2: https://www.redhat.com/archives/libvir-list/2018-July/msg00655.html
v1: https://www.redhat.com/archives/libvir-list/2018-July/msg00103.html

Rename virFileIsCDROM to virFileCheckCDROM and add new argument filled
with cdrom status. Now it could be used to get cdrom status.

In libvirt, it gives general Input/Output error when reading a not ready
cdrom backend vol. This error should be replaced with warning just like the
warning of lacking backing file.

Han Han (2):
  util: Refactor virFileIsCDROM to virFileCheckCDROM
  storage: Improve error handling on cdrom backend

 src/libvirt_private.syms   |  2 +-
 src/qemu/qemu_domain.c     |  4 ++--
 src/storage/storage_util.c |  8 ++++++++
 src/util/virfile.c         | 41 ++++++++++++++++++++++++++++++++------
 src/util/virfile.h         | 13 +++++++++++-
 5 files changed, 58 insertions(+), 10 deletions(-)

-- 
2.17.1

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list