From owner-svn-src-all@FreeBSD.ORG  Tue Dec 16 16:16:57 2008
Return-Path: <owner-svn-src-all@FreeBSD.ORG>
Delivered-To: svn-src-all@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id A1B511065677;
	Tue, 16 Dec 2008 16:16:57 +0000 (UTC)
	(envelope-from scottl@samsco.org)
Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57])
	by mx1.freebsd.org (Postfix) with ESMTP id 5729A8FC28;
	Tue, 16 Dec 2008 16:16:57 +0000 (UTC)
	(envelope-from scottl@samsco.org)
Received: from phobos.local ([192.168.254.200]) (authenticated bits=0)
	by pooker.samsco.org (8.14.2/8.14.2) with ESMTP id mBGGGq6O066613;
	Tue, 16 Dec 2008 09:16:53 -0700 (MST)
	(envelope-from scottl@samsco.org)
Message-ID: <4947D474.9040802@samsco.org>
Date: Tue, 16 Dec 2008 09:16:52 -0700
From: Scott Long <scottl@samsco.org>
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X; en-US;
	rv:1.8.1.13) Gecko/20080313 SeaMonkey/1.1.9
MIME-Version: 1.0
To: Alexander Motin <mav@FreeBSD.org>
References: <200812161604.mBGG4fBd041647@svn.freebsd.org>
In-Reply-To: <200812161604.mBGG4fBd041647@svn.freebsd.org>
X-Enigmail-Version: 0.95.6
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
X-Spam-Status: No, score=-4.4 required=3.8 tests=ALL_TRUSTED,BAYES_00
	autolearn=ham version=3.1.8
X-Spam-Checker-Version: SpamAssassin 3.1.8 (2007-02-13) on pooker.samsco.org
Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org,
	src-committers@FreeBSD.org
Subject: Re: svn commit: r186182 - head/sys/dev/ata
X-BeenThere: svn-src-all@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: "SVN commit messages for the entire src tree \(except for &quot;
	user&quot; and &quot; projects&quot; \)" <svn-src-all.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/svn-src-all>,
	<mailto:svn-src-all-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-src-all>
List-Post: <mailto:svn-src-all@freebsd.org>
List-Help: <mailto:svn-src-all-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/svn-src-all>,
	<mailto:svn-src-all-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Tue, 16 Dec 2008 16:16:57 -0000

Alexander Motin wrote:
> Author: mav
> Date: Tue Dec 16 16:04:40 2008
> New Revision: 186182
> URL: http://svn.freebsd.org/changeset/base/186182
> 
> Log:
>   Call ata_legacy() only once on attach and save it's result. Scanning PCI
>   configuration registers (which are not going to change) on every interrupt
>   looks expensive, especially when interrupt is shared. Profiling shows me 3%
>   of time spent by atapci0 on pure network load due to IRQ sharing with em0.
> 

Nice change.  PCI Config registers are exceptionally slow to access on 
most systems.

Scott