All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steve French <smfrench@austin.rr.com>
To: unlisted-recipients:; (no To-header on input)
Cc: linux-fsdevel@vger.kernel.org, linux-cifs-client@lists.samba.org
Subject: Re: writethrough of mmapped pages
Date: Sat, 31 Jan 2004 18:16:45 -0600	[thread overview]
Message-ID: <401C456D.3050709@austin.rr.com> (raw)
In-Reply-To: 401C3E2D.7040806@austin.rr.com

Well the filemap_fdatawrite after generic_write seems to cause the 
problem. Removing the call to it there worked around the problem - fsx 
works fine as long as I don't flush the dirty data with 
filemap_fdatawrite after the call to generic_write in my write wrapper 
routine. Removing it fixed the problem (but that doesn't seem to be very 
useful since when I lose the caching token - I really do need to flush 
the data to the server after every write - and if filemap_fdatawrite is 
not safe to call after every write - what routine should I call?)

Steve French wrote:

> The experiment described at the bottom (always calling the generic 
> read and write but doing filemap_fdatasync and invalidate_remote_inode 
> when file not cacheable) failed - same symptom. It looks like the 
> portion of page implicitly extended is not zeroed properly but I am 
> not sure why calling filemap_fdatasync early would make a difference. 
> In the note below there is a typo - I meant invalidate_remote_inode 
> not invalidate_inode_pages (since when the file is not cacheable I 
> have to force the client to reread the pages).
>


  reply	other threads:[~2004-02-01  0:16 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-01-31 23:15 writethrough of mmapped pages Steve French
2004-01-31 23:45 ` Steve French
2004-02-01  0:16   ` Steve French [this message]
  -- strict thread matches above, loose matches on Subject: below --
2004-01-30 23:30 Steve French
2004-01-31  0:02 ` Andrew Morton
2004-01-31 10:21   ` David Woodhouse

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=401C456D.3050709@austin.rr.com \
    --to=smfrench@austin.rr.com \
    --cc=linux-cifs-client@lists.samba.org \
    --cc=linux-fsdevel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.