Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1677693 > unrolled thread

Re: [1/3] cpuidle: powerpc: cpuidle set polling before enabling irqs

Started byMichael Ellerman <patch-notifications@ellerman.id.au>
First post2017-06-29 14:30 +0200
Last post2017-06-30 15:00 +0200
Articles 4 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [1/3] cpuidle: powerpc: cpuidle set polling before enabling irqs Michael Ellerman <patch-notifications@ellerman.id.au> - 2017-06-29 14:30 +0200
    Re: [1/3] cpuidle: powerpc: cpuidle set polling before enabling irqs "Rafael J. Wysocki" <rafael@kernel.org> - 2017-06-29 22:40 +0200
      Re: [1/3] cpuidle: powerpc: cpuidle set polling before enabling irqs Michael Ellerman <mpe@ellerman.id.au> - 2017-06-30 05:50 +0200
        Re: [1/3] cpuidle: powerpc: cpuidle set polling before enabling irqs "Rafael J. Wysocki" <rafael@kernel.org> - 2017-06-30 15:00 +0200

#1677693 — Re: [1/3] cpuidle: powerpc: cpuidle set polling before enabling irqs

FromMichael Ellerman <patch-notifications@ellerman.id.au>
Date2017-06-29 14:30 +0200
SubjectRe: [1/3] cpuidle: powerpc: cpuidle set polling before enabling irqs
Message-ID<tXGK7-7yG-25@gated-at.bofh.it>
On Wed, 2017-06-14 at 13:02:39 UTC, Nicholas Piggin wrote:
> local_irq_enable can cause interrupts to be taken which could
> take significant amount of processing time. The idle process
> should set its polling flag before this, so another process that
> wakes it during this time will not have to send an IPI.
> 
> Expand the TIF_POLLING_NRFLAG coverage to as large as possible.
> 
> Reviewed-by: Gautham R. Shenoy <ego@linux.vnet.ibm.com>
> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>

Series applied to powerpc next, thanks.

https://git.kernel.org/powerpc/c/3fc5ee927ff4ffed6aa2fcd44d2fbf

cheers

[toc] | [next] | [standalone]


#1678181

From"Rafael J. Wysocki" <rafael@kernel.org>
Date2017-06-29 22:40 +0200
Message-ID<tXOoi-40D-17@gated-at.bofh.it>
In reply to#1677693
On Thu, Jun 29, 2017 at 2:21 PM, Michael Ellerman
<patch-notifications@ellerman.id.au> wrote:
> On Wed, 2017-06-14 at 13:02:39 UTC, Nicholas Piggin wrote:
>> local_irq_enable can cause interrupts to be taken which could
>> take significant amount of processing time. The idle process
>> should set its polling flag before this, so another process that
>> wakes it during this time will not have to send an IPI.
>>
>> Expand the TIF_POLLING_NRFLAG coverage to as large as possible.
>>
>> Reviewed-by: Gautham R. Shenoy <ego@linux.vnet.ibm.com>
>> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
>
> Series applied to powerpc next, thanks.
>
> https://git.kernel.org/powerpc/c/3fc5ee927ff4ffed6aa2fcd44d2fbf

OK

I've applied it too, so I guess I should drop it?

Thanks,
Rafael

[toc] | [prev] | [next] | [standalone]


#1678440

FromMichael Ellerman <mpe@ellerman.id.au>
Date2017-06-30 05:50 +0200
Message-ID<tXV6p-8tl-11@gated-at.bofh.it>
In reply to#1678181
"Rafael J. Wysocki" <rafael@kernel.org> writes:

> On Thu, Jun 29, 2017 at 2:21 PM, Michael Ellerman
> <patch-notifications@ellerman.id.au> wrote:
>> On Wed, 2017-06-14 at 13:02:39 UTC, Nicholas Piggin wrote:
>>> local_irq_enable can cause interrupts to be taken which could
>>> take significant amount of processing time. The idle process
>>> should set its polling flag before this, so another process that
>>> wakes it during this time will not have to send an IPI.
>>>
>>> Expand the TIF_POLLING_NRFLAG coverage to as large as possible.
>>>
>>> Reviewed-by: Gautham R. Shenoy <ego@linux.vnet.ibm.com>
>>> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
>>
>> Series applied to powerpc next, thanks.
>>
>> https://git.kernel.org/powerpc/c/3fc5ee927ff4ffed6aa2fcd44d2fbf
>
> OK
>
> I've applied it too, so I guess I should drop it?

Erk sorry. I hadn't heard anything so I picked it up.

If you can drop it that would be good, but if not git will probably work
it out mostly :)

cheers

[toc] | [prev] | [next] | [standalone]


#1678790

From"Rafael J. Wysocki" <rafael@kernel.org>
Date2017-06-30 15:00 +0200
Message-ID<tY3GF-5AN-11@gated-at.bofh.it>
In reply to#1678440
On Fri, Jun 30, 2017 at 5:45 AM, Michael Ellerman <mpe@ellerman.id.au> wrote:
> "Rafael J. Wysocki" <rafael@kernel.org> writes:
>
>> On Thu, Jun 29, 2017 at 2:21 PM, Michael Ellerman
>> <patch-notifications@ellerman.id.au> wrote:
>>> On Wed, 2017-06-14 at 13:02:39 UTC, Nicholas Piggin wrote:
>>>> local_irq_enable can cause interrupts to be taken which could
>>>> take significant amount of processing time. The idle process
>>>> should set its polling flag before this, so another process that
>>>> wakes it during this time will not have to send an IPI.
>>>>
>>>> Expand the TIF_POLLING_NRFLAG coverage to as large as possible.
>>>>
>>>> Reviewed-by: Gautham R. Shenoy <ego@linux.vnet.ibm.com>
>>>> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
>>>
>>> Series applied to powerpc next, thanks.
>>>
>>> https://git.kernel.org/powerpc/c/3fc5ee927ff4ffed6aa2fcd44d2fbf
>>
>> OK
>>
>> I've applied it too, so I guess I should drop it?
>
> Erk sorry. I hadn't heard anything so I picked it up.
>
> If you can drop it that would be good, but if not git will probably work
> it out mostly :)

I've dropped it, no problem.

Thanks,
Rafael

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web