All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mike Christie <michaelc@cs.wisc.edu>
To: linux-scsi@vger.kernel.org, david.somayajulu@qlogic.com,
	andrew.vasquez@qlogic.com, James.Smart@Emulex.com,
	Eric.Moore@lsil.com
Subject: Re: [PATCH 1/9] scsi: Add helper code so transport classes/driver can control queueing
Date: Sun, 02 Mar 2008 03:08:13 -0600	[thread overview]
Message-ID: <47CA6E7D.8000205@cs.wisc.edu> (raw)
In-Reply-To: <1204448602-5435-1-git-send-email-michaelc@cs.wisc.edu>

Oh yeah, these patches were made over scsi-rc-fixes and can be applied 
over the target resets ones (the target reset ones can be applied over 
these too).

michaelc@cs.wisc.edu wrote:
> From: Mike Christie <michaelc@cs.wisc.edu>
> 
> SCSI-ml manages the queueing limits for the device and host, but
> does not do so at the target level. However something something similar
> can come in userful when a driver is transitioning a transport object to
> the the blocked state, becuase at that time we do not want to queue
> io and we do not want the queuecommand to be called again.
> 
> The patch adds code similar to the exisiting SCSI_ML_*BUSY handlers.
> You can now return SCSI_MLQUEUE_TARGET_BUSY when we hit
> a transport level queueing issue like the hw cannot allocate some
> resource at the iscsi session/connection level, or the target has temporarily
> closed or shrunk the queueing window, or if we are transitioning
> to the blocked state.
> 
> For bnx2i we will also need to be able to limit queueing at this level.
> bnx2i will hook into libiscsi, but will allocate a scsi host per
> netdevice/hba, so unlike pure software iscsi/iser which is allocating
> a host per session, it cannot set the scsi_host->can_queue and return
> SCSI_MLQUEUE_HOST_BUSY to reflect queueing limits on the transport.
> 
> The iscsi class/driver can also set a scsi_target->can_queue value which
> reflects the max commands the driver/class can support. For iscsi this
> reflects the number of commands we can support for each session due to
> session/connection hw limits, driver limits, and to also reflect the
> session/targets's queueing window.
> 

  parent reply	other threads:[~2008-03-02  9:08 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-03-02  9:03 [PATCH 1/9] scsi: Add helper code so transport classes/driver can control queueing michaelc
2008-03-02  9:03 ` [PATCH 2/9] iscsi: user per target can_queue and return SCSI_MLQUEUE_TARGET_BUSY michaelc
2008-03-02  9:03   ` [PATCH 3/9] iscsi class: add iscsi queueing helper michaelc
2008-03-02  9:03     ` [PATCH 4/9] qla4xxx: use SCSI_ML_TARGET_BUSY and iscsi helper michaelc
2008-03-02  9:03       ` [PATCH 5/9] libiscsi: use iscsi queueing helper michaelc
2008-03-02  9:03         ` [PATCH 6/9] fc class: add fc " michaelc
2008-03-02  9:03           ` [PATCH 7/9] qla2xxx: convert to use fc helper and SCSI_MLQUEUE_TARGET_BUSY michaelc
2008-03-02  9:03             ` [PATCH 8/9] lpfc: " michaelc
2008-03-02  9:03               ` [PATCH 9/9] mptfc: convert mptfc to use fc queueing helper michaelc
2008-03-13 18:53                 ` Michael Reed
2008-03-17 18:32                   ` Mike Christie
2008-03-17 18:39                     ` Mike Christie
2008-03-13 14:35           ` [PATCH 6/9] fc class: add " Michael Reed
2008-03-13 15:36             ` Mike Christie
2008-03-03 17:34       ` [PATCH 4/9] qla4xxx: use SCSI_ML_TARGET_BUSY and iscsi helper David C Somayajulu
2008-03-02  9:08 ` Mike Christie [this message]
2008-03-18  4:38 ` [PATCH 1/9] scsi: Add helper code so transport classes/driver can control queueing Mike Christie

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=47CA6E7D.8000205@cs.wisc.edu \
    --to=michaelc@cs.wisc.edu \
    --cc=Eric.Moore@lsil.com \
    --cc=James.Smart@Emulex.com \
    --cc=andrew.vasquez@qlogic.com \
    --cc=david.somayajulu@qlogic.com \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.