Re: [PATCH] Re: test10-pre7

John Kennedy
2000-10-31T10:06:17Z

[-- Attachment #1: Type: text/plain, Size: 326 bytes --]

On Mon, Oct 30, 2000 at 03:34:44PM -0500, Alexander Viro wrote:
> Unfortunately, it doesn't fix the thing. ->sync_page() is called ...
> Minimal patch (against -pre7) follows. It still leaves sync_page() problem
> open - any suggestions on that one are very welcome. ...

I needed to patch your patch to get it to compile.

[-- Attachment #2: PATCH --]
[-- Type: text/plain, Size: 301 bytes --]

--- ./mm/filemap.c.OLD Mon Oct 30 23:00:35 2000
+++ ./mm/filemap.c Mon Oct 30 23:11:26 2000
@@ -2313,9 +2313,9 @@
void *data)
{
struct page *page;
+ int err;
retry:
page = __read_cache_page(mapping, index, filler, data);
- int err;

if (IS_ERR(page) || Page_Uptodate(page))
goto out;