14 years agoAdd the "static" qualifier to function that isn't being declared
vlnb [Wed, 9 Jul 2008 17:35:11 +0000 (17:35 +0000)]
Add the "static" qualifier to function that isn't being declared
(in h file) or exported to fix the following sparse warning:

/scst/trunk/iscsi-scst/kernel/iscsi.c:166:6: warning: symbol
'cmnd_free' was not declared. Should it be static?

Signed-off-by: Dotan Barak <dotanba@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@448 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago- Added a proper Makefile.PL to SCST::SCST so now we have a also a
mgandalf [Wed, 9 Jul 2008 17:32:42 +0000 (17:32 +0000)]
- Added a proper Makefile.PL to SCST::SCST so now we have a also a
  proper man page.
- Renamed Changes to ChangeLog
- Updated README to include instructions for installing the perl module.
- Renamed SCST perl module directory tp scst-0.7.4

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@447 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoChange the name of local variable to prevent the following compilation warning:
vlnb [Wed, 9 Jul 2008 17:32:16 +0000 (17:32 +0000)]
Change the name of local variable to prevent the following compilation warning:
/scst/trunk/iscsi-scst/kernel/iscsi.c:190:30: warning: symbol 'req'
shadows an earlier one
/scst/trunk/iscsi-scst/kernel/iscsi.c:182:29: originally declared here

Signed-off-by: Dotan Barak <dotanba@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@446 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoThe patch below implements the following changes:
vlnb [Wed, 9 Jul 2008 17:28:36 +0000 (17:28 +0000)]
The patch below implements the following changes:
- The script specialize-patch now acts as if #define INSIDE_KERNEL_TREE 1
  is effective.
- The script generate-kernel-patch now renames sBUG_ON() into BUG_ON()
  and sBUG() into BUG(). The same script removes the definitions of the
  sBUG_ON(), sBUG() and WARN_ON_ONCE() macro's from the header file
  scst_debug.h. The last macro is only removed for kernel versions later than

This patch has been tested as follows:
- Compared the output of the command "scripts/generate-kernel-patch 2.6.25"
  before and after the changes.
- Verified the newly generated patch with checkpatch. Checkpatch statistics
  are now as follows:
    521 WARNING: line over 80 characters
     12 WARNING: consider using strict_strtoul in preference to simple_strtoul
     10 WARNING: printk() should include KERN_ facility level
      3 WARNING: CVS style keyword markers, these will _not_ be updated
      2 WARNING: do not add new typedefs
      1 WARNING: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
- Verified that the generated kernel patch still applies cleanly to the kernel and that this kernel still compiles, installs and boots fine.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@445 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoThe patch below contains the following changes:
vlnb [Wed, 9 Jul 2008 17:19:24 +0000 (17:19 +0000)]
The patch below contains the following changes:
- Substituted lun_t by uint64_t and removed typedef lun_t.
- Fixed the following class of checkpatch errors (two instances):
    ERROR: space required before the open parenthesis '('

This patch has been verified as follows:
- Checked that the following command still works and did not produce any
  new compiler warnings:
    make -s clean && make -C srpt -s clean && make -s scst iscsi-scst && make -C srpt -s
- Checked that the patch generated by generate-kernel-patch still applies
  cleanly to the kernel, and that the patched kernel tree still
  compiles, installs and boots fine, and that the iscsi-scst, ib_srpt,
  scst_disk and scst_vdisk modules still load.
- Verified that checkpatch does not report any new warnings or errors.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@444 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - Forgotten piece of changes for returning to the release mode
vlnb [Wed, 9 Jul 2008 16:58:56 +0000 (16:58 +0000)]
 - Forgotten piece of changes for returning to the release mode
 - QLA_ISP targets in the main Makefile temporary disabled, since they don't support building with specified kernel version

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@443 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - Version changed to 1.0.1
vlnb [Wed, 9 Jul 2008 16:44:01 +0000 (16:44 +0000)]
 - Version changed to 1.0.1
 - Switched back to the debug mode

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@442 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoTag 1.0.0 added
vlnb [Wed, 9 Jul 2008 09:46:36 +0000 (09:46 +0000)]
Tag 1.0.0 added

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@440 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoCosmetics
vlnb [Wed, 9 Jul 2008 09:14:21 +0000 (09:14 +0000)]

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@439 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoCosmetics
vlnb [Wed, 9 Jul 2008 07:00:50 +0000 (07:00 +0000)]

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@438 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - Known issues added
vlnb [Wed, 9 Jul 2008 06:53:41 +0000 (06:53 +0000)]
 - Known issues added
 - Cosmetics

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@437 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPending changes to match the 1.0.0 released packeges
vlnb [Tue, 8 Jul 2008 19:21:22 +0000 (19:21 +0000)]
Pending changes to match the 1.0.0 released packeges

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@436 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoWEB updates for release 1.0.0
vlnb [Tue, 8 Jul 2008 16:41:52 +0000 (16:41 +0000)]
WEB updates for release 1.0.0

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@435 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoAdd release2debug and debug2release Makefile targets for qla_isp.
swgruszka [Tue, 8 Jul 2008 07:59:42 +0000 (07:59 +0000)]
Add release2debug and debug2release Makefile targets for qla_isp.
Leave qla_isp in release 'state' as other stuff.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@434 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - Version changed to 1.0.0
vlnb [Mon, 7 Jul 2008 18:00:32 +0000 (18:00 +0000)]
 - Version changed to 1.0.0
 - Build configuration changed to "Release"
 - iscsi-scst-howto.txt added
 - ChangeLog's updated
 - Minor docs changes
 - scstadm_install and scstadm_uninstall main Makefile targets added

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@433 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - Comment about 2008 Linux Storage & Filesystem Workshop added
vlnb [Mon, 7 Jul 2008 17:54:49 +0000 (17:54 +0000)]
 - Comment about 2008 Linux Storage & Filesystem Workshop added
 - scst_user_spec.txt removed, it was committed here by mistake
 - Minor changes

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@432 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - Install/uninstall added
vlnb [Mon, 7 Jul 2008 17:51:53 +0000 (17:51 +0000)]
 - Install/uninstall added
 - Version fixed

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@431 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoAdd uninstall makefile target.
swgruszka [Sun, 6 Jul 2008 10:45:25 +0000 (10:45 +0000)]
Add uninstall makefile target.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@430 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoCopy Modules.symvers stuff from qla2x00t makefiles to avoid undefined symbols warrning.
swgruszka [Sat, 5 Jul 2008 20:19:11 +0000 (20:19 +0000)]
Copy Modules.symvers stuff from qla2x00t makefiles to avoid undefined symbols warrning.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@429 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoAdd warrning qla_isp do not support > 2.6.24
swgruszka [Sat, 5 Jul 2008 19:11:16 +0000 (19:11 +0000)]
Add warrning qla_isp do not support > 2.6.24

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@428 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoAdd "extraclean" entry to the qla_isp Makefiles.
vlnb [Fri, 4 Jul 2008 17:53:30 +0000 (17:53 +0000)]
Add "extraclean" entry to the qla_isp Makefiles.

