Commit Graph

542 Commits

Author SHA1 Message Date
Vladislav Bolkhovitin
3d4486cfb9 The patch below fixes a few more warnings about lines that exceed 80 columns.
This patch has been verified by rereading the patch carefully, and by verifying
that SCST source code still compiles fine.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@553 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-30 18:59:23 +00:00
Vladislav Bolkhovitin
75d2a4b1a2 The patch below adds the first version of a script that runs regression tests
to the SCST source tree. All tests are run in an automated fashion, but result
interpretation is not yet completely automated.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@552 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-30 12:43:20 +00:00
Vladislav Bolkhovitin
0b2be958e9 The patch below makes the srpt driver compile against the 2.6.27 kernel
headers.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
Acked-by: Vu Pham <vu@mellanox.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@551 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-30 12:34:48 +00:00
Vladislav Bolkhovitin
b35c83dbe2 Fixes wrong sess_count decrement on session registration failure
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@549 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-30 12:22:48 +00:00
Vladislav Bolkhovitin
fe84b1f26b The iscsi-scst kernel module does not get compiled even with CONFIG_SCST_ISCSI
set to m or y. The patch below fixes this.

This patch has been verified by rerunning scripts/run-regression-tests -k 2.6.27.3
and by verifying that the kernel build produced a iscsi-scst.ko kernel module.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@548 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-28 20:43:38 +00:00
Vladislav Bolkhovitin
d5dc2fed6a By testing kernel patch generation for the mpt driver I found out that the
Kconfig and Makefile were read from the wrong directory. The patch below fixes
this.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@547 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-28 20:41:29 +00:00
Vladislav Bolkhovitin
7782dc0768 Release/debug/perf patches updated
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@546 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-28 20:38:49 +00:00
Vladislav Bolkhovitin
0411713796 Small improvement
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@545 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-28 20:30:47 +00:00
Vladislav Bolkhovitin
3fc14278b5 Fixed accidentally enabled superfluous warnings.
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@544 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-28 20:19:07 +00:00
Vladislav Bolkhovitin
905eb033a4 Release/debug/perf patches updated + cosmetics in scst/src/dev_handlers/Makefile
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@543 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-25 18:49:47 +00:00
Vladislav Bolkhovitin
54b2a0a5b9 -Wno-missing-field-initializers added. Small cleanup.
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@542 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-25 17:31:10 +00:00
Vladislav Bolkhovitin
c76083f036 Fixes 2 incorrectly used variables
Signed off by: Richard Sharpe <realrichardsharpe@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@541 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-25 17:14:08 +00:00
Vladislav Bolkhovitin
79e315a82f A typo fixed
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@540 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-24 15:04:20 +00:00
Vladislav Bolkhovitin
2688d2b8c4 The patch below fixes the sparse warnings in ib_srpt.c about missing
declarations by inserting the keyword static where appropriate.

This patch has been verified by rereading the patch carefully, and by verifying
that SCST source code still compiles fine.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@539 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-24 12:14:11 +00:00
Vladislav Bolkhovitin
6077bb04e5 The patch below fixes more checkpatch warnings about lines that are over
eighty columns, and more sparse warnings about missing declarations by
inserting the keyword static where appropriate. A sparse warning about a
shadowed variable (ucmd) is also fixed.

This patch has been verified by rereading the patch carefully, and by verifying
that SCST source code still compiles fine.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@538 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-24 12:11:17 +00:00
Vladislav Bolkhovitin
6544d6716b - Minor context cleanup
- Minor doc update


git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@537 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-24 11:52:12 +00:00
Vladislav Bolkhovitin
20658918dc Web updates
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@536 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-24 11:50:36 +00:00
Dotan Barak
d7d2f38d5e Fix missing removal of DECLARE_MUTEX_UNLOCKED noticed by Shai.
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@535 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-24 00:19:11 +00:00
Vladislav Bolkhovitin
8e234f8824 Fixes problems spotted by lockdep
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@534 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-23 12:06:39 +00:00
Vladislav Bolkhovitin
7f62d55dde Replace SCSI_TRY_RESET_BUS by SCSI_TRY_RESET_TARGET, which is more appropriate for TARGET RESET TM
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@533 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-23 12:03:22 +00:00
Dotan Barak
d247bbbd5e Get rid of obsolete DECLARE_MUTEX_LOCKED usage.
Call device_create with extra driver data argument (NULL)
on 2.6.27 and later.


git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@532 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-22 21:45:16 +00:00
Vladislav Bolkhovitin
80ea13642f Patch from Bart Van Assche <bart.vanassche@gmail.com> with some minor changes/additions:
The patch below fixes the checkpatch warnings about lines that are over eighty
columns, and the sparse warnings about missing declarations by inserting the
keyword static where appropriate.

