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?



Reply via email to