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: 3b876140-c035-dd39-75d0-d54c48128...@redhat.com > Reported-by: BALATON Zoltan <bala...@eik.bme.hu> > Suggested-by: Thomas Huth <th...@redhat.com> > Tested-by: Laurent Desnogues <laurent.desnog...@gmail.com> > Signed-off-by: Philippe Mathieu-Daudé <f4...@amsat.org> > --- > 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 <th...@redhat.com>