<div dir="ltr">Hello, could anyone help review?<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul 3, 2018 at 10:29 AM, Han Han <span dir="ltr"><<a href="mailto:hhan@redhat.com" target="_blank">hhan@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In libvirt, it gives general Input/Output error when reading a not ready<br>
cdrom. This error could be more detailed on cdrom, such as drive not<br>
ready, no disc, trey open, no information.<br>
These patches provide private API virFileCdromStatus to check cdrom storage backend<br>
status and improve error handling of virStorageBackendVolOpen on cdrom.<br>
<br>
Han Han (2):<br>
  util: Introduce virFileCdromStatus<br>
  storage: Improve error handling on cdrom backend<br>
<br>
 src/libvirt_private.syms   |  1 +<br>
 src/storage/storage_util.c | 38 ++++++++++++++++++++++++++++<br>
 src/util/virfile.c         | 52 ++++++++++++++++++++++++++++++<wbr>++++++++<br>
 src/util/virfile.h         | 10 ++++++++<br>
 4 files changed, 101 insertions(+)<br>
<span class="HOEnZb"><font color="#888888"><br>
-- <br>
2.17.1<br>
<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr">Best regards,</div><div dir="ltr">-----------------------------------<br></div><div dir="ltr">Han Han<br>Quality Engineer<br>Redhat.<br><br>Email: <a href="mailto:hhan@redhat.com" target="_blank">hhan@redhat.com</a><br>Phone: +861065339333<br></div></div></div></div></div></div>
</div>