Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751765AbdGEPOX (ORCPT ); Wed, 5 Jul 2017 11:14:23 -0400 Received: from verein.lst.de ([213.95.11.211]:54954 "EHLO newverein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751636AbdGEPOW (ORCPT ); Wed, 5 Jul 2017 11:14:22 -0400 Date: Wed, 5 Jul 2017 17:14:20 +0200 From: Christoph Hellwig To: Linus Torvalds Cc: Thomas Gleixner , Jens Axboe , Max Gurtovoy , Christoph Hellwig , LKML , Andrew Morton , Ingo Molnar , "H. Peter Anvin" Subject: Re: [GIT pull] irq updates for 4.13 Message-ID: <20170705151420.GA2371@lst.de> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1392 Lines: 26 On Mon, Jul 03, 2017 at 05:00:03PM -0700, Linus Torvalds wrote: > I'm not at all understanding why that second commit came in through > the irq tree at all, in fact. Very annoying. Why was that not sent > through the block tree? It doesn't seem to have anything fundamentally > to do with irqs, really: it's a driver CPU choice for irq chocie. It depends on a major IRQ layer rework, and it had at that time no clash with the block tree at all. That's why I suggested to Thomas and Jens that we should take it through the irq code. Then Max' code showed up last minute and wrecked that plan, and no one noticed in time. Blame it on me for not noticing this in time. > Anyway, I absolutely detested that code, and the obvious resolution > was too disgusting to live. So I did an evil merge and moved some > things around in the merge to make it at least not cause me to dig my > eyes out. > > But I'd like people to look at that - not so much due to the evil > merge itself (but check that too, by any means), but just because the > code seems fundamentally broken for the hotplug case. We end up > picking a possible metric shit-ton of CPU's for queue 0, if they were > "possible but not online". I think this code also needs to move over to cpu_possible instead of cpu_online to match what we did for the MSI-X based mapping. But I'll need a little more coffee and get back into it.