lkml.org 
[lkml]   [2017]   [Jan]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v6 01/18] iommu/dma: Allow MSI-only cookies
From
Date
On 06/01/17 11:46, Auger Eric wrote:
>
>
> On 06/01/2017 11:59, Joerg Roedel wrote:
>> 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.
>
> Sure
>
> Thank you for the review.

FWIW I already had a cleaned up version of this patch, I just hadn't
mentioned it. I've pushed out an update with that change added too[1].

Robin.

[1]:http://linux-arm.org/git?p=linux-rm.git;a=shortlog;h=refs/heads/iommu/misc

>
> Best regards
>
> Eric
>>
>>
>>
>> Joerg
>>

\
 
 \ /
  Last update: 2017-01-06 13:14    [W:0.076 / U:0.672 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site