[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot
From: |
Ryan Harper |
Subject: |
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot |
Date: |
Wed, 8 Jul 2009 10:58:10 -0500 |
User-agent: |
Mutt/1.5.6+20040907i |
* Anthony Liguori <address@hidden> [2009-07-08 10:50]:
> Ryan Harper wrote:
> >>That suffers from the same problem as a system_reboot command in that
> >>the guest may never start the shutdown. If that's an acceptable
> >>limitation, then we might as well implement it directly in QEMU as
> >>system_reboot so its easily available to everyone. Make 'system_reboot'
> >>take an optional timeout arg (default to 60 seconds) after which it
> >>cancels its plans.
> >>
> >
> >This seems the most reasonable thing to me. As already mentioned,
> >system_powerdown already is non-deterministic since the guest could
> >ignore the ACPI event.
> >
>
> It's a different class of non-determinism though. With
> system_powerdown, you make a request to the guest. You don't know
> whether the guest will comply with the request. The same level of
> non-determinism exists with ballooning.
>
> With system_reboot, you're taking an action (system_reset) and what's
> not deterministic is whether you're taking that action based on what
> someone requested on the monitor (system_reboot) verses whether you're
> taking that action based on something the user did (shutdown). In the
> later case, you're action is functionally incorrect.
Only if the shutdown from the user occurs in timeout period. So, yes,
it's still present, but the window of the behavior can be much
smaller.
I still think this is a reasonable approach even with that window.
--
Ryan Harper
Software Engineer; Linux Technology Center
IBM Corp., Austin, Tx
address@hidden
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, (continued)
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Gleb Natapov, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Anthony Liguori, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Daniel P. Berrange, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Anthony Liguori, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Daniel P. Berrange, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Anthony Liguori, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Ryan Harper, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Anthony Liguori, 2009/07/08
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot,
Ryan Harper <=
- Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Anthony Liguori, 2009/07/08
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Avi Kivity, 2009/07/08
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Amit Shah, 2009/07/16
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Jamie Lokier, 2009/07/16
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Amit Shah, 2009/07/16
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Gleb Natapov, 2009/07/08
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Avi Kivity, 2009/07/08
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Daniel P. Berrange, 2009/07/08
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Avi Kivity, 2009/07/08
Re: [Qemu-devel] [PATCH][RESEND] Add monitor command for system_reboot, Stefano Stabellini, 2009/07/08