<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 15, 2019 at 4:04 PM Michal Privoznik <<a href="mailto:mprivozn@redhat.com">mprivozn@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 10/15/19 7:23 AM, Han Han wrote:<br>
> Hi Michal,<br>
> Any more advice update?<br>
<br>
Well, as I've said earlier, since we document that -1 is accepted value <br>
and it means that it suppresses automatic reboots we need a way to <br>
preserve this behaviour. For instance, what happens if you don't put <br>
reboot-timeout onto the cmd line at all? Does qemu use some default and <br></blockquote><div>If no  reboot-timeout, qemu will not reboot by default. I have updated the qemu doc:</div><div><a href="https://github.com/qemu/qemu/commit/bbd9e6985ff342cbe15b9cb7eb30e842796fbbe8">https://github.com/qemu/qemu/commit/bbd9e6985ff342cbe15b9cb7eb30e842796fbbe8</a></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
reboot anyways? If it doesn't reboot then -1 should mean to not put <br></blockquote><div>I think they use the same defaults and will not reboot by default. That can be checked by</div><div>the code before and after qemu commit ee5d0f8. The values passed to fw_cfg_add_file are<br></div><div>same by default.</div><div><br></div><div>Markus Armbruster, <br></div><div>Could you please confirm that, for cases of<br></div><div>- reboot-timeout=-1 before ee5d0f8</div><div>- reboot-timeout by default before ee5d0f8</div><div>- reboot-timeout by default after  ee5d0f8</div><div>they all will not reboot?</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
reboot-timeout onto the cmd line. However, if it does reboot then we <br>
need to talk to qemu developers to provide us a way to suppress <br>
automatic reboots.<br>
<br>
Michal<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>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>