linux-kernel - Re: Regard of thermal power allocator's coefficients

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: <20160215044751.GA5660@leoy-linaro>
Date:	Mon, 15 Feb 2016 12:47:51 +0800
From:	Leo Yan <leo.yan@...aro.org>
To:	Eduardo Valentin <edubezval@...il.com>
Cc:	linux-pm@...r.kernel.org, Javi Merino <javi.merino@....com>,
	Zhang Rui <rui.zhang@...el.com>, linux-kernel@...r.kernel.org
Subject: Re: Regard of thermal power allocator's coefficients

Hi Eduardo,

Thanks for replying.

On Sun, Feb 14, 2016 at 09:12:19AM -0800, Eduardo Valentin wrote:
> On Sun, Feb 14, 2016 at 07:00:41PM +0800, Leo Yan wrote:

[...]

> > So want to firstly to confirm if should we pass coefficients by using
> > device tree? Is someone working on related work for this?
> 
> We pass the sustainable power, but not the coefficients. IIRC, the IPA
> coefficients were considered (SW) implementation details. Sustainable power,
> on the other hand, still describes hardware capabilities.
> 
> So, I don't think they will go via DT.

I'd like clarify one thing: "coefficients" are used to calculate
static leakage and dynamic power but not PID's term constants. They
are quite dependent on silicon's process; this is similiar with
sustainable power. So do you think we should take these "coefficients"
as software paramters?

The code is written in drivers/thermal/hisi_thermal.c like below:

---8<---

struct cluster_power_coefficients cluster_data = {
	.dyn_coeff      = 266,
	.static_cpu     = 14,
	.static_cluster = 5,
};

/* voltage in uV and temperature in mC */
static int get_static_power(cpumask_t *cpumask, int interval,
		unsigned long u_volt, u32 *power)
{
	int nr_cpus = cpumask_weight(cpumask);

	*power  = nr_cpus * cluster_data.static_cpu;
	*power += 2 * cluster_data.static_cluster;

	return 0;
}

static int hisi_thermal_register_cooling_device(struct platform_device *pdev,
						struct hisi_thermal_data *data)
{
	struct device_node *np;

	np = of_find_node_by_name(NULL, "cluster0");
	if (!np) {
		dev_err(&pdev->dev, "Cluster0 node not founds\n");
		return -ENODEV;
	}

	data->cdevs = of_cpufreq_power_cooling_register(np,
			cpu_present_mask, cluster_data.dyn_coeff,
			get_static_power);
	if (IS_ERR(data->cdevs)) {
		dev_err(&pdev->dev,
			"Error registering cooling device: %ld\n",
			PTR_ERR(data->cdevs));
		return PTR_ERR(data->cdevs);
	}

	return 0;
}

Thanks,
Leo Yan