[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] qtest: use synchronous I/O for char device
From: |
Wenchao Xia |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] qtest: use synchronous I/O for char device |
Date: |
Wed, 27 Mar 2013 15:22:00 +0800 |
User-agent: |
Mozilla/5.0 (Windows NT 5.1; rv:17.0) Gecko/20130307 Thunderbird/17.0.4 |
Reviewed-by: Wenchao Xia <address@hidden>
> Peter reported that rtc-test would periodically hang. It turns out
> this was due to an EAGAIN occurring on qemu_chr_fe_write.
>
> Instead of heavily refactoring qtest, just use a synchronous version
> of the write operation for qemu_chr_fe_write to address this problem.
>
> Reported-by: Peter Maydell <address@hidden>
> Signed-off-by: Anthony Liguori <address@hidden>
> ---
> qtest.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/qtest.c b/qtest.c
> index 5e0e9ec..b03b68a 100644
> --- a/qtest.c
> +++ b/qtest.c
> @@ -191,7 +191,7 @@ static void GCC_FMT_ATTR(2, 3) qtest_send(CharDriverState
> *chr,
> len = vsnprintf(buffer, sizeof(buffer), fmt, ap);
> va_end(ap);
>
> - qemu_chr_fe_write(chr, (uint8_t *)buffer, len);
> + qemu_chr_fe_write_all(chr, (uint8_t *)buffer, len);
> if (qtest_log_fp && qtest_opened) {
> fprintf(qtest_log_fp, "%s", buffer);
> }
>
--
Best Regards
Wenchao Xia