On Sun, Jan 8, 2017 at 2:45 AM, Rafael J. Wysocki <rafael@xxxxxxxxxx> wrote:
> On Sun, Jan 8, 2017 at 2:01 AM, Borislav Petkov <bp@xxxxxxxxx> wrote:
>> On Sun, Jan 08, 2017 at 01:52:50AM +0100, Rafael J. Wysocki wrote:
>>> So we get the table, but apparently we crash when we attempt to put it.
>>
>> Right, except on 4.10-rc2 we don't crash but we freeze early. These are
>> the last lines:
>>
>> ...
>> [ 0.004778] mce: CPU supports 7 MCE banks
>> [ 0.004861] LVT offset 1 assigned for vector 0xf9
>> [ 0.004945] Last level iTLB entries: 4KB 512, 2MB 1024, 4MB 512
>> [ 0.005025] Last level dTLB entries: 4KB 1024, 2MB 1024, 4MB 512, 1GB 0
>> [ 0.005165] Freeing SMP alternatives memory: 24K
>> [ 0.211154] ftrace: allocating 25022 entries in 98 pages
>> [ 0.219614] smpboot: Max logical packages: 2
>> <EOF>
>>
>>> Let's try to check the obvious just to rule it out (see attached), but
>>> honestly I'm not sure what's going on in there.
>>
>> No change, same freeze.
>
> I was afraid that that would be the case.
>
> Can you try to comment out the acpi_put_table() in
> early_amd_iommu_init() and see if that makes any difference?
Well, there is a bug in early_amd_iommu_init() that may matter in
theory if the table checksum is incorrect.
Please see if the attached makes any difference.
Thanks,
Rafael
---
drivers/iommu/amd_iommu_init.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Index: linux-pm/drivers/iommu/amd_iommu_init.c
===================================================================
--- linux-pm.orig/drivers/iommu/amd_iommu_init.c
+++ linux-pm/drivers/iommu/amd_iommu_init.c
@@ -2230,7 +2230,7 @@ static int __init early_amd_iommu_init(v
*/
ret = check_ivrs_checksum(ivrs_base);
if (ret)
- return ret;
+ goto out;
amd_iommu_target_ivhd_type = get_highest_supported_ivhd_type(ivrs_base);
DUMP_printk("Using IVHD type %#x\n", amd_iommu_target_ivhd_type);
