nuttx icon indicating copy to clipboard operation
nuttx copied to clipboard
verified publisher icon apache

fs/sync: add sync and syncfs api

Open Donny9 opened this issue 3 years ago • 5 comments

Summary

sync: add sync and syncfs api

Signed-off-by: Jiuzhu Dong [email protected]

Impact

support sync and syncfs

Testing

Ci

Donny9 avatar Apr 07 '22 02:04 Donny9

do you have a plan to actually implement them?

yamt avatar Apr 07 '22 07:04 yamt

If there are no plans to implement then both could be handled as an empty defines at application layer.

pkarashchenko avatar Apr 07 '22 08:04 pkarashchenko

do you have a plan to actually implement them?

No, add them for iozone tools only.

Donny9 avatar Apr 07 '22 12:04 Donny9

There is currently no good idea to implement them, so assume they will always succeed

Donny9 avatar Apr 07 '22 12:04 Donny9

Then maybe it is more easy to handle this at iozone tools integration level only?

pkarashchenko avatar Apr 07 '22 13:04 pkarashchenko

@xiaoxiang781216 @Donny9 can we close this PR?

pkarashchenko avatar Jan 25 '23 22:01 pkarashchenko

No, sync isn't implemented yet.

xiaoxiang781216 avatar Jan 26 '23 02:01 xiaoxiang781216

@xiaoxiang781216 @Donny9 can we close this PR?

@pkarashchenko sync is ready for review now.

xiaoxiang781216 avatar Feb 07 '23 03:02 xiaoxiang781216