public inbox for linux-hyperv@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] x86/hyperv: Cosmetic changes for hv_spinlock.c
@ 2024-03-18 12:09 Purna Pavan Chandra Aekkaladevi
  2024-03-18 16:21 ` Saurabh Singh Sengar
  0 siblings, 1 reply; 3+ messages in thread
From: Purna Pavan Chandra Aekkaladevi @ 2024-03-18 12:09 UTC (permalink / raw)
  To: kys, haiyangz, wei.liu, decui, tglx, mingo, bp, dave.hansen, x86,
	hpa, linux-hyperv
  Cc: paekkaladevi, Purna Pavan Chandra Aekkaladevi

Fix issues reported by checkpatch.pl script for hv_spinlock.c file.
- Place __initdata after variable name
- Add missing blank line after enum declaration

No functional chagnes intended.

Signed-off-by: Purna Pavan Chandra Aekkaladevi <paekkaladevi@linux.microsoft.com>
---
 arch/x86/hyperv/hv_spinlock.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/arch/x86/hyperv/hv_spinlock.c b/arch/x86/hyperv/hv_spinlock.c
index 737d6f7a6155..151e851bef09 100644
--- a/arch/x86/hyperv/hv_spinlock.c
+++ b/arch/x86/hyperv/hv_spinlock.c
@@ -16,7 +16,7 @@
 #include <asm/paravirt.h>
 #include <asm/apic.h>
 
-static bool __initdata hv_pvspin = true;
+static bool hv_pvspin __initdata = true;
 
 static void hv_qlock_kick(int cpu)
 {
@@ -64,6 +64,7 @@ __visible bool hv_vcpu_is_preempted(int vcpu)
 {
 	return false;
 }
+
 PV_CALLEE_SAVE_REGS_THUNK(hv_vcpu_is_preempted);
 
 void __init hv_init_spinlocks(void)
-- 
2.34.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] x86/hyperv: Cosmetic changes for hv_spinlock.c
  2024-03-18 12:09 [PATCH] x86/hyperv: Cosmetic changes for hv_spinlock.c Purna Pavan Chandra Aekkaladevi
@ 2024-03-18 16:21 ` Saurabh Singh Sengar
  2024-03-18 21:57   ` Wei Liu
  0 siblings, 1 reply; 3+ messages in thread
From: Saurabh Singh Sengar @ 2024-03-18 16:21 UTC (permalink / raw)
  To: Purna Pavan Chandra Aekkaladevi
  Cc: kys, haiyangz, wei.liu, decui, tglx, mingo, bp, dave.hansen, x86,
	hpa, linux-hyperv, paekkaladevi

On Mon, Mar 18, 2024 at 05:09:11AM -0700, Purna Pavan Chandra Aekkaladevi wrote:
> Fix issues reported by checkpatch.pl script for hv_spinlock.c file.
> - Place __initdata after variable name
> - Add missing blank line after enum declaration
> 
> No functional chagnes intended.

s/chagnes/changes/

with this,
Reviewed-by: Saurabh Sengar <ssengar@linux.microsoft.com>

> 
> Signed-off-by: Purna Pavan Chandra Aekkaladevi <paekkaladevi@linux.microsoft.com>
> ---
>  arch/x86/hyperv/hv_spinlock.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/x86/hyperv/hv_spinlock.c b/arch/x86/hyperv/hv_spinlock.c
> index 737d6f7a6155..151e851bef09 100644
> --- a/arch/x86/hyperv/hv_spinlock.c
> +++ b/arch/x86/hyperv/hv_spinlock.c
> @@ -16,7 +16,7 @@
>  #include <asm/paravirt.h>
>  #include <asm/apic.h>
>  
> -static bool __initdata hv_pvspin = true;
> +static bool hv_pvspin __initdata = true;
>  
>  static void hv_qlock_kick(int cpu)
>  {
> @@ -64,6 +64,7 @@ __visible bool hv_vcpu_is_preempted(int vcpu)
>  {
>  	return false;
>  }
> +
>  PV_CALLEE_SAVE_REGS_THUNK(hv_vcpu_is_preempted);
>  
>  void __init hv_init_spinlocks(void)
> -- 
> 2.34.1
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] x86/hyperv: Cosmetic changes for hv_spinlock.c
  2024-03-18 16:21 ` Saurabh Singh Sengar
@ 2024-03-18 21:57   ` Wei Liu
  0 siblings, 0 replies; 3+ messages in thread
From: Wei Liu @ 2024-03-18 21:57 UTC (permalink / raw)
  To: Saurabh Singh Sengar
  Cc: Purna Pavan Chandra Aekkaladevi, kys, haiyangz, wei.liu, decui,
	tglx, mingo, bp, dave.hansen, x86, hpa, linux-hyperv,
	paekkaladevi

On Mon, Mar 18, 2024 at 09:21:28AM -0700, Saurabh Singh Sengar wrote:
> On Mon, Mar 18, 2024 at 05:09:11AM -0700, Purna Pavan Chandra Aekkaladevi wrote:
> > Fix issues reported by checkpatch.pl script for hv_spinlock.c file.
> > - Place __initdata after variable name
> > - Add missing blank line after enum declaration
> > 
> > No functional chagnes intended.
> 
> s/chagnes/changes/
> 
> with this,
> Reviewed-by: Saurabh Sengar <ssengar@linux.microsoft.com>
> 

Applied. Thanks.

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2024-03-18 21:57 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-03-18 12:09 [PATCH] x86/hyperv: Cosmetic changes for hv_spinlock.c Purna Pavan Chandra Aekkaladevi
2024-03-18 16:21 ` Saurabh Singh Sengar
2024-03-18 21:57   ` Wei Liu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox