linux-kernel - Re: [PATCH] acerhdf: Fix fan activation with new thermal governor

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20130225102526.GB12067@pd.tnic>
Date:	Mon, 25 Feb 2013 11:25:26 +0100
From:	Borislav Petkov <bp@...en8.de>
To:	Zhang Rui <rui.zhang@...el.com>
Cc:	Peter Feuerer <peter@...e.net>,
	LKML <linux-kernel@...r.kernel.org>,
	Durgadoss R <durgadoss.r@...el.com>,
	Andreas Mohr <andi@...as.de>,
	Alexander Lam <azl@...rew.cmu.edu>, bp@...e.de
Subject: Re: [PATCH] acerhdf: Fix fan activation with new thermal governor

On Mon, Feb 25, 2013 at 11:21:54AM +0800, Zhang Rui wrote:
> the reason we keep on calling it because the thermal zone trip point
> info may be changed at runtime.
> 
> > Can we pass trip points and temperature levels upon registration time
> > instead?
> > 
> no, but we can cache it in the thermal layer, and do update when
> necessary.

Right, so this would make sense for fan, etc drivers which have static
trip points.

> why? I do not think the stepwise scheme will break it. If it happens, we
> should fix stepwise governor instead.

Ok, see my earlier mail for what acerhdf would need to be able to do.

> >  Therefore, add two more trip points - an
> > active one where we turn on the fan, and a critical one.
> > 
> I think you add a passive trip point and a critical one here.

I got 3 in total:

    if (trip == 0)
+           *type = THERMAL_TRIP_PASSIVE;
+   else if (trip == 1)
            *type = THERMAL_TRIP_ACTIVE;
+   else if (trip == 2)
+           *type = THERMAL_TRIP_CRITICAL;

> has anybody checked if the patch at lkml.org/lkml/2012/12/30/47 fixes
> the problem, without any other patch?

Ok, let me try it out.

-- 
Regards/Gruss,
    Boris.

Sent from a fat crate under my desk. Formatting is fine.
--
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/