On Fri, Jan 28, 2022 at 11:10:02AM +0800, Lu Baolu wrote: > From: Guoqing Jiang <[email protected]> > > After commit e3beca48a45b ("irqdomain/treewide: Keep firmware node > unconditionally allocated"). For tear down scenario, fn is only freed > after fail to allocate ir_domain, though it also should be freed in case > dmar_enable_qi returns error. > > Besides free fn, irq_domain and ir_msi_domain need to be removed as well > if intel_setup_irq_remapping fails to enable queued invalidation. > > Improve the rewinding path by add out_free_ir_domain and out_free_fwnode > lables per Baolu's suggestion. > > Fixes: e3beca48a45b ("irqdomain/treewide: Keep firmware node unconditionally > allocated") > Suggested-by: Lu Baolu <[email protected]> > Signed-off-by: Guoqing Jiang <[email protected]> > Link: https://lore.kernel.org/r/[email protected] > Signed-off-by: Lu Baolu <[email protected]> > --- > drivers/iommu/intel/irq_remapping.c | 13 ++++++++++--- > 1 file changed, 10 insertions(+), 3 deletions(-)
Applied this one, thanks. _______________________________________________ iommu mailing list [email protected] https://lists.linuxfoundation.org/mailman/listinfo/iommu
