Re: [patch] sys_sync livelock fix

[Daniel Phillips]

On February 14, 2002 01:53 am, Andrew Morton wrote:
> Daniel Phillips wrote:
> > 
> > What's the theory behind writing the data both before and after the commit?
> 
> see fsync_dev().  It starts I/O against existing dirty data, then
> does various fs-level syncy things which can produce more dirty
> data - this is where ext3 runs its commit, via brilliant reverse
> engineering of its calling context :-(.

OK, so it sounds like cleaning that up with an ext3-specific super->sync would
be cleaner for what it's worth, and save a little cpu.

> It then again starts I/O against new dirty data then waits on it again.  And
> then again.  There's quite a lot of overkill there.  But that's OK, as long
> as it terminates sometime.

/me doesn't comment

-- 
Daniel