[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH v2 3/3] char: emit the OPENED event only whe
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] Re: [PATCH v2 3/3] char: emit the OPENED event only when a new char connection is opened |
Date: |
Mon, 26 Oct 2009 08:40:12 +0100 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
Amit Shah wrote:
> On (Sat) Oct 24 2009 [12:36:54], Jan Kiszka wrote:
>> Amit Shah wrote:
>>> The OPENED event gets sent also when qemu resets its state initially.
>>> The consumers of the event aren't interested in receiving this event
>>> on reset.
>> The monitor was. Now its initial prompt on activation is broken.
>
> The patch in Anthony's queue, titled
>
> 'console: call qemu_chr_reset() in text_console_init'
You may also want to rename qemu_chr_reset - unless there is still a
need for real "reset".
>
> fixed that.
>
> However, with the qcow2 synchronous patch, the monitor prompt doesn't
> come up again -- which shows there is a problem with the way the bhs
> work and also the initial resets.
Then the qcow2 patch is already in? At least applying your patch doesn't
change the picture.
>
> I think the initial resets are a hack to work around something from my
> reading of it; do you have a better idea of why it's there and how it's
> all supposed to work?
From the monitor's POV, it's not a hack, it's simply the requirement to
receive an indication that the console was opened.
>
>> Does this patch fix/improve something for a different user? If not,
>> please let us revert it.
>
> There's another question too: is a separate 'reset' event needed in
> addition to an 'opened' event?
Not for the monitor, but I cannot speak for other users. I think it
would be good to check them in details before changing the reset/open
semantic.
>
> I have a few apps (that are coming as part of the virtio-console work)
> that need just an 'opened' event and are not interested in the 'reset'
> event.
>
> Amit
Jan
signature.asc
Description: OpenPGP digital signature
- [Qemu-devel] [PATCH v2 0/3] Send out OPENED events only on chardev open, Amit Shah, 2009/10/07
- [Qemu-devel] [PATCH v2 1/3] char: check for initial_reset_issued unnecessary, Amit Shah, 2009/10/07
- [Qemu-devel] [PATCH v2 2/3] char: rename CHR_EVENT_RESET to CHR_EVENT_OPENED, Amit Shah, 2009/10/07
- [Qemu-devel] [PATCH v2 3/3] char: emit the OPENED event only when a new char connection is opened, Amit Shah, 2009/10/07
- [Qemu-devel] Re: [PATCH v2 3/3] char: emit the OPENED event only when a new char connection is opened, Jan Kiszka, 2009/10/24
- Re: [Qemu-devel] Re: [PATCH v2 3/3] char: emit the OPENED event only when a new char connection is opened, Amit Shah, 2009/10/25
- Re: [Qemu-devel] Re: [PATCH v2 3/3] char: emit the OPENED event only when a new char connection is opened,
Jan Kiszka <=
- Re: [Qemu-devel] Re: [PATCH v2 3/3] char: emit the OPENED event only when a new char connection is opened, Amit Shah, 2009/10/26
- Re: [Qemu-devel] Re: [PATCH v2 3/3] char: emit the OPENED event only when a new char connection is opened, Jan Kiszka, 2009/10/26
- Re: [Qemu-devel] Re: [PATCH v2 3/3] char: emit the OPENED event only when a new char connection is opened, Amit Shah, 2009/10/27
- Re: [Qemu-devel] Re: [PATCH v2 3/3] char: emit the OPENED event only when a new char connection is opened, Kevin Wolf, 2009/10/27
- Re: [Qemu-devel] Re: [PATCH v2 3/3] char: emit the OPENED event only when a new char connection is opened, Amit Shah, 2009/10/27
- Re: [Qemu-devel] Re: [PATCH v2 3/3] char: emit the OPENED event only when a new char connection is opened, Anthony Liguori, 2009/10/27
- Re: [Qemu-devel] Re: [PATCH v2 3/3] char: emit the OPENED event only when a new char connection is opened, Amit Shah, 2009/10/27
- Re: [Qemu-devel] Re: [PATCH v2 3/3] char: emit the OPENED event only when a new char connection is opened, Kevin Wolf, 2009/10/27