From: Dan Carpenter Date: Thu, 15 Oct 2015 18:25:15 +0000 (+0300) Subject: iommu/vt-d: shift wrapping bug in prq_event_thread() X-Git-Url: http://git.osdn.net/view?a=commitdiff_plain;h=95fb6144bb2222b4c0189e76c1aae006b0a02bff;p=sagit-ice-cold%2Fkernel_xiaomi_msm8998.git iommu/vt-d: shift wrapping bug in prq_event_thread() The "req->addr" variable is a bit field declared as "u64 addr:52;". The "address" variable is a u64. We need to cast "req->addr" to a u64 before the shift or the result is truncated to 52 bits. Fixes: a222a7f0bb6c ('iommu/vt-d: Implement page request handling') Signed-off-by: Dan Carpenter Signed-off-by: David Woodhouse --- diff --git a/drivers/iommu/intel-svm.c b/drivers/iommu/intel-svm.c index b7e923aae4d8..99a78030857b 100644 --- a/drivers/iommu/intel-svm.c +++ b/drivers/iommu/intel-svm.c @@ -490,7 +490,7 @@ static irqreturn_t prq_event_thread(int irq, void *d) req = &iommu->prq[head / sizeof(*req)]; result = QI_RESP_FAILURE; - address = req->addr << PAGE_SHIFT; + address = (u64)req->addr << PAGE_SHIFT; if (!req->pasid_present) { pr_err("%s: Page request without PASID: %08llx %08llx\n", iommu->name, ((unsigned long long *)req)[0],