From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nivedita Singhvi Subject: Re: [PATCH] qemu pcnet emulation fixes Date: Fri, 24 Mar 2006 13:10:14 -0800 Message-ID: <44246036.8020303@us.ibm.com> References: <20060324202640.GA27291@us.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20060324202640.GA27291@us.ibm.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Don Fry Cc: xen-devel@lists.xensource.com List-Id: xen-devel@lists.xenproject.org Don Fry wrote: > The attached patch to the qemu emulation of the pcnet hardware fixes > several problems. It will now only read and write a transmit or receive > descriptor once. It will correctly handle transmitting frames with more > than two fragments. It will discard oversize frames instead of > corrupting memory. I have tested all the changes I have made and even > seen an improvement in receive performance from 1.7MB/s to 3.3MB/s. > Your mileage will vary. Don, the better performance due to the fact that we aren't reading some descriptors multiple times for no good reason any more, correct? > The code could be simplified if multi-fragment frames were deleted. It > appears that both Linux and Windows XP don't use fragmented frames. I'd suggest we keep this for at least long enough to validate that the other OSs we want to support don't use them either. Thanks very much! thanks, Nivedita