public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
From: Luben Tuikov <luben@splentec.com>
To: Patrick Mansfield <patmans@us.ibm.com>
Cc: linux-scsi@vger.kernel.org
Subject: Re: [RFC][PATCH] scsi-misc-2.5 software enqueue when can_queue reached
Date: Tue, 04 Mar 2003 00:48:11 -0500	[thread overview]
Message-ID: <3E643E1B.9080608@splentec.com> (raw)
In-Reply-To: <20030303154119.A31562@beaverton.ibm.com>

Patrick Mansfield wrote:
> Luben -
> 
> Well they are just names, the important issue is that we understand how
> the code uses them.
> 
> It is a struct list_head, you can call it a queue but it is coded as a
> list_head. Adding _q or _queue does not change anything, or make the code
> easier to understand.

Ah, Patrick, mea culpa.  I see your point of view.

The reason I'm so finicky on names/labels/designators is because
I subscribe to the theory that symbols themselves carry meaning,
rather than the opposing camp of thought, where the claim is that
symbols just label ideas.

The saying ``it's just semantics'' stems from the second
school of thought, which is in recess, after the first one
(cf. Platonism) made its point in the latter 20th century, mainly
through math and philosophy.
Also, the second one gets into infinite regression when/if trying to
explain ``meaning'' itself.

I've mentioned this a few times and given examples of
how hard it is doing arithmetic in Roman numerals, as opposed
to Arabic numerals, which *are* the numbers.

So for this reason I try to name things as closely as possible
so that their names (which is they, themselves) represent what they
are used for or stand for -- which is again they themselves.

If something doesn't have a name, like this object:


then it doesn't really exist. :-)

Else we might just give names like 
	struct list_head c3po;   /* queue of done commands */
which would hardly be appropriate.

BTW, long time ago I used to be one of those ppl that didn't really
care for what the names were -- then I got converted.
 
> There is not much difference between "pending" and "deferred" (damn I
> actually had to go look those up in a dictionary),

(I love reading the dictionary.)

Anyway, something pending is always deferred, but something
deferred doesn't have to be pending.

So in your case, your pending commands may never get executed
if the LUN had died in due time, because the host was buggy
when it's queue limit got hit.  So put them in a deferred_q.

But all commands submitted to the LLDDs, via queuecommand()
are pending for their execution status via scsi_done().

> though SAM-3 references
> pending, I don't see any SAM references to deferred.

Forget about SAM-3.  The queuing model there is for a
different purpose, and we're talking about a [SCSI kernel]
subsystem here. What I'm trying to say is, do what makes sense.
 
> The name can always be modified via future patches.

Theoretically, yes.  Practially, I don't think anyone has
the time and/or desire -- well, maybe Christoph. ;-)

But really, whatever goes in now, will stay there for a
long, long time, so I say, just as we think throughout
the logic, might as well do the same for the names.

-- 
Luben
P.S. Meet the man behind the keyboard, last week, Le Massif, Quebec:
http://www.splentec.com/~luben/dsc00091-re.jpg




  reply	other threads:[~2003-03-04  5:48 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-02-28 19:19 [RFC][PATCH] scsi-misc-2.5 software enqueue when can_queue reached Patrick Mansfield
2003-03-02  8:57 ` Christoph Hellwig
2003-03-02 18:15   ` Patrick Mansfield
2003-03-03 15:52   ` Randy.Dunlap
2003-03-03 18:17   ` Luben Tuikov
2003-03-04  1:11     ` Andrew Morton
2003-03-04  4:49       ` Luben Tuikov
2003-03-02 20:57 ` Luben Tuikov
2003-03-02 21:08   ` Luben Tuikov
2003-03-03 20:52   ` Patrick Mansfield
2003-03-03 22:40     ` Luben Tuikov
2003-03-03 23:41       ` Patrick Mansfield
2003-03-04  5:48         ` Luben Tuikov [this message]
2003-03-05  3:02 ` James Bottomley
2003-03-05 18:43   ` Patrick Mansfield
2003-03-06 15:57     ` James Bottomley
2003-03-06 17:41       ` Patrick Mansfield
2003-03-06 18:04         ` James Bottomley

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=3E643E1B.9080608@splentec.com \
    --to=luben@splentec.com \
    --cc=linux-scsi@vger.kernel.org \
    --cc=patmans@us.ibm.com \
    /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