fabtests/hmem: change ZE memset to use uint8

Match the behavior of memset() where the value passed in
is an int, but it is interpreted as a char.
While ZE can technically handle this scenario, others may not
so we need to standardize across ifaces

Signed-off-by: Alexia Ingerson <[email protected]>

fabtests/common/hmem.c

@@ -185,6 +185,10 @@ int ft_hmem_free_host(enum fi_hmem_iface iface, void *buf)
 	return hmem_ops[iface].free_host(buf);
 }
 
+/*
+ * Matches the behavior of memset where value is an int but
+ * used as a unsigned char
+ */
 int ft_hmem_memset(enum fi_hmem_iface iface, uint64_t device, void *buf,
 		   int value, size_t size)
 {

fabtests/common/hmem_ze.c

@@ -382,14 +382,16 @@ int ft_ze_free(void *buf)
 
 int ft_ze_memset(uint64_t device, void *buf, int value, size_t size)
 {
+	unsigned char set_value = (unsigned char) value;
 	ze_result_t ze_ret;
 
 	ze_ret = (*libze_ops.zeCommandListReset)(cmd_list);
 	if (ze_ret)
 		return -FI_EINVAL;
 
 	ze_ret = (*libze_ops.zeCommandListAppendMemoryFill)(
-					cmd_list, buf, &value, sizeof(value),
+					cmd_list, buf, &set_value,
+					sizeof(set_value),
 					size, NULL, 0, NULL);
 	if (ze_ret)
 		return -FI_EINVAL;