Re: [PATCH] Fix races in 2.4.2-ac22 SysV shared memory

Linus Torvalds (torvalds@transmeta.com)
Fri, 23 Mar 2001 14:27:12 -0800 (PST)


On Fri, 23 Mar 2001, Alan Cox wrote:
> >
> > + spin_lock (&info->lock);
> > +
> > + /* The shmem_swp_entry() call may have blocked, and
> > + * shmem_writepage may have been moving a page between the page
> > + * cache and swap cache. We need to recheck the page cache
> > + * under the protection of the info->lock spinlock. */
> > +
> > + page = find_lock_page(mapping, idx);
> >
> > Ehh.. Sleeping with the spin-lock held? Sounds like a truly bad idea.
>
> Umm find_lock_page doesnt sleep does it ?

Sure it does. Note the "lock" in find_lock_page(). It will lock the page,
which implies sleeping if somebody is accessing it at the same time.

If you don't want to sleep, you need to use one of the wrappers for
"__find_page_nolock()". Something like "find_get_page()", which only
"gets" the page.

The naming actually does make sense in this area.

Linus

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/