From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:46679) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SulJO-0001sw-UJ for qemu-devel@nongnu.org; Fri, 27 Jul 2012 10:14:43 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SulJI-0005Ct-Sk for qemu-devel@nongnu.org; Fri, 27 Jul 2012 10:14:38 -0400 Received: from mail-ob0-f173.google.com ([209.85.214.173]:38141) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SulJI-0005Cn-NW for qemu-devel@nongnu.org; Fri, 27 Jul 2012 10:14:32 -0400 Received: by obbta14 with SMTP id ta14so4116335obb.4 for ; Fri, 27 Jul 2012 07:14:31 -0700 (PDT) Message-ID: <50129B8A.3060102@landley.net> Date: Fri, 27 Jul 2012 08:45:46 -0500 From: Rob Landley MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] [PATCH] Fixing sh4 serial abort List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: aboriginal@lists.landley.net, qemu-devel If you grab the current aboriginal linux build scripts: http://landley.net/hg/aboriginal/archive/tip.tar.bz2 And "./build.sh sh4", then cd to build/system-image-sh4 and "./run-emulator.sh" you get this: sh_serial: unsupported read from 0x18 Aborted The bug was triggered by linux kernel commit 73c3d53f38e0a8e6 back between v3.2 and v3.3, which did this: --- a/drivers/tty/serial/sh-sci.c +++ b/drivers/tty/serial/sh-sci.c @@ -1771,18 +1771,25 @@ static void sci_set_termios(struct uart_port *port, struct ktermios *termios, sci_init_pins(port, termios->c_cflag); - if (s->cfg->capabilities & SCIx_HAVE_RTSCTS) { - reg = sci_getreg(port, SCFCR); - if (reg->size) { - unsigned short ctrl; + reg = sci_getreg(port, SCFCR); + if (reg->size) { + unsigned short ctrl = sci_in(port, SCFCR); - ctrl = sci_in(port, SCFCR); + if (s->cfg->capabilities & SCIx_HAVE_RTSCTS) { I.E. sci_getreg(port, SCFCR) move to before checking whether or not we'll ever possibly use the result. SCFCR is 0x18 and QEMU calls abort() on an attempt to read from an unimplemented register. I can patch the kernel to work around this (and probably will for this release), but the _proper_ fix is to get qemu not to abort on a register read that works fine if it just returns 0. It turns out the qemu fix (in current git) is just: --- a/hw/sh_serial.c +++ b/hw/sh_serial.c @@ -248,11 +248,9 @@ static uint64_t sh_serial_read(void *opaque, target_phys s->flags &= ~SH_SERIAL_FLAG_RDF; } break; -#if 0 case 0x18: ret = s->fcr; break; -#endif case 0x1c: ret = s->rx_cnt; break; (It can also just do ret = 0; and that works too. Or comment out the abort() near the end of the function.) Doing that, qemu boots the sh4 system image to a shell prompt, and the result compiles "hello world" natively. By the way, this board emulation (r2d) only has 64 megs ram. Is there an easy way to get 256 megs out of it? Rob -- GNU/Linux isn't: Linux=GPLv2, GNU=GPLv3+, they can't share code. Either it's "mere aggregation", or a license violation. Pick one.