On Thu, Jan 05, 2017 at 07:04:29PM +0000, Eric Auger wrote:
>  struct iommu_dma_cookie {
> -     struct iova_domain      iovad;
> -     struct list_head        msi_page_list;
> -     spinlock_t              msi_lock;
> +     union {
> +             struct iova_domain      iovad;
> +             dma_addr_t              msi_iova;
> +     };
> +     struct list_head                msi_page_list;
> +     spinlock_t                      msi_lock;
> +     enum iommu_dma_cookie_type      type;

Please move the type to the beginning of the struct and add a comment
how the type relates to the union.



        Joerg

Reply via email to