Subject: Re: [PATCH v2] powerpc: re-enable dynticks

From: Paul Clarke <[email-masked]>
Date: Fri, 27 Feb 2015 07:27:39 -0600

It appears the discussion wrapped up on this, favorably.  Is there 
anything else I need to do to get this merged?

Regards,
PC

On 02/20/2015 11:18 PM, Paul E. McKenney wrote:
> On Fri, Feb 20, 2015 at 11:13:33AM -0600, Paul Clarke wrote:
>>
>> implement arch_irq_work_has_interrupt() for powerpc
>>
>> (resending because I messed up the e-mail addresses)
>>
>> Commit 9b01f5bf3 introduced a dependency on "IRQ work self-IPIs" for
>> full dynamic ticks to be enabled, by expecting architectures to
>> implement a suitable arch_irq_work_has_interrupt() routine.
>>
>> Several arches have implemented this routine, including x86
>> (3010279f) and arm (09f6edd4), but powerpc was omitted.
>>
>> This patch implements this routine for powerpc.
>>
>> The symptom, at boot (on powerpc arch systems) with "nohz_full=<CPU
>> list>" is displayed:
>>      NO_HZ: Can't run full dynticks because arch doesn't support irq
>> work self-IPIs
>>
>> after this patch:
>>      NO_HZ: Full dynticks CPUs: <CPU list>.
>>
>> Tested against 3.19.
>>
>> v2: changed "return 1" to "return true", per Michael Ellerman
>>
>> CC: Frederic Weisbecker <[email-masked]>
>> CC: Paul E. McKenney <[email-masked]>
>> Signed-off-by: Paul A. Clarke <[email-masked]>
>
> Reviewed-by: Paul E. McKenney <[email-masked]>
>
>> diff --git a/arch/powerpc/include/asm/irq_work.h
>> b/arch/powerpc/include/asm/irq_work.h
>> new file mode 100644
>> index 0000000..99cc0aa
>> --- /dev/null
>> +++ b/arch/powerpc/include/asm/irq_work.h
>> @@ -0,0 +1,11 @@
>> +#ifndef _ASM_IRQ_WORK_H
>> +#define _ASM_IRQ_WORK_H
>> +
>> +#include <asm/processor.h>
>> +
>> +static inline bool arch_irq_work_has_interrupt(void)
>> +{
>> +    return true;
>> +}
>> +
>> +#endif /* _ASM_IRQ_WORK_H */
>
> _______________________________________________
> Linuxppc-dev mailing list
> [email-masked]
> https://lists.ozlabs.org/listinfo/linuxppc-dev
>