From: James Bottomley <James.Bottomley@steeleye.com>
To: Alan Stern <stern@rowland.harvard.edu>
Cc: Jeff Garzik <jgarzik@pobox.com>,
SCSI development list <linux-scsi@vger.kernel.org>
Subject: Re: Status of patch as168 (sysfs readonly max_sectors attribute)
Date: 15 Jan 2004 14:42:35 -0500 [thread overview]
Message-ID: <1074195756.1868.222.camel@mulgrave> (raw)
In-Reply-To: <Pine.LNX.4.44L0.0401151425590.1316-100000@ida.rowland.org>
On Thu, 2004-01-15 at 14:30, Alan Stern wrote:
> If I understand this, you would like to see the low-level drivers manage
> these things for themselves by talking directly to the block layer,
> without having to go through the intermediate SCSI layers. This would
> imply that any sysfs interface to max_sectors would then be the
> responsibility of the driver, since the block layer itself is much too
> generic to permit users to fiddle with the settings.
>
> Correct?
Well, no, not exactly. At issue are some of the SCSI template
parameters which are copied to the host structure and then simply passed
straight through to the block queue.
If you look in scsi_alloc_queue, it simply passes through these queue
parameters. It's pretty pointless to have SCSI do anything about them
at all, so Jeff was saying we could just strip this out of the template
and the drivers and have them call the relevant blk_queue_ parameter
functions directly.
James
I personally think this would have to be done *before* any queue
activity, so probably in slave_alloc rather than slave_configure
(although there's no reason not to readjust in slave_configure).
James
prev parent reply other threads:[~2004-01-15 19:42 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-01-15 18:03 Status of patch as168 (sysfs readonly max_sectors attribute) Alan Stern
2004-01-15 18:18 ` James Bottomley
2004-01-15 18:58 ` Jeff Garzik
2004-01-15 19:30 ` Alan Stern
2004-01-15 19:42 ` James Bottomley [this message]
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=1074195756.1868.222.camel@mulgrave \
--to=james.bottomley@steeleye.com \
--cc=jgarzik@pobox.com \
--cc=linux-scsi@vger.kernel.org \
--cc=stern@rowland.harvard.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