]> git.baikalelectronics.ru Git - kernel.git/commit
iommu/vt-d: Fix infinite loop in free_all_cpu_cached_iovas
authorAaron Campbell <aaron@monkey.org>
Sun, 3 Jul 2016 00:23:24 +0000 (21:23 -0300)
committerJoerg Roedel <jroedel@suse.de>
Mon, 4 Jul 2016 11:34:52 +0000 (13:34 +0200)
commit80f142eedbf6860d70ac0cae456f44398d6c53a3
treecaa80d64be89509628aea83babbc05dc3fcb2539
parent94d7a1f5db336400539d53c4e1493f05ad047d95
iommu/vt-d: Fix infinite loop in free_all_cpu_cached_iovas

Per VT-d spec Section 10.4.2 ("Capability Register"), the maximum
number of possible domains is 64K; indeed this is the maximum value
that the cap_ndoms() macro will expand to.  Since the value 65536
will not fix in a u16, the 'did' variable must be promoted to an
int, otherwise the test for < 65536 will always be true and the
loop will never end.

The symptom, in my case, was a hung machine during suspend.

Fixes: 70b1c615d900 ("iommu/vt-d: Fix overflow of iommu->domains array")
Signed-off-by: Aaron Campbell <aaron@monkey.org>
Signed-off-by: Joerg Roedel <jroedel@suse.de>
drivers/iommu/intel-iommu.c