Hi Andrew,

On 03/01/2023 20:09, Andrew Cooper wrote:
This is technically an ABI change, but Xen doesn't operate in any environment
where "unsigned int" is differnet to uint32_t, so switch to the explicit form.

typo: s/differnet/different/

This avoids the need to derive (identical) compat logic for handling the
subop.

Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com>

Acked-by: Julien Grall <jgr...@amazon.com>

Cheers,

---
CC: George Dunlap <george.dun...@eu.citrix.com>
CC: Jan Beulich <jbeul...@suse.com>
CC: Stefano Stabellini <sstabell...@kernel.org>
CC: Wei Liu <w...@xen.org>
CC: Julien Grall <jul...@xen.org>
---
  xen/include/public/version.h | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/xen/include/public/version.h b/xen/include/public/version.h
index 9c78b4f3b6a4..0ff8bd9077c6 100644
--- a/xen/include/public/version.h
+++ b/xen/include/public/version.h
@@ -50,7 +50,7 @@ typedef struct xen_platform_parameters 
xen_platform_parameters_t;
#define XENVER_get_features 6
  struct xen_feature_info {
-    unsigned int submap_idx;    /* IN: which 32-bit submap to return */
+    uint32_t     submap_idx;    /* IN: which 32-bit submap to return */
      uint32_t     submap;        /* OUT: 32-bit submap */
  };
  typedef struct xen_feature_info xen_feature_info_t;

--
Julien Grall

Reply via email to