[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: more about serial ports: do they even work?
From: |
Michael Tokarev |
Subject: |
[Qemu-devel] Re: more about serial ports: do they even work? |
Date: |
Tue, 03 Feb 2009 11:13:46 +0300 |
User-agent: |
Mozilla-Thunderbird 2.0.0.19 (X11/20090103) |
David S. Ahern wrote:
> Michael Tokarev wrote:
>> David S. Ahern wrote:
>>> I don't recall seeing a followup to this post.
>>>
>>> To put Michael's second suggestion into patch form, the following fixes
>>> the problem for me:
>>>
>>> --- kvm-81.orig/qemu/qemu-char.c 2008-12-14 06:16:27.000000000 -0700
>>> +++ kvm-81/qemu/qemu-char.c 2009-02-02 14:12:20.000000000 -0700
>>> @@ -1078,20 +1078,21 @@
>>> if (sarg | TIOCM_DTR)
>>> *targ |= CHR_TIOCM_DTR;
>>> if (sarg | TIOCM_RTS)
>>> *targ |= CHR_TIOCM_RTS;
>>> }
>>> break;
>>> case CHR_IOCTL_SERIAL_SET_TIOCM:
>>> {
>>> int sarg = *(int *)arg;
>>> int targ = 0;
>>> + ioctl(s->fd_in, TIOCMGET, &targ);
>> here, one more operation is necessary:
>> targ &= ~(TIOCM_DTR|TIOCM_RTS);
>
> Interesting. that change was not needed to fix my problem.
It just means you (or, rather, your guests) never really needed to
DROP those signal lines, only to raise them.
>>> if (sarg | CHR_TIOCM_DTR)
>>> targ |= TIOCM_DTR;
>>> if (sarg | CHR_TIOCM_RTS)
>>> targ |= TIOCM_RTS;
Without that line above, the code never drops the two bits, once
set they can't be "removed" anymore.
By the way, this is upstream qemu issue, not kvm one, and has to be
pushed as such. Good you CC'd qemu list.
/mjt
- [Qemu-devel] Re: more about serial ports: do they even work?, David S. Ahern, 2009/02/02
- [Qemu-devel] Re: more about serial ports: do they even work?, Michael Tokarev, 2009/02/03
- [Qemu-devel] Re: more about serial ports: do they even work?, David S. Ahern, 2009/02/02
- [Qemu-devel] Re: more about serial ports: do they even work?,
Michael Tokarev <=
- [Qemu-devel] Re: more about serial ports: do they even work?, Mark Marshall, 2009/02/03
- [Qemu-devel] Re: more about serial ports: do they even work?, Marcelo Tosatti, 2009/02/04
- [Qemu-devel] Re: more about serial ports: do they even work?, Michael Tokarev, 2009/02/04
- [Qemu-devel] Re: more about serial ports: do they even work?, Marcelo Tosatti, 2009/02/04
- [Qemu-devel] Re: more about serial ports: do they even work?, Michael Tokarev, 2009/02/04
- [Qemu-devel] Re: more about serial ports: do they even work?, Stefano Stabellini, 2009/02/04
- [Qemu-devel] Re: more about serial ports: do they even work?, David S. Ahern, 2009/02/05
- [Qemu-devel] Re: more about serial ports: do they even work?, Stefano Stabellini, 2009/02/05
- Re: [Qemu-devel] Re: more about serial ports: do they even work?, Stefano Stabellini, 2009/02/04