mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Andi Kleen <ak@suse.de>
To: Venkatesh Ramamurthy <Venkateshr@ami.com>
Cc: "'Stephen C. Tweedie'" <sct@redhat.com>,
	"'Pavel Machek'" <pavel@suse.cz>,
	adefacc@tin.it, linux-kernel@vger.kernel.org
Subject: Re: Confirmation request about new 2.4.x. kernel limits
Date: Tue, 9 Jan 2001 16:51:05 +0100	[thread overview]
Message-ID: <20010109165105.A3868@gruyere.muc.suse.de> (raw)
In-Reply-To: <1355693A51C0D211B55A00105ACCFE64E95139@ATL_MS1>
In-Reply-To: <1355693A51C0D211B55A00105ACCFE64E95139@ATL_MS1>; from Venkateshr@ami.com on Tue, Jan 09, 2001 at 10:15:34AM -0500

On Tue, Jan 09, 2001 at 10:15:34AM -0500, Venkatesh Ramamurthy wrote:
> > Any memory over 1GB is bounce-buffered, but we don't use that memory
> > for anything other than process data pages or file cache, so only
> > swapping and disk IO to regular files gets the extra copy.  In
> > particular, things like network buffers are still all kept in the low
> > 1GB so never need to be buffered.
> 	[Venkatesh Ramamurthy]  If anything over 1GB is bounce buffered than
> what is the purpose of setting the pci_dev->dma_mask field.  On a IA32
> system we set it to 32 1's and IA64 to 64 1's - Venkat

On 2.4 on 32bit hosts it has no direct effect because the driver interface has not 
been revised.  It's only useful on 64bit atm.

2.5 will hopefully move the bounce test into your driver. Currently it tests
PageHighMem() which is bogus (and gives the 1GB boundary), because it should test 
the physical address against the dma_mask.

On 64bit PageHighMem always returns false, so the bounce buffer is never create in
the upper level and the final decision is down in your driver.

Currently create_bounce() in the block layer is at a very unfortunate place on the
top half of ll_rw_blk, because there it has not even a chance to know about the
final device (e.g. when LVM or MD come into play) 

I suspect someone will quickly do a limited 2.4 patch that moves create_bounce a bit
down for selected devices on 32bit hosts -- it is a very obvious optimization. 
It probably also could be done with minor changes.


-Andi

-
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/

  reply	other threads:[~2001-01-09 15:51 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-01-09 15:15 Venkatesh Ramamurthy
2001-01-09 15:51 ` Andi Kleen [this message]
  -- strict thread matches above, loose matches on Subject: below --
2001-01-09 18:00 Venkatesh Ramamurthy
2001-01-09 17:35 Venkatesh Ramamurthy
2001-01-09 17:47 ` Andi Kleen
2001-01-09  4:11 Venkatesh Ramamurthy
2001-01-09 12:21 ` Matti Aarnio
2001-01-09 16:27   ` Tim Wright
2001-01-09 16:44     ` Andi Kleen
2001-01-09 16:50       ` Tim Wright
2001-01-10 13:16     ` Kurt Garloff
2001-01-09 14:05 ` Stephen C. Tweedie
2001-01-04 11:50 A.D.F.
2001-01-04 11:23 ` Tigran Aivazian
2001-01-04 11:50 ` Anton Blanchard
2001-01-05 22:46 ` Pavel Machek
2001-01-09 14:03   ` Stephen C. Tweedie

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=20010109165105.A3868@gruyere.muc.suse.de \
    --to=ak@suse.de \
    --cc=Venkateshr@ami.com \
    --cc=adefacc@tin.it \
    --cc=linux-kernel@vger.kernel.org \
    --cc=pavel@suse.cz \
    --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®