lkml.org 
[lkml]   [2021]   [Jun]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v12 3/5] iommu/vt-d: Add support for IOMMU default DMA mode build options
From
Date
On 2021-06-14 15:19, John Garry wrote:
> On 14/06/2021 15:11, Robin Murphy wrote:
>> On 2021-06-14 08:53, John Garry wrote:
>>> On 12/06/2021 03:22, Lu Baolu wrote:
>>>> On 2021/6/11 20:20, John Garry wrote:
>>>>> @@ -453,8 +452,7 @@ static int __init intel_iommu_setup(char *str)
>>>>>                pr_warn("intel_iommu=forcedac deprecated; use
>>>>> iommu.forcedac instead\n");
>>>>>                iommu_dma_forcedac = true;
>>>>>            } else if (!strncmp(str, "strict", 6)) {
>>>>> -            pr_info("Disable batched IOTLB flush\n");
>>>>> -            intel_iommu_strict = 1;
>>>>> +            iommu_set_dma_strict(true);
>>>> I would like to deprecate this command line and ask users to use
>>>> iommu.strict instead.
>>> ok, but then I should probably make the AMD driver also officially
>>> support this.
>> Oops, I should have documented that "iommu.strict" already applies to
>> x86 as well. The interaction with the driver-specific parameters is a
>> bit weird and unintuitive, but it was done knowingly. Let me quickly
>> spin a kernel-parameters.txt fix for that...
>
> So I already had a pending patch here for the same to be included in
> this series:
> https://github.com/hisilicon/kernel-dev/commit/2375a2d888d78de9eb7d91d6f2c5891395300a96
>
>
> If you want to do it, then ok. I might have to reorder the series though...

Yeah, sorry for the complication. Your subsequent deprecation of the x86
parameters doesn't need to conflict, but we should probably also update
that the default value now depends on the kernel config, which will :(

Thanks,
Robin.

\
 
 \ /
  Last update: 2021-06-14 17:06    [W:0.244 / U:0.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site