Revert "hsakmt: Only set exec flag when requested"

This reverts commit 7514355.

Reason for revert: 
This is currently breaking some tools. Will put it back as soon as tools update their code.

Change-Id: I05c82d443f3a274a618d05e6dc5a87943f5dc7a4

Author: dayatsin-amd

libhsakmt/src/fmm.c

@@ -1401,8 +1401,11 @@ static uint32_t fmm_translate_hsa_to_ioc_flags(HsaMemFlags flags)
 			      KFD_IOC_ALLOC_MEM_FLAGS_UNCACHED);
 	if (!flags.ui32.ReadOnly)
 		ioc_flags |= KFD_IOC_ALLOC_MEM_FLAGS_WRITABLE;
-	if (flags.ui32.ExecuteAccess)
-		ioc_flags |= KFD_IOC_ALLOC_MEM_FLAGS_EXECUTABLE;
+	/* TODO: Since, ROCr interfaces doesn't allow caller to set page
+	 * permissions, mark all user allocations with exec permission.
+	 * Check for flags.ui32.ExecuteAccess once ROCr is ready.
+	 */
+	ioc_flags |= KFD_IOC_ALLOC_MEM_FLAGS_EXECUTABLE;
 	return ioc_flags;
 }