DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=mime-version:in-reply-to:references:date:message-id:subject:from:to
         :cc:content-type:content-transfer-encoding;
        b=R7an6e0snh1zfVQobYDXxikwWf3CRel6QIsGCwmBskIuYWW4ZgRaLiePlsevo6BhCY
         hReJKtaqAnO1CvZSEve42UCYDHv9TGDMFKrVSFIVxppwFUbJMfO6v7Geuva7IO6pC23n
         6+Wbge0/8GRxjbFi95/+nuFeMQY8WhlG2UNIE=
MIME-Version: 1.0
In-Reply-To: <43e72e890911090832v1c809482yde7e767502271376@mail.gmail.com>
References: <1257537660-5301-1-git-send-email-lrodriguez@atheros.com>
	 <200911072152.57596.trenn@suse.de> <4AF5F47C.5000402@linux.intel.com>
	 <200911072347.05786.trenn@suse.de> <4AF5FBC7.9000105@linux.intel.com>
	 <43e72e890911090832v1c809482yde7e767502271376@mail.gmail.com>
Date: Tue, 10 Nov 2009 13:12:21 +0100
Message-ID: <4e5e476b0911100412yb51d472r24be8b2068204348@mail.gmail.com>
Subject: Re: [PATCH] cpu-freq: add troubleshooting section for FSB changes
From: Corrado Zoccolo <czoccolo@gmail.com>
To: "Luis R. Rodriguez" <mcgrof@gmail.com>
Cc: Arjan van de Ven <arjan@linux.intel.com>,
       Senthil Balasubramanian <senthilkumar@atheros.com>,
       Aeolus Yang <Aeolus.Yang@atheros.com>,
       Jonathan May <Jonathan.May@atheros.com>,
       Thomas Renninger <trenn@suse.de>, davej@redhat.com,
       cpufreq@vger.kernel.org, linux-kernel@vger.kernel.org,
       Matthew Garrett <mjg@redhat.com>,
       Reinette Chatre <reinette.chatre@intel.com>,
       Amod Bodas <Amod.Bodas@atheros.com>,
       David Quan <David.Quan@atheros.com>,
       Kishore Jotwani <Kishore.Jotwani@atheros.com>,
       linux-wireless <linux-wireless@vger.kernel.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8BIT
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2975
Lines: 70

Hi,
On Mon, Nov 9, 2009 at 5:32 PM, Luis R. Rodriguez <mcgrof@gmail.com> wrote:
>
> And we tested this (reducing the min cpu freq to one less than the
> highest supported P state to avoid an FSB speed change) and it seems
> doing the steps described here did not fix the issue. But at least now
> if anyone else wants to verify this they can with some sort of
> documentaiton.
>
> So to confirm though -- we are seeing a huge performance depredation
> mainly on RX on an Intel Pine Trail platform with SpeedStep enabled on
> the BIOS.
>
> Let me get into the specifics in case anyone is able to help. The
> issue is with ath9k on RX and the CPU on C3 state requesting DMA over
> PCI-E. We typically would get about 110 Mbps with an AR9285 (single
> stream) but when SpeedStep is enabled it goes down to 25 Mbps. At the
> PCI-E level we are seeing huge latencies introduced when SpeedStep is
> used for DMA requests to the Intel root complex on the Intel Pine
> trail platform. Latencies are about 20-60 us.

You mentioned speedstep and cpufreq, but the problem is with C3 state
and cpuidle (probably the BIOS mixes the two concepts, but we should
keep them separated).
C3 is not related to the core or FSB frequency, it is an idle state.
When in C3, the CPU is not ready to perform any operation, not just
slower, and depending on the CPU hw, it may take several us to wake up
(even 85us, on an Atom).

>
> Is there a timeout threshold change that will cause the Intel chipset
> wait for some time after completion before going into a C3 state? Are
> there any other explanations for seeing such huge latencies on C3
> state?
There is a patch from Arjan for the cpuidle menu governor, that may fix it.
http://linux.derkeiler.com/Mailing-Lists/Kernel/2009-09/msg05276.html
It is already present in 2.6.32.

Thanks,
Corrado

>
> Thanks for your review and help with this. Any suggestions are greatly
> appreciated.
>
>  Luis
> --
> To unsubscribe from this list: send the line "unsubscribe cpufreq" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>


-- 
__________________________________________________________________________

dott. Corrado Zoccolo                          mailto:czoccolo@gmail.com
PhD - Department of Computer Science - University of Pisa, Italy
--------------------------------------------------------------------------
The self-confidence of a warrior is not the self-confidence of the average
man. The average man seeks certainty in the eyes of the onlooker and calls
that self-confidence. The warrior seeks impeccability in his own eyes and
calls that humbleness.
                               Tales of Power - C. Castaneda
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/