From: Suparna Bhattacharya <suparna@in.ibm.com>
To: arjan@fenrus.demon.nl
Cc: linux-kernel@vger.kernel.org
Subject: Re: Bio pool & scsi scatter gather pool usage
Date: Mon, 22 Apr 2002 13:24:03 +0530 [thread overview]
Message-ID: <20020422132403.A1764@in.ibm.com> (raw)
In-Reply-To: <3CC3B2AA.80217EA0@in.ibm.com> <200204220706.g3M76lm32442@fenrus.demon.nl>
On Mon, Apr 22, 2002 at 08:06:47AM +0100, arjan@fenrus.demon.nl wrote:
> In article <3CC3B2AA.80217EA0@in.ibm.com> you wrote:
>
> > or maybe have a way pass back an error to retry with smaller size.
> > Maybe 2 limits (one that indicates that anything bigger than this is
> > sure to
> > get split, so always break it up, and another that says that anything
> > smaller
> > than this is sure not to be split, so use this size when you can't
> > afford a
> > split).
>
> Unfortionatly it's not always size that's the issue. For example in my
> code I need to split when a request crosses a certain boundary, and without
> going into too much detail, that boundary is 62 Kb aligned, not 64
> (for technical reasons ;().
Yes, I know .. Size alone isn't the only constraint - this was what
the earlier grow_bio discussion (about max BIO sizes) was all
about. Actually, not only that, in cases where the queue already has
requests which we can merge, even the size the decision gets more complex ...
That's why allowing for the exception cases when we do need to split
seemed like an option to take.
>
> Size won't catch this and while a 64Kb Kb block will always be split, that
> you can be sure of, even a 4Kb request, if unlucky, can have the need to
> split up.
>
This would as you observe be caught in alignment checks. The limit
doesn't have to be a fixed size (e.g function of block) or even size
only thing. Conceptually the question is if it can be generalized into
a compound worst case constraint through layers of lvm et al (at bind
time). It could be expressed as multiple checks.
Regards
Suparna
next prev parent reply other threads:[~2002-04-22 7:57 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-04-18 22:58 Bio pool & scsi scatter gather pool usage Mark Peloquin
2002-04-18 23:36 ` Alan Cox
2002-04-18 23:48 ` Andrew Morton
2002-04-19 7:29 ` Stephen Lord
2002-04-19 8:08 ` Joe Thornber
2002-04-19 8:51 ` Alan Cox
2002-04-19 8:58 ` Alan Cox
2002-04-19 15:27 ` Steve Lord
2002-04-19 15:57 ` Alan Cox
2002-04-19 15:51 ` Rik van Riel
2002-04-22 6:50 ` Suparna Bhattacharya
2002-04-22 7:06 ` arjan
2002-04-22 7:54 ` Suparna Bhattacharya [this message]
2002-04-24 10:20 ` Helge Hafting
2002-04-19 18:15 ` Oliver Xymoron
-- strict thread matches above, loose matches on Subject: below --
2002-04-18 23:11 Douglas Gilbert
2002-04-18 18:23 Mark Peloquin
2002-04-18 18:57 ` Andrew Morton
2002-04-19 15:44 ` Denis Vlasenko
2002-04-25 19:43 ` Mike Fedyk
2002-04-25 19:56 ` Andrew Morton
2002-04-25 19:59 ` David Mansfield
2002-04-18 13:58 Mark Peloquin
2002-04-18 16:17 ` Andrew Morton
2002-04-18 17:35 ` Andrew Morton
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=20020422132403.A1764@in.ibm.com \
--to=suparna@in.ibm.com \
--cc=arjan@fenrus.demon.nl \
--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