On 10/2/2026 3:37 AM, Lisa Wang wrote:
> +#define tdx_vm_ioctl(vm, cmd, flags, arg) \
> +({ \
> + u64 hw_error; \
> + int ret = __tdx_vm_ioctl(vm, cmd, flags, arg, &hw_error); \
> + \
> + TEST_ASSERT(!ret, \
> + "%s failed, rc: %d errno: %i (%s) hw_error: 0x%llx",\
> + #cmd, ret, errno, strerror(errno), \
> + (unsigned long long)hw_error); \
> +})The cast to (unsigned long long) catches me eye. Can we use "hw_error: 0x%" PRIx64 instead?

