From: Jason Wessel <jason.wessel@windriver.com>
To: qemu-devel@nongnu.org
Cc: Ian Jackson <Ian.Jackson@eu.citrix.com>,
stefano.stabellini@eu.citrix.com
Subject: Re: [Qemu-devel] [PATCH] upgrading emulated UART to 16550A
Date: Tue, 05 Aug 2008 10:02:33 -0500 [thread overview]
Message-ID: <48986B89.8000608@windriver.com> (raw)
In-Reply-To: <48985284.9060001@eu.citrix.com>
Stefano Stabellini wrote:
> This patch upgrades the emulated UART to 16550A, the code comes from
> xen-unstable.
> The main improvement was introduced with the following patch and
> subsequent email thread:
>
> http://lists.xensource.com/archives/html/xen-devel/2007-12/msg00129.html
>
> I only made few changes to use qemu_get_clock instead of
> clock_gettime, as Anthony suggested a little while ago.
>
>
I didn't spend the time to figure out what was wrong, but I thought I'd
try your new port emulation against KGDB which is known to work with
real hardware, as well as the existing 16450 qemu uart support.
For some reason, the initial interrupt if a character comes into the
uart does not seem to get raised, so the kernel routine which handles
the traffic never gets invoked. It is possible that the that the kgdb
kernel driver has a defect, but this does seem to work on real hardware,
so perhaps it is a timing problem or the qemu emulation is not quite
accurate with respect to the real hardware.
If you like, I can provide you a bzImage and the qemu invocation, such
that you could see the problem first hand.
Jason.
next prev parent reply other threads:[~2008-08-05 15:02 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-08-05 13:15 [Qemu-devel] [PATCH] upgrading emulated UART to 16550A Stefano Stabellini
2008-08-05 13:57 ` Thiemo Seufer
2008-08-05 14:17 ` Stefano Stabellini
2008-08-05 15:02 ` Jason Wessel [this message]
2008-08-05 15:15 ` Stefano Stabellini
2008-08-06 2:28 ` Anthony Liguori
-- strict thread matches above, loose matches on Subject: below --
2008-08-07 17:35 Stefano Stabellini
2008-08-07 18:58 ` Anthony Liguori
2008-08-07 21:02 ` Samuel Thibault
2008-08-07 21:13 ` Anthony Liguori
2008-08-07 21:39 ` Samuel Thibault
2008-08-08 14:36 ` Stefano Stabellini
2008-08-07 19:26 ` Jason Wessel
2008-08-08 14:55 Stefano Stabellini
2008-08-08 15:15 ` Stefano Stabellini
2008-08-08 16:58 Stefano Stabellini
2008-08-09 18:26 ` Anthony Liguori
2008-08-09 18:44 ` Samuel Thibault
2008-08-11 14:17 ` Anthony Liguori
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=48986B89.8000608@windriver.com \
--to=jason.wessel@windriver.com \
--cc=Ian.Jackson@eu.citrix.com \
--cc=qemu-devel@nongnu.org \
--cc=stefano.stabellini@eu.citrix.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).