From patchwork Wed May 26 20:06:11 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Frank Mehnert X-Patchwork-Id: 53661 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [199.232.76.165]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id C27DEB7D17 for ; Thu, 27 May 2010 06:52:18 +1000 (EST) Received: from localhost ([127.0.0.1]:41348 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OHNaI-0005Em-J0 for incoming@patchwork.ozlabs.org; Wed, 26 May 2010 16:52:14 -0400 Received: from [140.186.70.92] (port=47178 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OHMs6-00079O-6B for qemu-devel@nongnu.org; Wed, 26 May 2010 16:06:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OHMs2-0007ey-Hg for qemu-devel@nongnu.org; Wed, 26 May 2010 16:06:32 -0400 Received: from gmp-eb-inf-1.sun.com ([192.18.6.21]:60733) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OHMs2-0007eP-AM for qemu-devel@nongnu.org; Wed, 26 May 2010 16:06:30 -0400 Received: from fe-emea-10.sun.com (gmp-eb-lb-1-fe1.eu.sun.com [192.18.6.7] (may be forged)) by gmp-eb-inf-1.sun.com (8.13.7+Sun/8.12.9) with ESMTP id o4QK6Sxs002190 for ; Wed, 26 May 2010 20:06:28 GMT MIME-version: 1.0 Content-disposition: inline Content-type: text/plain; charset=iso-8859-1 Received: from conversion-daemon.fe-emea-10.sun.com by fe-emea-10.sun.com (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul 2 2009)) id <0L3100000M9SX200@fe-emea-10.sun.com> for qemu-devel@nongnu.org; Wed, 26 May 2010 21:06:16 +0100 (BST) Received: from vpn-129-150-124-91.germany.sun.com ([unknown] [129.150.124.91]) by fe-emea-10.sun.com (Sun Java(tm) System Messaging Server 7u2-7.04 64bit (built Jul 2 2009)) with ESMTPSA id <0L3100L4KMIFVDF0@fe-emea-10.sun.com>; Wed, 26 May 2010 21:06:16 +0100 (BST) Date: Wed, 26 May 2010 22:06:11 +0200 From: Frank Mehnert Subject: Re: [Qemu-devel] Re: hw/serial.c: Xmit fifo never used In-reply-to: To: qemu-devel@nongnu.org Message-id: <201005262206.11919.frank.mehnert@sun.com> Organization: Sun Microsystems Content-transfer-encoding: QUOTED-PRINTABLE References: <201005261207.31310.frank.mehnert@sun.com> <4BFD02FC.6020401@siemens.com> User-Agent: KMail/1.9.9 X-detected-operating-system: by eggs.gnu.org: Solaris 10 (beta) Cc: Jan Kiszka , Stefano Stabellini X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org On Wednesday 26 May 2010, Stefano Stabellini wrote: > I think the patch is correct. serial: fixed bug which prevented the use of the xmit fifo Signed-off-by: Frank Mehnert --- hw/serial.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) return; -- 1.5.6.5 Hope this is now the correct format. Kind regards, Frank diff --git a/hw/serial.c b/hw/serial.c index 9102edb..0b1550b 100644 --- a/hw/serial.c +++ b/hw/serial.c @@ -327,7 +327,7 @@ static void serial_xmit(void *opaque) /* in loopback mode, say that we just received a char */ serial_receive1(s, &s->tsr, 1); } else if (qemu_chr_write(s->chr, &s->tsr, 1) != 1) { - if ((s->tsr_retry > 0) && (s->tsr_retry <= MAX_XMIT_RETRY)) { + if ((s->tsr_retry >= 0) && (s->tsr_retry <= MAX_XMIT_RETRY)) { s->tsr_retry++; qemu_mod_timer(s->transmit_timer, new_xmit_ts + s->char_transmit_time);