uberbaum and fileattr

[Ian Lance Taylor]

Angela Marie Thomas <angela@dragaera.ReleaseDominatrix.com> writes:

> > IIRC, there was some discussion about dropping our local fileattr mods to
> > cvs, anyway.
> 
> Since Ian wrote it, he should probably say whether it's a good idea to nuke
> the support or not.  I'm not sure it gives us a big improvement these days
> compared to some of the other changes (increased RAM for disk cache, ramdisk
> for cvs-tmep, etc)

I measured it as giving a big improvement in CPU usage and in overall
time.  That's why I wrote it.  You'll have to decide for yourself
whether you want to keep it.

A bigger disk cache always helps, but the fileattr file means that CVS
doesn't have to open all the RCS files.  Reading a single file
vs. opening and reading hundreds of files in every gcc update is a big
difference whether or not you have a disk cache.

Using a ramdisk for temporary storage is irrelevant to the fileattr
code.  The fileattr code addresses the files in the repository, not
the files in the temporary working directory.

Ian