From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rahul Lakkireddy Subject: [PATCH] vfio: Fix overflow while assigning vfio BAR region offset and size Date: Tue, 16 Jun 2015 19:46:09 +0530 Message-ID: <61d1c9174f7f9159d4005dd2cea16c7719cec964.1434462470.git.rahul.lakkireddy@chelsio.com> Cc: Felix Marti , Kumar Sanghvi , Nirranjan Kirubaharan To: dev@dpdk.org Return-path: Received: from stargate3.asicdesigners.com (stargate.chelsio.com [67.207.112.58]) by dpdk.org (Postfix) with ESMTP id 997425A9B for ; Tue, 16 Jun 2015 16:17:12 +0200 (CEST) List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" After the commit-id 90a1633b2 (eal/linux: allow to map BARs with MSI-X tables), VFIO stopped working. On further debug, found that although BAR region offset and size from vfio are read as u64, they are truncated when assigned to uint32_t variables resulting in wrong offset being passed for mmap. The fix is to use uint64_t for offset and size. Signed-off-by: Rahul Lakkireddy Signed-off-by: Kumar Sanghvi --- lib/librte_eal/linuxapp/eal/eal_pci_vfio.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/librte_eal/linuxapp/eal/eal_pci_vfio.c b/lib/librte_eal/linuxapp/eal/eal_pci_vfio.c index 426953a..d0385ff 100644 --- a/lib/librte_eal/linuxapp/eal/eal_pci_vfio.c +++ b/lib/librte_eal/linuxapp/eal/eal_pci_vfio.c @@ -728,7 +728,7 @@ pci_vfio_map_resource(struct rte_pci_device *dev) struct vfio_region_info reg = { .argsz = sizeof(reg) }; void *bar_addr; struct memreg { - uint32_t offset, size; + uint64_t offset, size; } memreg[2] = {}; reg.index = i; @@ -771,7 +771,7 @@ pci_vfio_map_resource(struct rte_pci_device *dev) RTE_LOG(DEBUG, EAL, "Trying to map BAR %d that contains the MSI-X " "table. Trying offsets: " - "%04x:%04x, %04x:%04x\n", i, + "%04lx:%04lx, %04lx:%04lx\n", i, memreg[0].offset, memreg[0].size, memreg[1].offset, memreg[1].size); } -- 2.4.1