[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V2] Use clean shutdown request for ctrl-a x
From: |
Fabien Chouteau |
Subject: |
Re: [Qemu-devel] [PATCH V2] Use clean shutdown request for ctrl-a x |
Date: |
Tue, 10 Jul 2012 12:16:16 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux i686; rv:13.0) Gecko/20120615 Thunderbird/13.0.1 |
On 07/09/2012 02:18 PM, Andreas Färber wrote:
> Am 09.07.2012 12:19, schrieb Fabien Chouteau:
>> Any comment?
>>
>> On 07/04/2012 01:04 PM, Fabien Chouteau wrote:
>>> The goal is to make ctrl-a x to close Qemu in a clean way. The current
>>> exit(0) skips a lot of cleanup/close functions, for example in block
>>> drivers.
>>>
>>> Signed-off-by: Fabien Chouteau <address@hidden>
>>> ---
>>> qemu-char.c | 2 +-
>>> sysemu.h | 1 +
>>> vl.c | 5 +++++
>>> 3 files changed, 7 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/qemu-char.c b/qemu-char.c
>>> index c2aaaee..7732846 100644
>>> --- a/qemu-char.c
>>> +++ b/qemu-char.c
>>> @@ -353,7 +353,7 @@ static int mux_proc_byte(CharDriverState *chr,
>>> MuxDriver *d, int ch)
>>> {
>>> const char *term = "QEMU: Terminated\n\r";
>>> chr->chr_write(chr,(uint8_t *)term,strlen(term));
>>> - exit(0);
>>> + qemu_system_force_shutdown();
>>> break;
>>> }
>>> case 's':
>
> FWIW there was a recent patch by Hervé that exposed further occurrences
> of exit(), probably all would need to be reviewed and fixed.
>
There's a lot of exit() in Qemu (~660 with a quick grep), but it doesn't
always make sense to change them all.
In my opinion this one is a clean user request to shutdown the system,
as opposed to an error state that requires exit().
--
Fabien Chouteau