From: Hans de Goede <hdegoede@redhat.com>
To: "Limonciello, Mario" <mario.limonciello@amd.com>,
Shyam Sundar S K <Shyam-sundar.S-k@amd.com>,
markgross@kernel.org
Cc: platform-driver-x86@vger.kernel.org, Patil.Reddy@amd.com,
Mark Pearson <markpearson@lenovo.com>
Subject: Re: [PATCH v1 13/15] platform/x86/amd/pmf: Handle AMT and CQL events for Auto mode
Date: Thu, 28 Jul 2022 15:03:27 +0200 [thread overview]
Message-ID: <828a7b8e-2375-a9d0-7b00-d59e84cfaa2b@redhat.com> (raw)
In-Reply-To: <d01e52cb-31c5-7d20-4f18-1e4bb70c1355@amd.com>
Hi,
On 7/28/22 01:52, Limonciello, Mario wrote:
> On 7/27/2022 16:46, Hans de Goede wrote:
>> <resend with Cc list fixed>
>>
>> Hi,
>>
>> On 7/12/22 16:58, Shyam Sundar S K wrote:
>>> The transition to auto-mode happens when the PMF driver receives
>>> AMT (Auto Mode transition) event. transition logic will reside in the
>>> PMF driver but the events would come from other supported drivers[1].
>>>
>>> The thermal parameters would vary between when a performance "on-lap" mode
>>> is detected and versus when not. The CQL event would get triggered from
>>> other drivers, so that PMF driver would adjust the system thermal config
>>> based on the ACPI inputs.
>>>
>>> OEMs can control whether or not to enable AMT or CQL via other supported
>>> drivers[1] but the actual transition logic resides in the AMD PMF driver.
>>> When an AMT event is received the automatic mode transition RAPL algorithm
>>> will run. When a CQL event is received an performance "on-lap" mode will
>>> be enabled and thermal parameters will be adjusted accordingly.
>>>
>>> [1]
>>> Link: https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgit.kernel.org%2Fpub%2Fscm%2Flinux%2Fkernel%2Fgit%2Fpdx86%2Fplatform-drivers-x86.git%2Fcommit%2F%3Fh%3Dreview-hans%26id%3D755b249250df1b612d982f3b702c831b26ecdf73&data=05%7C01%7Cmario.limonciello%40amd.com%7Cd72d8627f7ad4088aff308da7019afbf%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637945552890705546%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=4gLIOjMo%2FYO%2BkKdqtk4pcLHlsBrLiUb41cTKvuYrcHQ%3D&reserved=0
>>>
>>> Signed-off-by: Shyam Sundar S K <Shyam-sundar.S-k@amd.com>
>>> ---
>>> drivers/platform/x86/amd/pmf/acpi.c | 90 +++++++++++++++++++++++-
>>> drivers/platform/x86/amd/pmf/auto-mode.c | 22 ++++++
>>> drivers/platform/x86/amd/pmf/cnqf.c | 4 +-
>>> drivers/platform/x86/amd/pmf/core.c | 18 ++++-
>>> drivers/platform/x86/amd/pmf/pmf.h | 29 +++++++-
>>> 5 files changed, 156 insertions(+), 7 deletions(-)
>>>
>>> diff --git a/drivers/platform/x86/amd/pmf/acpi.c b/drivers/platform/x86/amd/pmf/acpi.c
>>> index e9f33e61659f..4bde86aeafa0 100644
>>> --- a/drivers/platform/x86/amd/pmf/acpi.c
>>> +++ b/drivers/platform/x86/amd/pmf/acpi.c
>>> @@ -12,6 +12,8 @@
>>> #include "pmf.h"
>>> #define APMF_METHOD "\\_SB_.PMF_.APMF"
>>> +#define APMF_CQL_NOTIFICATION 2
>>> +#define APMF_AMT_NOTIFICATION 3
>>> static unsigned long supported_func;
>>> @@ -55,6 +57,7 @@ static void apmf_if_parse_functions(struct apmf_if_functions *func, u32 mask)
>>> {
>>> func->system_params = mask & APMF_FUNC_GET_SYS_PARAMS;
>>> func->static_slider_granular = mask & APMF_FUNC_STATIC_SLIDER_GRANULAR;
>>> + func->sbios_requests = mask & APMF_FUNC_SBIOS_REQUESTS;
>>> func->auto_mode_def = mask & APMF_FUNC_AUTO_MODE;
>>> func->fan_table_idx = mask & APMF_FUNC_SET_FAN_IDX;
>>> func->dyn_slider_ac = mask & APMF_FUNC_DYN_SLIDER_GRANULAR_AC;
>>> @@ -292,6 +295,36 @@ int apmf_get_dyn_slider_def_dc(struct apmf_if *ampf_if, struct apmf_dyn_slider_o
>>> return err;
>>> }
>>> +int apmf_get_sbios_requests(struct apmf_if *ampf_if, struct apmf_sbios_req *req)
>>> +{
>>> + union acpi_object *info;
>>> + size_t size;
>>> + int err = 0;
>>> +
>>> + info = apmf_if_call(ampf_if, APMF_FUNC_SBIOS_REQUESTS, NULL);
>>> + if (!info)
>>> + return -EIO;
>>> +
>>> + size = *(u16 *)info->buffer.pointer;
>>> +
>>> + if (size < sizeof(union acpi_object)) {
>>> + pr_err("PMF: buffer too small %zu\n", size);
>>> + err = -EINVAL;
>>> + goto out;
>>> + }
>>> +
>>> + size = min(sizeof(*req), size);
>>> + memset(req, 0, sizeof(*req));
>>> + memcpy(req, info->buffer.pointer, size);
>>> +
>>> + pr_debug("PMF: %s: pending_req:%d cql:%d amt:%d\n", __func__,
>>> + req->pending_req, req->cql_event, req->amt_event);
>>> +
>>> +out:
>>> + kfree(info);
>>> + return err;
>>> +}
>>> +
>>> static acpi_handle apmf_if_probe_handle(void)
>>> {
>>> acpi_handle handle = NULL;
>>> @@ -312,18 +345,62 @@ static acpi_handle apmf_if_probe_handle(void)
>>> return handle;
>>> }
>>> +static void apmf_event_handler(acpi_handle handle, u32 event, void *data)
>>> +{
>>> + struct amd_pmf_dev *pmf_dev = data;
>>> + struct apmf_if *apmf_if = pmf_dev->apmf_if;
>>> + int ret;
>>> +
>>> + if (apmf_if->func.sbios_requests) {
>>> + struct apmf_sbios_req req;
>>> +
>>> + ret = apmf_get_sbios_requests(apmf_if, &req);
>>> + if (ret) {
>>> + dev_err(pmf_dev->dev, "Failed to get SBIOS requests:%d\n", ret);
>>> + return;
>>> + }
>>> + if (req.pending_req & BIT(APMF_AMT_NOTIFICATION)) {
>>> + pr_debug("PMF: AMT is supported and notifications %s\n",
>>> + req.amt_event ? "Enabled" : "Disabled");
>>> + if (req.amt_event)
>>> + pmf_dev->is_amt_event = true;
>>> + else
>>> + pmf_dev->is_amt_event = !!req.amt_event;
>>> + }
>>> +
>>> + if (req.pending_req & BIT(APMF_CQL_NOTIFICATION)) {
>>> + pr_debug("PMF: CQL is supported and notifications %s\n",
>>> + req.cql_event ? "Enabled" : "Disabled");
>>> + if (req.cql_event)
>>> + pmf_dev->is_cql_event = true;
>>> + else
>>> + pmf_dev->is_cql_event = !!req.cql_event;
>>> +
>>> + /* update the target mode information */
>>> + amd_pmf_update_2_cql(pmf_dev);
>>> + }
>>> + }
>>> +}
>>> +
>>> void apmf_acpi_deinit(struct amd_pmf_dev *pmf_dev)
>>> {
>>> + acpi_handle ahandle = ACPI_HANDLE(pmf_dev->dev);
>>> +
>>> if (pmf_dev->apmf_if->func.sbios_heartbeat)
>>> cancel_delayed_work_sync(&pmf_dev->heart_beat);
>>> +
>>> + if (is_apmf_func_supported(APMF_FUNC_AUTO_MODE))
>>> + acpi_remove_notify_handler(ahandle, ACPI_ALL_NOTIFY,
>>> + apmf_event_handler);
>>> }
>>> int apmf_acpi_init(struct amd_pmf_dev *pmf_dev)
>>> {
>>> + acpi_handle ahandle = ACPI_HANDLE(pmf_dev->dev);
>>> struct apmf_notification_cfg *n;
>>> acpi_handle apmf_if_handle;
>>> struct apmf_if *apmf_if;
>>> - int ret;
>>> + int ret, status;
>>> apmf_if_handle = apmf_if_probe_handle();
>>> if (!apmf_if_handle)
>>> @@ -360,6 +437,17 @@ int apmf_acpi_init(struct amd_pmf_dev *pmf_dev)
>>> schedule_delayed_work(&pmf_dev->heart_beat, 0);
>>> }
>>> + /* Install the APMF Notify handler */
>>> + if (is_apmf_func_supported(APMF_FUNC_AUTO_MODE)) {
>>> + status = acpi_install_notify_handler(ahandle,
>>> + ACPI_ALL_NOTIFY,
>>> + apmf_event_handler, pmf_dev);
>>> + if (ACPI_FAILURE(status)) {
>>> + dev_err(pmf_dev->dev, "failed to install notify handler\n");
>>> + return -ENODEV;
>>> + }
>>> + }
>>> +
>>> out:
>>> return ret;
>>> }
>>> diff --git a/drivers/platform/x86/amd/pmf/auto-mode.c b/drivers/platform/x86/amd/pmf/auto-mode.c
>>> index 954fde25e71e..a85ef4b95cdb 100644
>>> --- a/drivers/platform/x86/amd/pmf/auto-mode.c
>>> +++ b/drivers/platform/x86/amd/pmf/auto-mode.c
>>> @@ -111,6 +111,13 @@ void amd_pmf_trans_automode(struct amd_pmf_dev *dev, int socket_power, ktime_t t
>>> bool update = false;
>>> int i, j;
>>> + if (!dev->amt_running && dev->is_amt_event) {
>>> + dev_dbg(dev->dev, "setting AMT thermals\n");
>>> + amd_pmf_handle_automode(dev, SLIDER_OP_SET, config_store.current_mode, NULL);
>>> + dev->amt_running = true;
>>> + return;
>>> + }
>>> +
>>> /* Get the average moving average computed by auto mode algorithm */
>>> avg_power = amd_pmf_get_moving_avg(socket_power);
>>> @@ -161,6 +168,21 @@ void amd_pmf_trans_automode(struct amd_pmf_dev *dev, int socket_power, ktime_t t
>>> }
>>> }
>>> +void amd_pmf_update_2_cql(struct amd_pmf_dev *dev)
>>> +{
>>> + config_store.transition[AUTO_TRANSITION_TO_PERFORMANCE].target_mode =
>>> + dev->is_cql_event ? AUTO_PERFORMANCE_ON_LAP : AUTO_PERFORMANCE;
>>> + if ((config_store.current_mode == AUTO_PERFORMANCE ||
>>> + config_store.current_mode == AUTO_PERFORMANCE_ON_LAP) &&
>>> + config_store.current_mode !=
>>> + config_store.transition[AUTO_TRANSITION_TO_PERFORMANCE].target_mode) {
>>> + config_store.current_mode =
>>> + config_store.transition[AUTO_TRANSITION_TO_PERFORMANCE].target_mode;
>>> + amd_pmf_handle_automode(dev, SLIDER_OP_SET, config_store.current_mode, NULL);
>>> + }
>>> + dev_dbg(dev->dev, "updated CQL thermals\n");
>>> +}
>>> +
>>> static void amd_pmf_get_power_threshold(void)
>>> {
>>> config_store.transition[AUTO_TRANSITION_TO_QUIET].power_threshold =
>>> diff --git a/drivers/platform/x86/amd/pmf/cnqf.c b/drivers/platform/x86/amd/pmf/cnqf.c
>>> index 2b03ae1ad37f..eba8f0d79a62 100644
>>> --- a/drivers/platform/x86/amd/pmf/cnqf.c
>>> +++ b/drivers/platform/x86/amd/pmf/cnqf.c
>>> @@ -101,7 +101,7 @@ static const char *state_as_str(unsigned int state)
>>> }
>>> }
>>> -void amd_pmf_trans_cnqf(struct amd_pmf_dev *dev, int socket_power, ktime_t time_lapsed_ms)
>>> +void amd_pmf_trans_cnqf(struct amd_pmf_dev *dev, int socket_power, ktime_t time_elapsed_ms)
>>> {
>>> struct cnqf_tran_params *tp;
>>> int src, i, j, index = 0;
>>> @@ -117,7 +117,7 @@ void amd_pmf_trans_cnqf(struct amd_pmf_dev *dev, int socket_power, ktime_t time_
>>> }
>>> for (i = 0; i < CNQF_TRANSITION_MAX; i++) {
>>> - config_store.trans_param[src][i].timer += time_lapsed_ms;
>>> + config_store.trans_param[src][i].timer += time_elapsed_ms;
>>> config_store.trans_param[src][i].total_power += socket_power;
>>> config_store.trans_param[src][i].count++;
>>> diff --git a/drivers/platform/x86/amd/pmf/core.c b/drivers/platform/x86/amd/pmf/core.c
>>> index 674ddf599135..2a3dacfb2005 100644
>>> --- a/drivers/platform/x86/amd/pmf/core.c
>>> +++ b/drivers/platform/x86/amd/pmf/core.c
>>> @@ -109,10 +109,15 @@ static void amd_pmf_get_metrics(struct work_struct *work)
>>> enum platform_profile_option current_profile;
>>> ktime_t time_elapsed_ms;
>>> int socket_power;
>>> + u8 mode;
>>> /* Get the current profile information */
>>> platform_profile_get(¤t_profile);
>>> + if (!dev->is_amt_event)
>>> + dev_dbg(dev->dev, "%s amt event: %s\n", __func__,
>>> + dev->is_amt_event ? "Enabled" : "Disabled");
>>> +
>>> /* Transfer table contents */
>>> memset(&dev->m_table, 0, sizeof(dev->m_table));
>>> amd_pmf_send_cmd(dev, SET_TRANSFER_TABLE, 0, 7, NULL);
>>> @@ -123,8 +128,17 @@ static void amd_pmf_get_metrics(struct work_struct *work)
>>> socket_power = dev->m_table.apu_power + dev->m_table.dgpu_power;
>>> if (current_profile == PLATFORM_PROFILE_BALANCED) {
>>
>> Hmm, I guess this is also why the platform_profile_get() is necessary ? Because
>> on Thinkpads thinkpad_acpi is expected to be the platform_profile provider and
>> then the PMF code wants to know the platform_profile setting from thinkpad_acpi ?
>>
>> Can you please explain the expected interactions between thinkpad_acpi and
>> this code here a bit more ?
>>
>> Since we now only call amd_pmf_trans_automode() based on the AMT flag and
>> that flag is controlled by the thinkpad BIOS/EC can we not expect that flag
>> to be cleared when the profile is not balanced and can we thus not just drop
>> the current_profile == PLATFORM_PROFILE_BALANCED check all together?
>>
>> It seems to me that if current_profile == PLATFORM_PROFILE_BALANCED
>> then enable AMT, else disable it, logic belongs inside thinkpad_acpi
>> and not here?
>>
>
> It actually already lives in thinkpad_acpi.
>
> https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/tree/drivers/platform/x86/thinkpad_acpi.c?h=review-hans#n10489
>
> By the control point from thinkpad_acpi BIOS events will be emitted controlling whether AMT is running in a given mode.
>
> So yes; there is no need for this here anymore.
Right. There still are some open questions / things which need fixing here though:
1. If I understand things right, then on ThinkPads /sys/firmware/apci/platform_profile
will be registered by thinkpad_acpi. But in version 1 of this patchset nothing is
stopping the amd-pmf code from registering /sys/firmware/apci/platform_profile if
the amd-pmf module gets loaded first. So if the intend is for it to always be owned
by thinkpad_acpi then the amd-pmf code must check for this and not even try to
register its platform_profile support. We cannot rely on module ordering ensuring
that thinkpad_acpi registers first and then amd-pmf will get an -EBUSY error,
since there are no module load ordering guarantees.
2. So when the thinkpad_acpi platform_profile is set to balanced, then it will
enable AMT and then the periodically run workqueue function from amd-pmf
will do its AMT thing. But what when the thinkpad_acpi platform_profile is
set to low-power or performance. Should the amd-pmf code then apply the static
slider settings for low-power/performance which it has read from the ACPI
tables? Or will the ACPI/EC code on thinkpads take care of this themselves ?
3. If the answer to 2. is "Yes the amd-pmf code should apply the static-slider
settings" then we will still need patch 1/15 to allow the amd-pmd code to
read the platform-profile setting from the thinkpad_acpi platform-profile
provider;
And if the answer is "No, the thinkpad ACPI/EC will take care of this"
then we should probably make sure that the static slider code never runs
at all on thinkpads.
Regards,
Hans
>
>> Regards,
>>
>> Hans
>>
>>> - /* Apply the Auto Mode transition */
>>> - amd_pmf_trans_automode(dev, socket_power, time_elapsed_ms);
>>> + if (dev->is_amt_event) {
>>> + /* Apply the Auto Mode transition */
>>> + amd_pmf_trans_automode(dev, socket_power, time_elapsed_ms);
>>> + } else if (!dev->is_amt_event && dev->amt_running) {
>>> + pr_debug("resetting AMT thermals\n");
>>> + mode = amd_pmf_get_pprof_modes(dev);
>>> + amd_pmf_update_slider(dev, SLIDER_OP_SET, mode, NULL);
>>> + dev->amt_running = false;
>>> + }
>>> + } else {
>>> + dev->amt_running = false;
>>> }
>>> if (dev->cnqf_feat) {
>>> diff --git a/drivers/platform/x86/amd/pmf/pmf.h b/drivers/platform/x86/amd/pmf/pmf.h
>>> index 0532f49e9613..9ae9812353cd 100644
>>> --- a/drivers/platform/x86/amd/pmf/pmf.h
>>> +++ b/drivers/platform/x86/amd/pmf/pmf.h
>>> @@ -17,6 +17,7 @@
>>> /* APMF Functions */
>>> #define APMF_FUNC_VERIFY_INTERFACE 0
>>> #define APMF_FUNC_GET_SYS_PARAMS 1
>>> +#define APMF_FUNC_SBIOS_REQUESTS 2
>>> #define APMF_FUNC_SBIOS_HEARTBEAT 4
>>> #define APMF_FUNC_AUTO_MODE 5
>>> #define APMF_FUNC_SET_FAN_IDX 7
>>> @@ -51,6 +52,7 @@
>>> /* AMD PMF BIOS interfaces */
>>> struct apmf_if_functions {
>>> bool system_params;
>>> + bool sbios_requests;
>>> bool sbios_heartbeat;
>>> bool auto_mode_def;
>>> bool fan_table_idx;
>>> @@ -84,6 +86,24 @@ struct apmf_system_params {
>>> u32 heartbeat_int;
>>> } __packed;
>>> +struct apmf_sbios_req {
>>> + u16 size;
>>> + u32 pending_req;
>>> + u8 rsd;
>>> + u8 cql_event;
>>> + u8 amt_event;
>>> + u32 fppt;
>>> + u32 sppt;
>>> + u32 fppt_apu_only;
>>> + u32 spl;
>>> + u32 stt_min_limit;
>>> + u8 skin_temp_apu;
>>> + u8 skin_temp_hs2;
>>> + u8 dps_enable;
>>> + u32 custom_policy_1;
>>> + u32 custom_policy_2;
>>> +} __packed;
>>> +
>>> struct apmf_fan_idx {
>>> u16 size;
>>> u8 fan_ctl_mode;
>>> @@ -171,6 +191,9 @@ struct amd_pmf_dev {
>>> #endif /* CONFIG_DEBUG_FS */
>>> bool cnqf_feat;
>>> bool cnqf_running;
>>> + bool is_amt_event;
>>> + bool is_cql_event;
>>> + bool amt_running;
>>> };
>>> struct apmf_sps_prop_granular {
>>> @@ -417,9 +440,11 @@ int apmf_update_fan_idx(struct apmf_if *ampf_if, bool manual, u32 idx);
>>> /* Auto Mode Layer */
>>> void amd_pmf_load_defaults_auto_mode(struct amd_pmf_dev *dev);
>>> int apmf_get_auto_mode_def(struct apmf_if *ampf_if, struct apmf_auto_mode *data);
>>> +int apmf_get_sbios_requests(struct apmf_if *ampf_if, struct apmf_sbios_req *req);
>>> void amd_pmf_init_auto_mode(struct amd_pmf_dev *dev);
>>> void amd_pmf_deinit_auto_mode(struct amd_pmf_dev *dev);
>>> -void amd_pmf_trans_automode(struct amd_pmf_dev *dev, int socket_power, ktime_t time_lapsed_ms);
>>> +void amd_pmf_update_2_cql(struct amd_pmf_dev *dev);
>>> +void amd_pmf_trans_automode(struct amd_pmf_dev *dev, int socket_power, ktime_t time_elapsed_ms);
>>> /* CnQF Layer */
>>> int apmf_get_dyn_slider_def_ac(struct apmf_if *ampf_if, struct apmf_dyn_slider_output *data);
>>> @@ -427,6 +452,6 @@ int apmf_get_dyn_slider_def_dc(struct apmf_if *ampf_if, struct apmf_dyn_slider_o
>>> void amd_pmf_init_cnqf(struct amd_pmf_dev *dev);
>>> void amd_pmf_deinit_cnqf(struct amd_pmf_dev *dev);
>>> void amd_pmf_load_defaults_cnqf(struct amd_pmf_dev *dev);
>>> -void amd_pmf_trans_cnqf(struct amd_pmf_dev *dev, int socket_power, ktime_t time_lapsed_ms);
>>> +void amd_pmf_trans_cnqf(struct amd_pmf_dev *dev, int socket_power, ktime_t time_elapsed_ms);
>>> #endif /* PMF_H */
>>
>
next prev parent reply other threads:[~2022-07-28 13:03 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-07-12 14:58 [PATCH v1 00/15] platform/x86/amd/pmf: Introduce AMD PMF Driver Shyam Sundar S K
2022-07-12 14:58 ` [PATCH v1 01/15] ACPI: platform_profile: Add support for notification chains Shyam Sundar S K
2022-07-12 15:03 ` Limonciello, Mario
2022-07-27 13:24 ` Hans de Goede
2022-07-27 20:38 ` Hans de Goede
2022-07-12 14:58 ` [PATCH v1 02/15] platform/x86/amd/pmf: Add support for PMF core layer Shyam Sundar S K
2022-07-27 13:57 ` Hans de Goede
2022-07-12 14:58 ` [PATCH v1 03/15] platform/x86/amd/pmf: Add support for PMF APCI layer Shyam Sundar S K
2022-07-27 13:57 ` Hans de Goede
2022-07-12 14:58 ` [PATCH v1 04/15] platform/x86/amd/pmf: Add support SPS PMF feature Shyam Sundar S K
2022-07-27 19:29 ` Hans de Goede
2022-07-27 20:26 ` Hans de Goede
2022-07-12 14:58 ` [PATCH v1 05/15] platform/x86/amd/pmf: Add debugfs information Shyam Sundar S K
2022-07-27 19:50 ` Hans de Goede
2022-07-12 14:58 ` [PATCH v1 06/15] platform/x86/amd/pmf: Add heartbeat signal support Shyam Sundar S K
2022-07-27 19:53 ` Hans de Goede
2022-07-12 14:58 ` [PATCH v1 07/15] platform/x86/amd/pmf: Add fan control support Shyam Sundar S K
2022-07-27 20:11 ` Hans de Goede
2022-07-12 14:58 ` [PATCH v1 08/15] platform/x86/amd/pmf: Get performance metrics from PMFW Shyam Sundar S K
2022-07-27 20:36 ` Hans de Goede
2022-07-12 14:58 ` [PATCH v1 09/15] platform/x86/amd/pmf: Add support for CnQF Shyam Sundar S K
2022-07-27 20:51 ` Hans de Goede
2022-07-27 21:00 ` Hans de Goede
2022-07-12 14:58 ` [PATCH v1 10/15] platform/x86/amd/pmf: Add sysfs to toggle CnQF Shyam Sundar S K
2022-07-27 20:52 ` Hans de Goede
2022-07-27 21:12 ` Hans de Goede
2022-07-12 14:58 ` [PATCH v1 11/15] Documentation/ABI/testing/sysfs-amd-pmf: Add ABI doc for AMD PMF Shyam Sundar S K
2022-07-27 20:52 ` Hans de Goede
2022-07-12 14:58 ` [PATCH v1 12/15] platform/x86/amd/pmf: Add support for Auto mode feature Shyam Sundar S K
2022-07-27 21:22 ` Hans de Goede
2022-07-28 12:57 ` Shyam Sundar S K
2022-07-28 13:15 ` Hans de Goede
2022-07-12 14:58 ` [PATCH v1 13/15] platform/x86/amd/pmf: Handle AMT and CQL events for Auto mode Shyam Sundar S K
2022-07-27 21:33 ` Hans de Goede
2022-07-27 21:44 ` Hans de Goede
2022-07-27 21:46 ` Hans de Goede
2022-07-27 23:52 ` Limonciello, Mario
2022-07-28 13:03 ` Hans de Goede [this message]
2022-07-28 13:43 ` Limonciello, Mario
2022-07-28 14:09 ` Hans de Goede
2022-07-28 14:38 ` Limonciello, Mario
2022-07-28 17:46 ` Hans de Goede
2022-07-28 18:06 ` Limonciello, Mario
2022-07-28 18:17 ` Hans de Goede
2022-07-28 21:01 ` Limonciello, Mario
2022-07-29 11:03 ` Hans de Goede
2022-07-29 15:43 ` Limonciello, Mario
2022-07-29 17:40 ` Shyam Sundar S K
2022-07-29 17:59 ` Hans de Goede
2022-08-01 10:29 ` Shyam Sundar S K
2022-08-01 11:08 ` Hans de Goede
2022-07-12 14:58 ` [PATCH v1 14/15] platform/x86/amd/pmf: Force load driver on older supported platforms Shyam Sundar S K
2022-07-27 21:40 ` Hans de Goede
2022-07-12 14:58 ` [PATCH v1 15/15] MAINTAINERS: Add AMD PMF driver entry Shyam Sundar S K
2022-07-27 21:41 ` Hans de Goede
2022-07-28 17:44 ` Shyam Sundar S K
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=828a7b8e-2375-a9d0-7b00-d59e84cfaa2b@redhat.com \
--to=hdegoede@redhat.com \
--cc=Patil.Reddy@amd.com \
--cc=Shyam-sundar.S-k@amd.com \
--cc=mario.limonciello@amd.com \
--cc=markgross@kernel.org \
--cc=markpearson@lenovo.com \
--cc=platform-driver-x86@vger.kernel.org \
/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).