From: James Bottomley <James.Bottomley@HansenPartnership.com>
To: Hannes Reinecke <hare@suse.de>
Cc: Christoph Hellwig <hch@infradead.org>,
"Elliott, Robert (Server Storage)" <elliott@hp.com>,
SCSI Mailing List <linux-scsi@vger.kernel.org>,
Jens Axboe <axboe@kernel.dk>
Subject: Re: Question: request tag usage
Date: Fri, 26 Sep 2014 11:12:23 +0100 [thread overview]
Message-ID: <1411726343.2183.9.camel@jarvis> (raw)
In-Reply-To: <542521DB.1020605@suse.de>
On Fri, 2014-09-26 at 10:20 +0200, Hannes Reinecke wrote:
> On 09/26/2014 10:03 AM, Christoph Hellwig wrote:
> > On Fri, Sep 26, 2014 at 08:29:29AM +0200, Hannes Reinecke wrote:
> >> Hi Christoph,
> >>
> >> as discussed it would make sense to use the request->tag in eg
> >> scmd_printk() to identify the command.
> >> Which I duly did, only to figure out that the tag is always '-1', ie
> >> tagging is not in use.
> >> (Which is okay from the SCSI side, seeing the TCQ is basically a
> >> SCSI parallel thing).
> >
> > tag are still a live part of SAM for every transport, they've only
> > been renamed to "command identifier" in SAM-4 to confuse everyone.
> >
> >> Looking closer I found plenty of code for handling tags in the block
> >> layer (and the blk-mq stuff, of course), but virtually none of the
> >> non-SPI driver seems to be using them.
> >
> > A quick grep for scsi_activate_tcq disagrees with you.
> >
> Yeah, I've noticed after I've written the mail.
> However, main point still stands: using 'tag' to identify commands
> is pointless if not all of the LLDDs use tagging ...
Every non parallel LLD uses tagging; they all use the network request
response model, so they can no longer hold the bus until they get an
answer (which was how untagged commands work in SPI), so for most of the
transports, untagged commands aren't legal. Of course, some drivers
roll their own tags instead of using the block ones.
James
next prev parent reply other threads:[~2014-09-26 10:12 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-26 6:29 Question: request tag usage Hannes Reinecke
2014-09-26 8:03 ` Christoph Hellwig
2014-09-26 8:20 ` Hannes Reinecke
2014-09-26 10:12 ` James Bottomley [this message]
2014-09-26 12:41 ` Hannes Reinecke
2014-09-26 13:52 ` Elliott, Robert (Server Storage)
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=1411726343.2183.9.camel@jarvis \
--to=james.bottomley@hansenpartnership.com \
--cc=axboe@kernel.dk \
--cc=elliott@hp.com \
--cc=hare@suse.de \
--cc=hch@infradead.org \
--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