lkml.org 
[lkml]   [2016]   [Dec]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] iommu/vt-d: Flush old iotlb for kdump when the device gets context mapped
From
Date
On 12/01/2016 at 06:33 PM, Joerg Roedel wrote:
> On Thu, Dec 01, 2016 at 10:15:45AM +0800, Xunlei Pang wrote:
>> index 3965e73..624eac9 100644
>> --- a/drivers/iommu/intel-iommu.c
>> +++ b/drivers/iommu/intel-iommu.c
>> @@ -2024,6 +2024,25 @@ static int domain_context_mapping_one(struct dmar_domain *domain,
>> if (context_present(context))
>> goto out_unlock;
>>
>> + /*
>> + * For kdump cases, old valid entries may be cached due to the
>> + * in-flight DMA and copied pgtable, but there is no unmapping
>> + * behaviour for them, thus we need an explicit cache flush for
>> + * the newly-mapped device. For kdump, at this point, the device
>> + * is supposed to finish reset at its driver probe stage, so no
>> + * in-flight DMA will exist, and we don't need to worry anymore
>> + * hereafter.
>> + */
>> + if (context_copied(context)) {
>> + u16 did_old = context_domain_id(context);
>> +
>> + if (did_old >= 0 && did_old < cap_ndoms(iommu->cap))
>> + iommu->flush.flush_context(iommu, did_old,
>> + (((u16)bus) << 8) | devfn,
>> + DMA_CCMD_MASK_NOBIT,
>> + DMA_CCMD_DEVICE_INVL);
>> + }
>> +
>> pgd = domain->pgd;
> Yes, this looks better. Have you tested it the same way as the old
> patch?

Yes, I have tested and it works, will send v3 later.

Regards,
Xunlei

\
 
 \ /
  Last update: 2016-12-01 12:54    [W:2.308 / U:0.256 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site