From: "Eric Youngdale" <eric@andante.org>
To: "MEHTA,HIREN \(A-SanJose,ex1\)" <hiren_mehta@agilent.com>,
<linux-scsi@vger.kernel.org>, <linux-kernel@vger.kernel.org>
Subject: Re: who decrements can_queue in Scsi_Host structure ?
Date: Tue, 17 Jul 2001 17:57:38 -0400 [thread overview]
Message-ID: <004e01c10f0b$7ed500b0$4d0310ac@fairfax.mkssoftware.com> (raw)
In-Reply-To: <FEEBE78C8360D411ACFD00D0B7477971880AED@xsj02.sjs.agilent.com>
> /*
> * THis determines if we will use a non-interrupt driven
> * or an interrupt driver scheme, It is set to the maximum number
> * of simultaneous commands a given host adapter will accept.
> */
> int can_queue;
>
> Scsi_Host structure also has the same variable which gets initialized
> with the can_queue of SHT structure in the scsi_register().
> Let's say the can_queue in initialized to 16. Now, I could not find
> anybody decrementing this variable. So, when the time comes to
> send a new command to the hba, the can_queue of the host sturcture is
> checked and if that is non-zero, then the queuecommand() entry point
> is immediately called. So, the queuecommand will always be called
> even if the host is already given 'can_queue' commands.
>
> Any inputs on this ?
It should never be decremented or incremented - this one should remain
relatively static. The host_busy field indicates the number of commands
currently running on the HBA.
I am seeing a bug however - in the event that can_queue is 0, the theory
was that we would go ahead and send the command down anyways and if the HBA
driver was busy it would simply reject it. Instead we are falling through
into the ->command interface, which isn't at all what we want.
-Eric
-Eric
prev parent reply other threads:[~2001-07-17 22:00 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-07-17 21:35 MEHTA,HIREN (A-SanJose,ex1)
2001-07-17 21:57 ` Eric Youngdale [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='004e01c10f0b$7ed500b0$4d0310ac@fairfax.mkssoftware.com' \
--to=eric@andante.org \
--cc=hiren_mehta@agilent.com \
--cc=linux-kernel@vger.kernel.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
all inboxes | Powered by JetHome®