From: "Burakov, Anatoly" <anatoly.burakov@intel.com>
To: Liang Ma <liang.j.ma@intel.com>, david.hunt@intel.com
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH v2] libs/power: add p-state driver compatibility
Date: Fri, 14 Dec 2018 12:20:07 +0000 [thread overview]
Message-ID: <dc52b441-a1bf-f3df-b9d7-8d6a5cf3e9e8@intel.com> (raw)
In-Reply-To: <1544786028-10138-1-git-send-email-liang.j.ma@intel.com>
On 14-Dec-18 11:13 AM, Liang Ma wrote:
> Previously, in order to use the power library, it was necessary
> for the user to disable the intel_pstate driver by adding
> “intel_pstate=disable” to the kernel command line for the system,
> which causes the acpi_cpufreq driver to be loaded in its place.
>
> This patch adds the ability for the power library use the intel-pstate
> driver.
>
> It adds a new suite of functions behind the current power library API,
> and will seamlessly set up the user facing API function pointers to
> the relevant functions depending on whether the system is running with
> acpi_cpufreq kernel driver, intel_pstate kernel driver or in a guest,
> using kvm. The library API and ABI is unchanged.
>
> Signed-off-by: Liang Ma <liang.j.ma@intel.com>
> ---
<snip>
> sources = files('rte_power.c', 'power_acpi_cpufreq.c',
> 'power_kvm_vm.c', 'guest_channel.c',
> - 'rte_power_empty_poll.c')
> + 'rte_power_empty_poll.c',
> + 'power_pstate_cpufreq.c')
> headers = files('rte_power.h','rte_power_empty_poll.h')
> -deps += ['timer']
> diff --git a/lib/librte_power/power_pstate_cpufreq.c b/lib/librte_power/power_pstate_cpufreq.c
> new file mode 100644
> index 0000000..1711484
> --- /dev/null
> +++ b/lib/librte_power/power_pstate_cpufreq.c
> @@ -0,0 +1,770 @@
> +/* SPDX-License-Identifier: BSD-3-Clause
> + * Copyright(c) 2010-2018 Intel Corporation
Like i said in comments to previous revision, copyright date here is
wrong - you're creating a new file. This file wasn't created in 2010. It
should just say "2018".
Other than that, LGTM
Reviewed-by: Anatoly Burakov <anatoly.burakov@intel.com>
--
Thanks,
Anatoly
next prev parent reply other threads:[~2018-12-14 12:20 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-23 11:33 [dpdk-dev] [PATCH] " Liang Ma
2018-12-10 16:08 ` Burakov, Anatoly
2018-12-13 10:58 ` Liang, Ma
2018-12-13 11:16 ` Burakov, Anatoly
2018-12-13 13:46 ` Liang, Ma
2018-12-13 13:53 ` Burakov, Anatoly
2018-12-14 11:13 ` [dpdk-dev] [PATCH v2] " Liang Ma
2018-12-14 12:20 ` Burakov, Anatoly [this message]
2018-12-14 13:11 ` [dpdk-dev] [PATCH v3] " Liang Ma
2018-12-19 3:18 ` Thomas Monjalon
2018-12-19 9:09 ` Hunt, David
2018-12-19 20:31 ` Thomas Monjalon
2018-12-20 9:25 ` Burakov, Anatoly
2018-12-20 9:33 ` Burakov, Anatoly
2018-12-20 10:10 ` Thomas Monjalon
2018-12-20 10:42 ` Luca Boccassi
2018-12-20 10:44 ` Thomas Monjalon
2018-12-20 10:54 ` Liang, Ma
2018-12-20 14:52 ` Hunt, David
2018-12-21 0:30 ` Thomas Monjalon
2018-12-21 0:33 ` Thomas Monjalon
2018-12-20 14:43 ` [dpdk-dev] [PATCH v4] " Liang Ma
2018-12-21 0:34 ` Thomas Monjalon
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=dc52b441-a1bf-f3df-b9d7-8d6a5cf3e9e8@intel.com \
--to=anatoly.burakov@intel.com \
--cc=david.hunt@intel.com \
--cc=dev@dpdk.org \
--cc=liang.j.ma@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).