Re: Patch nag: Bug #363400

[Alexander Larsson <alexl redhat com>]

On Wed, 2007-03-21 at 14:28 -0600, Hans Petter Jansson wrote:
> http://bugzilla.gnome.org/show_bug.cgi?id=363400
> 
> Has a gnome-vfs patch that I believe to be correct. It works around a
> serious performance issue on XFS file systems. It has been tested by at
> least two users.

That patch removes the size limit for when using fadvice. I don't think
this is right. For small files we don't want to flush them from the
cache, as that could cause performance issues (we might be copying a
commonly used file).

=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
 Alexander Larsson                                            Red Hat, Inc 
                   alexl redhat com    alla lysator liu se 
He's a world-famous chivalrous paramedic haunted by memories of 'Nam. She's a 
bloodthirsty insomniac traffic cop who believes she is the reincarnation of an 
ancient Egyptian queen. They fight crime!