public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jens Axboe <axboe@suse.de>
To: Andrea Arcangeli <andrea@suse.de>
Cc: Martin Dalecki <dalecki@evision-ventures.com>,
	linux-kernel@vger.kernel.org,
	"Stephen C. Tweedie" <sct@redhat.com>,
	Linus Torvalds <torvalds@transmeta.com>,
	Alexander Viro <viro@math.psu.edu>
Subject: Re: blkdev in pagecache
Date: Wed, 9 May 2001 16:14:52 +0200	[thread overview]
Message-ID: <20010509161452.A521@suse.de> (raw)
In-Reply-To: <20010509043456.A2506@athlon.random> <3AF90A3D.7DD7A605@evision-ventures.com> <20010509151612.D2506@athlon.random>
In-Reply-To: <20010509151612.D2506@athlon.random>; from andrea@suse.de on Wed, May 09, 2001 at 03:16:12PM +0200

On Wed, May 09 2001, Andrea Arcangeli wrote:
> On Wed, May 09, 2001 at 11:13:33AM +0200, Martin Dalecki wrote:
> > >   (buffered and direct) to work with a 4096 bytes granularity instead of
> > 
> > You mean PAGE_SIZE :-).
> 
> In my first patch it is really 4096 bytes, but yes I agree we should
> change that to PAGE_CACHE_SIZE. The _only_ reason it's 4096 fixed bytes is that
> I wasn't sure all the device drivers out there can digest a bh->b_size of
> 8k/32k/64k (for the non x86 archs) and I checked the minimal PAGE_SIZE
> supported by linux is 4k. If Jens says I can sumbit 64k b_size without
> any problem for all the relevant blkdevices then I will change that in a
> jiffy ;). Anyways changing that is truly easy, just define

On IDE it should at least be possible, it can handle single segment
entries as big as 64kB for DMA. But apart from that, I think it's a lot
better to stay with PAGE_CACHE_SIZE and not get into dark country :-)

-- 
Jens Axboe


  reply	other threads:[~2001-05-09 14:15 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-05-09  2:34 blkdev in pagecache Andrea Arcangeli
2001-05-09  9:13 ` Martin Dalecki
2001-05-09 12:48   ` Jeff Garzik
2001-05-09 15:03     ` Reto Baettig
2001-05-09 15:04       ` Andrea Arcangeli
2001-05-09 13:16   ` Andrea Arcangeli
2001-05-09 14:14     ` Jens Axboe [this message]
2001-05-09 14:20       ` Andrea Arcangeli
2001-05-09 14:22         ` Jens Axboe
2001-05-09 14:43     ` Martin Dalecki

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=20010509161452.A521@suse.de \
    --to=axboe@suse.de \
    --cc=andrea@suse.de \
    --cc=dalecki@evision-ventures.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sct@redhat.com \
    --cc=torvalds@transmeta.com \
    --cc=viro@math.psu.edu \
    /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