On 1/18/26 2:03 PM, Richard Henderson wrote:
The target test is TARGET_I386, not __i386__.

Cc: Kyle Evans <[email protected]>
Reviewed-by: Warner Losh <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Pierrick Bouvier <[email protected]>
Signed-off-by: Richard Henderson <[email protected]>
---
  bsd-user/syscall_defs.h | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/bsd-user/syscall_defs.h b/bsd-user/syscall_defs.h
index 52f84d5dd1..c49be32bdc 100644
--- a/bsd-user/syscall_defs.h
+++ b/bsd-user/syscall_defs.h
@@ -247,7 +247,7 @@ struct target_freebsd11_stat {
      unsigned int:(8 / 2) * (16 - (int)sizeof(struct target_freebsd_timespec));
  } __packed;
-#if defined(__i386__)
+#if defined(TARGET_I386)
  #define TARGET_HAS_STAT_TIME_T_EXT       1
  #endif

This commit brings a regression, this patch fixes it:
https://lore.kernel.org/qemu-devel/[email protected]/T/#u

@Michael, the current patch is a good candidate for stable, but it has to come with the fix also, or it will be a regression.

Regards,
Pierrick

Reply via email to