lkml.org 
[lkml]   [2022]   [Jun]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.15 134/667] ACPI: CPPC: Assume no transition latency if no PCCT
    Date
    From: Pierre Gondois <Pierre.Gondois@arm.com>

    [ Upstream commit 6380b7b2b29da9d9c5ab2d4a265901cd93ba3696 ]

    The transition_delay_us (struct cpufreq_policy) is currently defined
    as:
    Preferred average time interval between consecutive invocations of
    the driver to set the frequency for this policy. To be set by the
    scaling driver (0, which is the default, means no preference).
    The transition_latency represents the amount of time necessary for a
    CPU to change its frequency.

    A PCCT table advertises mutliple values:
    - pcc_nominal: Expected latency to process a command, in microseconds
    - pcc_mpar: The maximum number of periodic requests that the subspace
    channel can support, reported in commands per minute. 0 indicates no
    limitation.
    - pcc_mrtt: The minimum amount of time that OSPM must wait after the
    completion of a command before issuing the next command,
    in microseconds.
    cppc_get_transition_latency() allows to get the max of them.

    commit d4f3388afd48 ("cpufreq / CPPC: Set platform specific
    transition_delay_us") allows to select transition_delay_us based on
    the platform, and fallbacks to cppc_get_transition_latency()
    otherwise.

    If _CPC objects are not using PCC channels (no PPCT table), the
    transition_delay_us is set to CPUFREQ_ETERNAL, leading to really long
    periods between frequency updates (~4s).

    If the desired_reg, where performance requests are written, is in
    SystemMemory or SystemIo ACPI address space, there is no delay
    in requests. So return 0 instead of CPUFREQ_ETERNAL, leading to
    transition_delay_us being set to LATENCY_MULTIPLIER us (1000 us).

    This patch also adds two macros to check the address spaces.

    Signed-off-by: Pierre Gondois <pierre.gondois@arm.com>
    Reviewed-by: Sudeep Holla <sudeep.holla@arm.com>
    Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/acpi/cppc_acpi.c | 17 ++++++++++++++++-
    1 file changed, 16 insertions(+), 1 deletion(-)

    diff --git a/drivers/acpi/cppc_acpi.c b/drivers/acpi/cppc_acpi.c
    index 71b87c16f92d..ed1341030684 100644
    --- a/drivers/acpi/cppc_acpi.c
    +++ b/drivers/acpi/cppc_acpi.c
    @@ -100,6 +100,16 @@ static DEFINE_PER_CPU(struct cpc_desc *, cpc_desc_ptr);
    (cpc)->cpc_entry.reg.space_id == \
    ACPI_ADR_SPACE_PLATFORM_COMM)

    +/* Check if a CPC register is in SystemMemory */
    +#define CPC_IN_SYSTEM_MEMORY(cpc) ((cpc)->type == ACPI_TYPE_BUFFER && \
    + (cpc)->cpc_entry.reg.space_id == \
    + ACPI_ADR_SPACE_SYSTEM_MEMORY)
    +
    +/* Check if a CPC register is in SystemIo */
    +#define CPC_IN_SYSTEM_IO(cpc) ((cpc)->type == ACPI_TYPE_BUFFER && \
    + (cpc)->cpc_entry.reg.space_id == \
    + ACPI_ADR_SPACE_SYSTEM_IO)
    +
    /* Evaluates to True if reg is a NULL register descriptor */
    #define IS_NULL_REG(reg) ((reg)->space_id == ACPI_ADR_SPACE_SYSTEM_MEMORY && \
    (reg)->address == 0 && \
    @@ -1378,6 +1388,9 @@ EXPORT_SYMBOL_GPL(cppc_set_perf);
    * transition latency for performance change requests. The closest we have
    * is the timing information from the PCCT tables which provides the info
    * on the number and frequency of PCC commands the platform can handle.
    + *
    + * If desired_reg is in the SystemMemory or SystemIo ACPI address space,
    + * then assume there is no latency.
    */
    unsigned int cppc_get_transition_latency(int cpu_num)
    {
    @@ -1403,7 +1416,9 @@ unsigned int cppc_get_transition_latency(int cpu_num)
    return CPUFREQ_ETERNAL;

    desired_reg = &cpc_desc->cpc_regs[DESIRED_PERF];
    - if (!CPC_IN_PCC(desired_reg))
    + if (CPC_IN_SYSTEM_MEMORY(desired_reg) || CPC_IN_SYSTEM_IO(desired_reg))
    + return 0;
    + else if (!CPC_IN_PCC(desired_reg))
    return CPUFREQ_ETERNAL;

    if (pcc_ss_id < 0)
    --
    2.35.1


    \
     
     \ /
      Last update: 2022-06-07 20:33    [W:4.063 / U:0.096 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site