Benjamin Herrenschmidt wrote:
Still, even with NOPAGE_REFAULT or the equivalent with the new fault() code,in the case we need to take this route, (and it looks like we won't have to), I guess we still need to restart from find_vma() in the fault()/nopage() handler to make sure the VMA is still present. The object mutex need to be dropped as well to avoid deadlocks. Sounds complicated.But as we said, it should be enough to do the flag change with the object mutex held as long as it's after unmap_mapped_ranges() Ben.
Agreed. /Thomas - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
- References:
- Re: [patch 3/3] mm: fault handler to replace nopage and populate
- From: Nick Piggin <[email protected]>
- Re: [patch 3/3] mm: fault handler to replace nopage and populate
- From: Benjamin Herrenschmidt <[email protected]>
- Re: [patch 3/3] mm: fault handler to replace nopage and populate
- From: Nick Piggin <[email protected]>
- Re: [patch 3/3] mm: fault handler to replace nopage and populate
- From: Benjamin Herrenschmidt <[email protected]>
- Re: [patch 3/3] mm: fault handler to replace nopage and populate
- From: Nick Piggin <[email protected]>
- Re: [patch 3/3] mm: fault handler to replace nopage and populate
- From: Benjamin Herrenschmidt <[email protected]>
- Re: [patch 3/3] mm: fault handler to replace nopage and populate
- From: Nick Piggin <[email protected]>
- Re: [patch 3/3] mm: fault handler to replace nopage and populate
- From: Benjamin Herrenschmidt <[email protected]>
- Re: [patch 3/3] mm: fault handler to replace nopage and populate
- From: Nick Piggin <[email protected]>
- Re: [patch 3/3] mm: fault handler to replace nopage and populate
- From: Thomas Hellstrom <[email protected]>
- Re: [patch 3/3] mm: fault handler to replace nopage and populate
- From: Nick Piggin <[email protected]>
- Re: [patch 3/3] mm: fault handler to replace nopage and populate
- From: Benjamin Herrenschmidt <[email protected]>
- Re: [patch 3/3] mm: fault handler to replace nopage and populate
- From: Thomas Hellström <[email protected]>
- Re: [patch 3/3] mm: fault handler to replace nopage and populate
- From: Benjamin Herrenschmidt <[email protected]>
- Re: [patch 3/3] mm: fault handler to replace nopage and populate
- Prev by Date: [PATCH] ext3: fsid for statvfs
- Next by Date: Re: [PATCH] firewire: handle sysfs errors
- Previous by thread: Re: [patch 3/3] mm: fault handler to replace nopage and populate
- Next by thread: Re: [patch 3/3] mm: fault handler to replace nopage and populate
- Index(es):