From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v3] libs/power: add p-state driver compatibility Date: Wed, 19 Dec 2018 04:18:58 +0100 Message-ID: <2602247.RKfj1taVKr@xps> References: <1544786028-10138-1-git-send-email-liang.j.ma@intel.com> <1544793093-9971-1-git-send-email-liang.j.ma@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Cc: dev@dpdk.org, anatoly.burakov@intel.com To: Liang Ma , david.hunt@intel.com Return-path: Received: from wout2-smtp.messagingengine.com (wout2-smtp.messagingengine.com [64.147.123.25]) by dpdk.org (Postfix) with ESMTP id E52DF1B5CE for ; Wed, 19 Dec 2018 04:19:01 +0100 (CET) In-Reply-To: <1544793093-9971-1-git-send-email-liang.j.ma@intel.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 14/12/2018 14:11, Liang Ma: > Previously, in order to use the power library, it was necessary > for the user to disable the intel_pstate driver by adding > =E2=80=9Cintel_pstate=3Ddisable=E2=80=9D to the kernel command line for t= he system, > which causes the acpi_cpufreq driver to be loaded in its place. >=20 > This patch adds the ability for the power library use the intel-pstate > driver. >=20 > 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. >=20 > Signed-off-by: Liang Ma >=20 > Reviewed-by: Anatoly Burakov > --- Please write a changelog when sending a new version. Dave, any comment on this patch? > --- /dev/null > +++ b/lib/librte_power/power_pstate_cpufreq.c > @@ -0,0 +1,770 @@ > +/* SPDX-License-Identifier: BSD-3-Clause > + * Copyright(c) 2018-2018 Intel Corporation Something wrong here :)