Your message dated Mon, 20 Feb 2006 00:02:19 -0800
with message-id <[EMAIL PROTECTED]>
and subject line Bug#348098: fixed in qemu 0.8.0-2
has caused the attached Bug report to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what I am
talking about this indicates a serious mail system misconfiguration
somewhere.  Please contact me immediately.)

Debian bug tracking system administrator
(administrator, Debian Bugs database)

--- Begin Message ---
Package: qemu
Version: 0.7.0-4
Severity: normal
Tags: patch

Hi,

Extract of hw/serial.c:

struct SerialState {
    uint8_t divider;

...

static uint32_t serial_ioport_read(void *opaque, uint32_t addr)
{   
    SerialState *s = opaque;
    ...
            ret = (s->divider >> 8) & 0xff;


This simply can't work with uint8_t divider! :)
Some systems use this as register probe, so that they can't find the
serial port...

Here is a patch for widening divider to 16bits.

-- System Information:
Debian Release: testing/unstable
  APT prefers testing
  APT policy: (900, 'testing'), (500, 'unstable'), (500, 'stable')
Architecture: i386 (i686)
Shell:  /bin/sh linked to /bin/bash
Kernel: Linux 2.6.15
Locale: [EMAIL PROTECTED], [EMAIL PROTECTED] (charmap=ISO-8859-15)

Versions of packages qemu depends on:
ii  bochsbios                     2.2.1-2    BIOS for the Bochs emulator
ii  libc6                         2.3.5-8    GNU C Library: Shared libraries an
ii  libsdl1.2debian               1.2.9-0.0  Simple DirectMedia Layer
ii  openhackware                  0.4.1-1    OpenFirmware emulator for PowerPC
ii  proll                         18-1       JavaStation PROM 2.x compatible re
ii  vgabios                       0.5d-1     VGA BIOS software for the Bochs an
ii  zlib1g                        1:1.2.3-9  compression library - runtime

Versions of packages qemu recommends:
ii  debootstrap                   0.3.3      Bootstrap a basic Debian system
ii  sharutils                     1:4.2.1-15 shar, unshar, uuencode, uudecode

-- no debconf information

-- 
Samuel Thibault <[EMAIL PROTECTED]>
I am the "ILOVEGNU" signature virus. Just copy me to your signature.
This email was infected under the terms of the GNU General Public License.
diff -ur qemu-0.8.0/hw/serial.c qemu-0.8.0-mine/hw/serial.c
--- qemu-0.8.0/hw/serial.c      2005-12-19 23:51:53.000000000 +0100
+++ qemu-0.8.0-mine/hw/serial.c 2006-01-14 20:33:41.000000000 +0100
@@ -71,7 +71,7 @@
 #define UART_LSR_DR    0x01    /* Receiver data ready */
 
 struct SerialState {
-    uint8_t divider;
+    uint16_t divider;
     uint8_t rbr; /* receive register */
     uint8_t ier;
     uint8_t iir; /* read only */
@@ -311,7 +311,7 @@
 {
     SerialState *s = opaque;
 
-    qemu_put_8s(f,&s->divider);
+    qemu_put_be16s(f,&s->divider);
     qemu_put_8s(f,&s->rbr);
     qemu_put_8s(f,&s->ier);
     qemu_put_8s(f,&s->iir);
@@ -329,7 +329,7 @@
     if(version_id != 1)
         return -EINVAL;
 
-    qemu_get_8s(f,&s->divider);
+    qemu_get_be16s(f,&s->divider);
     qemu_get_8s(f,&s->rbr);
     qemu_get_8s(f,&s->ier);
     qemu_get_8s(f,&s->iir);

--- End Message ---
--- Begin Message ---
Source: qemu
Source-Version: 0.8.0-2

We believe that the bug you reported is fixed in the latest version of
qemu, which is due to be installed in the Debian FTP archive:

qemu_0.8.0-2.diff.gz
  to pool/main/q/qemu/qemu_0.8.0-2.diff.gz
qemu_0.8.0-2.dsc
  to pool/main/q/qemu/qemu_0.8.0-2.dsc
qemu_0.8.0-2_i386.deb
  to pool/main/q/qemu/qemu_0.8.0-2_i386.deb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [EMAIL PROTECTED],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Guillem Jover <[EMAIL PROTECTED]> (supplier of updated qemu package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [EMAIL PROTECTED])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Mon, 20 Feb 2006 09:17:46 +0200
Source: qemu
Binary: qemu
Architecture: source i386
Version: 0.8.0-2
Distribution: unstable
Urgency: low
Maintainer: Debian QEMU Team <[EMAIL PROTECTED]>
Changed-By: Guillem Jover <[EMAIL PROTECTED]>
Description: 
 qemu       - fast processor emulator
Closes: 276276 345772 348098
Changes: 
 qemu (0.8.0-2) unstable; urgency=low
 .
   [ Guillem Jover ]
   * Switch away from cdbs to plain debhelper.
   * Upgrade to debhelper compat level 5.
   * Allow overriding CC compiler variable. (Closes: #345772)
   * Do not redefine 64 bit types on 64 bit arches.
     - debian/patches/61_safe_64bit_int.patch: New file.
   * Allow linux_boot.bin to be built on any arch by switching to nasm,
     and Build-Depending on it.
     - debian/patches/62_linux_boot_nasm.patch: New file.
   * The serial hw driver uses a small divider that gets zeroed when shifting
     bits to the right. (Closes: #276276, #348098)
     - debian/patches/51_serial_small_divider.patch: New file.
     Thanks to Samuel Thibault <[EMAIL PROTECTED]>.
   * Escaped hyphens in qemu-user manpage, use italics for filenames and
     parameters and bold for options.
   * Partial build failure fix for Sparc. (Bugs: #317145, #336970)
     Thanks to Jurij Smakov <[EMAIL PROTECTED]>.
Files: 
 9243972564b4051bd3b87d3175ff458c 954 misc optional qemu_0.8.0-2.dsc
 0a80a04dce2698d555f21f28b7d180b7 28596 misc optional qemu_0.8.0-2.diff.gz
 c52e8fb2281e7c9562454514734049da 2759948 misc optional qemu_0.8.0-2_i386.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (GNU/Linux)

iD8DBQFD+XS2uW9ciZ2SjJsRAviSAKDQXqcNiQPgwLZVPcA+H+0rHjWkzQCeNBL6
/5PsKxojby7PAK6kAX8lIX4=
=4p2e
-----END PGP SIGNATURE-----


--- End Message ---

Reply via email to