Signed-off-by: Dotan Barak <dotanba@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@427 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoAdded qla_isp support to the general Makefile.
vlnb [Wed, 2 Jul 2008 17:43:20 +0000 (17:43 +0000)]
Added qla_isp support to the general Makefile.

The following operations (parametgers to the make) are now supported:
qla_isp, qla_isp_clean, qla_isp_extraclean,
qla_isp_install, qla_isp_uninstall.

But at the moment qla_isp doesn't support all of the needed operations: uninstall and extraclean are not supported in its Makefile.

Signed-off-by: Dotan Barak <dotanba@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@426 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoFixes few harmless issues:
vlnb [Mon, 30 Jun 2008 09:50:27 +0000 (09:50 +0000)]
Fixes few harmless issues:
 1. The #define on line 228 of iscsi-scst/usr/isns.c triggers a compiler
warning on ppc.
 2. make iscsi && sudo make iscsi_install triggers the following error
message ( kernel, ppc), but doesn't break installation:
install: cannot stat `usr/iscsi-scst-adm': No such file or directory
 3. Unneeded "unlikely" removed

Issues 1 and 2 reported by Bart Van Assche <bart.vanassche@gmail.com>

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@425 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoMinor updates
vlnb [Mon, 30 Jun 2008 09:38:39 +0000 (09:38 +0000)]
Minor updates

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@424 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - A complete fix for the problem when a command comes with zero transfer length...
vlnb [Thu, 26 Jun 2008 16:39:05 +0000 (16:39 +0000)]
 - A complete fix for the problem when a command comes with zero transfer length and READ or WRITE data transfer direction
 - Version changed to 1.0.0-rc2

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@423 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoMinor cleanups and docs updates
vlnb [Thu, 26 Jun 2008 16:35:10 +0000 (16:35 +0000)]
Minor cleanups and docs updates

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@422 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoNew web look
vlnb [Thu, 26 Jun 2008 11:52:09 +0000 (11:52 +0000)]
New web look

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@421 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - Fixes incorrect allocation length 0 by scst_user module
vlnb [Mon, 23 Jun 2008 16:32:51 +0000 (16:32 +0000)]
 - Fixes incorrect allocation length 0 by scst_user module
 - Fixes missed case of "allocation length 0 for READ6/WRITE6 means 256".

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@420 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoReplace memory allocation from kzalloc to kmalloc because
vlnb [Thu, 19 Jun 2008 17:54:25 +0000 (17:54 +0000)]
Replace memory allocation from kzalloc to kmalloc because
this memory buffer will be zeroed later on in the code.

Signed-off-by: Dotan Barak <dotanba@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@419 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoFix compilation warning in kernels higher than 2.6.16
vlnb [Thu, 19 Jun 2008 17:52:18 +0000 (17:52 +0000)]
Fix compilation warning in kernels higher than 2.6.16
(the prototype of the function strncasecmp was changed in 2.6.16).

Signed-off-by: Dotan Barak <dotanba@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@418 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - Fixes not cleanly applies xxx2yyy patches
vlnb [Thu, 19 Jun 2008 17:48:08 +0000 (17:48 +0000)]
 - Fixes not cleanly applies xxx2yyy patches
 - Updated to IET r155
 - Compilation fixes
 - Typos fixes, including patch from Dotan Barak <dotanba@gmail.com>

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@417 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoAdded error message for a kernel bug, which doesn't allow for poll() to work with...
vlnb [Wed, 18 Jun 2008 09:54:38 +0000 (09:54 +0000)]
Added error message for a kernel bug, which doesn't allow for poll() to work with >256 file descriptors and prevents iscsi-scstd to start.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@416 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Fri, 13 Jun 2008 17:31:29 +0000 (17:31 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below fixes the following class of checkpatch warnings:
- WARNING: braces {} are not necessary for single statement blocks

How this patch has been verified:
- Checked that make -s clean && make -s scst iscsi-scst still works.
- Verified that checkpatch does no longer complain about the placement of

The patch below only changes comments, so it should be safe to include it in
the 1.0.0 release.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@415 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoAdded option mapping NV -> NV_CACHE.
mgandalf [Fri, 13 Jun 2008 13:38:46 +0000 (13:38 +0000)]
Added option mapping NV -> NV_CACHE.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@414 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Fri, 13 Jun 2008 11:08:23 +0000 (11:08 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The script scripts/specialize-patch'>scripts/specialize-patch is an awk script. Currently it specifies /usr/bin/awk as its interpreter although it doesn't use the standard AWK syntax but the extended GNU AWK syntax. The patch below changes the interpreter name such that the script also works on systems where /usr/bin/awk is not GNU AWK (e.g. Ubuntu 8.04). Note: GNU AWK is not installed by default on all Linux systems. E.g. on Debian systems it's in the gawk package.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@413 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - Complete the fix for broken REQUEST SENSE
vlnb [Fri, 13 Jun 2008 10:31:50 +0000 (10:31 +0000)]
 - Complete the fix for broken REQUEST SENSE
 - Cleanup: cleanup_lock shouldn't be IRQ

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@412 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - ISCSI sending response timeout increased to 30 seconds
vlnb [Thu, 12 Jun 2008 18:55:13 +0000 (18:55 +0000)]
 - ISCSI sending response timeout increased to 30 seconds
 - Fixed 2 problems in scst_user on release() cleanup
 - Added per-device memory limit and new scst.ko module parameter scst_max_dev_cmd_mem
 - Cleanups, including important ones
 - Version changed to 1.0.0-rc1

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@411 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoFixes broken REQUEST SENSE
vlnb [Thu, 12 Jun 2008 18:37:31 +0000 (18:37 +0000)]
Fixes broken REQUEST SENSE

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@410 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Thu, 12 Jun 2008 14:59:48 +0000 (14:59 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below fixes the following checkpatch warning:
__func__ should be used instead of gcc specific __FUNCTION__

This patch has been tested as follows:
- Checked that make -C ib_srpt still works.
- Checked that the patch generated by generate-kernel-patch still applies
  cleanly to the kernel, that the patched kernel still compiles,
  installs and boots and that the scst, iscsi-scst and ib_srpt modules still
  load (I performed rm -rf /lib/modules/* before starting kernel

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@409 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoCopyrights updated
vlnb [Thu, 12 Jun 2008 11:39:28 +0000 (11:39 +0000)]
Copyrights updated

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@408 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Arne Redlich <agr@powerkom-dd.de>:
vlnb [Thu, 12 Jun 2008 11:35:40 +0000 (11:35 +0000)]
Patch from Arne Redlich <agr@powerkom-dd.de>:

Removes unneeded kfree()

Signed-off-by: Arne Redlich <agr@powerkom-dd.de>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@407 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Dotan Barak <dotanba@gmail.com> (applied manually):
vlnb [Thu, 12 Jun 2008 11:32:12 +0000 (11:32 +0000)]
Patch from Dotan Barak <dotanba@gmail.com> (applied manually):

Fix compilation failure in kernel 2.6.19 and below
(bug.h was added only stafrting kernel 2.6.20).

Signed-off-by: Dotan Barak <dotanba@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@406 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Dotan Barak <dotanba@gmail.com>:
vlnb [Thu, 12 Jun 2008 11:25:38 +0000 (11:25 +0000)]
Patch from Dotan Barak <dotanba@gmail.com>:

Replace memory allocation from kzalloc to kmalloc because
this memory buffer will be zeroed later on in the code.

Signed-off by: Dotan Barak <dotanba@gmail.com>

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@405 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Thu, 12 Jun 2008 11:22:15 +0000 (11:22 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below fixes all remaining checkpatch errors:
- ERROR: do not use assignment in if condition
- ERROR: that open brace { should be on the previous line
- ERROR: trailing whitespace
- ERROR: use tabs not spaces
The last three had already been fixed before, but some occurences were
reintroduced in r403. It would be great if new modifications could be checked
for checkpatch errors before being committed.

This patch has been tested as follows:
- Reread the patch below carefully.
- Verified that make -s clean && make -s iscsi scst && make -s -C srpt
  still works and does not trigger any compiler warnings.
- Verified that checkpatch does no longer report any errors.
- Checked that the patch generated by generate-kernel-patch still applies
  cleanly to the kernel, and that the patched kernel tree still
  compiles, installs and boots fine, and that the iscsi-scst, ib_srpt,
  scst_disk and scst_vdisk modules still load.
- Tested that iSCSI login/logout still works from a remote system, and that
  iSCSI I/O still works.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@404 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoThe major TM processing cleanup in scst_user module which was possible after the...
vlnb [Sat, 31 May 2008 12:05:02 +0000 (12:05 +0000)]
The major TM processing cleanup in scst_user module which was possible after the recent SCST changes, to fix current problems. Also there are fixes for found during development/testing problems. Particularly:

 - PRIO queue was removed from scst_user. Instead, all priority commands now queued in the head of the regular queue. The corresponding code was removed from fileio_tgt as well. It necessary, in the future the priority queue can be easily restored from this patch.

 - pre_unreg_sess() was removed from struct scst_dev_type. The corresponding code was removed from SCST core as well

 - Almost all /proc/scsi_tgt commands now can fail after timeout (90 seconds) with EBUSY

 - Fixed possible incorrect command's retry if double RESET UA is detected.

 - Many minor changes and cleanups

Also docs were updated.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@403 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Thu, 29 May 2008 17:24:18 +0000 (17:24 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below fixes the checkpatch complaints on kfree():
WARNING: kfree(NULL) is safe this check is probabally not required

This patch has been verified as follows:
- Verified that checkpatch does no longer complain about kfree().
- Verified that make -s clean && make -s iscsi scst && make -s -C srpt
  still works and does not trigger any new compiler warnings.
- Checked that the patch generated by generate-kernel-patch still applies
  cleanly to the kernel, and that the patched kernel tree still
  compiles, installs and boots fine, and that the iscsi-scst, ib_srpt,
  scst_disk and scst_vdisk modules still load.

Note: I'm not sure any of the above verifications did trigger the code paths
changed by this patch.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@402 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago3 small patches from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Wed, 28 May 2008 11:17:08 +0000 (11:17 +0000)]
3 small patches from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below fixes the following category of checkpatch complaints:
ERROR: do not use C99 // comments

This patch has been verified as follows:
- Verified that checkpatch does no longer complain about C99 comments.
- Verified that make -s clean && make -s iscsi scst && make -s -C srpt
  still works.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
The patch below fixes the following category of checkpatch complaints:
WARNING: unnecessary cast may hide bugs, see http://c-faq.com/malloc/mallocnocast.html

This patch has been verified as follows:
- Verified that checkpatch does no longer complain about unnecessary casts.
- Verified that make -s clean && make -s iscsi scst && make -s -C srpt
  still works and does not trigger any new compiler warnings.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
The patch below fixes the following category of checkpatch warnings:
ERROR: Use of SPIN_LOCK_UNLOCKED is deprecated: see Documentation/spinlocks.txt

This patch has been verified as follows:
- Verified that checkpatch does no longer complain about spinlocks.
- Verified that make -s clean && make -s iscsi scst && make -s -C srpt
  still works and does not trigger any new compiler warnings.
- Verified that the patch generated by generate-kernel-patch still applies
  cleanly to the kernel, and that the patched kernel tree still
  compiles, installs and boots fine, and that the iscsi-scst, ib_srpt,
  scst_disk and scst_vdisk modules still load.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@401 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Tue, 27 May 2008 14:31:31 +0000 (14:31 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below fixes the following checkpatch warning:
WARNING: EXPORT_SYMBOL(foo); should immediately follow its function/variable

This patch has been tested as follows:
- Verified that the output of the following command did not change due to the
  patch below:
    find -name '*.[ch]'|xargs grep -h -w EXPORT_SYMBOL | sort
- Checked that make -s clean && make -s iscsi scst && make -s -C srpt still
- Checked that the patch generated by generate-kernel-patch still applies
  cleanly to the kernel, and that the patched kernel tree still
  compiles, installs and boots fine, and that the iscsi-scst, ib_srpt,
  scst_disk and scst_vdisk modules still load.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@400 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Tue, 27 May 2008 14:28:37 +0000 (14:28 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below fixes the following checkpatch warning:
WARNING: Use #include <linux/...> instead of <asm/...>

This patch has been tested as follows:
- Verified the patch by rereading it.
- Checked that make -s clean && make -s iscsi scst && make -s -C srpt still
- Checked that the patch generated by generate-kernel-patch still applies
  cleanly to the kernel, and that the patched kernel tree still
  compiles, installs and boots fine, and that the iscsi-scst, ib_srpt,
  scst_disk and scst_vdisk modules still load.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@399 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Tue, 27 May 2008 14:26:15 +0000 (14:26 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below fixes the following categories of checkpatch complaints:
- ERROR: switch and case should be at the same indent
- ERROR: do not initialise statics to 0 or NULL
- ERROR: use tabs not spaces

I have verified that the SCST source code still compiles.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@398 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Mon, 26 May 2008 11:13:04 +0000 (11:13 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

While all warnings on mismatches between printf-style format strings and
argument lists were fixed on i386 and x86_64, gcc prints a large number of
warnings about mismatches when compiling SCST on ppc64. The patch below
fixes these by casting integer arguments to the type (long long unsigned int)
where appropriate.

This patch has been tested as follows:
- Verified the patch by reading it.
- Checked that make -s clean && make -s iscsi scst && make -s -C srpt did not
  trigger any compiler warnings about format strings on i386, x86_64 and ppc.
- Checked that no previously fixed checkpatch warnings were reintroduced in
  the patch generated by generate-kernel-patch.
- Checked that the patch generated by generate-kernel-patch still applies
  cleanly to the kernel, and that the patched kernel tree still
  compiles, installs and boots fine.

Additionally, this patch splits a kernel version preprocessor test from one into two lines, such that
the generate-kernel-patch script can process the version check properly.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@397 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - Fixed one more 32/64 bits user/kernel incompatibility
vlnb [Mon, 26 May 2008 11:00:19 +0000 (11:00 +0000)]
 - Fixed one more 32/64 bits user/kernel incompatibility
 - Minor fixes

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@396 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoCheck preprocesor definitions when releasing firmware.
swgruszka [Sat, 24 May 2008 13:48:20 +0000 (13:48 +0000)]
Check preprocesor definitions when releasing firmware.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@395 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Erik Habbinga <erikhabbinga@inphase-tech.com> to fix out of tree building...
vlnb [Fri, 23 May 2008 16:00:56 +0000 (16:00 +0000)]
Patch from Erik Habbinga <erikhabbinga@inphase-tech.com> to fix out of tree building introduced in r387 with some changes to prevent such things in the future.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@394 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Fri, 23 May 2008 15:22:12 +0000 (15:22 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below fixes the following checkpatch warning:
__func__ should be used instead of gcc specific __FUNCTION__

This patch has been tested as follows:
- Checked that make iscsi scst still works.
- Checked that after loading the scst module the kernel messages generated
  because of the shell command "echo all >/proc/scsi_tg/trace_level" still
  make sense. An example:
  [  646.408035] [5845]: scst: scst_proc_log_entry_write:346:Changed trace level for "scsi_tgt": old 0x80005f0e, new 0xffffffff
- Checked that the patch generated by generate-kernel-patch still applies
  cleanly to the kernel, that the patched kernel still compiles,
  installs and boots and that the scst, iscsi-scst and ib_srpt modules still
  load (I performed rm -rf /lib/modules/* before starting kernel

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@393 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Fri, 23 May 2008 15:19:52 +0000 (15:19 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The current version of generate-kernel-patch does not remove preprocessor
statements that should have been removed because intended for another kernel
version (surrounded by #if LINUX_VERSION_CODE ... KERNEL_VERSION(...) / #endif).
The patch below fixes this.

I have verified the patch below by comparing the patch generated by the
old and the new versions of the generate-kernel-patch script:
$ diff p1 p2
< +#if defined(EXTRACHECKS)
< +#endif
> > +
> > +
< +#elif !defined(SCSI_EXEC_REQ_FIFO_DEFINED)
< +#else
< +#endif
> > +
> > +
> > +
> > +

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@392 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - Fixed problems with 64-bit platforms with 32-bit user space and with non-4K pages
vlnb [Fri, 23 May 2008 15:12:34 +0000 (15:12 +0000)]
 - Fixed problems with 64-bit platforms with 32-bit user space and with non-4K pages
 - From the main Makefile all all LSI/MPT related targets are commented out, because the recent changes in its Makefile have broken them.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@391 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Fri, 23 May 2008 09:56:13 +0000 (09:56 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch adds command-line options -m and -q to the script
generate-kernel-patch. These command line options make it possible to
choose at the time the script is invoked whether or not to include the
qla2x00t and/or mpt code in the generated kernel patch.

Please review.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
Index: scripts/generate-kernel-patch

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@390 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Fri, 23 May 2008 09:38:07 +0000 (09:38 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below makes sure the SCST code compiles again (it was broken by r386). The patch
includes the following changes:
- Make sure that macro's do not end in a semicolon, because otherwise the
  transformation if (condition) { MACRO; } else {...}
  into if (condition) MACRO; else {...}
  results in a compilation error.
- Aligned backslashes (whitespace-only change).
- One "else{" was incorectly changed into "else { {" -- fixed this.

I have verified that the following command again succeeds:
  make -s iscsi scst && make -s -C srpt && make -s -C qla2x00t

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@389 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoOops, forgot to add this file in commit 387
vlnb [Thu, 22 May 2008 17:01:44 +0000 (17:01 +0000)]
Oops, forgot to add this file in commit 387

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@388 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Erik Habbinga <erikhabbinga@inphase-tech.com>:
vlnb [Thu, 22 May 2008 16:36:59 +0000 (16:36 +0000)]
Patch from Erik Habbinga <erikhabbinga@inphase-tech.com>:

I've updated the MPT fusion target driver for LSI SCSI/FC/SAS HBAs.  The original SCST MPT driver was based on the 1.00.13 LSI
sample target driver.  LSI has since released version 1.00.14 of the sample target driver.  I've ported all the changes from .13 to
.14 into the SCST MPT driver for consistency.  I've also updated the SCST MPT driver to compile under 2.6.24 and  Finally,
I force the SCSI parallel transfer rate to very slow depending on the SCSI HBA firmware revision.  Newer versions of SCSI HBA
firmware have a bug where the incorrect amount of data is transferred for non-divisible-by-4 length transfers (like standard 14 byte
REQUEST SENSE).  LSI is aware of the problem and I'll update the driver to restore functionality upon a new good SCSI HBA firmware

I still have not confirmed operation with LSI SAS or FC HBAs.

Signed-off-by: Erik Habbinga <erikhabbinga@inphase-tech.com>


- differences between LSI sample target drivers 1.00.13 and 1.00.14


- remove some unnecessary white space


- differences between LSI sample target drivers 1.00.13 and 1.00.14

- changes for kernel 2.6.24 and

- force slow transfers for newer SCSI HBA firmware versions


- simplified for inclusion into kernel source tree


- patch against 2.6.24 and drivers/message/fusion/Kconfig


- changed to work with generate-kernel-patch script

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@387 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com> with one change ("/* To do...
vlnb [Thu, 22 May 2008 16:13:36 +0000 (16:13 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com> with one change ("/* To do */;;" replaced by "/* ToDo */"):

The patch below fixes the following checkpatch complaints:
- ERROR: else should follow close brace '}'
- ERROR: need consistent spacing around '&' (ctx:VxW)
- ERROR: open brace '{' following enum go on the same line
- ERROR: open brace '{' following struct go on the same line
- ERROR: that open brace { should be on the previous line
- ERROR: trailing statements should be on next line
- ERROR: trailing whitespace
- WARNING: braces {} are not necessary for any arm of this statement

The patch below has been verified as follows:
- Verified the patch below by reading it.
- Checked that checkpatch does no longer complain about the above categories
  of messages on the generated patch.
- Checked that the generated kernel patch applies cleanly to the
- Checked that the patched kernel compiles and installs cleanly,
  and that after reboot it was possible to load the iscsi-scst and ib_srpt
  kernel modules.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@386 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Thu, 22 May 2008 15:55:23 +0000 (15:55 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below implements the following changes:
- Make sure the output of the make command does not appear in the generated
  kernel patch.
- Changed interpreter of scripts/specialize-patch from /usr/bin/gawk to
  /usr/bin/awk in order to make sure this awk script also runs on systems
  where awk is installed but not gawk.
- Added an extra evaluation rule such that expressions like (0) are rewritten
  as 0.
- Fixed a spelling error.
- Expressions that can only be partially evaluated are now partially evaluated.
  E.g. #if defined(EXTRACHECKS) && (LINUX_VERSION_CODE < KERNEL_VERSION(2, 6, 18))
  becomes #if defined(EXTRACHECKS) && 0 in the output.
- Code between #if 0 / #endif or #if 1 / #endif is included in the generated
  kernel patch, including the surrounding preprocessor directives.
- Preprocessor statements with spaces behind # are now recognized (e.g. "#   if").
- Expression evaluator now supports ! (logical not) and () (braces).
- #elif is now evaluated correctly.
- Nested #if / #elif / #else / #endif are now interpreted correctly.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@385 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Wed, 21 May 2008 15:48:02 +0000 (15:48 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below contains a script that removes C preprocessor tests on the
LINUX_VERSION_CODE macro, depending on the kernel version that was passed as
an argument to that script. Furthermore, the generate-kernel-patch script has
been modified such that its output is filtered by the specialize-patch script.
This eliminates another class of checkpatch errors.

Note: due to the way the specialize-patch script is implemented, #if 0 and
#if 1 statements are also processed.

The patch below has been verified as follows:
- Checked that checkpatch does no longer complain about LINUX_VERSION_CODE
  on the generated patch.
- Checked that the generated kernel patch applies cleanly to the
- Checked that the patched kernel compiles and installs cleanly, and that
  after reboot it was possible to load the iscsi-scst and ib_srpt kernel modules.

I will wait with sending more patches until this and the previous two patches
have been reviewed and/or applied.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@384 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Wed, 21 May 2008 15:43:54 +0000 (15:43 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below fixes the following class of checkpatch.pl errors:
  ERROR("(foo*)" should be "(foo *)".
Or: another whitespace-only change.

I have verified that the output of svn diff -w -x is empty for this patch.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@383 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Wed, 21 May 2008 15:40:33 +0000 (15:40 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below fixes another category of checkpatch warnings, namely:
  ERROR: "foo * bar" should be "foo *bar"

This patch has been verified by checking that the output of svn diff -x -w is

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@382 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Wed, 21 May 2008 11:56:09 +0000 (11:56 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below implements the following changes:
- Fixes the remaining warnings reported by checkpatch.pl with regard to the use
  of whitespace in C code.
- Adds double qoutes around some #warning messages, such that checkpatch.pl
  does no longer try to interprete the text behind the #warning preprocessor
- Adds an extra conversion to the generate-kernel-patch script that removes
  the single space from before goto-labels while generating the kernel patch.

This patch has been tested as follows:
- Verified that svn diff -x -w only shows changes in the #warning preprocessor directives.
- Verified the output of svn diff by reading it.
- Verified that checkpatch.pl does no longer complain on the use of whitespace
  (searched through the checkpatch.pl output for the text ' space').
- Verified that the patch generated by the generate-kernel-patch script still
  applies cleanly to the 2.6.24 kernel, and that the patched kernel still compiles cleanly.

Please let me know if I have to resubmit parts of this patch to other SCST authors.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@381 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoleft align and remove indent 6 for goto labels
huongvp [Wed, 21 May 2008 00:11:38 +0000 (00:11 +0000)]
left align and remove indent 6 for goto labels

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@380 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Tue, 20 May 2008 14:17:25 +0000 (14:17 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

As discussed earlier today, GNU-style named initializers should be replaced
by C99-style named initializers. The patch below realizes this. The patch
below contains one additional change: double quotes are added around
a #warning message. These double qoutes are not required by the C89
or C99 language standards, but many tools expect these (e.g. checkpatch.pl).

This patch has been tested as follows:
* Verified that SCST still compiles and links (make scst iscsi && make -C srpt)
* Verified that checkpatch.pl does no longer complain on named initializers.
* Verified that the kernel patch generated by scripts/generate-kernel-patch
   applies cleanly to the kernel, that the patched kernel compiles
   cleanly, installs properly and that it was possible after reboot to load
   the modules scst, iscsi-scst and ib_srpt load.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@379 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Tue, 20 May 2008 09:32:14 +0000 (09:32 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The patch below fixes statements that were indented by spaces are now indented by tabs.
The patch only contains whitespace changes -- after having applied the patch below, the
output of svn diff -x -w is still empty.

How this patch was tested:
- Verified that the output of svn diff -x -w is empty.
- Verified that the patch generated by generate-kernel-patch still applies
  cleanly to the kernel, that the patched kernel still compiles
  and that the SCST modules load properly after having installed the newly
  compiled kernel and after having rebooted.

Signed-off-by: Bart Van Assche <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@378 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>, except few chuncks from qla2x0...
vlnb [Mon, 19 May 2008 10:21:41 +0000 (10:21 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>, except few chuncks from qla2x00t/ obviously not related to the target mode addon:

One of the Linux kernel patch submission requirements is that source files do
not contain trailing whitespace. The patch below removes trailing whitespace
from .c and .h source files.

Note: it might be more convenient to run the script I used to generate this
patch than to review and apply the patch below. This is how I generated and
verified the patch below:

cat <<EOF >./strip-trailing-whitespace
trap "rm -f $t" EXIT
for f in "$@"
  sed 's/[  ]*$//' <"$f" >"$t" && mv "$t" "$f"
chmod a+x ./strip-trailing-whitespace
find -name '*.[ch]' | xargs ./strip-trailing-whitespace
svn diff -x -w

Signed-off-by: <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@377 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Fri, 16 May 2008 17:47:31 +0000 (17:47 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

Disable the scst-user selection in Kconfig when HIGHMEM is enabled because
this module is incompatible with HIGHMEM. I have tested the modification
below by toggling HIGHMEM via "make ARCH=i386 xconfig".

Signed-off-by: <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@376 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Fri, 16 May 2008 10:29:34 +0000 (10:29 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

I don't know when this was broken, but with the current SVN version, after
having generated and applied a kernel patch for SCST, iscsi-scst doesn't
build anymore. The patch below fixes this and also includes a cleanup patch
for the generate-kernel-patch'>generate-kernel-patch script. The generated kernel patch has been
tested with the 2.6.24 and kernels (extracted kernel sources +
applied patch + built kernel + installed kernel + reboot + load SCST

Signed-off-by: <bart.vanassche@gmail.com>
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@375 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - Fixed sg_tablesize in qla2x00t to be per target card, not global
vlnb [Thu, 15 May 2008 17:27:50 +0000 (17:27 +0000)]
 - Fixed sg_tablesize in qla2x00t to be per target card, not global
 - Minor cleanups

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@374 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Thu, 15 May 2008 15:00:21 +0000 (15:00 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

By this time I have reworked the update of scripts/generate-kernel-patch'>scripts/generate-kernel-patch.
These are the changes that were already present in version 1 of this patch
and that have been kept in this patch:
- The SCST code has been moved from drivers/scsi/scsi_tgt to drivers/scst.
- Moved SCST header files from include/scsi_tgt to include/scst.
- Added iscsi-scst in drivers/scst/iscsi-scst.

For the time being the generate-kernel-patch does no longer try to add the
qla2x00t driver in drivers/scst/qla2x00-target -- I have not yet been able to
figure out how to get qla2x00t/qla2x00-target compiled without the initiator

Just as in version 1 of this patch, the following files in the Subversion
repository have been renamed to reflect the above changes:
- Renamed scst/kernel/in-tree/Makefile.scsi.Linux-2.6.24.patch'>scst/kernel/in-tree/Makefile.scsi.Linux-2.6.24.patch to
- Renamed Kconfig'>scst/kernel/in-tree/Kconfig.scsi.Linux-2.6.24.patch to
- Renamed scst/kernel/in-tree/Makefile.scsi_tgt'>scst/kernel/in-tree/Makefile.scsi_tgt to
- Renamed scst/kernel/in-tree/Kconfig.scsi_tgt'>scst/kernel/in-tree/Kconfig.scsi_tgt to

Because of the above changes the include path specified in the following
Makefiles had to be changed:
- scst/kernel/in-tree/Makefile.dev_handlers'>scst/kernel/in-tree/Makefile.dev_handlers
- srpt/src/Makefile.in_kernel'>srpt/src/Makefile.in_kernel

Other changes:
- I have added menu/endmenu entries in file scst/kernel/in-tree/Kconfig.scst
such that the SCST entries become visible when running make xconfig or make
- Elaborated the SCST help texts a little bit (in file

New in version 2 of this patch:
- Moved remaining patches in the generate-kernel-patch script to separate
- Added support for the 2.6.25 kernel.

This patch has been tested as follows:

mkdir -p tmp
cd tmp
rm -rf linux- /lib/modules/
tar xjf ~vanasscb/software/downloads/linux-
cd linux-
cp ../.config-2.6.25 .
(cd ~vanasscb/software/scst
&& ~vanasscb/software/scst/scripts/generate-kernel-patch \
2.6.25) | patch -p1
make oldconfig
diff ../.config .
make xconfig
make menuconfig
make -j5 bzImage modules
make modules_install install
cp -r /lib/firmware/$(uname -r) /lib/firmware/
update-initramfs -k -c
modprobe iscsi-scst
modprobe ib_srpt
lsmod | grep -E 'scst|ib_srpt'

Signed-off-by: bart.vanassche@gmail.com
git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@373 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoUpdate to the latest IET
vlnb [Thu, 15 May 2008 09:35:20 +0000 (09:35 +0000)]
Update to the latest IET

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@372 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoInterface versioning between iSCSI-SCST kernel and user space parts, iscsi_u.h rename...
vlnb [Wed, 14 May 2008 17:20:20 +0000 (17:20 +0000)]
Interface versioning between iSCSI-SCST kernel and user space parts, iscsi_u.h renamed to iscsi_scst.h. Property svn:ignore adjusted accordingly. Docs updated.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@371 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoCritical patch from anonymous. Fixes possible data corruption in BLOCKIO code. A...
vlnb [Wed, 14 May 2008 14:58:29 +0000 (14:58 +0000)]
Critical patch from anonymous. Fixes possible data corruption in BLOCKIO code. A pretty subtle bug. Can be triggered on very rare on practice combination of circumstances.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@370 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoPatch from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Wed, 14 May 2008 10:01:30 +0000 (10:01 +0000)]
Patch from Bart Van Assche <bart.vanassche@gmail.com>:

The script generate-kernel-patch generates a kernel tree based on the sources
and the in-tree kernel patches in the SCST Subversion repository. When I
submitted the first version of the generate-kernel-patch script most but not
all in-tree patches were present as separate files in the SCST Subversion
repository. The patch below moves the two remaining patches that are contained
inline in the generate-kernel-patch script to separate files. This should
make it possible to use the script on future Linux kernels without having to
modify the script itself. The patch below does not contain functional
changes, it only moves two patches out of the script.

* scripts/generate-kernel-patch'>scripts/generate-kernel-patch
* srpt/src/Makefile.infiniband.Linux-2.6.24.patch'>srpt/src/Makefile.infiniband.Linux-2.6.24.patch
* srpt/src/Kconfig.infiniband.Linux-2.6.24.patch'>srpt/src/Kconfig.infiniband.Linux-2.6.24.patch

Also Makefile and Kconfig files were renamed (".drivers." was replaced by ".scsi.")

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@369 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - In "extraclean" target new autogenerated files added
vlnb [Tue, 13 May 2008 17:18:07 +0000 (17:18 +0000)]
 - In "extraclean" target new autogenerated files added
 - Don't automatically build iscsi-scst-adm anymore until it will be fixed

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@368 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoReference to Open-FCoE added
vlnb [Tue, 13 May 2008 10:08:21 +0000 (10:08 +0000)]
Reference to Open-FCoE added

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@367 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoSeries of patches from Bart Van Assche <bart.vanassche@gmail.com>:
vlnb [Tue, 13 May 2008 09:27:16 +0000 (09:27 +0000)]
Series of patches from Bart Van Assche <bart.vanassche@gmail.com>:

 - Added Makefile.scsi.Linux-2.6.24.patch and Kconfig.scsi.Linux-2.6.24.patch
 - Updated Makefile.scsi_tgt to reflect move of the SCST device handlers from scst/src/ to scst/src/dev_handlers/
 - Added Makefile.dev_handlers
 - SCST include path fixed in srpt/ Makefile.in_kernel
 - Added script for converting the SCST source tree to a kernel patch. More comment on it:

The SCST source tree as it exists in the current Subversion repository will
have to be maintained for some time. And if SCST is submitted for inclusion
in the mainline kernel, this tree will have to be converted to a kernel
patch, reviewer comments will have to be processed, and the patch will have
to be resubmitted. So it's convenient to have a script available that
converts the Subversion source tree into a kernel patch. The patch below does
just that: convert the scst and srpt directories into a kernel patch
(iscsi-scst support will be added later on). I have tested the script
contained in the patch below as follows:

rm -rf linux-2.6.24
rm -rf /lib/modules/2.6.24-scst
tar xjf ~vanasscb/software/downloads/linux-2.6.24.tar.bz2
cd linux-2.6.24
cp ../.config .
(cd ~vanasscb/software/scst
&& ~vanasscb/software/scst/scripts/generate-kernel-patch \
2.6.24) | patch -p1
make -j5 bzImage modules && make modules_install install
cp -r /lib/firmware/$(uname -r) /lib/firmware/2.6.24-scst
update-initramfs -k 2.6.24-scst -c

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@366 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - Fixed GFP_KERNEL misuse. Reported independently by mbe1@charter.net and Erez Zilbe...
vlnb [Mon, 12 May 2008 15:58:34 +0000 (15:58 +0000)]
 - Fixed GFP_KERNEL misuse. Reported independently by mbe1@charter.net and Erez Zilber <erezz@Voltaire.COM>
 - Fixed not sufficient in_irq() to determine atomic allocation context. Reported by mbe1@charter.net

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@365 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago - Rejecting command reimplemented in a more simple, straightforward and readable...
vlnb [Wed, 7 May 2008 18:44:15 +0000 (18:44 +0000)]
 - Rejecting command reimplemented in a more simple, straightforward and readable way.
 - Minor external interface change: now target drivers should set for aborted commands SCST_CMD_DELIVERY_ABORTED status via scst_set_delivery_status(). In-tree drivers updated.
 - Fixed broken compilation if put_page_callback patch not applied to the kernel. Reported by Erez Zilber <erezz@Voltaire.COM>
 - Fixed several minor problems reported by David Berton <davidbrt@yahoo.com>
 - Fixed __exit misuse, when such functions called from __init functions.
 - Docs updated.
 - Other minor changes and cleanups.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@364 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoDouble "repository" word fixed
vlnb [Sun, 4 May 2008 10:31:05 +0000 (10:31 +0000)]
Double "repository" word fixed

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@363 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoWeb updates
vlnb [Fri, 2 May 2008 11:40:26 +0000 (11:40 +0000)]
Web updates

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@362 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoRemove request sense warning and fix typo.
swgruszka [Thu, 1 May 2008 06:49:33 +0000 (06:49 +0000)]
Remove request sense warning and fix typo.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@361 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoNullify f/w pointer if load fails.
swgruszka [Thu, 1 May 2008 06:46:23 +0000 (06:46 +0000)]
Nullify f/w pointer if load fails.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@360 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoBugfix - disallow to free initiator data when disabling target mode,
swgruszka [Wed, 30 Apr 2008 14:19:38 +0000 (14:19 +0000)]
Bugfix - disallow to free initiator data when disabling target mode,
as long as all reference to it is dropped.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@359 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoAdd missing definition of strncasecmp() for older kernels.
swgruszka [Thu, 24 Apr 2008 13:12:08 +0000 (13:12 +0000)]
Add missing definition of strncasecmp() for older kernels.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@358 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago- Update README.scst.
swgruszka [Thu, 24 Apr 2008 12:50:51 +0000 (12:50 +0000)]
- Update README.scst.
- Add comment about overall isp_scst.c design.
- Put some compile time selectable values in one place.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@357 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoBugfix: put initiator reference counter when finish task management.
swgruszka [Thu, 24 Apr 2008 09:31:02 +0000 (09:31 +0000)]
Bugfix: put initiator reference counter when finish task management.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@356 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoSynchronize with Feral CVS repository:
swgruszka [Thu, 24 Apr 2008 06:34:08 +0000 (06:34 +0000)]
Synchronize with Feral CVS repository:
- use kthread
- MBOX_ACQUIRE macro redefined

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@355 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoRemove unnecessary variables.
swgruszka [Wed, 23 Apr 2008 11:17:33 +0000 (11:17 +0000)]
Remove unnecessary variables.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@354 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago- Compile time conditinal debug.
swgruszka [Wed, 23 Apr 2008 09:23:43 +0000 (09:23 +0000)]
- Compile time conditinal debug.
- Initialize variable.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@353 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years ago- Avoid tmd's leakage when disable target mode when pending I/O (some work still
swgruszka [Wed, 23 Apr 2008 09:15:14 +0000 (09:15 +0000)]
- Avoid tmd's leakage when disable target mode when pending I/O (some work still
  needed in low level driver).
- Properly free channels on error case.
- Start HBA unregistation after proc file is removed.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@352 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoFixed wrong state assigmnent in scst_xmit_response()
vlnb [Tue, 22 Apr 2008 15:04:09 +0000 (15:04 +0000)]
Fixed wrong state assigmnent in scst_xmit_response()

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@351 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoThe bunch of pending changes. Sorry, it's quite big.
vlnb [Tue, 22 Apr 2008 10:30:53 +0000 (10:30 +0000)]
The bunch of pending changes. Sorry, it's quite big.

 - "RECEIVE DIAGNOSTIC RESULTS" command handling fixed
 - Obtaining device queue parameters in scst_obtain_device_parameters() changed to handle NOT READY sense
 - Fixed possible dev_cmd_count underflow
 - Minor iSCSI-SCST connection closing cleanups and fixes
 - Semantic of other *_atomic flags changed a bit. Now they are intended only to allow SCST to optimize execution context. The corresponing functions now can *always* be called on atomic context and, if necessary, should manually determine that using scst_cmd_atomic() function and restart using corresponding *_NEED_THREAD_* return value.
 - scst_cmd_get_tgt_resp_flags() and SCST_TSC_FLAG_STATUS flag made obsolete and replaced by scst_cmd_get_is_send_status(). All in-tree drivers updated accordingly
 - Cleanup: preprocessing_done_atomic removed, now preprocessing_done() callback always called on thread context
 - 64-bit cleanups
 - Documented that iscsi-scst-adm is currently broken and explained why
 - In user space Makefile's added possibility to specify external linker flags
 - Minor performance optimizations
 - Other minor cleanups

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@350 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoType fix.
swgruszka [Mon, 21 Apr 2008 17:37:43 +0000 (17:37 +0000)]
Type fix.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@349 d57e44dd-8a1f-0410-8b47-8ef2f437770f

14 years agoRemove not necessary headers and definitions.
swgruszka [Mon, 21 Apr 2008 17:34:42 +0000 (17:34 +0000)]
Remove not necessary headers and definitions.

git-svn-id: https://scst.svn.sourceforge.net/svnroot/scst/trunk@348 d57e44dd-8a1f-0410-8b47-8ef2f437770f