From owner-freebsd-net@FreeBSD.ORG Wed Mar 16 16:43:01 2011 Return-Path: Delivered-To: freebsd-net@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id CC8B11065670 for ; Wed, 16 Mar 2011 16:43:01 +0000 (UTC) (envelope-from pyunyh@gmail.com) Received: from mail-iw0-f182.google.com (mail-iw0-f182.google.com [209.85.214.182]) by mx1.freebsd.org (Postfix) with ESMTP id 853418FC08 for ; Wed, 16 Mar 2011 16:43:01 +0000 (UTC) Received: by iwn33 with SMTP id 33so2159317iwn.13 for ; Wed, 16 Mar 2011 09:43:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:from:date:to:cc:subject:message-id:reply-to :references:mime-version:content-type:content-disposition :in-reply-to:user-agent; bh=Wv9bbi5q7L9JnC1q5SzTiyAnuBtZhMA48jV8z9WQGsw=; b=xxgJjmLd40bIj+krYy46R5cZwmmBssrV1VcUFs02avy9sJ5u38U5eWvjEqB6zgs/6O Ui3bmc+2FtOmFPeNMmRQUHccu4vMjyTdB+i7Vs1Y6GH8RDoqa2xkQ8jhij80NSSIYYVn k5eIjvG8oeXHx759Nk0raaXXAGMCOjF/+iz+k= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=from:date:to:cc:subject:message-id:reply-to:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; b=DPiVutr0fnzdvn6EQQVFZaTQASDONp8sJ6RtJLIY+BQR+2pezta0n30Y8xrNQBTl++ UWT5a2xnWvrm4Flx5wI4VbxM+XpiMa1z4Iq4/nZW55f/gTBLf1AgJz8aHbch1vvEXWf3 CaU7SF7YeHlHkZxvDOX5XFUQ1gl/dmGbSwBYI= Received: by 10.42.142.133 with SMTP id s5mr330327icu.64.1300293780478; Wed, 16 Mar 2011 09:43:00 -0700 (PDT) Received: from pyunyh@gmail.com ([174.35.1.224]) by mx.google.com with ESMTPS id i20sm36207iby.48.2011.03.16.09.42.55 (version=TLSv1/SSLv3 cipher=OTHER); Wed, 16 Mar 2011 09:42:57 -0700 (PDT) Received: by pyunyh@gmail.com (sSMTP sendmail emulation); Wed, 16 Mar 2011 09:41:49 -0700 From: YongHyeon PYUN Date: Wed, 16 Mar 2011 09:41:49 -0700 To: Vijay Singh Message-ID: <20110316164149.GA9004@michelle.cdnetworks.com> References: <20110315024325.GG1577@michelle.cdnetworks.com> <20110315233844.GE5076@michelle.cdnetworks.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20110315233844.GE5076@michelle.cdnetworks.com> User-Agent: Mutt/1.4.2.3i Cc: freebsd-net@freebsd.org Subject: Re: Jumbo frame support for BGE_ASICREV_BCM5714 X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: pyunyh@gmail.com List-Id: Networking and TCP/IP with FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 16 Mar 2011 16:43:02 -0000 On Tue, Mar 15, 2011 at 04:38:44PM -0700, YongHyeon PYUN wrote: > On Mon, Mar 14, 2011 at 09:40:36PM -0700, Vijay Singh wrote: > > > As you know, BCM5714, BCM5715 and BCM5780 use unique jumbo frame > > > scheme that is not compatible with other controllers. All other > > > Broadcom controllers have better jumbo frame scheme. These > > > controllers have one send ring, one standard receive producer ring > > > and one receive return ring. In order to receive jumbo frames on > > > these controllers you have to increase Rx buffer size to hold 9k > > > sized jumbo frame. Two Rx modes(standard Rx BDs and extended Rx > > > BDs) are supported for these controllers. Using extended Rx BDs on > > > BCM5714/BCM5715/BCM5780 reduces the number of Rx BDs to 256 entries > > > which shall reduce the performance. I would use standard Rx BDs to > > > hold 512 entries for RX buffers. > > > > > > I think I received jumbo frame support request for these > > > controllers in past. At that time I had no interests on > > > implementing it due to severe implementation differences. What is > > > your main reason to use jumbo frame on this controller? What is > > > your expectation on performance numbers? I guess no other OSes > > > support jumbo frame on this controller. > > > > Hi Pyun, I am stuck with this NIC due to it being the one present in > > the HW platform that I have to support. The performance is expectation > > is mainly for apps that use large payloads (where something TSO would > > have helped). > > > > Here is experimental patch which I tried not to penalize other > controllers. I don't have BCM5714 controllers so I don't know > whether it works or not. The patch was generated against HEAD and > it would be cleanly applied to 8.2R/7.4R. Due to large bge(4) > changes I guess it wouldn't be applied to 7.2R. But I guess you can > install 8.2R/7.4R to one of your box and experiment this patch for > a while then you can backport this to 7.2R. > You can find the patch at the following URL. > http://people.freebsd.org/~yongari/bge/bge.5714.jumbo.diff > There was a bug in the diff. I updated the diff but URL is the same as before. If you have downloaded the file, please try again. > > -vijay