15 Dec, 2011
19 commits
-
One ';' at the end of the return statement is enough.
Signed-off-by: Jesper Juhl
Acked-by: Chad Dupuis
Signed-off-by: James Bottomley -
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Some variables need to be initialized before we config PCI I/O config space
or else strange firmware initialization errors may occur.Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
The driver should ensure there's a sufficient number of IOCBs
to satisfy the number of scatter-gather entries specified in the
command. Add a 'count' to the control structure, srb_ctx, to use
in qla2x00_alloc_iocbs().Signed-off-by: Andrew Vasquez
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
This provides a capability to set a priority in the FCP packet.
Signed-off-by: Saurav Kashyap
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Signed-off-by: Giridhar Malavali
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
When multiple request queues are created process the marker IOCB request
on request queue 0.Signed-off-by: Giridhar Malavali
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Add a new function to ISP specific pointers structure to take care of ISP
specific PCI IO space configuration.Signed-off-by: Giridhar Malavali
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Signed-off-by: Giridhar Malavali
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Signed-off-by: Giridhar Malavali
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Correct the erroneous return codes introduced by the following patch:
"Return sysfs error codes appropriate to conditions".Signed-off-by: Joe Carnuccio
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Signed-off-by: Giridhar Malavali
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Signed-off-by: Giridhar Malavali
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
The 'continue' cases neglected to place the thread in an
interruptible state, causing the DPC routine to wake immediately.Signed-off-by: Andrew Vasquez
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
When reading the incoming mailbox registers, read only the specified ones.
Signed-off-by: Andrew Vasquez
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
[jejb: fixed up checkpatch and casting errors]
Signed-off-by: Giridhar Malavali
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
This patch contains minor fixes to our new logging infrastructure:
- Remove extranous messages.
- Re-add 'nexus' and 'hdl' information.
- Adjusted the message ids to fill up the holes.
- Display FCP_CMND priority on update.
- Log only mail box error conditions.
- Do not print "Firmware ready **** FAILED ****" if cable is unplugged.
- Drop noisy 'fw_state...curr time...' message.
- Correct nexus display during abort.
- Add a special case error-logging set to '1'.
- Catagorize I/O exception display handling.
- Correct the bsg msg code printing.
- Dont use dynamic logging after host is removed.Signed-off-by: Saurav Kashyap
Signed-off-by: Andrew Vasquez
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
…current broken uses as appropriate.
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Saurav Kashyap <saurav.kashyap@qlogic.com>
Signed-off-by: Chad Dupuis <chad.dupuis@qlogic.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com> -
Return early when logging level too low.
Use normal kernel codeing style for function braces.
Add const where appropriate to logging functions.
Remove now unused #define QL_DBG_BUF_LEN.Signed-off-by: Joe Perches
Signed-off-by: Saurav Kashyap
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley
12 Dec, 2011
14 commits
-
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Signed-off-by: Giridhar Malavali
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
[jejb: checkpatch fixes]
Add more fine grain parsing of vha->loop_state to export a more accurate
fc_host port_state.Signed-off-by: Saurav Kashyap
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Signed-off-by: Giridhar Malavali
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Signed-off-by: Giridhar Malavali
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Signed-off-by: Giridhar Malavali
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
[jejb: fixup checkpatch error]
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
We need to return QLA_FUNCTION_TIMEOUT immediately otherwise we mess up the
mailbox command state machine.Signed-off-by: Andrew Vasquez
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Signed-off-by: Giridhar Malavali
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
…rrupt mode during firmware hang.
Signed-off-by: Giridhar Malavali <giridhar.malavali@qlogic.com>
Signed-off-by: Chad Dupuis <chad.dupuis@qlogic.com>
Signed-off-by: James Bottomley <JBottomley@Parallels.com> -
If there is an error creating multiple response queues then we need to revert
the request queue mapping back to request queue 0.Signed-off-by: Giridhar Malavali
Signed-off-by: Andrew Vasquez
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Signed-off-by: Saurav Kashyap
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Signed-off-by: Arun Easi
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
This function can wait for 5min under certain scenarios. One of them is when
the port is down from switch and bus reset is issued. The bus reset used to
wait for 5 minutes for the loop and upper layer callers used to hang and give
stack trace because of getting stuck for 120 sec. It is legacy code that was
used when the driver used to do queuing of the commands.Signed-off-by: Saurav Kashyap
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley
30 Oct, 2011
1 commit
-
During heavy I/O (CPU-affinity mode enabled) and CLI/Agent
interactions, the driver would report periodic mailbox command
timeout statuses. Within the CPU-affinity ISR handler, the
driver should check the 'disable-msix-handshake' flag in deciding
whether or not to clear HCCRX_CLR_RISC_INT. The mode is not
specific to a dedicated queue, instead, applies to the current
'ha' context.Signed-off-by: Andrew Vasquez
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley
29 Oct, 2011
1 commit
-
* git://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi-misc-2.6: (204 commits)
[SCSI] qla4xxx: export address/port of connection (fix udev disk names)
[SCSI] ipr: Fix BUG on adapter dump timeout
[SCSI] megaraid_sas: Fix instance access in megasas_reset_timer
[SCSI] hpsa: change confusing message to be more clear
[SCSI] iscsi class: fix vlan configuration
[SCSI] qla4xxx: fix data alignment and use nl helpers
[SCSI] iscsi class: fix link local mispelling
[SCSI] iscsi class: Replace iscsi_get_next_target_id with IDA
[SCSI] aacraid: use lower snprintf() limit
[SCSI] lpfc 8.3.27: Change driver version to 8.3.27
[SCSI] lpfc 8.3.27: T10 additions for SLI4
[SCSI] lpfc 8.3.27: Fix queue allocation failure recovery
[SCSI] lpfc 8.3.27: Change algorithm for getting physical port name
[SCSI] lpfc 8.3.27: Changed worst case mailbox timeout
[SCSI] lpfc 8.3.27: Miscellanous logic and interface fixes
[SCSI] megaraid_sas: Changelog and version update
[SCSI] megaraid_sas: Add driver workaround for PERC5/1068 kdump kernel panic
[SCSI] megaraid_sas: Add multiple MSI-X vector/multiple reply queue support
[SCSI] megaraid_sas: Add support for MegaRAID 9360/9380 12GB/s controllers
[SCSI] megaraid_sas: Clear FUSION_IN_RESET before enabling interrupts
...
03 Oct, 2011
1 commit
-
I hit a crash in qla2x00_abort_all_cmds() if the qla2xxx module is
unloaded right after it is loaded. I debugged this down to the abort
handling improperly treating a command of type SRB_ADISC_CMD as if it
had a bsg_job to complete when that command actually uses the iocb_cmd
part of the union. (I guess to hit this one has to unload the module
while the async FC initialization is still in progress)It seems we should only look for a bsg_job if type is SRB_ELS_CMD_RPT,
SRB_ELS_CMD_HST or SRB_CT_CMD, so switch the test to make that explicit.Signed-off-by: Roland Dreier
Acked-by: Chad Dupuis
Cc: stable@kernel.org
Signed-off-by: James Bottomley
24 Sep, 2011
1 commit
-
sector_t can be different types, so cast it to its largest possible
type.drivers/scsi/qla2xxx/qla_isr.c:1509:5: warning: format '%lx' expects type 'long unsigned int', but argument 5 has type 'sector_t'
Signed-off-by: Randy Dunlap
Signed-off-by: Linus Torvalds
29 Aug, 2011
3 commits
-
Since active_mask is first assigned in an "else" block, the compiler throws
a warning saying that the active_mask variable may be used uninitialized in
a print statement later. Initialize active_mask to 0 in the declaration to
stop the warning.Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Signed-off-by: Giridhar Malavali
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley -
Return sysfs error codes that match the conditions encountered.
This prevents sysfs from retrying a request which is conditioned
to fail under the current circumstance.Signed-off-by: Joe Carnuccio
Signed-off-by: Chad Dupuis
Signed-off-by: James Bottomley