From owner-cvs-all Wed Nov 28 13:16:55 2001 Delivered-To: cvs-all@freebsd.org Received: from peter3.wemm.org (c1315225-a.plstn1.sfba.home.com [24.14.150.180]) by hub.freebsd.org (Postfix) with ESMTP id A627037B417; Wed, 28 Nov 2001 13:16:41 -0800 (PST) Received: from overcee.netplex.com.au (overcee.wemm.org [10.0.0.3]) by peter3.wemm.org (8.11.0/8.11.0) with ESMTP id fASLGfM82447; Wed, 28 Nov 2001 13:16:41 -0800 (PST) (envelope-from peter@wemm.org) Received: from wemm.org (localhost [127.0.0.1]) by overcee.netplex.com.au (Postfix) with ESMTP id 5FF9E3808; Wed, 28 Nov 2001 13:16:41 -0800 (PST) (envelope-from peter@wemm.org) X-Mailer: exmh version 2.5 07/13/2001 with nmh-1.0.4 To: Luigi Rizzo Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/pci if_sis.c In-Reply-To: <200111281610.fASGAcU63381@freefall.freebsd.org> Date: Wed, 28 Nov 2001 13:16:41 -0800 From: Peter Wemm Message-Id: <20011128211641.5FF9E3808@overcee.netplex.com.au> Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG Luigi Rizzo wrote: > luigi 2001/11/28 08:10:37 PST > > Modified files: > sys/pci if_sis.c > Log: > Remove the need for an expensive m_devget on the i386, which does not > have alignment problems. Yes it does, just not fatal ones. Different models of the x86 family have different characteristics doing these. > On small boxes (e.g. the net4501 from Soekris, featuring a 486/133) > this provides huge performance benefits: the peak forwarding rate > with avg.sized packets goes up by 50-70% because of this change > alone. Faster CPUs might benefit less from this change, but in any > case the CPU has better things to do than waste time on useless > memory-to-memory copies. While this helps things like packet forwarding, it hurts things like NFS which now have to do lots and lots of unaligned accesses. Have you benchmarked anything else besides packet forwarding? > Right now the new behaviour is controlled by a sysctl variable, > hw.sis_quick which defaults to 1 (on), you can set it to 0 to > reintroduce the old behaviour (and compare the results). The > variable is only there to show how much you can gain with this > change, it will go away soon. Please do not remove this yet. Cheers, -Peter -- Peter Wemm - peter@FreeBSD.org; peter@yahoo-inc.com; peter@netplex.com.au "All of this is for nothing if we don't go to the stars" - JMS/B5 To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message