[MPlayer-dev-eng] [PATCH] Use posix_fadvise in stream_file if available

Reimar Döffinger Reimar.Doeffinger at gmx.de
Mon Nov 9 18:40:40 CET 2009


On Mon, Nov 09, 2009 at 06:26:37PM +0100, Tobias Diedrich wrote:
> > The name PREFETCH_LEN is bad though, it should probably be READAHEAD_DISTANCE.
> 
> prefetch: "(computing) To load data or instructions in anticipation
> of their need"
> "readahead is the file prefetching technology used in the Linux
> operating system"
> (Well, from wiktionary and wikipedia respectively, to be taken with
> a grain of salt :))
> 
> I'd say that prefetch and read-ahead are more or less synonyms...
> With the change distance is clearly better than len though.
> Anyway I don't really care if it's PREFETCH_DISTANCE or
> READAHEAD_DISTANCE... ^^;

PREFETCH_DISTANCE is okay, too, I only minded the _LEN, sorry for
being confusing.
>From my point of view it can be applied (not the STREAM_MAX_BUFFER_SIZE,
that is too messy if there's no significant benefit).



More information about the MPlayer-dev-eng mailing list