On Thu, Jul 2, 2020 at 9:51 AM Andreas Gruenbacher <agruenba@xxxxxxxxxx> wrote:
>
> Of this patch queue, either only the first patch or all four patches can
> be applied to fix gfs2's current issues in 5.8. Please let me know what
> you think.
I think the IOCB_NOIO flag looks fine (apart from the nit I pointed
out), abnd we could do that.
However, is the "revert and reinstate" looks odd. Is the reinstate so
different front he original that it makes sense to do that way?
Or was it done that way only to give the choice of just doing the revert?
Because if so, I think I'd rather just see a "fix" rather than
"revert+reinstate".
But I didn't look that closely at the gfs2 code itself, maybe there's
some reason you did it that way.
Linus
