Re: [PATCH 3/5] f2fs: report readonly status in ->fsync

[Jaegeuk Kim <jaegeuk@kernel.org>]

Hi Chao,

...

> > > And one more thing is how do you think of moving inline dot recovery into
> > > fsck?
> > 
> > Hmm, we can't move that part entirely into fsck regarding to backward
> > compatibility.
> 
> IMO, dynamical repair could be optional, not a must. Because when touching
> such incomplete directory in ->lookup, we could leave some dmesg to remind
> user to repair. Or if user found this issue by himself, also he could
> repair it by using fsck. So this shouldn't case a backward compatibility
> issue, or am I missing something?

Hmm, it seems that maybe I misunderstood your *moving it into fsck*.
Anyway, I'm fine to add kernel messages as well.
And, in any case, the only way to fix this case is to run fsck at first.
One approach, as I implemented, is to set a flag and then repair it at runtime.

The other one is to repair by fsck, as you suggested.
In this case, my concern is that current fsck is working without allocating
blocks, since it is a little bit risky right now.
For example, if there is no valid user block, not free space, fsck should handle
ENOSPC while changing the existing disk, which has not been fully validated so
far.

Ah, otherwise, we can activate inline_dentry by fsck for this special case, once
inline_dentry in f2fs becomes pretty much stable later. :)

Thanks,

> 
> > We can add that in fsck, but it needs another block
> > allocation part which we need to handle -ENOSPC in fsck as well.
> 
> That's right, is it possible to do defragment (gc) if there is no
> more space when allocating block?
> 
> Thanks,
> 
> > 
> > Thanks,
> > 
> > >
> > > Thanks,
> > >
> > > >
> > > > I only looked at the diff, so if I missed the context and my reasoning
> > > > doesn't apply, forgive and ignore me :)
> > > >
> > > > --
> > > >                 The choice of a       Deliantra, the free code+content MORPG
> > > >       -----==-     _GNU_              http://www.deliantra.net
> > > >       ----==-- _       generation
> > > >       ---==---(_)__  __ ____  __      Marc Lehmann
> > > >       --==---/ / _ \/ // /\ \/ /      schmorp@schmorp.de
> > > >       -=====/_/_//_/\_,_/ /_/\_\

------------------------------------------------------------------------------