commit 77b7055a1678b5c1d0d0e44d33cda12d86ca193a
author Satyajit Sahu <satyajit.sahu@amd.com> Thu May 03 16:35:24 2018 +0530
committer chrome-bot <chrome-bot@chromium.org> Thu May 10 15:09:50 2018 -0700
tree d282025d8d2e9fd7c169ddc190be6053855f7772
parent 62a9c4ee5a1daa0bd23d1104b64c8625f2eecf93

minigbm: Use the stride value returned by mapImage

mapImage can return a different stride value. This value must be
used for the mapped address.

BUG=b:38152101 b:79346377
TEST=graphics_Sanity autotest passed

Change-Id: Ie0eb716291366ae6a047d704f66d24ec41738713
Signed-off-by: Satyajit Sahu <satyajit.sahu@amd.com>
Reviewed-on: https://chromium-review.googlesource.com/1041369
Commit-Ready: Bernie Thompson <bhthompson@chromium.org>
Tested-by: Drew Davenport <ddavenport@chromium.org>
Reviewed-by: Gurchetan Singh <gurchetansingh@chromium.org>

dri.c

diff --git a/dri.c b/dri.c
index 8ca00c6..ae491bb 100644
--- a/dri.c
+++ b/dri.c
@@ -260,11 +260,11 @@
 void *dri_bo_map(struct bo *bo, struct vma *vma, size_t plane, uint32_t map_flags)
 {
 	struct dri_driver *dri = bo->drv->priv;
-	int stride;
 
 	/* GBM flags and DRI flags are the same. */
-	vma->addr = dri->image_extension->mapImage(dri->context, bo->priv, 0, 0, bo->width,
-						   bo->height, map_flags, &stride, &vma->priv);
+	vma->addr =
+	    dri->image_extension->mapImage(dri->context, bo->priv, 0, 0, bo->width, bo->height,
+					   map_flags, (int *)&vma->map_strides[plane], &vma->priv);
 	if (!vma->addr)
 		return MAP_FAILED;
 

drv.c

diff --git a/drv.c b/drv.c
index f6420e5..bc1f782 100644
--- a/drv.c
+++ b/drv.c
@@ -440,6 +440,7 @@
 	}
 
 	mapping.vma = calloc(1, sizeof(*mapping.vma));
+	memcpy(mapping.vma->map_strides, bo->strides, sizeof(mapping.vma->map_strides));
 	addr = bo->drv->backend->bo_map(bo, mapping.vma, plane, map_flags);
 	if (addr == MAP_FAILED) {
 		*map_data = NULL;

drv.h

diff --git a/drv.h b/drv.h
index 67d763e..9ca77d3 100644
--- a/drv.h
+++ b/drv.h
@@ -84,6 +84,7 @@
 	uint32_t handle;
 	uint32_t map_flags;
 	int32_t refcount;
+	uint32_t map_strides[DRV_MAX_PLANES];
 	void *priv;
 };
 

gbm.c

diff --git a/gbm.c b/gbm.c
index a720461..c12c269 100644
--- a/gbm.c
+++ b/gbm.c
@@ -233,7 +233,6 @@
 	if (!bo || width == 0 || height == 0 || !stride || !map_data)
 		return NULL;
 
-	*stride = gbm_bo_get_plane_stride(bo, plane);
 	map_flags = (transfer_flags & GBM_BO_TRANSFER_READ) ? BO_MAP_READ : BO_MAP_NONE;
 	map_flags |= (transfer_flags & GBM_BO_TRANSFER_WRITE) ? BO_MAP_WRITE : BO_MAP_NONE;
 
@@ -241,7 +240,9 @@
 	if (addr == MAP_FAILED)
 		return MAP_FAILED;
 
-	offset = gbm_bo_get_plane_stride(bo, plane) * rect.y;
+	*stride = ((struct mapping *)*map_data)->vma->map_strides[plane];
+
+	offset = *stride * rect.y;
 	offset += drv_stride_from_format(bo->gbm_format, rect.x, plane);
 	return (void *)((uint8_t *)addr + offset);
 }