From: NIIBE Yutaka <gniibe@m17n.org>
To: Marcelo Tosatti <marcelo@conectiva.com.br>
Cc: "Stephen C. Tweedie" <sct@redhat.com>,
lkml <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] swapin flush cache bug
Date: Thu, 28 Jun 2001 09:07:52 +0900 (JST) [thread overview]
Message-ID: <200106280007.f5S07qQ04446@mule.m17n.org> (raw)
In-Reply-To: <Pine.LNX.4.21.0106270710050.1291-100000@freak.distro.conectiva>
In-Reply-To: <200106270051.f5R0pkl19282@mule.m17n.org> <Pine.LNX.4.21.0106270710050.1291-100000@freak.distro.conectiva>
Marcelo Tosatti wrote:
> I think Stephen C. Tweedie has some considerations about the cache
> flushing calls on do_swap_page().
Yup. IIRC, he said that flushing cache at do_swap_page() (which I've
tried at first) is not good, because it's the hot path and it causes
another performance problem in apache or sendmail, where many
processes share the pages in swap cache.
Then, the fix I sent yesterday is second try. The flush is moved to
I/O routine, instead of do_swap_page().
Actually, I really wonder why current code causes no problem in other
architectures.
--
next prev parent reply other threads:[~2001-06-28 0:08 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-02-12 23:21 Marcelo Tosatti
2001-02-13 9:50 ` Russell King
2001-02-13 10:53 ` NIIBE Yutaka
2001-02-13 11:16 ` Russell King
2001-02-13 11:26 ` Alan Cox
2001-02-13 23:50 ` NIIBE Yutaka
2001-02-14 2:08 ` NIIBE Yutaka
2001-02-14 10:12 ` Marcelo Tosatti
2001-06-27 0:51 ` NIIBE Yutaka
2001-06-27 10:11 ` Marcelo Tosatti
2001-06-28 0:42 ` David S. Miller
2001-06-28 0:07 ` NIIBE Yutaka [this message]
2001-06-27 22:41 ` Marcelo Tosatti
2001-06-28 0:23 ` Stephen C. Tweedie
2001-06-28 0:47 ` David S. Miller
2001-06-28 1:10 ` David S. Miller
2001-06-29 14:18 ` NIIBE Yutaka
2001-07-02 22:47 ` Cache issues David S. Miller
2001-06-28 0:41 ` [PATCH] swapin flush cache bug NIIBE Yutaka
2001-06-28 1:04 ` NIIBE Yutaka
2001-07-02 11:23 ` Cache issues NIIBE Yutaka
2001-07-03 0:04 ` NIIBE Yutaka
2001-06-28 0:46 ` [PATCH] swapin flush cache bug David S. Miller
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=200106280007.f5S07qQ04446@mule.m17n.org \
--to=gniibe@m17n.org \
--cc=linux-kernel@vger.kernel.org \
--cc=marcelo@conectiva.com.br \
--cc=sct@redhat.com \
/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®