diff --git a/CONTRIBUTORS b/CONTRIBUTORS index cc12503f..56049c1b 100644 --- a/CONTRIBUTORS +++ b/CONTRIBUTORS @@ -1275,3 +1275,7 @@ Wolfgang Rohdewald Chad Flynt for suggestions and experiments regarding the buffer reserve in cTransfer + +Chris Warren + for pointing out that the call to system("sync") in SpinUpDisk() should be + replaced with fsync(f) to avoid problems on NPTL systems diff --git a/HISTORY b/HISTORY index df11f471..2015f001 100644 --- a/HISTORY +++ b/HISTORY @@ -3424,3 +3424,5 @@ Video Disk Recorder Revision History (thanks to Marco Schlüßler). - Fixed calling cStatus::MsgChannelSwitch() in cDevice::SetChannel() (thanks to Marco Schlüßler). +- Replaced the call to system("sync") in SpinUpDisk() with fdatasync(f) to avoid + problems on NPTL systems (thanks to Chris Warren for pointing this out). diff --git a/tools.c b/tools.c index 0501d76d..aee71ecf 100644 --- a/tools.c +++ b/tools.c @@ -4,7 +4,7 @@ * See the main source file 'vdr.c' for copyright information and * how to reach the author. * - * $Id: tools.c 1.89 2005/02/05 10:10:30 kls Exp $ + * $Id: tools.c 1.90 2005/02/19 13:43:03 kls Exp $ */ #include "tools.h" @@ -454,8 +454,9 @@ bool SpinUpDisk(const char *FileName) int f = open(buf, O_WRONLY | O_CREAT, S_IRUSR | S_IWUSR | S_IRGRP | S_IROTH); // O_SYNC doesn't work on all file systems if (f >= 0) { + if (fdatasync(f) < 0) + LOG_ERROR_STR(buf); close(f); - system("sync"); remove(buf); gettimeofday(&tp2, NULL); double seconds = (((long long)tp2.tv_sec * 1000000 + tp2.tv_usec) - ((long long)tp1.tv_sec * 1000000 + tp1.tv_usec)) / 1000000.0;