Received: by 2002:a05:6a10:206:0:0:0:0 with SMTP id 6csp1233143pxj; Wed, 19 May 2021 01:06:13 -0700 (PDT) X-Google-Smtp-Source: ABdhPJxS8ApaTjPAOUskGfTnG6EfssLmxb5jkNRG8EUhB7UXVFV1XTCWIuN5LY6JRF2eX8PP5hIw X-Received: by 2002:a05:6402:518f:: with SMTP id q15mr12851407edd.345.1621411573442; Wed, 19 May 2021 01:06:13 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1621411573; cv=none; d=google.com; s=arc-20160816; b=MOK/rkdzCU44QskXpEqLEqvq8S6OqNKXD41Ly1AfsFLqLI1FpcM2Gd5uTEroNNTXtw WbssXCN89QU6ySkGCMr8InmnoBIs/VWTfFiIZC0VT2GO/anyq8rBkiRp19ZlKUtKcgn2 ZzDGB6EopYTWnoIDQUoVjakFO17hHZCGe8WGzdSSrCANfEz4jARJ1eRWo2kSTchMSkS9 MzFecL/mLVaCC199JyWMqnRIj8DSbIAyK4K0ewZslAS+ze71XLOInl48YQMu7EDy+j3w vCjaN9VoNbnP9GXYbEamF/7fgm3JwXNB7fM3lJtjfnZYwacSBGao2ody5EP5AR3vZCSp hpCw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:mime-version:message-id:date:in-reply-to:subject :cc:to:dkim-signature:dkim-signature:from; bh=N9j0B7b156ySsfMwDJsNRd7Lot/GWP1durNGX4F30bE=; b=cW7kyZK+3dSGBSFXMyXg7wJdHsCQYZKb3ygIH5Ww3h9y4OnmOBaChJNGYJaxUM3RnK KTKpiR/BC0hUUz211nsPxuAGkCgMyBHw1H0lLMPPm/mW+z90RQHdGMnxWh4NMvKjRFb8 W67MoUJwfVCr0JFqFjc7x8Q/28HBTHM7/wndEHd6sRDio0LJWZRfiMCRjzLqOGk7B0Zf KgnQz4JrvXuKtO4k0k0OvQmvcie4YeXZnLf6DXR+Zl7aYsQ2w4lL4MrPv/Ttv8qmfWix QHAryHSy+HgZfRIobQB2yGUrb4RxxDGEwQ6WJj3RGadOhiZ5bAkMzbox92BN9uJs1u94 2kLQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linutronix.de header.s=2020 header.b=1ZpNYLsT; dkim=neutral (no key) header.i=@linutronix.de header.b=Cq5gGnOu; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=linutronix.de Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id cc28si14068793edb.100.2021.05.19.01.05.50; Wed, 19 May 2021 01:06:13 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linutronix.de header.s=2020 header.b=1ZpNYLsT; dkim=neutral (no key) header.i=@linutronix.de header.b=Cq5gGnOu; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=linutronix.de Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237012AbhEQUtx (ORCPT + 99 others); Mon, 17 May 2021 16:49:53 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:41818 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229776AbhEQUtx (ORCPT ); Mon, 17 May 2021 16:49:53 -0400 Received: from galois.linutronix.de (Galois.linutronix.de [IPv6:2a0a:51c0:0:12e:550::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 31F46C061573; Mon, 17 May 2021 13:48:36 -0700 (PDT) From: Thomas Gleixner DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020; t=1621284514; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to; bh=N9j0B7b156ySsfMwDJsNRd7Lot/GWP1durNGX4F30bE=; b=1ZpNYLsTCuxDd2w4It1l3dHPjwwAqXAEgSY+DEPNfEPoHcR+Tj8HSX/lM6OuPLk7dDZQVW W1mANCzHmyx/moYnbSqJIsfksQzHubt7JiZM2zKDIcbWHzozO2ubMZ64DlHO+vvi6SI3/u kzgUMF1r1LiMVSf6GAg4a80TbFVuMBAThFvwPAqShd0TKE41wvAv3ecwduS+Wu1jdbDg0I AbAfl/Kx/MAGDQmSaMwvUS+//Ee8ZAgjroZ+SaUJfqQ5dTIfSI6X60+ohc6ZLvj7Z1TR8t CqFlk/za3NV3QACI21PFNIS7CqVMAmhb4Di/XCXQ8oCYUumaEBmQw4P267xG8Q== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020e; t=1621284514; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to; bh=N9j0B7b156ySsfMwDJsNRd7Lot/GWP1durNGX4F30bE=; b=Cq5gGnOum9YEtmqRoidToVZ0cJKjg3O1ZDCn247dy6VTZmmSgwiY8J34jS0r/5mY/3Ru7v WBWf4Bc9muZaDeDA== To: Jesse Brandeburg , Robin Murphy Cc: Ingo Molnar , linux-kernel@vger.kernel.org, intel-wired-lan@lists.osuosl.org, jbrandeb@kernel.org, "frederic\@kernel.org" , "juri.lelli\@redhat.com" , Marcelo Tosatti , abelits@marvell.com, "linux-api\@vger.kernel.org" , "bhelgaas\@google.com" , "linux-pci\@vger.kernel.org" , "rostedt\@goodmis.org" , "peterz\@infradead.org" , "davem\@davemloft.net" , "akpm\@linux-foundation.org" , "sfr\@canb.auug.org.au" , "stephen\@networkplumber.org" , "rppt\@linux.vnet.ibm.com" , "jinyuqi\@huawei.com" , "zhangshaokun\@hisilicon.com" , netdev@vger.kernel.org, chris.friesen@windriver.com, Nitesh Lal , Marc Zyngier Subject: Re: [PATCH tip:irq/core v1] genirq: remove auto-set of the mask when setting the hint In-Reply-To: <20210504092340.00006c61@intel.com> Date: Mon, 17 May 2021 22:48:33 +0200 Message-ID: <87pmxpdr32.ffs@nanos.tec.linutronix.de> MIME-Version: 1.0 Content-Type: text/plain Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, May 04 2021 at 09:23, Jesse Brandeburg wrote: > I'd add in addition that irqbalance daemon *stopped* paying attention > to hints quite a while ago, so I'm not quite sure what purpose they > serve. The hint was added so that userspace has a better understanding where it should place the interrupt. So if irqbalanced ignores it anyway, then what's the point of the hint? IOW, why is it still used drivers? Now there is another aspect to that. What happens if irqbalanced does not run at all and a driver relies on the side effect of the hint setting the initial affinity. Bah... While none of the drivers (except the perf muck) actually prevents userspace from fiddling with the affinity (via IRQF_NOBALANCING) a deeper inspection shows that they actually might rely on the current behaviour if irqbalanced is disabled. Of course every driver has its own convoluted way to do that and all of those functions are well documented. What a mess. If the hint still serves a purpose then we can provide a variant which solely applies the hint and does not fiddle with the actual affinity, but if the hint is useless anyway then we have a way better option to clean that up. Most users are in networking, there are a few in crypto, a couple of leftovers in scsi, virtio and a handfull of oddball drivers. The perf muck wants to be cleaned up anyway as it's just crystal clear abuse. Thanks, tglx