From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1Mgfk1-0003lr-DY for qemu-devel@nongnu.org; Thu, 27 Aug 2009 10:14:17 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1Mgfjw-0003gh-Jc for qemu-devel@nongnu.org; Thu, 27 Aug 2009 10:14:17 -0400 Received: from [199.232.76.173] (port=53015 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Mgfjw-0003gU-FS for qemu-devel@nongnu.org; Thu, 27 Aug 2009 10:14:12 -0400 Received: from e35.co.us.ibm.com ([32.97.110.153]:37142) by monty-python.gnu.org with esmtps (TLS-1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.60) (envelope-from ) id 1Mgfjw-0005NX-4i for qemu-devel@nongnu.org; Thu, 27 Aug 2009 10:14:12 -0400 Received: from d03relay05.boulder.ibm.com (d03relay05.boulder.ibm.com [9.17.195.107]) by e35.co.us.ibm.com (8.14.3/8.13.1) with ESMTP id n7RE4fUs031462 for ; Thu, 27 Aug 2009 08:04:41 -0600 Received: from d03av04.boulder.ibm.com (d03av04.boulder.ibm.com [9.17.195.170]) by d03relay05.boulder.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id n7REDekW104422 for ; Thu, 27 Aug 2009 08:13:41 -0600 Received: from d03av04.boulder.ibm.com (loopback [127.0.0.1]) by d03av04.boulder.ibm.com (8.12.11.20060308/8.13.3) with ESMTP id n7REDXJw015430 for ; Thu, 27 Aug 2009 08:13:40 -0600 From: Ryan Arnold In-Reply-To: <20090827065251.GB3218@amit-x200.redhat.com> References: <1251181044-3696-1-git-send-email-amit.shah@redhat.com> <20090826112718.GA11117@amit-x200.redhat.com> <20090826154552.GA31910@amit-x200.redhat.com> <1251349485.6267.6.camel@concordia> <20090827065251.GB3218@amit-x200.redhat.com> Content-Type: text/plain Date: Thu, 27 Aug 2009 09:13:28 -0500 Message-Id: <1251382408.3063.9.camel@localhost.localdomain> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [Qemu-devel] Re: Extending virtio_console to support multiple ports Reply-To: rsa@us.ibm.com List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Amit Shah Cc: kvm@vger.kernel.org, linuxppc-dev@ozlabs.org, linux-kernel@vger.kernel.org, miltonm@bga.com, qemu-devel@nongnu.org, Michael Ellerman , borntraeger@de.ibm.com, brueckner@linux.vnet.ibm.com, virtualization@lists.linux-foundation.org, alan@linux.intel.com On Thu, 2009-08-27 at 12:22 +0530, Amit Shah wrote: > On (Thu) Aug 27 2009 [15:04:45], Michael Ellerman wrote: > Ryan you called his code "pure legacy baggage" if you > > don't ;) > > > > http://git.kernel.org/?p=linux/kernel/git/torvalds/old-2.6-bkcvs.git;a=commitdiff;h=d450b4ae023fb4be175389c18f4f87677da03020 > > Thanks for the link! > > (and this general area might be the one that doesn't get major upheavals > in 5-yr spans :-) Actually, quite the opposite... too many cooks in the kitchen in the last 6 years have left this code with some historical oddities. When I added the interrupt context code in the first place I made sure that the driver did NOTHING in the interrupt handler except awaken the working thread. Someone in the subsequent years thought it'd be a good idea to have the interrupt handler actually do the work. Furthermore the front and backends were separated so that new front ends can be used. This was sometime in the last 5 years as well. Let's just say I'm happy to pass the torch on this one. I would suggest that someone perform throughput tests on this driver at some point. At the point where I stopped maintaining it I'd gotten the driver to the point where there was no jerking output or lost data under heavy I/O load. I'm not sure where the driver's at now. Ryan S. Arnold