[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH] chardev: Restore CR,LF on stdio
From: |
Peter Maydell |
Subject: |
Re: [Qemu-trivial] [PATCH] chardev: Restore CR,LF on stdio |
Date: |
Fri, 8 Jun 2018 16:25:07 +0100 |
On 8 June 2018 at 06:47, Thomas Huth <address@hidden> wrote:
> On 07.06.2018 23:08, Philippe Mathieu-Daudé wrote:
>> Remove the 'stair-step output' on stdio.
>>
>> This partially reverts commit 12fb0ac05, which was correct
>> on the mailing list but got corrupted by the maintainer :p
>>
>> Introduced-by: address@hidden
>> Reported-by: BALATON Zoltan <address@hidden>
>> Suggested-by: Thomas Huth <address@hidden>
>> Tested-by: Laurent Desnogues <address@hidden>
>> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
>> ---
>> See:
>> http://lists.nongnu.org/archive/html/qemu-devel/2018-05/msg06202.html (bug)
>> http://lists.nongnu.org/archive/html/qemu-devel/2018-06/msg01309.html
>> (report)
>>
>> Peter, Can this enters directly as bug-fix?
>>
>> chardev/char-stdio.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/chardev/char-stdio.c b/chardev/char-stdio.c
>> index d83e60e787..96375f2ab8 100644
>> --- a/chardev/char-stdio.c
>> +++ b/chardev/char-stdio.c
>> @@ -59,7 +59,7 @@ static void qemu_chr_set_echo_stdio(Chardev *chr, bool
>> echo)
>> if (!echo) {
>> tty.c_iflag &= ~(IGNBRK | BRKINT | PARMRK | ISTRIP
>> | INLCR | IGNCR | ICRNL | IXON);
>> - tty.c_oflag &= ~OPOST;
>> + tty.c_oflag |= OPOST;
>> tty.c_lflag &= ~(ECHO | ECHONL | ICANON | IEXTEN);
>> tty.c_cflag &= ~(CSIZE | PARENB);
>> tty.c_cflag |= CS8;
>>
>
> I think this is the right way to go.
>
> Reviewed-by: Thomas Huth <address@hidden>
Applied to master, thanks.
-- PMM