[libvirt] [PATCH v2 4/6] qemu: make error messages device specific

Boris Fiuczynski posted 6 patches 6 years, 5 months ago
[libvirt] [PATCH v2 4/6] qemu: make error messages device specific
Posted by Boris Fiuczynski 6 years, 5 months ago
Changing the error messages to report the problem encountered.

Signed-off-by: Boris Fiuczynski <fiuczy@linux.ibm.com>
Reviewed-by: Marc Hartmayer <mhartmay@linux.ibm.com>
---
 src/qemu/qemu_domain.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index ce5334aa67..9a9b6676e3 100644
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -5267,16 +5267,16 @@ qemuDomainDeviceDefValidateHostdev(const virDomainHostdevDef *hostdev,
         case VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_SCSI_HOST:
             if (hostdev->info->bootIndex) {
                 virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
-                               _("booting from assigned devices is only "
-                                 "supported for PCI, USB and SCSI devices"));
+                               _("booting from assigned devices is not "
+                                 "supported by vhost SCSI devices"));
                 return -1;
             }
             break;
         case VIR_DOMAIN_HOSTDEV_SUBSYS_TYPE_MDEV:
             if (hostdev->info->bootIndex) {
                 virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
-                               _("booting from assigned devices is only "
-                                 "supported for PCI, USB and SCSI devices"));
+                               _("booting from assigned devices is not "
+                                 "supported by mediated devices"));
                 return -1;
             }
             mdevsrc = &hostdev->source.subsys.u.mdev;
-- 
2.17.0

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH v2 4/6] qemu: make error messages device specific
Posted by Ján Tomko 6 years, 5 months ago
On Tue, Aug 13, 2019 at 05:40:28PM +0200, Boris Fiuczynski wrote:
>Changing the error messages to report the problem encountered.
>
>Signed-off-by: Boris Fiuczynski <fiuczy@linux.ibm.com>
>Reviewed-by: Marc Hartmayer <mhartmay@linux.ibm.com>
>---
> src/qemu/qemu_domain.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

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