<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Aug 31, 2020 at 9:48 PM Ján Tomko <<a href="mailto:jtomko@redhat.com">jtomko@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On a Monday in 2020, Daniel Henrique Barboza wrote:<br>
><br>
><br>
>On 8/21/20 7:19 AM, Han Han wrote:<br>
>>Since QEMU 1.5.3, the ib700 watchdog device has no options for address,<br>
>>and not address in device tree:<br>
>><br>
>>$ /usr/libexec/qemu-kvm -version<br>
>>QEMU emulator version 1.5.3 (qemu-kvm-1.5.3-175.el7), Copyright (c) 2003-2008 Fabrice Bellard<br>
>>$ /usr/libexec/qemu-kvm -device ib700,\?<br>
>>$ virsh qemu-monitor-command seabios --hmp info qtree|grep ib700 -A 2<br>
>>         dev: ib700, id "watchdog0"<br>
>>         dev: isa-serial, id "serial0"<br>
>>           index = 0<br>
>><br>
>>So only allow it to use none address.<br>
>><br>
>><a href="https://bugzilla.redhat.com/show_bug.cgi?id=1509908" rel="noreferrer" target="_blank">https://bugzilla.redhat.com/show_bug.cgi?id=1509908</a><br>
><br>
>I'm not able to open this bug :/<br>
><br>
<br>
Fixed.<br>
<br>
>><br>
>>Signed-off-by: Han Han <<a href="mailto:hhan@redhat.com" target="_blank">hhan@redhat.com</a>><br>
<br>
Fixes: 8a54cc1d08a333283c9cfc3fd7788be2642ca71a<br>
<br>
>>---<br>
><br>
>Reviewed-by: Daniel Henrique Barboza <<a href="mailto:danielhb413@gmail.com" target="_blank">danielhb413@gmail.com</a>><br>
><br>
>>  src/qemu/qemu_validate.c | 3 +--<br>
>>  1 file changed, 1 insertion(+), 2 deletions(-)<br>
>><br>
>>diff --git a/src/qemu/qemu_validate.c b/src/qemu/qemu_validate.c<br>
>>index 488f258d00..0e5ca81ab4 100644<br>
>>--- a/src/qemu/qemu_validate.c<br>
>>+++ b/src/qemu/qemu_validate.c<br>
>>@@ -1637,8 +1637,7 @@ qemuValidateDomainWatchdogDef(const virDomainWatchdogDef *dev,<br>
>>          break;<br>
>>      case VIR_DOMAIN_WATCHDOG_MODEL_IB700:<br>
>>-        if (dev->info.type != VIR_DOMAIN_DEVICE_ADDRESS_TYPE_NONE &&<br>
>>-            dev->info.type != VIR_DOMAIN_DEVICE_ADDRESS_TYPE_ISA) {<br>
>>+        if (dev->info.type != VIR_DOMAIN_DEVICE_ADDRESS_TYPE_NONE) {<br>
>>              virReportError(VIR_ERR_CONFIG_UNSUPPORTED,<br>
>>                             _("%s model of watchdog can go only on ISA bus"),<br>
<br>
This error message would be then misleading.<br>
<br></blockquote><div>Thanks for your suggestions. I will update it in v2 <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
"'%s' model of watchdog does not support configuring the address"<br>
<br>
Jano<br>
<br>
>>                             virDomainWatchdogModelTypeToString(dev->model));<br>
>><br>
><br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="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>Senior 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>