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 62ED5C54EBD for ; Thu, 12 Jan 2023 13:36:00 +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:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=GmcGyvU/jeDVyppJleFrD32ALQ4JK5eT7yLpmTZ/jWw=; b=DtxEQiWi9W4eOa suIvlvLLM3luy6rmFL4e49pzDqPoVDW2GaH+izSInrnE3glWRER5HckWidepmGiUU4/TExKTE/O2n rb+hYHk652xDgTfIFImYoEnz8UgKNaogJEDn3kxuKiWgQIgDmu0J0SySlyllYTxO+FUK02CRsIQoS 3cp1nxJnNZzts9eAyIZRyigv2CEkgRlAQ3/di5pNtUzlwRaGt9v2gZMOimfqd/G7q2/6L3S0B2eOu NkweJxI+5F6EW6xGDExzV+m2NlhGScZ9Um9YAAEKghuSv+44BGP1h0VGBxkv9FUh05WdBlUPlEgpZ 3rU/GOnDay+CSqqYNSzg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pFxjM-00FAaJ-2t; Thu, 12 Jan 2023 13:34:56 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pFxjB-00FAZc-AE; Thu, 12 Jan 2023 13:34:46 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id D657C61F28; Thu, 12 Jan 2023 13:34:44 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 366CBC433D2; Thu, 12 Jan 2023 13:34:44 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1673530484; bh=j0YIDz5bHF2dFE2r4pGCMvZWfbTDEfD2sZPAljVVdPM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=oVh35eywb5oK7iIixDKPwxnygS6i9OfZxaRlgXZVZSaUI2xBUtrtHDIUB350VJI1c JK8EQ9nzK1wAVVCzb5Au21M7c+h9dTyP23XudUduDXGrppEhSIDJp/WGoB5uLyFd9W 57K2iAQni9rcMi60meEtbkPnvxgIczX1aXRmp7I7L2GZAbLGkkAETYSDwJLFDmBIjC nfTIUiC9euE0vKEvft7Mu4mgCqgqDxlTvI6Pa7kXTCZ0RGOgRlpOVXJprKzRaChlzJ DgfIdidyjphtq0P40ueCY+WmQXLY4xRo9s0zzepVrezR5J69gSc8mIIWqX0BAx3aHE 5rfJGq222ZZGQ== Received: from johan by xi.lan with local (Exim 4.94.2) (envelope-from ) id 1pFxjG-0001vA-Mi; Thu, 12 Jan 2023 14:34:51 +0100 Date: Thu, 12 Jan 2023 14:34:50 +0100 From: Johan Hovold To: Thomas Gleixner Cc: Johan Hovold , Marc Zyngier , Hector Martin , Sven Peter , Alyssa Rosenzweig , Andrew Lunn , Gregory Clement , Sebastian Hesselbarth , Florian Fainelli , Broadcom internal kernel review list , Ray Jui , Scott Branden , asahi@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-rpi-kernel@lists.infradead.org Subject: Re: [PATCH 2/2] genirq/irqdomain/msi: Add irq_domain_alloc_irqs_affinity() wrapper Message-ID: References: <20221213140844.15470-1-johan+linaro@kernel.org> <20221213140844.15470-3-johan+linaro@kernel.org> <87358ggayt.ffs@tglx> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230112_053445_409972_76E51F4B X-CRM114-Status: GOOD ( 20.98 ) 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 Thu, Jan 12, 2023 at 02:24:58PM +0100, Johan Hovold wrote: > On Wed, Jan 11, 2023 at 07:52:42PM +0100, Thomas Gleixner wrote: > > On Tue, Dec 13 2022 at 15:08, Johan Hovold wrote: > > > The full __irq_domain_alloc_irqs() interface was initially only intended > > > for some legacy (x86) use cases while most callers should use the > > > irq_domain_alloc_irqs() wrapper. > > > > > > Add a matching irq_domain_alloc_irqs_affinity() wrapper that can be used > > > when an affinity hint needs to be provided and use it for MSI > > > allocations. > > > > I definitely like the irqchip cleanup, but this one is core code and > > having the extra wrapper is not really buying us much. > > Fair enough. If you don't expect there to be any further users then > perhaps it's not worth it even if it arguably makes the call site a bit > more readable (e.g. by dropping the base and realloc arguments). And having this wrapper would also limit the use of the full (internal) helper to places that actually need the realloc parameter (e.g. making those stand out more). Johan _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel