Add function to check cdrom status

[libvirt] [PATCH 0/2] Add function to check cdrom status

Han Han posted 2 patches 5 years, 10 months ago

Download series mbox

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

Test syntax-check passed

src/libvirt_private.syms   |  1 +
src/storage/storage_util.c | 38 ++++++++++++++++++++++++++++
src/util/virfile.c         | 52 ++++++++++++++++++++++++++++++++++++++
src/util/virfile.h         | 10 ++++++++
4 files changed, 101 insertions(+)

Expand all Fold all

[libvirt] [PATCH 0/2] Add function to check cdrom status

Posted by Han Han 5 years, 10 months ago

In libvirt, it gives general Input/Output error when reading a not ready
cdrom. This error could be more detailed on cdrom, such as drive not
ready, no disc, trey open, no information.
These patches provide private API virFileCdromStatus to check cdrom storage backend
status and improve error handling of virStorageBackendVolOpen on cdrom.

Han Han (2):
  util: Introduce virFileCdromStatus
  storage: Improve error handling on cdrom backend

 src/libvirt_private.syms   |  1 +
 src/storage/storage_util.c | 38 ++++++++++++++++++++++++++++
 src/util/virfile.c         | 52 ++++++++++++++++++++++++++++++++++++++
 src/util/virfile.h         | 10 ++++++++
 4 files changed, 101 insertions(+)

-- 
2.17.1

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

Re: [libvirt] [PATCH 0/2] Add function to check cdrom status

Posted by Han Han 5 years, 10 months ago

Hello, could anyone help review?

On Tue, Jul 3, 2018 at 10:29 AM, Han Han <hhan@redhat.com> wrote:

> In libvirt, it gives general Input/Output error when reading a not ready
> cdrom. This error could be more detailed on cdrom, such as drive not
> ready, no disc, trey open, no information.
> These patches provide private API virFileCdromStatus to check cdrom
> storage backend
> status and improve error handling of virStorageBackendVolOpen on cdrom.
>
> Han Han (2):
>   util: Introduce virFileCdromStatus
>   storage: Improve error handling on cdrom backend
>
>  src/libvirt_private.syms   |  1 +
>  src/storage/storage_util.c | 38 ++++++++++++++++++++++++++++
>  src/util/virfile.c         | 52 ++++++++++++++++++++++++++++++++++++++
>  src/util/virfile.h         | 10 ++++++++
>  4 files changed, 101 insertions(+)
>
> --
> 2.17.1
>
>


-- 
Best regards,
-----------------------------------
Han Han
Quality Engineer
Redhat.

Email: hhan@redhat.com
Phone: +861065339333
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list