Re: [PATCH v2] x86: irq_remapping: Fix the regression of hpet irq remapping

[Joerg Roedel <joro-zLv9SwRftAIdnm+yROfE0A@public.gmane.org>]

On Wed, Sep 17, 2014 at 05:32:19PM +0800, Yijing Wang wrote:
> Commit 71054d8841b4 ("x86, hpet: Introduce x86_msi_ops.setup_hpet_msi")
> introduced x86_msi_ops.setup_hpet_msi to setup hpet MSI irq 
> when irq remapping enabled. This caused a regression of 
> hpet MSI irq remapping.
> 
> Original code flow before commit 71054d8841b4:
> hpet_setup_msi_irq()
> 	arch_setup_hpet_msi()
> 		setup_hpet_msi_remapped()
> 			remap_ops->setup_hpet_msi()
> 				alloc_irte()
> 		msi_compose_msg()
> 		hpet_msi_write()
> 		...
> 
> Current code flow after commit 71054d8841b4:
> hpet_setup_msi_irq()
> 	x86_msi.setup_hpet_msi()
> 		setup_hpet_msi_remapped()
> 			intel_setup_hpet_msi()
> 				alloc_irte()
> 
> Currently, we only call alloc_irte() for hpet MSI, but
> do not composed and wrote its msg...
> 
> Signed-off-by: Yijing Wang <wangyijing-hv44wF8Li93QT0dZR+AlfA@public.gmane.org>
> ---
> v1->v2: rename remap_ops->setup_hpet_msi to remap_ops->alloc_hpet_msi.
> ---
>  drivers/iommu/amd_iommu.c           |    4 ++--
>  drivers/iommu/intel_irq_remapping.c |    4 ++--
>  drivers/iommu/irq_remapping.c       |   11 +++++++++--
>  drivers/iommu/irq_remapping.h       |    2 +-
>  4 files changed, 14 insertions(+), 7 deletions(-)

Applied, thanks.