This patch has been tested by rereading the patch carefully, and by verifying
that SCST source code still compiles fine.

Also extracheck in_interrupt() was added in scst_cmd_atomic()



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@531 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-22 18:45:05 +00:00
Vladislav Bolkhovitin
9af236707e The patch below implements the following changes:
- Add a dummy Signed-off-by line to the generated patch, such that checkpatch
  doesn't complain that it's missing.
- Sort the names of source files (through ls) before using these, such that
  the generated patch is the same during each run.
- Fixed a bug in #elif handling (this bug is not triggered by the current SCST
  code).

This patch has been tested by comparing the output of the old and the new
generate-kernel-patch script.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@530 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-22 17:47:28 +00:00
Vladislav Bolkhovitin
1eeffbb3da The first approximation to update to 2.6.27 + minor fixes, particularly remove of unused variable and add full path to depmod in Makefiles. Partially done by Bart Van Assche <bart.vanassche@gmail.com>.
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@529 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-21 11:51:52 +00:00
Vladislav Bolkhovitin
66bc3f06a3 The patch below implements the following changes:
- Converted tabs to spaces in the help text.
- Removed command-line option '-s' -- the srpt driver is now always included.
- Recognizes now both kernel versions with three components (x.y.z) and kernel
  versions with four components (x.y.z.p).
- Empty files for Kconfig and Makefile are added for those components that are
  not included in the kernel patch, such that make allyesconfig
  / make allmodconfig keeps working.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@528 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-21 10:23:42 +00:00
Vladislav Bolkhovitin
2c112a2544 Property svn:ignore set
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@527 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-21 10:19:05 +00:00
Vladislav Bolkhovitin
3fb2634b6e scst_local module added.
Signed-off-by: Richard Sharpe <realrichardsharpe@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@526 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-16 19:23:22 +00:00
Vladislav Bolkhovitin
775ddf6f18 The patch below implements the following changes, because an
upcoming checkpatch version will complain about these:
* Make sure that a single tab is used for indenting instead of four spaces.
* Removed spaces from around the colon used for specifying field widths.
* Remove trailing whitespace in .txt files.

This patch has been tested by verifying that:
- the output of svn diff -x -w is empty, i.e. that the patch only affects
  whitespace.
- the SCST out-of-tree source code still compiles fine against the 2.6.25 kernel
  (make -s clean && make -C srpt -s clean && make -s scst iscsi-scst && make -C srpt -s).

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@525 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-16 17:46:37 +00:00
Vladislav Bolkhovitin
54cfaac7ee Compiler warning fixed
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@524 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-16 17:42:54 +00:00
Vladislav Bolkhovitin
ad6366bbf9 A major cleanup of sending commands for execution code path fixing found problems in it: handling double UA and internal REQUEST SENSE + possible wrong context assignment. Plus several other minor cleanups and fixes.
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@523 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-16 17:34:33 +00:00
Vladislav Bolkhovitin
d747aa2fa4 Allows enabling target mode support without SCST enabled. Otherwise it is impossible to use the target driver with out of tree SCST.
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@522 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-10 10:00:08 +00:00
Vladislav Bolkhovitin
fdf6e710e1 Unused files removed as requested by Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@521 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-09 17:58:04 +00:00
Vladislav Bolkhovitin
6181fda851 Fixes SAM violation, noticed by Sudhakar <ssudhakarp@gmail.com>. Plus, Windows requires it to automatically support LUNs >8, see http://support.microsoft.com/kb/310072/en-us
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@520 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-08 17:59:11 +00:00
Vladislav Bolkhovitin
50af04800a Cleanups
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@519 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-08 17:36:08 +00:00
Vladislav Bolkhovitin
2f3236e013 Until now the kernel config option to enable the SRP target was called
CONFIG_INFINIBAND_SRPT. This is a reminiscent from the time when the SRPT code
resided in the directory drivers/infiniband/ulp/srpt. Since this code has been moved
into the directory drivers/scst/srpt, the path below renames this config option
into CONFIG_SCST_SRPT. The other changes present in this patch are:
- Removed srpt/src/Makefile.infiniband.Linux-2.6.2[45].patch since these files are obsolete.
- Rewrote Kconfig help text slightly.

The patch below has been tested by verifying that the 2.6.26.5 kernel + SCST
patch still compiles cleanly with CONFIG_SCST_SRPT=m.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@518 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-08 13:44:27 +00:00
Vladislav Bolkhovitin
3e3424fb4d Web fixes and updates
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@517 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-07 14:56:42 +00:00
Vladislav Bolkhovitin
38897729bd The patch below enables compilation of the SRPT driver with the 2.6.26.x kernel.
This patch has been tested by verifying that:
- the patched Linux 2.6.25.17 and 2.6.26.5 kernel trees still compile when
  CONFIG_INFINIBAND_SRPT=m is present in .config (verified that
  make -k modules only complains about a missing qla2xxx-target directory).
