Re: [patch v3] splice: fix race with page invalidation
From: Miklos Szeredi
Date: Wed Jul 30 2008 - 17:16:48 EST
> On Wed, 30 Jul 2008, Miklos Szeredi wrote:
> >
> > It _is_ a bug fix.
>
> No it's not.
>
> It's still papering over a totally unrelated bug. It's not a "fix", it's a
> "paper-over". It doesn't matter if _behaviour_ changes.
>
> Can you really not see the difference?
>
> Afaik, everybody pretty much agreed what the real fix should be (don't
> mark the page not up-to-date, just remove it from the radix tree)
Huh? I did exactly that, and that patch got NAK'ed by you and Nick:
http://lkml.org/lkml/2008/6/25/230
http://lkml.org/lkml/2008/7/7/21
Confused.
> I'm not seeing why you continue to push the patch that ALREADY GOT
> NAK'ed.
And later ACK'ed by Nick.
There's everything here but agreement about a solution :)
Miklos
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/