mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@digeo.com>
To: Andre Hedrick <andre@linux-ide.org>
Cc: keitha@edp.fastfreenet.com, linux-kernel@vger.kernel.org, axboe@suse.de
Subject: Re: bdflush flushing memory mapped pages.
Date: Wed, 9 Apr 2003 02:27:26 -0700	[thread overview]
Message-ID: <20030409022726.1ec93a0f.akpm@digeo.com> (raw)
In-Reply-To: <Pine.LNX.4.10.10304090209440.12558-100000@master.linux-ide.org>

Andre Hedrick <andre@linux-ide.org> wrote:
>
> 
> Funny you mention this point!
> 
> I just spent 30-45 minutes on the phone talking to Jens about this very
> issue.  Jens states he can map the model in to 2.5. and will give it a
> fling in a bit.  This issue is a must; however, I had given up on the idea
> until 2.7.  However, the issues he and I addressed, in combination to your
> request jive in sync.

noooo.....   This isn't going to happen.  There are many reasons.

Firstly, how can bdflush even know what pages to write?  The dirtiness of
these pages is recorded *only* in some processor's hardware pte cache and/or
the software pagetables.  Someone needs to go tell all the CPUs to writeback
their pte caches into the pagetables and then someone needs to walk the
pagetables propagating the pte dirty bit into the pageframes before we can
even start the I/O.

That's what msync does, in filemap_sync().


And even if bdflush did this automagically, it's the wrong thing to do
because the application could very well be repeatedly dirtying the pages. 
Very probably.  So we've just gone and done a ton of pointless I/O, over and
over.

You can view MAP_SHARED as an IPC mechanism which uses the filesystem
namespace for naming.  No way do these people want bdflush pointlessly
hammering the disk.

You can also view MAP_SHARED as a (strange) way of writing files out.  If you
want to do that then fine, but you need to tell the kernel when you've
finished, just like write() does.   You do that with msync.




  reply	other threads:[~2003-04-09  9:15 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-04-09 19:20 Keith Ansell
2003-04-09  9:13 ` Andre Hedrick
2003-04-09  9:27   ` Andrew Morton [this message]
2003-04-09  9:33     ` Jens Axboe
2003-04-10 20:09     ` Keith Ansell
2003-04-10  8:16       ` Andre Hedrick
2003-04-10  9:02       ` Nick Piggin
2003-04-10 17:04       ` Alan Cox
2003-04-09  9:22 ` Arjan van de Ven
2003-04-09 10:39 ` Alan Cox

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=20030409022726.1ec93a0f.akpm@digeo.com \
    --to=akpm@digeo.com \
    --cc=andre@linux-ide.org \
    --cc=axboe@suse.de \
    --cc=keitha@edp.fastfreenet.com \
    --cc=linux-kernel@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®