[mythtv-firehose] mythtv/master commit: e4c590915 by Daniel Kristjansson (daniel-kristjansson)

MythTV noreply at mythtv.org
Tue May 3 22:08:23 UTC 2011


      Author:  Daniel Kristjansson <danielk at cuymedia.net>
 Change Date:  2011-05-03T15:04:12-07:00
   Push Date:  2011/05/03 15:08:16 -0700
  Repository:  mythtv
      Branch:  master
New Revision:  e4c590915daf6f9132d8c950748cc1e8ba7a67a4
   Changeset:  https://github.com/MythTV/mythtv/commit/e4c590915

Log:

Some fadvise changes.

First this gets rid of DONTNEED advice. These calls a global, so in order for these to be safe in the MythTV context we'd need to make sure that no other process is currently accessing the file. This is complicated on 64 bit platforms where you can memory map the whole file and extreemely complicated on 32 bit platforms.. Instead lets leave this task to the OS.

Second we change some of the WILLNEED calls. On Linux these are unfortunately implemented as blocking calls; that is they block until the requested number of bytes have been read, instead of just telling the OS to start reading in those bytes in a kernel thread. Consequently, this drops the calls in some cases and reduces the number of bytes to read in other cases.

Modified:

   mythtv/libs/libmythtv/ThreadedFileWriter.cpp
   mythtv/libs/libmythtv/fileringbuffer.cpp
   mythtv/libs/libmythtv/ringbuffer.cpp



More information about the mythtv-firehose mailing list