Re: [PATCH V3 2/2] cpufreq: AMD "frequency sensitivity feedback" powersave bias for ondemand governor

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Apr 02, 2013 at 03:03:37PM -0500, Jacob Shin wrote:
> Well, so this powersave_bias also works as a tunable knob.
>
> From ondemand side, if /sys/../ondemand/powersave_bias is 0, then we
> (AMD sensitivity) don't get called and you get the default ondemand
> behavior.
>
> Like existing powersave_bias, users can tune the value to whatever
> they want, to get a specturum of less to more aggressive power savings
> vs performance.
>
> I thought tunable would be more flexible .. out in the field or what
> not .. no?

Ok, yes, that is the default on current systems which don't have hw
feedback.

But, on hw with such counters, I think the default should be to use
the hw feedback feature so that hardware can already do more informed
decisions for users.

As Thomas said, I hardly doubt users even know about that knob. So if we
can make the freq sensitivity thing work out of the box and without user
intervention, then we should strive to do that, no?

IOW:

	if (!powersave_bias) {
		/* user hasn't touched knob */

		if (HAS_FEEDBACK_INTERFACE)
			od_ops.powersave_bias_target(...);

		__cpufreq_driver_target(..)
	else
		od_ops.powersave_bias_target(..)
		__cpufreq_driver_target(..)
	}

The only change is that on hw feedback systems, you don't get the old
behavior with powersave_bias == 0. Question is, do you even want it all
that much but would rather leave the hw do much more informed decisions
than the ondemand governor.

Hmmm.

-- 
Regards/Gruss,
    Boris.

Sent from a fat crate under my desk. Formatting is fine.
--
--
To unsubscribe from this list: send the line "unsubscribe cpufreq" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel Devel]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Forum]     [Linux SCSI]

  Powered by Linux