- the SCST out-of-tree source code still compiles fine against the 2.6.25 kernel
  (make -s clean && make -C srpt -s clean && make -s scst iscsi-scst && make -C srpt -s).
- checkpatch output is identical for the output of scripts/generate-kernel-patch 2.6.25
  and scripts/generate-kernel-patch 2.6.26.

Based on patch from Vu Pham <vu@mellanox.com>

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@516 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-07 08:57:46 +00:00
Vladislav Bolkhovitin
3c270e713f The patch below fixes the following class of checkpatch warnings:
WARNING: externs should be avoided in .c files

This patch has been tested by verifying that the SCST sources still compile
fine (make -s clean && make -C srpt -s clean && make -s scst iscsi-scst && make -C srpt -s).

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@515 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-06 17:24:49 +00:00
Vladislav Bolkhovitin
01f7f50dc4 The patch below fixes the following class of checkpatch warnings:
WARNING: %Ld/%Lu are not-standard C, use %lld/%llu

This patch has been tested by verifying that the SCST sources still compile
fine (make -s clean && make -C srpt -s clean && make -s scst iscsi-scst && make -C srpt -s).

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@514 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-06 17:22:17 +00:00
Vladislav Bolkhovitin
a6be4c1abe The patch below fixes the following checkpatch error:
ERROR: return is not a function, parentheses are not required

This patch has been tested by verifying that the newly generated patch still
applies without warnings to the 2.6.25.17 kernel and that the patched kernel
still compiles.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@513 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-06 17:19:48 +00:00
Vladislav Bolkhovitin
7b02f74ee0 The patch below fixes the following classes of checkpatch errors:
* ERROR: return is not a function, parentheses are not required
* ERROR: that open brace { should be on the previous line

This patch has been tested by verifying that the newly generated patch still
applies without warnings to the 2.6.25.17 kernel and that the patched kernel
still compiles.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@512 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-06 17:18:04 +00:00
Vladislav Bolkhovitin
eb7c33a0b0 As known the script generate-kernel-patch generates a patch for the specified
vanilla Linux kernel version. When I submitted this script for inclusion in the
SCST Subversion repository the SRPT target driver was excluded by default
because of checkpatch errors triggered by the SRPT source code. These errors
have been fixed considerable time ago. Hence the patch below makes inclusion
of the SRPT driver the default.

This patch has been tested by verifying that the generated patch still applies
cleanly to the 2.6.25.17 kernel, and that the patched kernel still compiles cleanly.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@511 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-06 17:15:08 +00:00
Vladislav Bolkhovitin
f876bba99e The patch below fixes the following two checkpatch warnings:
WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
#290: FILE: net/core/utils.c:316:
+EXPORT_SYMBOL(net_get_page_callback);
WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable
#291: FILE: net/core/utils.c:317:
+EXPORT_SYMBOL(net_put_page_callback);

This patch has been tested by verifying that the generated patch still applies
without warnings to the 2.6.25.17 kernel and that the patched kernel still
compiles.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@510 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-06 17:13:11 +00:00
Vladislav Bolkhovitin
f0da641f63 Web updates. Particularly, "Contributing" page was added
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@509 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-03 18:09:00 +00:00
Vladislav Bolkhovitin
35754e7f26 Minor fixes and changes
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@508 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-10-02 10:46:47 +00:00
Vladislav Bolkhovitin
8570113f7a VERIFY commands that comes with BYTCHK unset (i.e. no data to write)
shouldn't fail

Signed-off-by: Gal Rosen <galr@storwize.com>



git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@506 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-09-30 18:15:06 +00:00
Vladislav Bolkhovitin
c1f04d3ad9 Sponsorship related updates
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@505 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-09-30 17:29:31 +00:00
Vladislav Bolkhovitin
34687a4030 As pointed out by Arne Redlich, size of dummy_data is too small as well as some checks in noop_out_start() are not needed
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@503 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-09-29 16:58:36 +00:00
Vladislav Bolkhovitin
2d8ac6b3ee Fixes possible oops in double reset UAs handling
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@500 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-09-26 16:03:50 +00:00
Vladislav Bolkhovitin
3785302eaa Handling of double reset UAs made more reliable
git-svn-id: http://svn.code.sf.net/p/scst/svn/trunk@499 d57e44dd-8a1f-0410-8b47-8ef2f437770f
2008-09-26 16:01:53 +00:00