| Message ID | 20240514132457.292865-1-gautam@linux.ibm.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [v2] arch/powerpc: Remove unused cede related functions | expand |
| Context | Check | Description |
|---|---|---|
| snowpatch_ozlabs/github-powerpc_ppctests | success | Successfully ran 8 jobs. |
| snowpatch_ozlabs/github-powerpc_selftests | success | Successfully ran 8 jobs. |
| snowpatch_ozlabs/github-powerpc_clang | success | Successfully ran 6 jobs. |
| snowpatch_ozlabs/github-powerpc_sparse | success | Successfully ran 4 jobs. |
| snowpatch_ozlabs/github-powerpc_kernel_qemu | success | Successfully ran 23 jobs. |
On Tue, May 14, 2024 at 06:54:55PM GMT, Gautam Menghani wrote: > Remove extended_cede_processor() and its helpers as > extended_cede_processor() has no callers since > commit 48f6e7f6d948("powerpc/pseries: remove cede offline state for CPUs") > > Signed-off-by: Gautam Menghani <gautam@linux.ibm.com> > --- > v1 -> v2: > 1. Remove helpers of extended_cede_processor() Acked-by: Naveen N Rao <naveen@kernel.org> > > arch/powerpc/include/asm/plpar_wrappers.h | 28 ----------------------- > 1 file changed, 28 deletions(-) > > diff --git a/arch/powerpc/include/asm/plpar_wrappers.h b/arch/powerpc/include/asm/plpar_wrappers.h > index b3ee44a40c2f..71648c126970 100644 > --- a/arch/powerpc/include/asm/plpar_wrappers.h > +++ b/arch/powerpc/include/asm/plpar_wrappers.h > @@ -18,16 +18,6 @@ static inline long poll_pending(void) > return plpar_hcall_norets(H_POLL_PENDING); > } > > -static inline u8 get_cede_latency_hint(void) > -{ > - return get_lppaca()->cede_latency_hint; > -} > - > -static inline void set_cede_latency_hint(u8 latency_hint) > -{ > - get_lppaca()->cede_latency_hint = latency_hint; > -} > - > static inline long cede_processor(void) > { > /* > @@ -37,24 +27,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 >
diff --git a/arch/powerpc/include/asm/plpar_wrappers.h b/arch/powerpc/include/asm/plpar_wrappers.h index b3ee44a40c2f..71648c126970 100644 --- a/arch/powerpc/include/asm/plpar_wrappers.h +++ b/arch/powerpc/include/asm/plpar_wrappers.h @@ -18,16 +18,6 @@ static inline long poll_pending(void) return plpar_hcall_norets(H_POLL_PENDING); } -static inline u8 get_cede_latency_hint(void) -{ - return get_lppaca()->cede_latency_hint; -} - -static inline void set_cede_latency_hint(u8 latency_hint) -{ - get_lppaca()->cede_latency_hint = latency_hint; -} - static inline long cede_processor(void) { /* @@ -37,24 +27,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) {
Remove extended_cede_processor() and its helpers as extended_cede_processor() has no callers since commit 48f6e7f6d948("powerpc/pseries: remove cede offline state for CPUs") Signed-off-by: Gautam Menghani <gautam@linux.ibm.com> --- v1 -> v2: 1. Remove helpers of extended_cede_processor() arch/powerpc/include/asm/plpar_wrappers.h | 28 ----------------------- 1 file changed, 28 deletions(-)