Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 9895ece1 authored by Jordan Crouse's avatar Jordan Crouse
Browse files

msm: kgsl: Don't print memory address debugging on permission fault 



There isn't any point in printing the memory address debugging for a
permission MMU fault and in fact it might be misleading.

Change-Id: Ic0dedbad3a6320bea60886782eb105f3697a388f
Signed-off-by: default avatarJordan Crouse <jcrouse@codeaurora.org>
parent 374a4729

drivers/gpu/msm/kgsl_iommu.c

+17 −12
Original line number Diff line number Diff line
@@ -342,9 +342,12 @@ static int kgsl_iommu_fault_handler(struct iommu_domain *domain, 
			ctx->ctx_id, &ptbase,

			write ? "write" : "read", fault_type);



		/* Don't print the debug if this is a permissions fault */

		if (!(flags & IOMMU_FAULT_PERMISSION)) {

			_check_if_freed(ctx, addr, ptname);



		KGSL_LOG_DUMP(ctx->kgsldev, "---- nearby memory ----\n");

			KGSL_LOG_DUMP(ctx->kgsldev,

				"---- nearby memory ----\n");



			_find_mem_entries(mmu, addr, ptbase, &prev, &next);
@@ -353,7 +356,8 @@ static int kgsl_iommu_fault_handler(struct iommu_domain *domain, 
			else

				KGSL_LOG_DUMP(ctx->kgsldev, "*EMPTY*\n");



		KGSL_LOG_DUMP(ctx->kgsldev, " <- fault @ %8.8lX\n", addr);

			KGSL_LOG_DUMP(ctx->kgsldev, " <- fault @ %8.8lX\n",

				addr);



			if (next.gpuaddr != (uint64_t) -1)

				_print_entry(ctx->kgsldev, &next);
@@ -361,6 +365,7 @@ static int kgsl_iommu_fault_handler(struct iommu_domain *domain, 
				KGSL_LOG_DUMP(ctx->kgsldev, "*EMPTY*\n");



		}

	}



	trace_kgsl_mmu_pagefault(ctx->kgsldev, addr,

			kgsl_mmu_get_ptname_from_ptbase(mmu, ptbase),