Re: [PATCH v4 4/5] cramfs: add mmap support
Nicolas Pitre <[email protected]>
| Newsgroups | org.kernel.vger.linux-embedded,org.kernel.vger.linux-fsdevel,org.kernel.vger.linux-kernel,org.kvack.linux-mm |
|---|---|
| Message-ID | <[email protected]> |
On Tue, 3 Oct 2017, Richard Weinberger wrote: > On Mon, Oct 2, 2017 at 12:29 AM, Nicolas Pitre <[email protected]> wrote: > > On Sun, 1 Oct 2017, Christoph Hellwig wrote: > > > >> up_read(&mm->mmap_sem) in the fault path is a still a complete > >> no-go, > >> > >> NAK > > > > Care to elaborate? > > > > What about mm/filemap.c:__lock_page_or_retry() then? > > As soon you up_read() in the page fault path other tasks will race > with you before > you're able to grab the write lock. But I _know_ that. Could you highlight an area in my code where this is not accounted for? Nicolas -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to [email protected]. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"[email protected]"> [email protected] </a>