Re: CVS commit: src/sys/ufs/ffs

[Antti Kantee <pooka%cs.hut.fi@localhost>]

On Wed Nov 12 2008 at 19:08:37 +0100, Joerg Sonnenberger wrote:
> On Wed, Nov 12, 2008 at 07:38:22PM +0200, Antti Kantee wrote:
> > > The PR is about the optional reallocation fcntl?
> > 
> > The what?
> 
> The ancient FFS code had a sysctl doreallocblks. That allowed on a
> per-inode base to force reallocation of the blocks. I think this was
> meant to be used for some online defragmentation. The actual code for it
> was what I ultimately removed.

Try to decide between sysctl and fcntl ;)

As far as I understood it, the point was to reallocate blocks while a
file wasn't yet flushed to disk.  This allowed better locality for files
written in increments.  Since vfs_cluster, VOP_REALLOCBLKS() and with your
commit even the backend code gone, the PR could use a little updating.
You struck the final blow, so it's your job ;)