linux-kernel - Re: [PATCH V2 0/8] iommu/amd: enable ACPI hardware ID device support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Sat, 5 Mar 2016 16:00:25 +0700
From:	Suravee Suthikulpanit <Suravee.Suthikulpanit@....com>
To:	"Wan, Vincent" <Vincent.Wan@....com>,
	Joerg Roedel <joro@...tes.org>
CC:	"iommu@...ts.linux-foundation.org" <iommu@...ts.linux-foundation.org>,
	Borislav Petkov <bp@...e.de>, "Huang, Ray" <Ray.Huang@....com>,
	"Xue, Ken" <Ken.Xue@....com>, "vw@...mu.org" <vw@...mu.org>,
	"mcuos.com@...il.com" <mcuos.com@...il.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"Long, Gang" <Gang.Long@....com>
Subject: Re: [PATCH V2 0/8] iommu/amd: enable ACPI hardware ID device support

Hi Joerg,

On 03/04/2016 03:49 PM, Wan, Vincent wrote:
>>> Joerg,
>>> Can you provide me which branch you are using to test those patches.
>>>
>>> We are validating the patches in v4.5.0-rc6, and no issues like your
>> description.
>>
>> I tested them on v4.5-rc5.
>>
> Thanks!
> We will test the patches basing on this v4.5-rc5 version again.
>
> Vincent.
>>
>> 	Joerg
>

I found the issue now. I'll work with Vincent to send out V3.

The problem is because in the information in the IOMMU Attributes field 
of the IVHD header type 40h (offset byte 20 = struct 
ivhd_header.efr_attr) is incorrect. It was correct in the IVHD type 
10h.... Sigh :(

However, the same information in the MMIO offset 4000h Counter 
configuration Register is correct. So, I'll try to have the patch get 
information from here instead.

Earlier, we had another issue with IOMMU PERF intialization, and it was 
disabled. So, it hides this bug when we were testing the patch earlier.
Now that we had a workaround for that bug n v4.5-rc5, it bombs in the 
init_iommu_perf_ctr().

So, if you want to test out the rest of this patch series, just simply 
comment out this function, and it should boot.

Regards,
Suravee