浏览代码

该问题在Keil-MDK中没有一个太好的解决方案。本PR为权衡利弊之后得到的一套解决方案。Keil曾经提供了底层桩函数_sys_ensure,但是目前已经被废弃。因此唯一的解决方案就是每写完一句,就调用fsync去同步一次。
该问题在mbedos社区也进行过讨论,他们也面临和我们相同的两个问题:
1. Keil-MDK无有效的fflush解决方案,mbedos依然在使用淘汰的_sys_ensure桩函数。
2. Keil-MDK没有fileno函数,我方少部分软件包因此而无法使用Keil平台编译。
参考:https://github.com/ARMmbed/mbed-os/issues/1601

issue: https://github.com/RT-Thread/rt-thread/issues/4928


经过进一步测试发现,Keil-MDK中的fflush函数已经退化成服务于STDOUT的函数,Keil-MDK内部有一个buffer可以存放fputc(stdout) 发过来的数据,如果调用fflush,这些数据是可以flush出去的。但是如果是真正的文件数据,fflush函数并没有提供任何桩函数接口可以让文件flush数据出去。

Meco Man 2 年之前
父节点
当前提交
bd33d37bc4
共有 1 个文件被更改,包括 18 次插入1 次删除
  1. 18 1
      components/libc/compilers/armlibc/syscalls.c

+ 18 - 1
components/libc/compilers/armlibc/syscalls.c

@@ -1,5 +1,5 @@
 /*
 /*
- * Copyright (c) 2006-2021, RT-Thread Development Team
+ * Copyright (c) 2006-2022, RT-Thread Development Team
  *
  *
  * SPDX-License-Identifier: Apache-2.0
  * SPDX-License-Identifier: Apache-2.0
  *
  *
@@ -226,6 +226,11 @@ int _sys_write(FILEHANDLE fh, const unsigned char *buf, unsigned len, int mode)
         size = write(fh, buf, len);
         size = write(fh, buf, len);
         if (size >= 0)
         if (size >= 0)
         {
         {
+            /*
+            fflush doesn't have a good solution in Keil-MDK,
+            so it has to sync/flush when for each writen.
+            */
+            fsync(fh);
             return len - size; /* success */
             return len - size; /* success */
         }
         }
         else
         else
@@ -239,6 +244,18 @@ int _sys_write(FILEHANDLE fh, const unsigned char *buf, unsigned len, int mode)
     }
     }
 }
 }
 
 
+/*
+ * Flush any OS buffers associated with fh, ensuring that the file
+ * is up to date on disk. Result is >=0 if OK, negative for an
+ * error.
+ * This function is deprecated. It is never called by any other library function,
+ * and you are not required to re-implement it if you are retargeting standard I/O (stdio).
+ */
+int _sys_ensure(FILEHANDLE fh)
+{
+    return fsync(fh);
+}
+
 /*
 /*
  * Move the file position to a given offset from the file start.
  * Move the file position to a given offset from the file start.
  * Returns >=0 on success, <0 on failure.
  * Returns >=0 on success, <0 on failure.