public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* A question about ll_10byte_cmd_build
@ 2002-04-18  2:41 peterc
  2002-04-18  5:55 ` Jens Axboe
  0 siblings, 1 reply; 2+ messages in thread
From: peterc @ 2002-04-18  2:41 UTC (permalink / raw)
  To: linux-kernel



(Linux 2.5.8)
In ll_rw_blk.c there's a function, ll_10byte_cmd_build() which is
supposed to be used to generate `10-byte commands'.

It appears to generate a SCSI READ_10 or WRITE_10 command (which
happen to be identical in format to the ATAPI GPCMD_{READ,WRITE}_10 commands)

Is this IDE specific, or is it meant to cover all block devices?
If it's IDE specific, why is it in ll_rw_blk.c, which is meant to be
common to all block devices?


As far as I can tell, only ide-cd.c actually uses the function in a
stock 2.5.8 kernel --- so it could theoretically be moved to ide-cd.c.

Peter C

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: A question about ll_10byte_cmd_build
  2002-04-18  2:41 A question about ll_10byte_cmd_build peterc
@ 2002-04-18  5:55 ` Jens Axboe
  0 siblings, 0 replies; 2+ messages in thread
From: Jens Axboe @ 2002-04-18  5:55 UTC (permalink / raw)
  To: peterc; +Cc: linux-kernel

On Thu, Apr 18 2002, peterc@gelato.unsw.edu.au wrote:
> 
> 
> (Linux 2.5.8)
> In ll_rw_blk.c there's a function, ll_10byte_cmd_build() which is
> supposed to be used to generate `10-byte commands'.
> 
> It appears to generate a SCSI READ_10 or WRITE_10 command (which
> happen to be identical in format to the ATAPI GPCMD_{READ,WRITE}_10 commands)
> 
> Is this IDE specific, or is it meant to cover all block devices?
> If it's IDE specific, why is it in ll_rw_blk.c, which is meant to be
> common to all block devices?

It's not IDE specific, since when does IDE use packet commands? It's
not ATAPI specific either.

> As far as I can tell, only ide-cd.c actually uses the function in a
> stock 2.5.8 kernel --- so it could theoretically be moved to ide-cd.c.

It's meant to be an example of a generic prep_rq_fn() queue function, a
start the conversion of using struct request as the generic passer of
cdb's. Only ide-cd uses it for now as you see, that's merely because
lots of bits of the infrastructure are still missing.

-- 
Jens Axboe


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2002-04-18  5:57 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2002-04-18  2:41 A question about ll_10byte_cmd_build peterc
2002-04-18  5:55 ` Jens Axboe

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox