Bart Van Assche
7eee1392b8
Compiles again without warnings.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1501 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-02-12 18:59:15 +00:00
Vladislav Bolkhovitin
82e07ddb73
Commented out required by iSCSI RFC, but not needed for processing, check of LUN field in Data-Out PDU, because some initiators (Windows, at least) don't care to correctly set it.
...
Reported and tested by Ladislav Pašek <pasek@valbek.cz >
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1500 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-02-12 13:12:58 +00:00
Vladislav Bolkhovitin
8a4b3a1732
Those files were deleted moving to new pass-through implementation, but should have been resurrected when that implementation was limited to 2.6.30+ kernels only. Fixing.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1499 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-02-12 13:07:23 +00:00
Bart Van Assche
492e570df0
Made ib_srpt easier to build on a multiboot system where multiple kernels
...
have been installed -- one kernel with OFED and one or more kernels without
OFED.
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1498 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-02-11 19:28:58 +00:00
Bart Van Assche
7c04f47a6b
Fixed two MODULE_PARM_DESC() clauses.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1497 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-02-11 19:27:49 +00:00
Vladislav Bolkhovitin
b611a22ed0
Patch from Christian Motschke <motschke@itso-berlin.de>:
...
I had some problems compiling scst due to some bashisms in the Makefiles. As
you will know, Debian uses /bin/dash as default shell and /bin/sh is only a
link to /bin/dash. I can either change the default shell, (or in my opinion
the better solution) use SHELL=/bin/bash in the Makefile.
I have made a patch for the Makefiles in question (I hope, that are all
Makefiles).
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1496 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-02-10 19:32:31 +00:00
Vladislav Bolkhovitin
d880bdab9d
cmnd_reject() should return supplied reason, not hardcoded ISCSI_REASON_INVALID_PDU_FIELD
...
Noticed by Alexey Obitotskiy <alexeyo1@open-e.com >
This line, and those below, will be ignored--
M trunk/iscsi-scst/usr/iscsid.c
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1495 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-02-10 19:22:39 +00:00
Bart Van Assche
8ed82e1dbe
Updated kernel revisions.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1494 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-02-09 15:46:10 +00:00
Bart Van Assche
2032370cca
svn merge -c1492 https://scst.svn.sourceforge.net/svnroot/scst/branches/srpt-separate-rx-tx-buffers/scripts
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1493 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-02-07 09:06:03 +00:00
Vladislav Bolkhovitin
f42a81e94a
Web updates
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1490 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-02-02 17:16:58 +00:00
Bart Van Assche
0b3861da48
Added a blank line.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1483 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-29 19:09:06 +00:00
Bart Van Assche
773ec6e974
Updated kernel revisions.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1482 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-29 11:54:38 +00:00
Vladislav Bolkhovitin
28ed0ed7e3
session_active_commands_attr attribute must have active_commands name.
...
Reported by Joe Eykholt <jeykholt@cisco.com >
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1478 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-26 13:02:13 +00:00
Bart Van Assche
f7f8c13456
Updated kernel revisions.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1477 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-26 11:52:01 +00:00
Bart Van Assche
4c3a1c3e9e
Updated kernel revisions.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1476 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-23 08:54:06 +00:00
Bart Van Assche
fa91abb965
Updated kernel revisions.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1475 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-19 11:37:41 +00:00
Bart Van Assche
30fef168be
Moved comma's in table header to the proper place.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1474 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-17 17:36:50 +00:00
Bart Van Assche
421dbafa7c
Changes:
...
- The entire receive queue can be used again instead of only SRPT_RQ_SIZE - 1
elements (as in r1470).
- Added initiator-lockup-avoidance code to SRP_TSK_MGMT processing code too.
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1473 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-17 13:07:46 +00:00
Bart Van Assche
5ce3870372
Updated test instructions.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1472 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-16 19:38:08 +00:00
Bart Van Assche
e3d9724359
Updated copyright statement.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1471 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-16 19:36:29 +00:00
Bart Van Assche
c654261e03
Changes:
...
- Updated copyright statement.
- Micro-optimized the function srpt_req_lim_delta().
- Made sure that SRPT does not cause a lockup in (at least) the Linux SRP
initiator. See also http://bugzilla.kernel.org/show_bug.cgi?id=14235 .
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1470 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-14 20:35:37 +00:00
Bart Van Assche
3d9011443c
Removed a trailing space because checkpatch complained about it.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1469 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-14 13:37:56 +00:00
Bart Van Assche
6384c37e83
Enabled sparse checking for the 2.6.23.17 kernel.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1468 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-14 13:28:26 +00:00
Vladislav Bolkhovitin
5b77c89fbd
Patch from Gilad H <gilad.public2@gmail.com> with improvements fixing the following scenario:
...
1. Commands start going to the init_cmd_list probably because SCST was suspended
2. A command is now waiting in the init_cmd_list, it was not yet completely initialized (more impotently cmd->dev == NULL)
3. A clear task set for the same lun arrives from a different initiator
4. The command is marked as aborted from other initiator and released from the init list.
5. The command gets to the PRE_XMIT_RESPONSE state, since it was aborted the scst_xmit_process_aborted_cmd is called.
6. At this stage since the command is marked as aborted by other initiator the scst checks for the device TAS flag (cmd->dev->tas) however cmd->dev is NULL and we crash on Null pointer de-referencing.
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1466 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-14 13:14:30 +00:00
Vladislav Bolkhovitin
04cb08ec2d
Improve handling of aborts of done commands
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1465 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-14 12:59:13 +00:00
Bart Van Assche
ee0767206c
Removed SRP_CRED_REQ support again.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1464 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-13 20:19:54 +00:00
Bart Van Assche
354d74e675
Fixed the following category of checkpatch errors:
...
3 ERROR: code indent should use tabs where possible
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1463 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-13 19:33:21 +00:00
Vladislav Bolkhovitin
eada62f329
Fixed processing of REQUEST SENSE and INQUIRY commands for not existing LUNs
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1462 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-13 18:10:10 +00:00
Vladislav Bolkhovitin
138cd01268
Regular reservations cleanup
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1461 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-13 14:03:55 +00:00
Bart Van Assche
dc45f111c6
Fixed a compiler warning about a format specification mismatch.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1460 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-13 13:21:08 +00:00
Bart Van Assche
b9ad2bae2b
Made sure that the EXTRACHECKS_BUG_ON() macro does not trigger a compiler
...
warning about an empty controlled statement in a release or perf build.
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1459 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-13 13:20:00 +00:00
Bart Van Assche
e90930ed6d
Made sure that none of the EXTRACHECKS_BUG_ON(), EXTRACHECKS_WARN_ON()
...
or EXTRACHECKS_WARN_ON_ONCE() macros trigger a compiler warning about an
empty controlled statement in a release or perf build.
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1458 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-13 13:13:03 +00:00
Bart Van Assche
ddbaa159ca
Fixed the following checkpatch complaint:
...
1 WARNING: braces {} are not necessary for single statement blocks
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1457 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-13 08:21:39 +00:00
Vladislav Bolkhovitin
21476c4b9c
Docs about QLA FW version updated
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1456 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-12 20:10:56 +00:00
Vladislav Bolkhovitin
89345ab95c
Custom commands parsing cleanups
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1455 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-12 19:05:27 +00:00
Vladislav Bolkhovitin
e5a195516c
Link to PRs patch to test updated.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1454 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-12 15:05:25 +00:00
Vladislav Bolkhovitin
c32ef099ce
Patch from Daniel Fernandes <dfernandes1978@hotmail.com> fixing broken link to the Marvell logo after Marvell redesigned their site.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1453 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-12 14:53:33 +00:00
Vladislav Bolkhovitin
4a2fcb870b
- Logging cleanups. Particularly, TRACE_MGMT_MINOR level was removed
...
- Docs updates
- Other minor fixes and cleanups
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1452 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-12 13:29:01 +00:00
Bart Van Assche
2610bf1d0a
Fixed a race condition (use-after-free) that could trigger a kernel bug report during module unloading.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1451 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-12 08:52:08 +00:00
Vladislav Bolkhovitin
9b5aec7b7f
Web update
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1450 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-11 19:44:15 +00:00
Vladislav Bolkhovitin
5b930aa9a4
- Improved handling of commands with TASK ABORTED status
...
- Minor cleanup
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1449 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-11 19:41:23 +00:00
Vladislav Bolkhovitin
a0b91ecc55
Let's experimantally disable SCST_MAX_DEV_COMMANDS and cmd->dev_cmd_count functionality and see if users will complain about it. If not, we will remove it.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1448 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-11 19:27:07 +00:00
Vladislav Bolkhovitin
23accb2e12
When converting the SCST source code into a series of upstream patches, SCST
...
is built via the kbuild infastructure. In kernel version 2.6.24 (commit
f77bf01425b11947eeb3b5b54685212c302741b8) the kbuild variable ccflags-y has
been introduced and the use of EXTRA_CFLAGS in kbuild makefiles has been
deprecated. The patch below converts SCST's in-tree makefiles such that these
use ccflags-y instead of EXTRA_CFLAGS when generating kernel patches for
kernel 2.6.24 or later.
Additionally, this patch removes the "default n" from the file
scst/kernel/in-tree/Kconfig.scst since this statement is superfluous.
The patch below has been tested by verifying the output of the following
command:
for ((v=23;v<=32;v++)); do rm -rf re* && scripts/run-regression-tests 2.6.$v; done
Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com >
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1447 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-11 19:16:38 +00:00
Bart Van Assche
b85002b263
Made sure that if a work completion reports an error status that
...
completion queue processing is not aborted but that the remaining
completion queue entries are processed.
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1446 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-11 19:00:03 +00:00
Bart Van Assche
530ecf06c2
Converted a WARN_ON() into a PRINT_ERROR() statement.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1445 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-11 18:41:55 +00:00
Bart Van Assche
661ecd52d2
Close IB channel after session unregistration instead of before in order
...
to prevent the following errors to occur:
ib_srpt: srpt_completion:1737:***ERROR***: failed sending response status= 4
ib_srpt: srpt_completion:1737:***ERROR***: failed sending response status= 5
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1444 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-11 11:30:08 +00:00
Bart Van Assche
0cd53b65ef
Made sure that the pre-cflags patches are uniform to the other in-tree kernel
...
patches, i.e. that -p1 has to be used to apply these patches.
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1443 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-10 13:16:28 +00:00
Bart Van Assche
048a338747
Backported some 2.6.32 IB bug fixes to the 2.6.30 kernel.
...
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1442 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-10 13:14:28 +00:00
Bart Van Assche
5cc522712e
Fixed the following race conditions:
...
- With thread=1, the kernel thread was stopped before all I/O was stopped.
This could result in IB completions for responses sent to the initiator
not being processed, scst_tgt_cmd_done() not being called for certain
SCST commands and hence this race could cause "rmmod ib_srpt" to hang.
- ib_unregister_event_handler() was called after cancel_work_sync(). This
could cause srpt_event_handler() to queue new work, work that could be
executed after the data it operates on had been deallocated.
Change: simplified module unloading by calling ib_destroy_cm_id(sdev->cm_id)
before calling scst_unregister(). This made the variable "cleaning_up"
superfluous.
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1441 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-09 20:14:13 +00:00
Bart Van Assche
1660fe1c3c
Fixed the following category of checkpatch complaints:
...
2 ERROR: code indent should use tabs where possible
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@1440 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2010-01-09 15:01:59 +00:00