Re: [PATCH v2 3/5] semihosting: add qemu_semihosting_console_inc for SYS_READC

[Richard Henderson <richard.henderson@linaro.org>]

On 12/21/19 12:22 AM, Alex Bennée wrote:
> From: Keith Packard <keithp@keithp.com>
> 
> Provides a blocking call to read a character from the console using
> semihosting.chardev, if specified. This takes some careful command
> line options to use stdio successfully as the serial ports, monitor
> and semihost all want to use stdio. Here's a sample set of command
> line options which share stdio between semihost, monitor and serial
> ports:
> 
> 	qemu \
> 	-chardev stdio,mux=on,id=stdio0 \
> 	-serial chardev:stdio0 \
> 	-semihosting-config enable=on,chardev=stdio0 \
> 	-mon chardev=stdio0,mode=readline
> 
> This creates a chardev hooked to stdio and then connects all of the
> subsystems to it. A shorter mechanism would be good to hear about.
> 
> Signed-off-by: Keith Packard <keithp@keithp.com>
> Message-Id: <20191104204230.12249-1-keithp@keithp.com>
> [AJB: fixed up deadlock, minor commit title reword]
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> Cc: Paolo Bonzini <pbonzini@redhat.com>
> Reviewed-by: Keith Packard <keithp@keithp.com>
> Tested-by: Keith Packard <keithp@keithp.com>

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>


r~