From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1NiiGG-0004tH-66 for qemu-devel@nongnu.org; Sat, 20 Feb 2010 00:52:16 -0500 Received: from [199.232.76.173] (port=60057 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1NiiGF-0004sr-C2 for qemu-devel@nongnu.org; Sat, 20 Feb 2010 00:52:15 -0500 Received: from Debian-exim by monty-python.gnu.org with spam-scanned (Exim 4.60) (envelope-from ) id 1NiiGE-0003sb-J6 for qemu-devel@nongnu.org; Sat, 20 Feb 2010 00:52:15 -0500 Received: from mail-fx0-f214.google.com ([209.85.220.214]:48810) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1NiiGE-0003sX-Ao for qemu-devel@nongnu.org; Sat, 20 Feb 2010 00:52:14 -0500 Received: by fxm6 with SMTP id 6so791699fxm.2 for ; Fri, 19 Feb 2010 21:52:12 -0800 (PST) MIME-Version: 1.0 In-Reply-To: <4B7F1E3E.6090908@codemonkey.ws> References: <1266309345-2889-1-git-send-email-freddy77@gmail.com> <4B7F0796.4000706@codemonkey.ws> <4B7F1E3E.6090908@codemonkey.ws> Date: Sat, 20 Feb 2010 08:52:12 +0300 Message-ID: Subject: Re: [Qemu-devel] [PATCH] rewrote timer implementation for rtl8139. From: Igor Kovalenko Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony Liguori Cc: qemu-devel@nongnu.org, Alexander Graf , Frediano Ziglio On Sat, Feb 20, 2010 at 2:26 AM, Anthony Liguori wr= ote: > On 02/19/2010 04:22 PM, Alexander Graf wrote: >> >> On 19.02.2010, at 22:50, Anthony Liguori wrote: >> >> >>> >>> On 02/16/2010 02:35 AM, Frediano Ziglio wrote: >>> >>>> >>>> Add a QEMU timer only when needed (timeout status not set, timeout >>>> irq wanted and timer set). >>>> >>>> This patch is required for Darwin. Patch has been tested under >>>> FreeBSD, Darwin and Linux. >>>> >>>> Signed-off-by: Frediano Ziglio >>>> >>>> >>> >>> Alex, I know you've played with this feature for OS X in the past. >>> >>> Is this something we should be enabling unconditionally? >>> >> >> I've played with it and realized that >> >> 1) OSX's rtl8139 driver uses this timer instead of tx/rx interrupts to >> process the queue >> 2) it was broken >> >> I haven't gotten around to test if this patch makes it work, but I'd >> suppose so. >> >> So yes, I think it should be enabled by default. It's a feature real hw >> has, so we should emulate it. >> > > The thing that concerns me is why it was disabled by default. =A0Igor, wh= at > was the reasoning for disabling it if you recall? It was very inefficient pci timer implementation which fired callback frequently. Once it was found that (freebsd?) was relying on pci timer in realtek drive= r to communicate we just added missing piece. Now this patch fixes that inefficiency issue. --=20 Kind regards, Igor V. Kovalenko