Re: [PATCH 08/12] xfs: remove xfs_ifork_ops

Christoph Hellwig
2020-05-01T16:50:20Z

On Fri, May 01, 2020 at 06:38:09PM +0200, Christoph Hellwig wrote:
> That being said my approach here was a little too dumb. Once we are
> all in the same code base we can stop the stupid patching of the
> parent and just handle the case directly. Something like this
> incremental diff on top of the sent out version (not actually tested).
>
> Total diffstate with the original patch is:
>
> 4 files changed, 37 insertions(+), 35 deletions(-)
>
> and this should also help with online repair while killing a horrible
> kludge.

Btw, І wonder if for repair / online repair just skipping the verifiers
entirely would make more sense. But I think we can go there
incrementally and just keep the existing repair behavior for now.