From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754829Ab1CYSYw (ORCPT ); Fri, 25 Mar 2011 14:24:52 -0400 Received: from oproxy3-pub.bluehost.com ([69.89.21.8]:42516 "HELO oproxy3-pub.bluehost.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1754546Ab1CYSYu (ORCPT ); Fri, 25 Mar 2011 14:24:50 -0400 DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=default; d=virtuousgeek.org; h=Received:Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References:X-Mailer:Mime-Version:Content-Type:Content-Transfer-Encoding:X-Identified-User; b=DiN8koOtaPR8Z8Vo8/Td7sq77dJLIN6UbBf4Ub3yRkgKhfooFCMMmNBlTjljkgFHDlnOkhEf7BbKIM9FuiMA2pijstl0dgfW0BbVtWXuaFB9ISyCbHmOJ3Z9DAPtl3oS; Date: Fri, 25 Mar 2011 11:24:43 -0700 From: Jesse Barnes To: Andreas Hartmann Cc: linux-kernel@vger.kernel.org Subject: Re: intel_ips produces constant load of 1 Message-ID: <20110325112443.7caf4429@jbarnes-desktop> In-Reply-To: <4D8CDD93.3030203@dualc.maya.org> References: <20110321110404.6a71a404@jbarnes-desktop> <20110322132555.715dc096@jbarnes-desktop> <4D8CDD93.3030203@dualc.maya.org> X-Mailer: Claws Mail 3.7.6 (GTK+ 2.22.0; x86_64-pc-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Identified-User: {10642:box514.bluehost.com:virtuous:virtuousgeek.org} {sentby:smtp auth 67.161.37.189 authed with jbarnes@virtuousgeek.org} Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, 25 Mar 2011 19:23:15 +0100 Andreas Hartmann wrote: > Jesse Barnes wrote: > > On Mon, 21 Mar 2011 11:04:04 -0700 > > Jesse Barnes wrote: > > > >>> ps aux | grep ips > >>> root 593 0.0 0.0 0 0 ? S 17:20 0:00 > >>> [ips-adjust] > >>> root 594 0.0 0.0 0 0 ? D 17:20 0:00 > >>> [ips-monitor] > >>> > >>> If the module isn't loaded, the load of the machine in idle mode is 0 as > >>> expected. > >> > >> This is a reporting problem, and probably due to the schedule() call > >> and associated task state in the ips-monitor thread. I thought setting > >> the task state to interruptible would prevent this, but it seems like > >> it's not enough for the deferrable on-stack timers? > >> > >> At any rate, it's not actually causing increased CPU usage, so you can > >> safely ignore it until we have a fix. > > > > Oops, one task uses interruptible correctly, but the monitor thread > > doesn't. > > > > Does this patch fix your load average? > > The patch works fine: > > ps aux | grep ips > root 22427 0.0 0.0 0 0 ? S 19:15 0:00 > [ips-adjust] > root 22428 0.0 0.0 0 0 ? S 19:15 0:00 > [ips-monitor] Great, thanks for testing. I'll get the fix over to Matthew. > btw: what does this mean (first line): > > > kernel: [ 6610.030205] intel ips 0000:00:1f.6: CPU TDP doesn't match > expected value (found 25, expected 29) Each SKU has a default TDP. In your case that's 29W. But the BIOS can override that value with a lower one if the platform (chassis, board, fans, etc) isn't designed to dissipate the full amount. -- Jesse Barnes, Intel Open Source Technology Center