From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id D9C38C433F5 for ; Mon, 9 May 2022 13:14:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:References :In-Reply-To:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=G9zYVs+zCnU/TsZavZt81olHmZBsHB0pSW+qlKCR9z8=; b=kAsRUOMv/VpbVL 0Fb8KKZiy590uRGtWCpTxuEOmRL+Jb6z8bzQRHXcK4x0COOc1gCGKOlf+XhD1mUSD/12874n93Wfz dnpOUBck7v6bpBI7VnvN9sKn3upXrQyyELaxa7qyCqub4fOLA9ImYFSM2qBGTlxyfIJce0Z4qMjuD b2X81FeCRuw4M99lddc48s58xU4NNwEXnedsrDmnR5hRou0f7eyrsc3VN/pTmEPTwGSEOt+6eMpkO Q6HNDun0umPc9NNwEqihTRIDk3pX1Cv4EOGprAkAUt8nZQ/an66h2gpa4rG4gQiKp6gOHSCGyeB1h 4v6a0aMzN0vr9PjoBWuQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1no3CG-00EW8t-Lr; Mon, 09 May 2022 13:13:08 +0000 Received: from galois.linutronix.de ([193.142.43.55]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1no3CC-00EW7P-UO for linux-arm-kernel@lists.infradead.org; Mon, 09 May 2022 13:13:06 +0000 From: Thomas Gleixner DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020; t=1652101973; 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:references:references; bh=bBspkfCOiP10g+5drPmJg2fsEO9rzBTC2dvAOh7cjPw=; b=kbHC1p6RRwZ/BL4jPEXFzwHYmjXokSL+uS3cwwordw1EJV6wNjIGQPREMrngqZUm8qHdwV Yprr9120crP5C2G9QFQtp4xTRYus4VgivOnIfP0v1LdQe7hcYRwElLMb0JYFa2YjZcweGt Afw5OzTdqT2yxN9FHXGeK2vVPtz6hG0CzOZkw6XYnSNcHUVM/fPKFeANoOtDgILzwj/Gy/ Fjx3ScPzRkCJquINskW2r2WqrYQO5Yg6ORmSwupnfjmr8ImdWUFR4Z6KUoYi7OVsoT/tUH U9irj/6W/4qicC44sXMkY/LG0wMR5iFgI5alXKj2aFseBiunMELtbF5bRFZK6g== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020e; t=1652101973; 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:references:references; bh=bBspkfCOiP10g+5drPmJg2fsEO9rzBTC2dvAOh7cjPw=; b=Rq/x/sHbZy5rjclUvavUh0K3iWFJI4577Zfz96J+8/mfp7GtGy2EDV7p9Y8N0aZztlpsZR IEiFHShk1a3pk4CQ== To: Marc Zyngier , Mark Rutland Cc: Lukas Wunner , Linus Walleij , Bartosz Golaszewski , linux-gpio@vger.kernel.org, Octavian Purdila , linux-kernel@vger.kernel.org, aou@eecs.berkeley.edu, catalin.marinas@arm.com, deanbo422@gmail.com, green.hu@gmail.com, guoren@kernel.org, jonas@southpole.se, kernelfans@gmail.com, linux-arm-kernel@lists.infradead.org, linux@armlinux.org.uk, nickhu@andestech.com, palmer@dabbelt.com, paul.walmsley@sifive.com, shorne@gmail.com, stefan.kristiansson@saunalahti.fi, tsbogend@alpha.franken.de, vgupta@kernel.org, vladimir.murzin@arm.com, will@kernel.org Subject: Re: [PATCH v2 17/17] irq: remove handle_domain_{irq,nmi}() In-Reply-To: <87h75z6pix.wl-maz@kernel.org> References: <20211026092504.27071-1-mark.rutland@arm.com> <20211026092504.27071-18-mark.rutland@arm.com> <20220506203242.GA1855@wunner.de> <87h75z6pix.wl-maz@kernel.org> Date: Mon, 09 May 2022 15:12:53 +0200 Message-ID: <87ilqergru.ffs@tglx> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220509_061305_169172_2789FD78 X-CRM114-Status: GOOD ( 20.94 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, May 09 2022 at 10:09, Marc Zyngier wrote: > On Mon, 09 May 2022 09:54:21 +0100, > Mark Rutland wrote: >> >> On Fri, May 06, 2022 at 10:32:42PM +0200, Lukas Wunner wrote: >> > On Tue, Oct 26, 2021 at 10:25:04AM +0100, Mark Rutland wrote: >> > > int generic_handle_domain_irq(struct irq_domain *domain, unsigned int hwirq) >> > > { >> > > + WARN_ON_ONCE(!in_irq()); >> > > return handle_irq_desc(irq_resolve_mapping(domain, hwirq)); >> > > } >> > > EXPORT_SYMBOL_GPL(generic_handle_domain_irq); >> > >> > Why isn't the WARN_ON_ONCE() conditional on handle_enforce_irqctx()? >> > (See handle_irq_desc() and c16816acd086.) >> >> I did this for consistency with the in_nmi() check in >> generic_handle_domain_nmi(); I was unaware of commit c16816acd086 and >> IRQD_HANDLE_ENFORCE_IRQCTX. >> >> I'll have ot leave it to Marc and Thomas as to what we should do there. > > My preference would be to not introduce things that result in > different behaviours for drivers, specially for things that are > evidently cross-architecture such as USB drivers (which seems to be > the case here). > > I'd rather do something that allows these to be handled in the right > context such as a self-IPI. This would certainly work for the GIC. No > idea whether this is valid for x86, which is the other user. We have to differentiate between interrupts which require hard interrupt context due to constraints in the hardware, e.g. the horrors of affinity changes on x86, and interrupts like the dln one which are synthetic. The latter never go through the regular interrupt entry ASM muck. Thanks, tglx _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel