Re: [PATCH] powerpc/paravirt: correct preempt debug splat in vcpu_is_preempted()

2021-09-24 Thread Nathan Lynch
Michael Ellerman writes: > Srikar Dronamraju writes: >> * Michael Ellerman [2021-09-23 17:29:32]: >> >>> Nathan Lynch writes: >>> > Srikar Dronamraju writes: >>> > >>> >> * Nathan Lynch [2021-09-22 11:01:12]: >>> >> >>> >>> Srikar Dronamraju writes: > ... >>> >> Or can I understand how

Re: [PATCH] powerpc/paravirt: correct preempt debug splat in vcpu_is_preempted()

2021-09-23 Thread Michael Ellerman
Srikar Dronamraju writes: > * Michael Ellerman [2021-09-23 17:29:32]: > >> Nathan Lynch writes: >> > Srikar Dronamraju writes: >> > >> >> * Nathan Lynch [2021-09-22 11:01:12]: >> >> >> >>> Srikar Dronamraju writes: ... >> >> Or can I understand how debug_smp_processor_id() is useful if >> >>

Re: [PATCH] powerpc/paravirt: correct preempt debug splat in vcpu_is_preempted()

2021-09-23 Thread Srikar Dronamraju
* Michael Ellerman [2021-09-23 17:29:32]: > Nathan Lynch writes: > > Srikar Dronamraju writes: > > > >> * Nathan Lynch [2021-09-22 11:01:12]: > >> > >>> Srikar Dronamraju writes: > >>> > * Nathan Lynch [2021-09-20 22:12:13]: > >>> > > >>> >> vcpu_is_preempted() can be used outside of

Re: [PATCH] powerpc/paravirt: correct preempt debug splat in vcpu_is_preempted()

2021-09-23 Thread Michael Ellerman
Nathan Lynch writes: > Srikar Dronamraju writes: > >> * Nathan Lynch [2021-09-22 11:01:12]: >> >>> Srikar Dronamraju writes: >>> > * Nathan Lynch [2021-09-20 22:12:13]: >>> > >>> >> vcpu_is_preempted() can be used outside of preempt-disabled critical >>> >> sections, yielding warnings such

Re: [PATCH] powerpc/paravirt: correct preempt debug splat in vcpu_is_preempted()

2021-09-22 Thread Nathan Lynch
Srikar Dronamraju writes: > * Nathan Lynch [2021-09-22 11:01:12]: > >> Srikar Dronamraju writes: >> > * Nathan Lynch [2021-09-20 22:12:13]: >> > >> >> vcpu_is_preempted() can be used outside of preempt-disabled critical >> >> sections, yielding warnings such as: >> >> >> >> BUG: using

Re: [PATCH] powerpc/paravirt: correct preempt debug splat in vcpu_is_preempted()

2021-09-22 Thread Srikar Dronamraju
* Nathan Lynch [2021-09-22 11:01:12]: > Srikar Dronamraju writes: > > * Nathan Lynch [2021-09-20 22:12:13]: > > > >> vcpu_is_preempted() can be used outside of preempt-disabled critical > >> sections, yielding warnings such as: > >> > >> BUG: using smp_processor_id() in preemptible []

Re: [PATCH] powerpc/paravirt: correct preempt debug splat in vcpu_is_preempted()

2021-09-22 Thread Nathan Lynch
Srikar Dronamraju writes: > * Nathan Lynch [2021-09-20 22:12:13]: > >> vcpu_is_preempted() can be used outside of preempt-disabled critical >> sections, yielding warnings such as: >> >> BUG: using smp_processor_id() in preemptible [] code: >> systemd-udevd/185 >> caller is

Re: [PATCH] powerpc/paravirt: correct preempt debug splat in vcpu_is_preempted()

2021-09-22 Thread Nathan Lynch
Michael Ellerman writes: > Nathan Lynch writes: >> --- a/arch/powerpc/include/asm/paravirt.h >> +++ b/arch/powerpc/include/asm/paravirt.h >> @@ -97,7 +97,14 @@ static inline bool vcpu_is_preempted(int cpu) >> >> #ifdef CONFIG_PPC_SPLPAR >> if (!is_kvm_guest()) { >> -int

Re: [PATCH] powerpc/paravirt: correct preempt debug splat in vcpu_is_preempted()

2021-09-22 Thread Srikar Dronamraju
* Nathan Lynch [2021-09-20 22:12:13]: > vcpu_is_preempted() can be used outside of preempt-disabled critical > sections, yielding warnings such as: > > BUG: using smp_processor_id() in preemptible [] code: > systemd-udevd/185 > caller is rwsem_spin_on_owner+0x1cc/0x2d0 > CPU: 1 PID:

Re: [PATCH] powerpc/paravirt: correct preempt debug splat in vcpu_is_preempted()

2021-09-22 Thread Michael Ellerman
Nathan Lynch writes: > vcpu_is_preempted() can be used outside of preempt-disabled critical > sections, yielding warnings such as: > > BUG: using smp_processor_id() in preemptible [] code: > systemd-udevd/185 > caller is rwsem_spin_on_owner+0x1cc/0x2d0 > CPU: 1 PID: 185 Comm:

[PATCH] powerpc/paravirt: correct preempt debug splat in vcpu_is_preempted()

2021-09-20 Thread Nathan Lynch
vcpu_is_preempted() can be used outside of preempt-disabled critical sections, yielding warnings such as: BUG: using smp_processor_id() in preemptible [] code: systemd-udevd/185 caller is rwsem_spin_on_owner+0x1cc/0x2d0 CPU: 1 PID: 185 Comm: systemd-udevd Not tainted 5.15.0-rc2+ #33 Call