Re: [PATCH] filemap_fdatasync & related changes

From: Marcelo Tosatti (marcelo@conectiva.com.br)
Date: Thu Jan 04 2001 - 09:41:41 EST


On 4 Jan 2001, Christoph Rohland wrote:

> Chris Mason <mason@suse.com> writes:
> > Just noticed the filemap_fdatasync code doesn't check the return value from
> > writepage. Linus, would you take a patch that redirtied the page, puts it
> > back onto the dirty list (at the tail), and unlocks the page when writepage
> > returns 1?
> >
> > That would loop forever if the writepage func kept returning 1 though...I
> > think that's what we want, unless someone like ramfs made a writepage func
> > that always returned 1.
>
> shmem has such a writepage for locked shm segments.

The correct thing seems to be wait for this lock in case of
fdatasync/sync/etc, then.

What about a sync parameter to ->writepage() ?

> return 1 if the swap space is exhausted. So everybody using shared
> anonymous, SYSV shared or POSIX shared memory can hit this.

In case of swap exhausted I think we should just fail.

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



This archive was generated by hypermail 2b29 : Sun Jan 07 2001 - 21:00:18 EST