>>> On 16.09.15 at 15:23, <quan...@intel.com> wrote:
> Signed-off-by: Quan Xu <quan...@intel.com>

The title isn't really meaningful, and there's no description.

> @@ -1084,8 +1086,8 @@ static int __init iommu_set_interrupt(struct 
> acpi_drhd_unit *drhd)
>      }
>  
>      desc = irq_to_desc(irq);
> -    desc->handler = &dma_msi_type;
> -    ret = request_irq(irq, 0, iommu_page_fault, "dmar", iommu);
> +    desc->handler = irq_ctrl;

I suppose (also taking the title into account) any the planned second
user will also use dma_msi_type, so there's no real point in making
this a function parameter.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to