From: Jeff Garzik <jgarzik@pobox.com>
To: James Bottomley <James.Bottomley@steeleye.com>
Cc: Jens Axboe <axboe@suse.de>,
Andrew Vasquez <andrew.vasquez@qlogic.com>,
SCSI Mailing List <linux-scsi@vger.kernel.org>
Subject: Re: PATCH [5/15] qla2xxx: SG tablesize update
Date: Mon, 15 Mar 2004 11:09:51 -0500 [thread overview]
Message-ID: <4055D54F.8010908@pobox.com> (raw)
In-Reply-To: <1079303516.2108.73.camel@mulgrave>
James Bottomley wrote:
> On Sun, 2004-03-14 at 15:36, Jeff Garzik wrote:
>
>>On Sun, Mar 14, 2004 at 09:59:55AM -0500, James Bottomley wrote:
>>
>>>The Qla chips are rather weird in that they have a single issue queue
>>>(whose size you can vary) but whose entry formats are fixed. If I
>>>remember correctly, an initial command can have 4 SG elements, but a
>>>follow on entry can have 7 (not sure of the figures). But anyway, large
>>>SG commands end up having to find multiple entries in this queue (and
>>>being a single issue queue for the entire card, it has to be mutexed
>>>while you search). The more resources you need, the more difficult the
>>>search and the more contention you generate on the resource mutex.
>>
>>The block layer can handle this type of hardware just fine ;-)
>>
>>In my SATA hacking I am finding several devices that one must constrain
>>based on hardware-global resources, rather than just TCQ (or lack
>>thereof).
>
>
> I'm sorry, I must have missed the multiple queues down to a single HBA
> queue API in the block layer, which one is it again?
Not saying the block layer provides this capability, simply that it is
possible in any block driver. It required < 20 LOC in carmel.c to do
this in a hardware-specific fashion, so it seemed unnecessary to invent
a new API... I would like to see something similar in cciss.c, in fact...
Jeff
next prev parent reply other threads:[~2004-03-15 16:10 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-03-14 8:24 PATCH [5/15] qla2xxx: SG tablesize update Andrew Vasquez
2004-03-14 14:49 ` James Bottomley
2004-03-14 14:51 ` Jens Axboe
2004-03-14 14:59 ` James Bottomley
2004-03-14 15:15 ` Jens Axboe
2004-03-14 15:18 ` Anton Blanchard
2004-03-14 15:31 ` James Bottomley
2004-03-14 15:47 ` Anton Blanchard
2004-03-14 15:55 ` James Bottomley
2004-03-14 16:01 ` Anton Blanchard
2004-03-14 20:41 ` Jeff Garzik
2004-03-14 22:27 ` James Bottomley
2004-03-15 16:12 ` Jeff Garzik
2004-03-14 20:36 ` Jeff Garzik
2004-03-14 22:31 ` James Bottomley
2004-03-15 16:09 ` Jeff Garzik [this message]
-- strict thread matches above, loose matches on Subject: below --
2004-03-15 23:43 Andrew Vasquez
2004-03-16 3:37 ` James Bottomley
2004-03-16 6:40 ` Jeremy Higdon
2004-03-16 11:32 ` Anton Blanchard
2004-03-16 21:49 ` James Bottomley
2004-03-16 22:09 Andrew Vasquez
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=4055D54F.8010908@pobox.com \
--to=jgarzik@pobox.com \
--cc=James.Bottomley@steeleye.com \
--cc=andrew.vasquez@qlogic.com \
--cc=axboe@suse.de \
--cc=linux-scsi@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