Re: [GIT PATCH] final SCSI fixes for 2.6.15-rc5

From: Michael Reed
Date: Wed Dec 14 2005 - 19:19:41 EST


If possible, a comment on my previous email which identified two
other issues with SCSI would be appreciated before 2.6.15 goes
final.

Thanks,
Mike


> "Current Issues with 2.6.15-rc5" sent today:

> Hello Everyone,
>
> I'm concerned about the stability of the fibre channel infrastructure
> in 2.6.15-rc. There are current issues which I see effecting our (SGI's)
> customers' satisfaction. I feel that these issues must be addressed
> prior to the release of 2.6.15.
>
> 1 - fc transport change gets really noisy if targets disappear from
> the fabric due to recursion on the workqueue. I've seen recursion
> depths > 40 and it's only really limited by the number of targets
> on a fibre channel fabric. It's unknown (to me) what side effects
> this recursion might have if the number of targets is in the
> hundreds, which is not uncommon for our customers. James Smart
> is aware of this and is working on a solution.
> See my posting of 12/2/2005 and replies.
> "2.6.15-rc4 error messages with multiple qla2300 hba ports on fabric".
>
> 2 - qlogic fibre channel driver no longer properly handles targets coming
> and going. There is a race condition within the driver which can
> result in targets being deleted yet they are present and accounted for.
> This is a show stopper for SGI's customers. We have to have reliable
> target [re-]discovery. Andrew Vasquez is aware of this issue.
> See my posting on 12/5/2005. "qla2x00 driver serialization issue".
>
> In a private email, Andrew wrote:
>> > Yes, I was afraid of something like this happening (due to qla2xxx
>> > detections of ports being dropped from an interrupt context)... Codes
>> > in qla_rscn.c already queue-up rport_adds() via the standard
>> > kernel-workqueue:
>> >
>> > qla_init.c: INIT_WORK(&fcport->rport_add_work, qla2x00_rport_add, fcport);
>> >
>> > qla_rscn.c: schedule_work(&fcport->rport_add_work);
>> >
>> > With the additions though, I'm wondering if adding a special
>> > single-cpu qla2xxx-rport workqueue would make sense (at least we could
>> > enforce serialization).
>
>
> What can be done to assure fc stability in the 2.6.15 release? It appears
> that the revision of the fc-transport from 2.6.14 has caused some
> difficulty for a current driver, and potentially the entire system. Is
> it ready for prime time?
>
> Mike Reed
>


James Bottomley wrote:
> These should (hopefully) represent the last few urgent bug fixes that
> have shown up. The fixes are available here:
>
> master.kernel.org:/pub/scm/linux/kernel/git/jejb/scsi-rc-fixes-2.6.git
>
> The short changelog is:
>
> Andrew Vasquez:
> o qla2xxx: Correct short-WRITE status handling
> o qla2xxx: Correct mis-handling of AENs
>
> Brian King:
> o fix double free of scsi request queue
>
> Dave Boutcher:
> o ibmvscsi kexec fix
>
> James Bottomley:
> o Consolidate REQ_BLOCK_PC handling path (fix ipod panic)
>
> Jens Axboe:
> o fix panic when ejecting ieee1394 ipod
>
> Mark Lord:
> o Fix incorrect pointer in megaraid.c MODE_SENSE emulation
>
> Matthew Wilcox:
> o Negotiate correctly with async-only devices
>
> Michael Reed:
> o fix OOPS due to clearing eh_action prior to aborting eh command
>
> And the diffstat:
>
> drivers/scsi/ibmvscsi/ibmvscsi.h | 2 +-
> drivers/scsi/ibmvscsi/iseries_vscsi.c | 3 ++-
> drivers/scsi/ibmvscsi/rpa_vscsi.c | 8 +++++++-
> drivers/scsi/megaraid.c | 2 +-
> drivers/scsi/qla2xxx/qla_def.h | 10 +---------
> drivers/scsi/qla2xxx/qla_init.c | 6 +++---
> drivers/scsi/qla2xxx/qla_isr.c | 15 +++++++++++++++
> drivers/scsi/scsi_error.c | 7 ++++++-
> drivers/scsi/scsi_lib.c | 33 +++++++++++++++++++++------------
> drivers/scsi/scsi_scan.c | 1 -
> drivers/scsi/sd.c | 16 +---------------
> drivers/scsi/sr.c | 20 +++-----------------
> drivers/scsi/st.c | 19 +------------------
> drivers/scsi/sym53c8xx_2/sym_hipd.c | 4 ++--
> include/scsi/scsi_cmnd.h | 1 +
> 15 files changed, 65 insertions(+), 82 deletions(-)
>
> James
>
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/