[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <vfx2z4dn6afpkl4z7ygewadw7mohfd7l3tfazdya4sbxniurkg@vrubq25slkvs>
Date: Tue, 14 May 2024 16:20:04 +0530
From: Naveen N Rao <naveen@...nel.org>
To: Gautam Menghani <gautam@...ux.ibm.com>
Cc: mpe@...erman.id.au, npiggin@...il.com, christophe.leroy@...roup.eu,
linuxppc-dev@...ts.ozlabs.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arch/powerpc: Remove the definition of unused cede
function
On Tue, May 14, 2024 at 03:35:03PM GMT, Gautam Menghani wrote:
> Remove extended_cede_processor() definition as it has no callers since
> commit 48f6e7f6d948("powerpc/pseries: remove cede offline state for CPUs")
extended_cede_processor() was added in commit 69ddb57cbea0
("powerpc/pseries: Add extended_cede_processor() helper function."),
which also added [get|set]_cede_latency_hint(). Those can also be
removed if extended_cede_processor() is no longer needed.
- Naveen
>
> Signed-off-by: Gautam Menghani <gautam@...ux.ibm.com>
> ---
> arch/powerpc/include/asm/plpar_wrappers.h | 18 ------------------
> 1 file changed, 18 deletions(-)
>
> diff --git a/arch/powerpc/include/asm/plpar_wrappers.h b/arch/powerpc/include/asm/plpar_wrappers.h
> index b3ee44a40c2f..6431fa1e1cb1 100644
> --- a/arch/powerpc/include/asm/plpar_wrappers.h
> +++ b/arch/powerpc/include/asm/plpar_wrappers.h
> @@ -37,24 +37,6 @@ static inline long cede_processor(void)
> return plpar_hcall_norets_notrace(H_CEDE);
> }
>
> -static inline long extended_cede_processor(unsigned long latency_hint)
> -{
> - long rc;
> - u8 old_latency_hint = get_cede_latency_hint();
> -
> - set_cede_latency_hint(latency_hint);
> -
> - rc = cede_processor();
> -
> - /* Ensure that H_CEDE returns with IRQs on */
> - if (WARN_ON(IS_ENABLED(CONFIG_PPC_IRQ_SOFT_MASK_DEBUG) && !(mfmsr() & MSR_EE)))
> - __hard_irq_enable();
> -
> - set_cede_latency_hint(old_latency_hint);
> -
> - return rc;
> -}
> -
> static inline long vpa_call(unsigned long flags, unsigned long cpu,
> unsigned long vpa)
> {
> --
> 2.45.0
>
Powered by blists - more mailing lists