]> git.ipfire.org Git - thirdparty/binutils-gdb.git/commitdiff
sim: m32r: migrate ftime() to clock_gettime()
authorMike Frysinger <vapier@gentoo.org>
Thu, 11 Jan 2024 02:57:47 +0000 (21:57 -0500)
committerMike Frysinger <vapier@gentoo.org>
Thu, 11 Jan 2024 03:31:54 +0000 (22:31 -0500)
The ftime() function has been deprecated since POSIX-1-2004, and
removed in POSIX.1-2008.  It's also been deprecated/removed in glibc
since 2.33.  POSIX has always said the function is not portable, and
its return value, timezone, and dstflag fields are unspecified.  Even
if Linux/glibc & m32r had defined behavior, those aren't the host for
the sim runtime.

So let's stop using the function and switch to clock_gettime.  gnulib
already has detection support for it, and it's been around since at
least POSIX-1-2004.

sim/m32r/traps.c

index b779e0c3b9f5c038c9cf85bc8cc3576cdc0138dd..c5191989d7fd068b43a9d54b23638d6d46d90760 100644 (file)
@@ -386,17 +386,20 @@ m32r_trap (SIM_CPU *current_cpu, PCADDR pc, int num)
          case TARGET_LINUX_SYS_ftime:
            {
              struct timeb t;
+             struct timespec ts;
 
-             result = ftime (&t);
+             result = clock_gettime (CLOCK_REALTIME, &ts);
              errcode = errno;
 
              if (result != 0)
                break;
 
-             t.time = H2T_4 (t.time);
-             t.millitm = H2T_2 (t.millitm);
-             t.timezone = H2T_2 (t.timezone);
-             t.dstflag = H2T_2 (t.dstflag);
+             t.time = H2T_4 (ts.tv_sec);
+             t.millitm = H2T_2 (ts.tv_nsec / 1000000);
+             /* POSIX.1-2001 says the contents of the timezone and dstflag
+                members of tp after a call to ftime() are unspecified.  */
+             t.timezone = H2T_2 (0);
+             t.dstflag = H2T_2 (0);
              if ((s.write_mem) (cb, &s, arg1, (char *) &t, sizeof(t))
                  != sizeof(t))
                {