From owner-freebsd-net@FreeBSD.ORG Tue Apr 16 05:25:09 2013 Return-Path: Delivered-To: freebsd-net@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by hub.freebsd.org (Postfix) with ESMTP id BE467631; Tue, 16 Apr 2013 05:25:09 +0000 (UTC) (envelope-from pyunyh@gmail.com) Received: from mail-pb0-f43.google.com (mail-pb0-f43.google.com [209.85.160.43]) by mx1.freebsd.org (Postfix) with ESMTP id 95425D97; Tue, 16 Apr 2013 05:25:09 +0000 (UTC) Received: by mail-pb0-f43.google.com with SMTP id md4so89397pbc.30 for ; Mon, 15 Apr 2013 22:25:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:from:date:to:cc:subject:message-id:reply-to:references :mime-version:content-type:content-disposition:in-reply-to :user-agent; bh=OS99moBMGkxH2tA6F/vftfoCEVaeZAHweIH2qn4bEqU=; b=DsIiq7woR/poEZqZaSSllUH9xrMDK9OBTDv1UqIzAjLYKtR3ljK71xqMNt79dIhE+6 2bnmDASuxhNMUFIgsmQNZ/oPnsLBfrhsS0LifLmBZLuHpN+4v87rhOVNdSkRAmx1kvwX HNw09hrKR0oeYNiU8+j/Jfl3tYfJXJlUmt1kMpateb/O8DaYbUQwsIXdBLQ8NuHnUlbC Ajp5WuzzK64aQMJ07l9WYFP2aFzAAMIqjFJQUoIj54dJAX+NFcxE5vBX8SHOIw/tCEwa z+AVUeiRflC41ErNWraU6JojpmIXpXPlYexocDSybtw0bcJdl5AJOEoZOBvG+NE3xAFh G8sA== X-Received: by 10.66.122.167 with SMTP id lt7mr1517754pab.168.1366089909020; Mon, 15 Apr 2013 22:25:09 -0700 (PDT) Received: from pyunyh@gmail.com (lpe4.p59-icn.cdngp.net. [114.111.62.249]) by mx.google.com with ESMTPS id be7sm1094332pad.20.2013.04.15.22.25.05 (version=TLSv1 cipher=RC4-SHA bits=128/128); Mon, 15 Apr 2013 22:25:07 -0700 (PDT) Received: by pyunyh@gmail.com (sSMTP sendmail emulation); Tue, 16 Apr 2013 14:25:00 +0900 From: YongHyeon PYUN Date: Tue, 16 Apr 2013 14:25:00 +0900 To: Sean Bruno Subject: Re: bge(4) sysctl tuneables -- a blast from the past. Message-ID: <20130416052500.GA1428@michelle.cdnetworks.com> References: <1365781568.1418.1.camel@localhost> <20130413200512.G1165@besplex.bde.org> <1366065356.1350.7.camel@localhost> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1366065356.1350.7.camel@localhost> User-Agent: Mutt/1.4.2.3i Cc: "freebsd-net@freebsd.org" , bde X-BeenThere: freebsd-net@freebsd.org X-Mailman-Version: 2.1.14 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: Tue, 16 Apr 2013 05:25:09 -0000 On Mon, Apr 15, 2013 at 03:35:56PM -0700, Sean Bruno wrote: > > > FreeBSD has too many knobs, but it would be nice if the bge defaults weren't > > so broken, so that they don't need overriding. > > > > Bruce > > > So many knobs ... well here's more. :-) > > http://people.freebsd.org/~sbruno/bge_config_update.txt > > At least this gets a man page update with references to manuals. You have to change BGE_STD_RX_RING_CNT to change number of RX descriptors. It's hard-coded and it needs much more work to change that. And I don't see any reason to modify that though(Max # of RX descriptor is 512). I think bge(4) touches minimal set of coalescing parameters but publicly available bge(4) data sheet shows more coalescing parameters. These parameters could be programmed with different values(BDs & ticks) during interrupt. And some parameters are not applicable to certain controllers. In addition, the allowed value range for certain parameters vary on controller models. So I think it's good idea to mention allowed value range for each parameters as well as a warning that mentions possible connection lost caused by wrongly programmed value(i.e. no RX interrupt for bge_rx_coal_ticks == 0 && bge_rx_max_coal_bds == 0) It's common to see multiple instances of bge(4) in a box so I think it would be better to implement them as sysctl tunables rather than loader tunables(i.e. each controller may need different coalescing parameters). Except hw.bge.allow_asf tunable, all others were implemented to support multiple bge(4) instances. sysctl tunables also allow dynamic change so you don't have to reboot your box to change coalescing parameters. > > Sean