Commit f69503f1 authored by Pawel Wodkowski's avatar Pawel Wodkowski Committed by Jim Harris
Browse files

rte_vhost: add offset while mmaping log base address. 



QEMU always set offset to 0 but for sanity we should take the offset
into account.

Change-Id: I36213cd8fbeb85862b6de59c60bd6bcee7f9d1b2
Signed-off-by: default avatarPawel Wodkowski <pawelx.wodkowski@intel.com>
Reviewed-on: https://review.gerrithub.io/395740


Reviewed-by: default avatarDaniel Verkamp <daniel.verkamp@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarDariusz Stojaczyk <dariuszx.stojaczyk@intel.com>
Tested-by: default avatarSPDK Automated Test System <sys_sgsw@intel.com>
parent c1cfc10a

lib/vhost/rte_vhost/vhost_user.c

+1 −1
Original line number Diff line number Diff line
@@ -885,7 +885,7 @@ vhost_user_set_log_base(struct virtio_net *dev, struct VhostUserMsg *msg) 
	 * mmap from 0 to workaround a hugepage mmap bug: mmap will

	 * fail when offset is not page size aligned.

	 */

	addr = mmap(0, size, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0);

	addr = mmap(0, size + off, PROT_READ | PROT_WRITE, MAP_SHARED, fd, 0);

	close(fd);

	if (addr == MAP_FAILED) {

		RTE_LOG(ERR, VHOST_CONFIG, "mmap log base failed!\n");