Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752332AbdF3MvL (ORCPT ); Fri, 30 Jun 2017 08:51:11 -0400 Received: from mail-io0-f195.google.com ([209.85.223.195]:36136 "EHLO mail-io0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751741AbdF3MvJ (ORCPT ); Fri, 30 Jun 2017 08:51:09 -0400 MIME-Version: 1.0 In-Reply-To: <878tkaqgu2.fsf@concordia.ellerman.id.au> References: <20170614130241.19865-2-npiggin@gmail.com> <3wyzKL1g2Vz9s65@ozlabs.org> <878tkaqgu2.fsf@concordia.ellerman.id.au> From: "Rafael J. Wysocki" Date: Fri, 30 Jun 2017 14:51:08 +0200 X-Google-Sender-Auth: oiVNZDYpQ7yQ4N7HTNyspN075OE Message-ID: Subject: Re: [1/3] cpuidle: powerpc: cpuidle set polling before enabling irqs To: Michael Ellerman Cc: "Rafael J. Wysocki" , Michael Ellerman , Nicholas Piggin , linuxppc-dev , "Gautham R . Shenoy" , Linux PM , Daniel Lezcano , "Rafael J . Wysocki" , Linux Kernel Mailing List Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1138 Lines: 33 On Fri, Jun 30, 2017 at 5:45 AM, Michael Ellerman wrote: > "Rafael J. Wysocki" writes: > >> On Thu, Jun 29, 2017 at 2:21 PM, Michael Ellerman >> 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 >>>> Signed-off-by: Nicholas Piggin >>> >>> 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