Commit Graph

  • 11044c3033 Added kernel version 2.6.32.1. Bart Van Assche 2009-12-16 19:51:11 +00:00
  • ca47017056 Added to repository. Bart Van Assche 2009-12-16 19:49:19 +00:00
  • eaad0ab4fb Fixed kernel version numbers. Bart Van Assche 2009-12-16 19:49:00 +00:00
  • 0e0c9d1a42 Merge of trunk r1400: Vladislav Bolkhovitin 2009-12-16 19:27:45 +00:00
  • 67f0cefd7d Fix incorrect initialization of scst_mgmt_cmd_stub structure Vladislav Bolkhovitin 2009-12-16 19:24:42 +00:00
  • 2b5833d90e Update for 2.6.32 + fix context of vdisk_t10_dev_id_rwlock Vladislav Bolkhovitin 2009-12-16 19:21:41 +00:00
  • 05cf277910 Fixed incorrect "set_t10_dev_id " token size caused by rename of set_t10_device_id to set_t10_dev_id. Vladislav Bolkhovitin 2009-12-16 11:48:29 +00:00
  • 52ca079fb8 Patch from Daniel Fernandes <dfernandes1978@hotmail.com> adding missed links in 2 pages Vladislav Bolkhovitin 2009-12-16 11:38:54 +00:00
  • f519e2c265 Fix spelling typos. Vladislav Bolkhovitin 2009-12-16 11:35:12 +00:00
  • 8db1080d78 - Added -noprompt option. Use with care! - Added support for T10 device ID. Applied modified patch by witoldk1@open-e.com. Mark Buechler 2009-12-15 20:01:01 +00:00
  • 24c072d970 Patch from Witold Kowolik <witold.kowolik@open-e.com> ranaming SCSI ID to T10 device ID + small fix to handle spaces in t10dev_id with small changes. Particularly, I renamed everywhere t10_device_id in t10_dev_id. Vladislav Bolkhovitin 2009-12-15 19:32:20 +00:00
  • 1587872d58 Merge with r1392: Vladislav Bolkhovitin 2009-12-15 18:43:19 +00:00
  • 72eb1111f0 Patch from Smadar Gonen <smadar.gn@gmail.com> with minor changes fixing race in task managmement code if several TM functions affect the same command on different stages of processing. Vladislav Bolkhovitin 2009-12-15 18:32:13 +00:00
  • e76ee7f95f Updated kernel version 2.6.31.7 to 2.6.31.8. Bart Van Assche 2009-12-14 19:14:23 +00:00
  • 6ed8923406 Spelling fix. Bart Van Assche 2009-12-13 18:02:39 +00:00
  • 50dc64fd1c Updated SRPT documentation. Bart Van Assche 2009-12-13 18:01:14 +00:00
  • 9e72c9bade Bug fix: SRQ availability counter is no longer decremented when ib_post_send() fails. Bart Van Assche 2009-12-13 17:59:44 +00:00
  • a03c2176a8 Added an additional consistency check: an error message is now printed if the kernel-ib-devel RPM has been installed but not the kernel-ib RPM. Bart Van Assche 2009-12-13 13:00:39 +00:00
  • 345cd34aae Whitespace-only change: fixed a checkpatch complaint. Bart Van Assche 2009-12-09 12:03:35 +00:00
  • 914b26386a Updated kernel version 2.6.27.40 to 2.6.27.41. Bart Van Assche 2009-12-09 11:38:11 +00:00
  • 36321326d9 Updated kernel versions 2.6.27.39 and 2.6.31.6 to 2.6.27.40 and 2.6.31.7 respectively. Bart Van Assche 2009-12-08 20:47:50 +00:00
  • ca81d05dd7 Update: the __same_type() macro has been introduced in kernel version 2.6.31, not in 2.6.30. Bart Van Assche 2009-12-07 08:51:58 +00:00
  • e4b93a83c0 - Fixed a checkpatch complaint. - Made sure that the SRPT source code compiles again on systems with a 2.6.29 or older kernel. Bart Van Assche 2009-12-07 08:07:58 +00:00
  • 6a9b203dd2 FMR only applies to the SRP initiator, not to the SRP target. Bart Van Assche 2009-12-07 07:21:39 +00:00
  • 799820d44b - Simplified error handling in srpt_get_desc_tbl(): incorrect indirect descriptors are now reported by returning -EINVAL instead of setting the third argument to true. Removed the third argument of this function. - Added support for SRP_CMD requests with non-zero ADDITIONAL CDB LENGTH. Note: support for these requests has not yet been tested -- I do not know of any SRP initiator that generates such requests. Bart Van Assche 2009-12-06 19:00:25 +00:00
  • ffb2fb1f11 Updated SRPT documentation. Bart Van Assche 2009-12-06 12:44:47 +00:00
  • 001d47266c Updated to do list. Bart Van Assche 2009-12-06 12:37:21 +00:00
  • 0486e8404c - Moved two constants from ib_srpt.c to ib_srpt.h. - Changed default value of srp_max_message_size from 4096 to 2116. This size corresponds to the size of an SRP_CMD request message containing an indirect buffer list with 128 elements. - A warning is now printed when an SRP_CMD request is received containing more than one CDB -- this kind of messages is not yet supported. - Switched back from multiple ib_dma_sync_single_for_cpu() calls to a single call because the latter results in better performance. - An error message is now printed if a corrupt SRP_CMD request has been received. Before these were ignored silently. Bart Van Assche 2009-12-05 10:10:53 +00:00
  • 60d27d6e24 - Added new vdisk_blockio and vdisk_nullio introduced by the sysfs changes. Change by witoldk1@open-e.com. Mark Buechler 2009-12-04 19:08:32 +00:00
  • 7572933afe Validator fix Vladislav Bolkhovitin 2009-12-03 18:57:58 +00:00
  • 47e976b0d8 Web updates Vladislav Bolkhovitin 2009-12-03 18:43:13 +00:00
  • 24482f5d41 Some locking cleanups Vladislav Bolkhovitin 2009-11-30 19:17:59 +00:00
  • 7a330f2e09 Should have been included in the previous commit (r1371). Bart Van Assche 2009-11-30 10:59:40 +00:00
  • b371630185 Avoid dangling pointer dereferences by resetting ioctx->scmnd in srpt_on_free_cmd(). Bart Van Assche 2009-11-30 09:51:17 +00:00
  • 7cdcbc8c69 Cleanup: - Removed a spinlock that became superfluous. - Fixed checkpatch and sparse complaints. Bart Van Assche 2009-11-28 18:40:15 +00:00
  • 418e7fb3d6 - Split srpt_find_channel into two functions: srpt_find_channel() and srpt_release_channel_by_cmid(). - Removed tracing statements from srpt_unmap_sg_to_ib_sge() because this function is now invoked while performing I/O. Bart Van Assche 2009-11-28 17:20:22 +00:00
  • ec2e199060 Bug fixes (some introduced in the previous revision, some long-standing): - Fixed system lockup triggered by "rmmod ib_srpt" while the SRP was writing data. This lockup occurred because srpt_reset_ioctx() was not resetting the pointers to the memory it freed, which indirectly triggered a kernel oops in the IB interrupt handler. - Fixed BUG() during "rmmod ib_srpt" triggered by calling scst_rx_cmd() after scst_unregister_session(). This has been fixed by making sure that the channel state is set to RDMA_CHANNEL_DISCONNECTING before calling scst_unregister_session(). Performance improvements: - Simplified cmd_wait_list manipulation code. Bart Van Assche 2009-11-28 16:36:53 +00:00
  • f50ce17866 Bug fixes: - Fixed races on srpt_ioctx::state manipulation by declaring this variable atomic. - Fixed races on srpt_rdma_ch::state manipulation by declaring this variable atomic. - Fixed bug in srpt_abort_scst_cmd(): SRPT_STATE_DATA_IN was not yet handled. - Fixed races on srpt_rdma_ch::cm_id destruction. Clearly documented which object owns the cm_id at which time. Performance improvements: - Eliminated the list of active commands (active_scmnd_list) because this list was a duplicated of a list already kept by the SCST core. - Switched to double-checked locking in srpt_handle_new_iu(). This saves one spin_lock_irqsave() and one spin_unlock_irqrestore() call in the common case (that is, a live channel). - Eliminated RDMA channel state tests in SCST command handling callback functions -- removed the function srpt_abort_if_disconnecting(). Bart Van Assche 2009-11-28 14:03:55 +00:00
  • 2ff67c27cc Removed trailing whitespace. Bart Van Assche 2009-11-28 08:24:10 +00:00
  • 69d7b1eba7 Patch from Witold Kowolik <witoldk1@open-e.com> implementing manual setting of SCSI ID on vdisk devices (with some changes) Vladislav Bolkhovitin 2009-11-27 18:52:23 +00:00
  • 1de649c263 Followup for r1361 Vladislav Bolkhovitin 2009-11-27 18:22:45 +00:00
  • e6f5aa8865 Changes: - The message that sense data is truncated is now generated via the PRINT_WARNING() macro instead of TRACE_DBG(). - Cleaned up code for sense data truncation. - Compiles again on RHEL and CentOS systems. Bart Van Assche 2009-11-27 17:52:47 +00:00
  • 1065e2d317 Log sense data truncation when debugging is enabled. Bart Van Assche 2009-11-27 12:10:59 +00:00
  • f381dbb74c Device sysfs locking reconsidered to remove recently introduced deadlock possibility. Vladislav Bolkhovitin 2009-11-26 18:39:13 +00:00
  • a4f3af18bb Improve fairness between connections. Vladislav Bolkhovitin 2009-11-24 19:50:11 +00:00
  • d3665768f0 Logos rearranged Vladislav Bolkhovitin 2009-11-24 11:52:54 +00:00
  • 78ec60df89 Fix for warning: 'res' may be used uninitialized in this function Vladislav Bolkhovitin 2009-11-23 18:43:03 +00:00
  • f0a5a62fd4 - Patch from Daniel Fernandes <dfernandes1978@hotmail.com> adding validator logos - QStar Technologies added Vladislav Bolkhovitin 2009-11-23 16:38:44 +00:00
  • 2658982f05 - Patch from Daniel Fernandes <dfernandes1978@hotmail.com> fixing bug in URL reported by validator - Soul company name fixed Vladislav Bolkhovitin 2009-11-23 12:06:23 +00:00
  • 78f387fe1b Patch from Daniel Fernandes <dfernandes1978@hotmail.com>: Vladislav Bolkhovitin 2009-11-23 11:44:55 +00:00
  • 10530125a5 Whitespace-only change. Bart Van Assche 2009-11-23 08:40:51 +00:00
  • 495257b44e - Added counter that keeps track of the number of work requests posted on the per-channel queue pair. This counter is used to prevent that ib_post_send() triggers a queue overflow. - Fixed error handling in srpt_xfer_data(): srpt_unmap_sg_to_ib_sge() is now called before returning an error code. - Simplified implementation of the sysfs attribute login_info. Bart Van Assche 2009-11-22 18:42:35 +00:00
  • c918eb371e gcc 4.4.1 (as included in openSUSE 11.2) complains about the user-space iSCSI code because it breaks strict-aliasing rules. The reported warnings should be addressed because not doing so can result in the compiler generating incorrect code. The patch below fixes these compiler warnings. Vladislav Bolkhovitin 2009-11-20 19:53:12 +00:00
  • da022918e4 Patch from Daniel Fernandes <dfernandes1978@hotmail.com> fixing a bit incorrect layout on the users page Vladislav Bolkhovitin 2009-11-20 11:28:37 +00:00
  • 8219791b72 Ordered reads groupped under single config symbol. Vladislav Bolkhovitin 2009-11-19 11:57:57 +00:00
  • 4e65d67680 Fix bug preventing pass-through dev handlers be loaded if SCSI_EXEC_REQ_FIFO_DEFINED not defined. Vladislav Bolkhovitin 2009-11-19 10:22:25 +00:00
  • 96e4adfb03 Minor web updates and fixes Vladislav Bolkhovitin 2009-11-18 13:57:58 +00:00
  • e9e8884f41 Removed trailing whitespace. Bart Van Assche 2009-11-18 07:52:12 +00:00
  • b9402438ff Updated to do list once more. Bart Van Assche 2009-11-18 07:22:30 +00:00
  • b01b495735 Updated to do list. Bart Van Assche 2009-11-17 20:03:35 +00:00
  • a57d10033f Added more comments. Bart Van Assche 2009-11-17 18:01:16 +00:00
  • 9d65e4fe05 Removed a BUILD_BUG_ON() statement that was too restrictive. Bart Van Assche 2009-11-17 18:00:23 +00:00
  • ef9469cefb Fix for recently introduced incorrectly set expected data direction Vladislav Bolkhovitin 2009-11-17 13:53:03 +00:00
  • c3ad1a5075 Minor compiler and checkpatch fixes Vladislav Bolkhovitin 2009-11-17 12:59:49 +00:00
  • 42d9b02d49 Web fixes from Daniel Fernandes <dfernandes1978@hotmail.com> Vladislav Bolkhovitin 2009-11-17 12:10:10 +00:00
  • ed22024642 Patch from Daniel Fernandes <dfernandes1978@hotmail.com>: Vladislav Bolkhovitin 2009-11-16 18:19:46 +00:00
  • 3261614731 - Fix for memory allocation for scst_user and locally handled commands - Execution context cleanups and fixes Vladislav Bolkhovitin 2009-11-16 18:05:10 +00:00
  • 760b190ba2 Users page made logs based Vladislav Bolkhovitin 2009-11-16 17:22:27 +00:00
  • 9955ef7627 Since the expression "sizeof(*sense) + sizeof(*sense) % 4" did not make sense, replaced this expression by "sizeof(*sense)" and a BUILD_BUG_ON() statement that verifies that sizeof(*sense) is a multiple of four. Should the first expression have been "roundup(sizeof(*sense), 4)" ? Bart Van Assche 2009-11-15 13:14:40 +00:00
  • bcec50e065 - Small performance improvement: ib_dma_sync_for_device() and ib_dma_sync_for_cpu() calls do now only synchronize the data areas used. - Removed per-command mutex again because it's not a full solution with regard to avoiding races between module unloading / IB error handling and SCST's command handling. Bart Van Assche 2009-11-15 13:11:46 +00:00
  • 0a657760e0 Fixed a (false positive) compiler warning. Bart Van Assche 2009-11-14 19:30:27 +00:00
  • 4afbe006a5 Updated test instructions. Bart Van Assche 2009-11-11 18:56:05 +00:00
  • 5c208ca261 - Fixed races between command abortion code and regular command processing code. - Fixed the bug that new commands could be queued for a channel that was being closed. - Bug fix: ib_dma_unmap_sg() is now called by srpt_xmit_response() when aborting commands. - Changed default value of the kernel module parameter 'thread' from zero to one because using thread=0 can cause the Linux and the OFED SRP initiator to lock up -- see also http://bugzilla.kernel.org/show_bug.cgi?id=14235 and https://bugs.openfabrics.org/show_bug.cgi?id=1745. - Made disconnect logging more detailed -- added cm_id in output. - Changed argument type of second argument of srpt_release_channel() from int to bool. - Eliminated srpt_abort_scst_cmd's 'tell_initiator' argument because this argument always has the same value (i.e. true). - Simplified abortion of commands in state SRPT_STATE_NEW. Simplified implementation of srpt_on_free_cmd(). Bart Van Assche 2009-11-11 18:44:05 +00:00
  • b2223e664b Updated kernel versions 2.6.27.38 and 2.6.31.5 to 2.6.28.39 and 2.6.31.6 respectively. Bart Van Assche 2009-11-10 11:59:00 +00:00
  • cf54a13171 Web fixes from Daniel Fernandes <dfernandes1978@hotmail.com> Vladislav Bolkhovitin 2009-11-09 20:05:52 +00:00
  • e63a865d23 BIDI fixes Vladislav Bolkhovitin 2009-11-09 13:53:57 +00:00
  • a6e1b94e5b - Changed default maximum control message size from 996 to 4096 bytes. As a result, the maximum number of sg list elements that an initiator can pass in one message increased from 58 to 251. - Added kernel module parameter srp_max_message_size, which allows to specify the maximum size of SRP control messages. Bart Van Assche 2009-11-07 11:56:51 +00:00
  • 2d083413a1 Fixed a sparse error that was introduced in r1324. Bart Van Assche 2009-11-07 10:15:23 +00:00
  • eaed144f79 Fix for user space part of the iSCSI target, which refused connections in the procfs mode. Vladislav Bolkhovitin 2009-11-06 19:28:27 +00:00
  • b5ea323927 Logging cleanup Vladislav Bolkhovitin 2009-11-06 19:05:09 +00:00
  • 6153a079a3 iSCSI target sysfs "enabled" attribute implemented Vladislav Bolkhovitin 2009-11-06 18:41:37 +00:00
  • 1ef8de232b kobject tricky lifetime races fixed Vladislav Bolkhovitin 2009-11-06 18:39:52 +00:00
  • 76390b3d31 Patch from Daniel Fernandes <dfernandes1978@hotmail.com> fixing more Validator errors. Vladislav Bolkhovitin 2009-11-06 18:32:39 +00:00
  • 4a72d7efa3 Added command-line option -w. Bart Van Assche 2009-11-06 13:17:14 +00:00
  • a3460da98d Data Domain added Vladislav Bolkhovitin 2009-11-06 11:55:00 +00:00
  • 1ca3ccd06f Patch from Daniel Fernandes <dfernandes1978@hotmail.com> fixing issues discovered by validator. Vladislav Bolkhovitin 2009-11-05 14:34:38 +00:00
  • df8d5bb882 SVN access changed to trunk/ Vladislav Bolkhovitin 2009-11-04 19:44:50 +00:00
  • 90aedbc026 Pranah Storage Technologies added Vladislav Bolkhovitin 2009-11-04 19:29:11 +00:00
  • 7f1e235150 Added command-line options -m and -M. Bart Van Assche 2009-11-04 12:52:17 +00:00
  • b92c7d1c6f The patch below converts the #ifndef SYMBOL / #define SYMBOL / #endif symbol constructs in the SCST source code into one of the following: * These three lines are removed entirely if the symbol definition is present in the Linux 2.6.18 source tree. * The definition has been made unconditional if the symbol definition is not yet present in the 2.6.31 source tree. * Converted the definition into #if LINUX_VERSION_CODE < KERNEL_VERSION(...) / #define SYMBOL / #endif if a definition for the symbol has been added between kernel versions 2.6.18 and 2.6.31. Vladislav Bolkhovitin 2009-11-03 18:56:19 +00:00
  • 02bdb07c7e Fixes possible crash, caused by q2t_sess_work_fn() called on several CPUs at time. Reported by Gal Rosen <galr@storwize.com> Vladislav Bolkhovitin 2009-11-03 18:44:09 +00:00
  • cdbd65daeb Removed trailing whitespace. Bart Van Assche 2009-11-03 07:46:39 +00:00
  • d5cd0c893a - Fixed uninitialized variable read that was introduced in r1307. - Introduced new kernel module parameter srp_max_rdma_size. This parameter allows to configure the maximum size of SRP RDMA transfers. Bart Van Assche 2009-11-02 19:45:45 +00:00
  • 18c99cc2cf Merge of r1300: Vladislav Bolkhovitin 2009-11-02 18:43:22 +00:00
  • 2d25617396 Disable inlining in the debug build Vladislav Bolkhovitin 2009-11-02 18:37:53 +00:00
  • 3af8bc9a4c Various sysfs related fixes Vladislav Bolkhovitin 2009-11-02 18:07:54 +00:00
  • b9f64a7c61 - Fixed a bug introduced in the previous commit. - Added more debug tracing related to command abortion. Bart Van Assche 2009-11-01 20:11:30 +00:00
  • c9f49128bb Fixed the following issues found by Vladislav Bolkhovitin via source reading: - Inserted tests such that ib_dma_unmap_sg() is never called with a NULL second argument. - Fixed race conditions triggered by ioctx->state manipulations. Bart Van Assche 2009-11-01 17:01:08 +00:00
  • b9a0eb9386 Updates for the OFED 1.5 + kernel 2.6.30 combination. Bart Van Assche 2009-10-31 13:06:39 +00:00
  • 977a20ccba ISCSI target sysfs implementation, except "enabled" attribute. Vladislav Bolkhovitin 2009-10-30 20:15:38 +00:00