mirror of
https://github.com/SCST-project/scst.git
synced 2026-05-23 13:41:27 +00:00
svn+ssh://yanb123@svn.code.sf.net/p/scst/svn/trunk
........
r5409 | bvassche | 2014-04-06 23:13:53 +0300 (Sun, 06 Apr 2014) | 4 lines
scstadmin: Restore LUNs in "scstadmin -list_sessions" output
Signed-off-by: Dave Butler <tears.the@gmail.com>
........
r5410 | bvassche | 2014-04-06 23:26:08 +0300 (Sun, 06 Apr 2014) | 1 line
scstadmin: List keys alphabetically in the -list_sessions output
........
r5411 | vlnb | 2014-04-10 02:58:20 +0300 (Thu, 10 Apr 2014) | 10 lines
vdisk_blockio: Specify REQ_SYNC for synchronous I/O requests
Using READ_SYNC instead of READ increases the priority of read requests.
Using WRITE_SYNC instead of REQ_WRITE increases the priority of write
requests and avoids that the CFQ scheduler queues such writes waiting
for further write requests.
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
........
r5412 | vlnb | 2014-04-10 02:59:02 +0300 (Thu, 10 Apr 2014) | 18 lines
vdisk_blockio: Reenable COMPARE AND WRITE
The COMPARE AND WRITE implementation has been tested as follows against
an SCST vdisk_blockio device:
lba=7
bdev=$bdev
for ((i=0;i<4;i++)); do dd if=/dev/urandom of=b$i bs=8k count=1; done
for p in "0 1" "1 2" "2 3" "3 0"; do set $p; cat b$1 b$2 >b$1$2; done
dd if=/dev/urandom of=$bdev
dd if=b0 of=$bdev seek=$lba
md5sum $bdev
for ((i=0;i<10000;i++)); do for f in b01 b12 b23 b30; do sg_compare_and_write -l $lba -i $f -n 16 -x 16384 $bdev; done; done
md5sum $bdev
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
........
r5413 | bvassche | 2014-04-15 09:03:59 +0300 (Tue, 15 Apr 2014) | 1 line
ib_srpt: Fix a sparse warning
........
r5414 | vlnb | 2014-04-16 00:26:06 +0300 (Wed, 16 Apr 2014) | 5 lines
Fix READ(6)/WRITE(6)
LBA in those commands is 3 bytes long, not 2.
........
r5415 | vlnb | 2014-04-16 00:30:26 +0300 (Wed, 16 Apr 2014) | 3 lines
Add SYNCHRONIZE_CACHE(16)
........
r5416 | vlnb | 2014-04-16 01:04:54 +0300 (Wed, 16 Apr 2014) | 6 lines
Make HEAD OF QUEUE requests sync
Since the block layer has no way to specify bio as HQ, there's no
choice, but to use every measure to approximate it as close as possible.
........
r5417 | vlnb | 2014-04-16 01:17:34 +0300 (Wed, 16 Apr 2014) | 3 lines
Add NULLIO VERIFY
........
r5418 | vlnb | 2014-04-16 04:02:50 +0300 (Wed, 16 Apr 2014) | 3 lines
REPORT SUPPORTED OPERATION CODES added
........
r5419 | vlnb | 2014-04-16 04:05:49 +0300 (Wed, 16 Apr 2014) | 3 lines
Cleanup
........
r5420 | vlnb | 2014-04-16 05:24:44 +0300 (Wed, 16 Apr 2014) | 3 lines
Follow up for r5418: some cleanups and fixes
........
r5421 | bvassche | 2014-04-16 09:17:19 +0300 (Wed, 16 Apr 2014) | 6 lines
scst_targ: Fix a checkpatch complaint
Avoid that checkpatch reports the following message:
ERROR: space required before the open parenthesis '('
........
r5422 | bvassche | 2014-04-16 09:35:19 +0300 (Wed, 16 Apr 2014) | 1 line
scst_vdisk: Build fix for kernels < 2.6.36 (see also r5416)
........
r5423 | bvassche | 2014-04-16 10:30:34 +0300 (Wed, 16 Apr 2014) | 6 lines
ib_srpt: Disable RDMA access by the initiator
With the SRP protocol all RDMA operations are initiated by the target.
Since no RDMA operations are initiated by the initiator, do not grant
the initiator permission to submit RDMA reads or writes to the target.
........
r5424 | bvassche | 2014-04-16 11:01:58 +0300 (Wed, 16 Apr 2014) | 1 line
ib_srpt: Constify two arguments of srpt_get_nexus()
........
r5425 | bvassche | 2014-04-16 11:08:50 +0300 (Wed, 16 Apr 2014) | 1 line
ib_srpt: Clean up the code that prints the dgid during login
........
r5426 | bvassche | 2014-04-16 11:23:11 +0300 (Wed, 16 Apr 2014) | 1 line
ib_srpt: Cache P_Key lookups
........
r5427 | bvassche | 2014-04-16 12:14:31 +0300 (Wed, 16 Apr 2014) | 1 line
ib_srpt: Remove a superfluous assignment
........
r5428 | bvassche | 2014-04-16 12:31:56 +0300 (Wed, 16 Apr 2014) | 1 line
scst_vdisk: Avoid that checkpatch complains about unnecessary line continuations
........
r5429 | bvassche | 2014-04-16 12:35:18 +0300 (Wed, 16 Apr 2014) | 1 line
scst_lib: Avoid that checkpatch complains about unnecessary line continuations
........
r5430 | bvassche | 2014-04-17 09:09:41 +0300 (Thu, 17 Apr 2014) | 1 line
scst_vdisk: Kernel 2.6.27 build fix
........
r5431 | bvassche | 2014-04-17 09:12:07 +0300 (Thu, 17 Apr 2014) | 5 lines
scst_vdisk: Fix a kernel 2.6.27 compiler warning
Avoid that the compiler reports that variables 'start_sector' and 'nr_sects'
are set but not used when building against kernel 2.6.27.
........
r5432 | vlnb | 2014-04-18 03:32:51 +0300 (Fri, 18 Apr 2014) | 3 lines
Update to kernels 3.14
........
r5433 | bvassche | 2014-04-18 08:50:04 +0300 (Fri, 18 Apr 2014) | 1 line
nightly build: Add kernel 3.14 build infrastructure
........
r5434 | bvassche | 2014-04-18 08:54:29 +0300 (Fri, 18 Apr 2014) | 1 line
nightly build: Add kernel version 3.14.1 and update other kernel versions
........
r5435 | bvassche | 2014-04-18 08:56:35 +0300 (Fri, 18 Apr 2014) | 6 lines
iscsi-scst: Fix a checkpatch warning
Fix the following checkpatch 3.14 warning:
Unnecessary parentheses - maybe == should be = ?
........
r5436 | bvassche | 2014-04-18 09:03:34 +0300 (Fri, 18 Apr 2014) | 6 lines
scst.h: Fix a checkpatch warning
Fix the following checkpatch 3.14 warning:
Unnecessary space after function pointer name
........
r5437 | bvassche | 2014-04-18 09:08:07 +0300 (Fri, 18 Apr 2014) | 6 lines
scst: Fix a checkpatch warning
Fix the following checkpatch 3.14 warning:
Unnecessary space after function pointer name
........
r5438 | bvassche | 2014-04-18 09:13:03 +0300 (Fri, 18 Apr 2014) | 6 lines
scst: Fix a checkpatch 3.14 warning about whitespace
Fix the following checkpatch 3.14 warning:
missing space after return type
........
r5439 | bvassche | 2014-04-18 14:09:55 +0300 (Fri, 18 Apr 2014) | 1 line
ib_srpt: Update README
........
r5440 | bvassche | 2014-04-18 15:04:05 +0300 (Fri, 18 Apr 2014) | 2 lines
ib_srpt: Move IB/CM knowledge out of srpt_cm_req_recv()
........
r5441 | bvassche | 2014-04-18 15:08:00 +0300 (Fri, 18 Apr 2014) | 6 lines
ib_srpt: Remove a superfluous check from the REQ handler
ib_send_cm_rep() checks the connection state before sending a response.
Hence checking ch->state before calling ib_send_cm_rep() is superfluous,
so remove that check and also the locking that is no longer needed.
........
r5442 | bvassche | 2014-04-18 15:09:33 +0300 (Fri, 18 Apr 2014) | 5 lines
ib_srpt: Prepare RDMA/CM support
Move IB/CM members into a new struct. Report channel pointer instead of CM ID
pointer in diagnostic messages.
........
r5443 | bvassche | 2014-04-18 15:10:47 +0300 (Fri, 18 Apr 2014) | 5 lines
ib_srpt: Use a mutex instead of a spinlock to protect the channel list
This is allowed because all CM callback functions are invoked from thread
context.
........
r5444 | bvassche | 2014-04-18 15:11:18 +0300 (Fri, 18 Apr 2014) | 2 lines
ib_srpt: Move the code for checking the QP timeout
........
r5445 | bvassche | 2014-04-18 15:22:30 +0300 (Fri, 18 Apr 2014) | 4 lines
ib_srpt: Add RDMA/CM support
Or in other words, add RoCE and iWARP support.
........
r5446 | bvassche | 2014-04-18 15:38:39 +0300 (Fri, 18 Apr 2014) | 1 line
ib_srpt: Build fix for kernel versions < 3.0
........
r5447 | bvassche | 2014-04-18 15:44:48 +0300 (Fri, 18 Apr 2014) | 1 line
ib_srpt: RHEL 6.5 build fix
........
r5448 | bvassche | 2014-04-19 14:48:33 +0300 (Sat, 19 Apr 2014) | 1 line
scst_vdisk, COMPARE AND WRITE: Convert a kernel warning into a SCSI sense code
........
r5449 | bvassche | 2014-04-19 14:52:34 +0300 (Sat, 19 Apr 2014) | 33 lines
vdisk_blockio: Make COMPARE AND WRITE compatible with the scsi_debug driver
This patch fixes the following kernel oops:
BUG: unable to handle kernel paging request at ffffeae380000690
Call Trace:
[<ffffffff812f69f9>] sg_miter_next+0x9/0xd0
[<ffffffff812f6b60>] sg_copy_buffer+0xa0/0x100
[<ffffffffa0507fc6>] do_device_access.isra.8+0xa6/0x150 [scsi_debug]
[<ffffffffa05085f4>] resp_read+0xe4/0x240 [scsi_debug]
[<ffffffffa050b285>] scsi_debug_queuecommand_lck+0x11e5/0x2060 [scsi_debug]
[<ffffffffa050c130>] scsi_debug_queuecommand+0x30/0x48 [scsi_debug]
[<ffffffff813fe23f>] scsi_dispatch_cmd+0xaf/0x260
[<ffffffff814068ed>] scsi_request_fn+0x32d/0x540
[<ffffffff812bda2a>] __blk_run_queue+0x2a/0x40
[<ffffffff812c25a4>] blk_queue_bio+0x274/0x350
[<ffffffff812bd878>] generic_make_request+0xa8/0xf0
[<ffffffff812bd92c>] submit_bio+0x6c/0x140
[<ffffffffa06247f6>] blockio_rw_sync.isra.29+0x106/0x170 [scst_vdisk]
[<ffffffffa0629bf9>] vdisk_exec_caw+0xd9/0x3c0 [scst_vdisk]
[<ffffffffa061ff5e>] vdev_do_job+0x9e/0x320 [scst_vdisk]
[<ffffffffa0622c47>] non_fileio_exec+0x57/0xd0 [scst_vdisk]
[<ffffffffa054cde2>] scst_do_real_exec+0x92/0x3b0 [scst]
[<ffffffffa05502d2>] scst_exec_check_blocking+0xe2/0x300 [scst]
[<ffffffffa055066b>] scst_exec_check_sn+0x17b/0x2d0 [scst]
[<ffffffffa0553231>] scst_process_active_cmd+0x431/0x770 [scst]
[<ffffffffa0554a0a>] scst_do_job_active+0xea/0x180 [scst]
[<ffffffffa0554bc6>] scst_cmd_thread+0x126/0x290 [scst]
[<ffffffff81075241>] kthread+0xc1/0xe0
[<ffffffff81598ccc>] ret_from_fork+0x7c/0xb0
Reported-by: Sebastian Herbszt <herbszt@gmx.de>
........
r5450 | bvassche | 2014-04-20 09:24:23 +0300 (Sun, 20 Apr 2014) | 1 line
iscsi-scst/kernel/patches/put_page_callback-3.2.57.patch: Add
........
r5451 | bvassche | 2014-04-22 09:56:37 +0300 (Tue, 22 Apr 2014) | 1 line
scst: Revert r5438, a whitespace-only change
........
r5452 | bvassche | 2014-04-22 10:05:21 +0300 (Tue, 22 Apr 2014) | 1 line
scripts/run-regression-tests: Suppress the checkpatch warning "missing space after return type"
........
r5454 | bvassche | 2014-04-22 10:32:44 +0300 (Tue, 22 Apr 2014) | 1 line
scst/README: Update the section about Linux initiator ALUA support
........
git-svn-id: http://svn.code.sf.net/p/scst/svn/branches/iser@5455 d57e44dd-8a1f-0410-8b47-8ef2f437770f
SCST Local ...
Richard Sharpe, 30-Nov-2008
This is the SCST Local driver. Its function is to allow you to access devices
that are exported via SCST directly on the same Linux system that they are
exported from.
No assumptions are made in the code about the device types on the target, so
any device handlers that you load in SCST should be visible, including tapes
and so forth.
You can freely use any sg, sd, st, etc. devices imported from target,
except the following: you can't mount file systems or put swap on them
for all dev handlers, except BLOCKIO and pass-through, because it can
lead to recursive memory allocation deadlock. This is a limitation of
Linux memory/cache manager. See SCST README file for details. For
BLOCKIO and pass-through dev handlers there's no such limitation, so you
can freely mount file systems over them.
To build, simply issue 'make' in the scst_local directory.
Try 'modinfo scst_local' for a listing of module parameters so far.
Here is how I have used it so far:
1. Load up scst:
modprobe scst
modprobe scst_vdisk
2. Create a virtual disk (or your own device handler):
dd if=/dev/zero of=/some/path/vdisk1.img bs=16384 count=1000000
echo "add_device vm_disk1 filename=/some/path/vdisk1.img" >/sys/kernel/scst_tgt/handlers/vdisk_fileio/mgmt
3. Load the scst_local driver:
insmod scst_local
echo "add vm_disk1 0" >/sys/kernel/scst_tgt/targets/scst_local/scst_local_tgt/luns/mgmt
4. Check what you have
cat /proc/scsi/scsi
Attached devices:
Host: scsi0 Channel: 00 Id: 00 Lun: 00
Vendor: ATA Model: ST9320320AS Rev: 0303
Type: Direct-Access ANSI SCSI revision: 05
Host: scsi4 Channel: 00 Id: 00 Lun: 00
Vendor: TSSTcorp Model: CD/DVDW TS-L632D Rev: TO04
Type: CD-ROM ANSI SCSI revision: 05
Host: scsi7 Channel: 00 Id: 00 Lun: 00
Vendor: SCST_FIO Model: vm_disk1 Rev: 200
Type: Direct-Access ANSI SCSI revision: 04
Or instead of manually "add_device" in (2) and step (3) write a
scstadmin config:
HANDLER vdisk_fileio {
DEVICE vm_disk1 {
filename /some/path/vdisk1.img
}
}
TARGET_DRIVER scst_local {
TARGET scst_local_tgt {
LUN 0 vm_disk1
}
}
then:
insmod scst_local
scstadmin -config conf_file.cfg
More advanced examples:
For (3) you can:
insmod scst_local add_default_tgt=0
echo "add_target scst_local_tgt session_name=scst_local_host" >/sys/kernel/scst_tgt/targets/scst_local//mgmt
echo "add vm_disk1 0" >/sys/kernel/scst_tgt/targets/scst_local/scst_local_tgt/luns/mgmt
Scst_local module's parameter add_default_tgt disables creation of
default target "scst_local_tgt" and session "scst_local_host", so you
needed to create it manually.
There can be any number of targets and sessions created. Each SCST
session corresponds to SCSI host. You can change which LUNs assigned to
each session by using SCST access control. This mode is intended for
user space target drivers (see below).
Alternatively, you can write an scstadmin's config file conf_file.cfg:
HANDLER vdisk_fileio {
DEVICE vm_disk1 {
filename /some/path/vdisk1.img
}
}
TARGET_DRIVER scst_local {
TARGET scst_local_tgt {
session_name scst_local_host
LUN 0 vm_disk1
}
}
then:
insmod scst_local add_default_tgt=0
scstadmin -config conf_file.cfg
NOTE! Although scstadmin allows to create scst_local's sessions using
"session_name" expression, it doesn't save existing sessions during
writing config file by "write_config" command. If you need this
functionality, feel free to send a request for it in SCST development
mailing list.
5. Have fun.
Some of this was coded while in Santa Clara, some in Bangalore, and some in
Hyderabad. Noe doubt some will be coded on the way back to Santa Clara.
The code still has bugs, so if you encounter any, email me the fixes at:
realrichardsharpe@gmail.com
I am thinking of renaming this to something more interesting.
Sysfs interface
===============
See SCST's README for a common SCST sysfs description.
Root of this driver is /sys/kernel/scst_tgt/targets/scst_local. It has
the following additional entry:
- stats - read-only attribute with some statistical information.
Each target subdirectory contains the following additional entries:
- phys_transport_version - contains and allows to change physical
transport version descriptor. It determines by which physical
interface this target will look like. See SPC for more details. By
default, it is not defined (0).
- scsi_transport_version - contains and allows to change SCSI
transport version descriptor. It determines by which SCSI
transport this target will look like. See SPC for more details. By
default, it is SAS.
Each session subdirectory contains the following additional entries:
- transport_id - contains this host's TransportID. This TransportID
used to identify initiator in Persisten Reservation commands. If you
change scsi_transport_version for a target, make sure you set for all
its sessions correct TransportID. See SPC for more details.
- host - links to the corresponding SCSI host. Using it you can find
local sg/bsg/sd/etc. devices of this session. For instance, this
links points out to host12, so you can find your sg devices by:
$ lsscsi -g|grep "\[12:"
[12:0:0:0] disk SCST_FIO rd1 200 /dev/sdc /dev/sg2
[12:0:0:1] disk SCST_FIO nullio 200 /dev/sdd /dev/sg3
They are /dev/sg2 and /dev/sg3.
The following management commands available via /sys/kernel/scst_tgt/targets/scst_local/mgmt:
- add_target target_name [session_name=sess_name; [session_name=sess_name1;] [...]] -
creates a target with optionally one or more sessions.
- del_target target_name - deletes a target.
- add_session target_name session_name - adds to target target_name
session (SCSI host) with name session_name.
- del_session target_name session_name - deletes session session_name
from target target_name.
Note on performance
===================
Although this driver implemented in the most performance effective way,
including zero-copy passing data between SCSI/block subsystems and SCST,
in many cases it is NOT suited to measure performance as a NULL link.
For example, it is not suited for max IOPS measurements. This is because
for such cases not performance of the link between the target and
initiator is the bottleneck, but CPU or memory speed on the target or
initiator. For scst_local you have both initiator and target on the same
system, which means each your initiator and target are much less
CPU/memory powerful.
User space target drivers
=========================
Scst_local can be used to write full featured SCST target drivers in
user space:
1. For each SCSI target a user space target driver should create an
scst_local's target using "add_target" command.
2. Then the user space target driver should, if needed, set its SCSI and
physical transport version descriptors using attributes
scsi_transport_version and phys_transport_version correspondingly in
/sys/kernel/scst_tgt/targets/scst_local/target_name directory.
3. For incoming session (I_T nexus) from an initiator the user space
target driver should create scst_local's session using "add_session"
command.
4. Then, if needed, the user space target driver should set TransportID
for this session (I_T nexus) using attribute
/sys/kernel/scst_tgt/targets/scst_local/target_name/sessions/session_name/transport_id
5. Then the user space target driver should find out sg/bsg devices for
the LUNs the created session has using link
/sys/kernel/scst_tgt/targets/scst_local/target_name/sessions/session_name/host
as described above.
6. Then the user space target driver can start serving the initiator using
found sg/bsg devices.
For other connected initiators steps 3-6 should be repeated.
Compilation options
===================
There are the following compilation options, that could be commented
in/out in Makefile:
- CONFIG_SCST_LOCAL_DIRECT_PROCESSING - by default, when this option
is not defined, scst_local reschedules all commands for processing in
one of the SCST threads. If this option is defined, scst_local tries
to not do it, if possible (sometimes queuecommand() called under
various locks held), but instead process them in the submitter's
context. This is to increase performance, but as on 2.6.37 and below
Linux block layer doesn't work with such kind of reentrance, hence
this option disabled by default. Note! At the moment in
scst_estimate_context*() returning DIRECT contexts disabled, so this
option doesn't have any real effect.
Change log
==========
V0.1 24-Sep-2008 (Hyderabad) Initial coding, pretty chatty and messy,
but worked.
V0.2 25-Sep-2008 (Hong Kong) Cleaned up the code a lot, reduced the log
chatter, fixed a bug where multiple LUNs did not
work. Also, added logging control. Tested with
five virtual disks. They all came up as /dev/sdb
through /dev/sdf and I could dd to them. Also
fixed a bug preventing multiple adapters.
V0.3 26-Sep-2008 (Santa Clara) Added back a copyright plus cleaned up some
unused functions and structures.
V0.4 5-Oct-2008 (Santa Clara) Changed name to scst_local as suggested, cleaned
up some unused variables (made them used) and
change allocation to a kmem_cache pool.
V0.5 5-Oct-2008 (Santa Clara) Added mgmt commands to handle dev reset and
aborts. Not sure if aborts works. Also corrected
the version info and renamed readme to README.
V0.6 7-Oct-2008 (Santa Clara) Removed some redundant code and made some
changes suggested by Vladislav.
V0.7 11-Oct-2008 (Santa Clara) Moved into the scst tree. Cleaned up some
unused functions, used TRACE macros etc.
V0.9 30-Nov-2008 (Mtn View) Cleaned up an additional problem with symbols not
being defined in older version of the kernel. Also
fixed some English and cleaned up this doc.
V1.0 10-Sep-2010 (Moscow) Sysfs management added. Reviewed and cleaned up.
V2.1 Update for kernels up to 3.0. Cleanups.