On Mon, Oct 24, 2016 at 1:46 PM, Linus Torvalds <torva...@linux-foundation.org> wrote: > > So this is all some really subtle code, but I'm not seeing that it > would be wrong.
Ahh... Except maybe.. The vmalloc/vfree code itself is a bit scary. In particular, we have a rather insane model of TLB flushing. We leave the virtual area on a lazy purge-list, and we delay flushing the TLB and actually freeing the virtual memory for it so that we can batch things up. But we've free'd the physical pages that are *mapped* by that area when we do the vfree(). So there can be stale TLB entries that point to pages that have gotten re-used. They shouldn't matter, because nothing should be writing to those pages, but it strikes me that this may also be hurting the DEBUG_PAGEALLOC thing. Maybe we're not getting the page fautls that we *should* be getting, and there are hidden reads and writes to those paghes that already got free'd.\ There was some nasty reason why we did that insane thing. I think it was just that there are a few high-frequency vmalloc/vfree users and the TLB flushing was killing some performance. But it does strike me that we are playing very fast and loose with the TLB on the vmalloc area. So maybe all the new VMAP code is fine, and it's really vmalloc/vfree that has been subtly broken but nobody has ever cared before? Linus -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html