From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D38ED46EC77; Mon, 7 Sep 2026 13:13:17 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788786800; cv=none; b=fGAuTgMSwQD/63iUTqsHW1Tr16Ay/SVAz32lCiSGXkBljr/bcIjJ/HWF/RJK4RePPGflHLPeCAOlXcDS09TTugw+9xcpd/fgzRBQHsDw1xa3H4gjyuormbbgoFeQTJFMBw7JbvZqTYH6pz6pj/JjidzPCwuhm82B8MWgkBPLrsI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1788786800; c=relaxed/simple; bh=sCyqdA7XYz/cXsFKfyuwykcilF3WMPlvoJkerdQfDrs=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=O2YhCzUgDBNziiD1Gp9nOhXXKLk9p7MwXm+vRE0RyfDCpSxwyAxEP29fSXpHPkdPas+/+yBwvI6zdPj+jHA+13KPy51boRqZa+XSI3JxxMFUET8tBIm+UEDy127V+Esq2OHlNPhn9X4Ihhif+gdgd7g5AOTJnClbp9NUuZbJfGs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=mHWLJ5ee; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="mHWLJ5ee" Received: by smtp.kernel.org (Postfix) with ESMTPSA id D9BB81F00A3A; Mon, 7 Sep 2026 13:13:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1788786795; bh=sCyqdA7XYz/cXsFKfyuwykcilF3WMPlvoJkerdQfDrs=; h=From:To:Cc:Subject:In-Reply-To:References:Date; b=mHWLJ5eejw1DAWE24BKkr7csqEjBElRPKGHJPmtHXeCTNjccZW0Dnu3fn8UL955U3 fQB5ucZqA9OMqXI/Md+NP2OC/gXAYre7PULDNaJ8H2eeyCspoD4Z1829FeRSylPUQX vOLSNE8ULcuYlgf7uMpicB7OVcyrUV5VKD96apvjkgKtwBFP2ywujSY6a+DvyoDRk3 LSEZ3cm8B0atA6WRgQvHnQe1VR3EmE4xai945KgHQNXoFTOxrxmYuc8BfDrHEOCqKy XgGLXMdc9qrCCNrvYrOTCH1/lcg8eyRjljzDeJUDFYH7GAZ87WpC1ONUgaqhpviwJU 9bAWGKoVLzxJg== From: Thomas Gleixner To: Naman Jain , Ingo Molnar , Borislav Petkov , Dave Hansen , x86@kernel.org, "H . Peter Anvin" , Wei Liu , "K . Y . Srinivasan" , Haiyang Zhang , Dexuan Cui Cc: Michael Kelley , Long Li , Nam Cao , Joerg Roedel , Suravee Suthikulpanit , Vasant Hegde , Will Deacon , Robin Murphy , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Manivannan Sadhasivam , Rob Herring , Bjorn Helgaas , Kees Cook , "Rafael J . Wysocki" , Radu Rendec , Sean Christopherson , Kai Huang , Dmitry Ilvokhin , Shradha Gupta , Christophe JAILLET , Thierry Reding , Jiri Wiesner , Daniel J Blueman , linux-hyperv@vger.kernel.org, linux-kernel@vger.kernel.org, iommu@lists.linux.dev, linux-pci@vger.kernel.org Subject: Re: [PATCH v3 3/3] x86/irq, iommu/amd, x86/hyperv, PCI: Drop redundant irq_retrigger inits In-Reply-To: <20260907065129.269262-4-namjain@linux.microsoft.com> References: <20260907065129.269262-1-namjain@linux.microsoft.com> <20260907065129.269262-4-namjain@linux.microsoft.com> Date: Mon, 07 Sep 2026 15:13:12 +0200 Message-ID: <87o6e92oif.ffs@fw13> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain On Mon, Sep 07 2026 at 06:51, Naman Jain wrote: > With fixup_irqs() and the generic resend path (try_retrigger()) now both > falling back to irq_chip_retrigger_hierarchy(), installing it as the > irq_retrigger() callback on the outermost domain chips is no longer > required. The hierarchy walk reaches the x86 vector domain's > apic_retrigger_irq() for all of these stacked domain chips. > > Remove the now redundant irq_retrigger() initializations from the outermost > domain interrupt chips, including the Hyper-V root partition PCI/MSI chip. > msi_set_affinity() is the only site that invoked the outermost chip's > irq_retrigger() directly. Convert it to irq_chip_retrigger_hierarchy() so > the x86_init_dev_msi_info() initialization can be dropped as well. > > No functional change intended. > > Suggested-by: Thomas Gleixner > Reviewed-by: Shradha Gupta > Signed-off-by: Naman Jain Reviewed-by: Thomas Gleixner