Commit f8f91f3f authored by Martin K. Petersen's avatar Martin K. Petersen

scsi: libfc: Revert "[SCSI] libfc: use offload EM instance again instead jumping to next EM"

This reverts commit 3e22760d.

This revert came about because of efforts by Ewan Milne, Curtis Taylor
and I.  In researching this issue, significant performance issues were
seen on large CPU count systems using the software FCOE stack.  Hannes
also weighed in.

The same was not apparent on much smaller low count CPU systems.  The
behavior introduced by commit 3e22760d
lands sup with large count CPU systems seeing continual
blk_requeue_request() calls due to ML_QUEUE_HOST_BUSY.

fc_exch_alloc() used to try all the available exchange managers in the
list for an available exchange id, but this was changed in 2010 so that
if the first matched exchange manager couldn't allocate one, it fails
and we end up returning host busy.  This was due to commit:

Setting the ddp_min module parameter to fcoe to 128MB prevents the
->match function from permitting the use of the offload exchange manager
for the frame, and we no longer see the problem with host busy status,
since it uses the larger non-offloaded pool.

Reverting commit 3e22760d was tested to
also prevent the host busy issue due to failing allocations.
Suggested-by: default avatarEwan Milne <emilne@redhat.com>
Suggested-by: default avatarCurtis Taylor <cjt@us.ibm.com>
Tested-by: default avatarLaurence Oberman <loberman@redhat.com>
Signed-off-by: default avatarLaurence Oberman <loberman@redhat.com>
parent c74f95d6
...@@ -888,14 +888,19 @@ static struct fc_exch *fc_exch_em_alloc(struct fc_lport *lport, ...@@ -888,14 +888,19 @@ static struct fc_exch *fc_exch_em_alloc(struct fc_lport *lport,
* EM is selected when a NULL match function pointer is encountered * EM is selected when a NULL match function pointer is encountered
* or when a call to a match function returns true. * or when a call to a match function returns true.
*/ */
static inline struct fc_exch *fc_exch_alloc(struct fc_lport *lport, static struct fc_exch *fc_exch_alloc(struct fc_lport *lport,
struct fc_frame *fp) struct fc_frame *fp)
{ {
struct fc_exch_mgr_anchor *ema; struct fc_exch_mgr_anchor *ema;
struct fc_exch *ep;
list_for_each_entry(ema, &lport->ema_list, ema_list) list_for_each_entry(ema, &lport->ema_list, ema_list) {
if (!ema->match || ema->match(fp)) if (!ema->match || ema->match(fp)) {
return fc_exch_em_alloc(lport, ema->mp); ep = fc_exch_em_alloc(lport, ema->mp);
if (ep)
return ep;
}
}
return NULL; return NULL;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment