From mboxrd@z Thu Jan 1 00:00:00 1970 From: Peter Zijlstra Subject: Re: [PATCH 2/6] irq_work: Implement remote queueing Date: Wed, 25 Jun 2014 11:36:57 +0200 Message-ID: <20140625093657.GN19860@laptop.programming.kicks-ass.net> References: <1402413309-3415-1-git-send-email-fweisbec@gmail.com> <1402413309-3415-3-git-send-email-fweisbec@gmail.com> <53A9E0A5.5040402@wwwdotorg.org> <20140625051234.GD6758@twins.programming.kicks-ass.net> <20140625051721.GG3588@twins.programming.kicks-ass.net> <53AA6E11.1000404@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <53AA6E11.1000404@linux.vnet.ibm.com> Sender: linux-kernel-owner@vger.kernel.org To: "Srivatsa S. Bhat" Cc: Stephen Warren , Frederic Weisbecker , LKML , Andrew Morton , Eric Dumazet , Ingo Molnar , Kevin Hilman , "Paul E. McKenney" , Thomas Gleixner , Viresh Kumar , "linux-next@vger.kernel.org" List-Id: linux-next.vger.kernel.org On Wed, Jun 25, 2014 at 12:07:05PM +0530, Srivatsa S. Bhat wrote: > I don't think irqs_disabled() is the problematic condition, since > hotplug_cfg() invokes irq_work_run() from CPU_DYING context (which has > irqs disabled). I guess you meant to remove the in_irq() check inside > irq_work_run() instead? Yes, clearly I should not get up at 6am.. :-) Let me go do a new one.