352280 |
13-Sep-2019 |
bapt |
MFC r352119-r352124, r352130
Stop linking to libl by specifying we do not need yywrap |
332595 |
16-Apr-2018 |
trasz |
MFC r328337:
Add missing SPDX tags for ctld(8). |
332583 |
16-Apr-2018 |
trasz |
MFC r325390:
Use proper naming in a debug message. |
330449 |
05-Mar-2018 |
eadler |
MFC r326276:
various: general adoption of SPDX licensing ID tags.
Mainly focus on files that use BSD 2-Clause license, however the tool I was using misidentified many licenses so this was mostly a manual - error prone - task.
The Software Package Data Exchange (SPDX) group provides a specification to make it easier for automated tools to detect and summarize well known opensource licenses. We are gradually adopting the specification, noting that the tags are considered only advisory and do not, in any way, superceed or replace the license texts.
No functional change intended. |
324720 |
18-Oct-2017 |
bapt |
MFC r324623:
Fix ctld segfaulting when using ucl conf file format and having duplicated lun or target
Submitted by: Nikita Kozlov <nikita.kozlov at blade-group.com> Sponsored by: blade Differential Revision: https://reviews.freebsd.org/D12646 |
319619 |
06-Jun-2017 |
trasz |
MFC r319072:
Make ctld(8) mention cfiscsi(4).
Approved by: re (marius) |
319152 |
29-May-2017 |
ngie |
MFC r314659,r314676:
r314659:
usr.sbin: normalize paths using SRCTOP-relative paths or :H when possible
This simplifies make logic/output
r314676:
Fix build after r314656
Some of the changes I introduced to use .ALLSRC were correct in spirit, but incorrect in reality -- in particular, ../Makefile.inc hadn't been pulled in via bsd.init.mk (via bsd.lib.mk, bsd.prog.mk), so the value of .ALLSRC (evaluated immediately) was empty. .include bsd.init.mk explicitly so we can be certain that the values used as dependencies in the targets are defined when the target recipe has been evaluated.
Reminder: thou shalt separate out separate functional changes before committing them.
(YUGE) Pointyhat to: ngie In collaboration with: bdrewery |
319119 |
29-May-2017 |
ngie |
MFC r314659,r314676:
r314659:
usr.sbin: normalize paths using SRCTOP-relative paths or :H when possible
This simplifies make logic/output
r314676:
Fix build after r314656
Some of the changes I introduced to use .ALLSRC were correct in spirit, but incorrect in reality -- in particular, ../Makefile.inc hadn't been pulled in via bsd.init.mk (via bsd.lib.mk, bsd.prog.mk), so the value of .ALLSRC (evaluated immediately) was empty. .include bsd.init.mk explicitly so we can be certain that the values used as dependencies in the targets are defined when the target recipe has been evaluated.
Reminder: thou shalt separate out separate functional changes before committing them.
(YUGE) Pointyhat to: ngie In collaboration with: bdrewery |
317351 |
24-Apr-2017 |
mav |
MFC r316677: Do not register in CTL portal groups without portals.
From config synthax point of view such portal groups are not incorrect, but they are useless since can not receive any connection. And since CTL port resource is very limited, it is good to save it. |
317062 |
17-Apr-2017 |
araujo |
MFC r314987-r314989, r314993-r314994, r315046-r315049, r315102
r314987: Use nitems() from sys/param.h.
Reviewed by: asomers MFC after: 3 weeks. Differential Revision: https://reviews.freebsd.org/D9944
r314988: Use nitems() from sys/param.h.
Reviewed by: ume MFC after: 3 weeks. Differential Revision: https://reviews.freebsd.org/D9938
r314989: Use nitems() from sys/param.h and also remove the cast.
Reviewed by: markj MFC after: 3 weeks. Differential Revision: https://reviews.freebsd.org/D9937
r314993: Use nitems() from sys/param.h.
Reviewed by: ngie MFC after: 3 weeks. Differential Revision: https://reviews.freebsd.org/D9936
r314994: Use nitems() from sys/param.h.
Reviewed by: ngie MFC after: 3 weeks. Differential Revision: https://reviews.freebsd.org/D9940
r315046: Use nitems() from sys/param.h
Reviewed by: jhb MFC after: 3 weeks. Differential Revision: https://reviews.freebsd.org/D9941
r315047: Use nitems() from sys/param.h.
Reviewed by: np MFC after: 3 weeks. Differential Revision: https://reviews.freebsd.org/D9945
r315048: Use nitems() from sys/param.h.
MFC after: 3 weeks.
r315049: Bring back the cast removed in my previous commit to allow us build vmstat with WARNS 2. This cast was first introduced at r87690.
Reported by: bde, pfg and ngie MFC after: 3 weeks.
r315102: Use nitems() from sys/param.h.
Reviewed by: ngie Differential Revision: https://reviews.freebsd.org/D9943 |
316252 |
30-Mar-2017 |
ngie |
MFC r316050,r316051:
r316050:
ctld: sort #includes per style(9)
- Only include sys/types.h or sys/param.h, not both. - Sort alphabetically.
r316051:
ctldadm: sort #includes per style(9)
- Only include sys/types.h or sys/param.h, not both. - Sort alphabetically. |
316230 |
30-Mar-2017 |
ngie |
MFC r316049:
Allow WARNS to be overridden by the end-user |
311865 |
10-Jan-2017 |
mav |
MFC r310633: Add MAX_LUNS overflow safety checks.
While this MAX_LUNS limitation is too synthetic and should be removed, it is better to enforce it while it is here. |
309868 |
12-Dec-2016 |
delphij |
MFC r309241,309243:
Plug memory leaks. |
308645 |
14-Nov-2016 |
mav |
MFC r308133, r308134: Fix wrong copy/paste in error message. |
307697 |
21-Oct-2016 |
araujo |
MFC r303541, r303542, r303543, r303545, r303546, r303547, r304225, r304226, r304605, r304676, r305212, r305863:
r303541: Use nitems() from sys/param.h.
Sponsored by: gandi.net (BSD Day Taiwan) MFC after: 2 weeks.
r303542: Use nitems() from sys/param.h.
MFC after: 2 weeks. Sponsored by: gandi.net (BSD Day Taiwan)
r303543: Use nitems() from sys/param.h.
MFC after: 2 weeks. Sponsored by: gandi.net (BSD Day Taiwan)
r303545: Use nitems() from sys/param.h.
MFC after: 2 weeks. Sponsored by: gandi.net (BSD Day Taiwan)
r303546: Use nitems() from sys/param.h.
MFC after: 2 weeks. Sponsored by: gandi.net (BSD Day Taiwan)
r303547: Use nitems() from sys/param.h.
MFC after: 2 weeks. Sponsored by: gandi.net (BSD Day Taiwan)
r304225: Use nitems() from sys/param.h.
MFC after: 2 weeks.
r304226: Use nitems() from sys/param.h.
MFC after: 2 weeks.
r304605: Fix calloc(3) argument order.
Reviewed by: trasz MFC after: 4 weeks. Differential Revision: https://reviews.freebsd.org/D7532
r304676: Fix calloc(3) argument order.
MFC after: 4 weeks.
r305212: - Invert calloc(3) argument order.
MFC after: 4 weeks
r305863: Invert calloc(3) argument order.
Reviewed by: ed. MFC after: 4 weeks. Differential Revision: https://reviews.freebsd.org/D7902 |
303311 |
25-Jul-2016 |
np |
MFC r303199:
ctld(8): Fix MaxBurstLength negotiation.
The target must reply with the selected value of MaxBurstSize instead of just echoing back the initiator's offered value.
Reviewed by: mav@ Approved by: re@ (gjb@) Sponsored by: Chelsio Communications Differential Revision: https://reviews.freebsd.org/D7278 |
303262 |
24-Jul-2016 |
trasz |
MFC r303132:
Add documentation for "ctld -u", the UCL configuration file format.
Submitted by: jceel@ Approved by: re (kib) MFC after: 1 month Differential Revision: https://reviews.freebsd.org/D7222 |
302663 |
12-Jul-2016 |
trasz |
MFC r302503:
Fix ctld(8) to not exit when reloading configuration with invalid initiator-portal clause.
Approved by: re (gjb) |
302408 |
08-Jul-2016 |
gjb |
Copy head@r302406 to stable/11 as part of the 11.0-RELEASE cycle. Prune svn:mergeinfo from the new branch, as nothing has been merged here.
Additional commits post-branch will follow.
Approved by: re (implicit) Sponsored by: The FreeBSD Foundation |
301720 |
09-Jun-2016 |
trasz |
Don't cap FirstBurstLength to maximum MaxRecvDataSegmentLength claimed by the offload driver; there is no reason to do so, and it actually harms performance.
MFC after: 1 month
|
301437 |
05-Jun-2016 |
trasz |
Report negotiated MaxBurstLength and FirstBurstLength in "iscsictl -v" and "ctladm islist -v" outputs.
MFC after: 1 month
|
301118 |
01-Jun-2016 |
trasz |
Describe default value for "offload".
MFC after: 1 month
|
300210 |
19-May-2016 |
trasz |
Fix error message.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
299894 |
16-May-2016 |
truckman |
pdu_delete(request) frees request, so move the call after login_new_response(request) to avoid a use-after-free error
Reported by: Coverity Reviewed by: 1331219, 1331220
|
297860 |
12-Apr-2016 |
trasz |
Make the usage() mention the -u option added in r295212.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
296897 |
15-Mar-2016 |
trasz |
Use the log_errno value passed to the function instead of the errno global variable.
Suggested by: Daniel Braniss MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
296808 |
13-Mar-2016 |
jceel |
Check value returned from ucl_parser_add_file().
Reviewed by: trasz Approved by: trasz (mentor) MFC after: 1 month Sponsored by: iXsystems, Inc.
|
295989 |
24-Feb-2016 |
bdrewery |
DIRDEPS_BUILD: Regenerate without local dependencies.
These are no longer needed after the recent 'beforebuild: depend' changes and hooking DIRDEPS_BUILD into a subset of FAST_DEPEND which supports skipping 'make depend'.
Sponsored by: EMC / Isilon Storage Division
|
295650 |
16-Feb-2016 |
bdrewery |
DIRDEPS_BUILD: Update dependencies.
Sponsored by: EMC / Isilon Storage Division
|
295562 |
12-Feb-2016 |
mav |
Don't check CmdSN for initial login request.
This fixes connection errors for some initiators not starting CmdSN from zero.
While there, fix wrong status details reported for couple errors.
MFC after: 3 days
|
295212 |
03-Feb-2016 |
jceel |
Add an additional, libucl-based configuration file parser to ctld.
Default ctld behavior remains unchanged - libucl parser can be selected explicitly by adding -u switch to ctld command line.
Reviewed by: trasz Approved by: trasz (mentor) MFC after: 1 month Relnotes: yes Sponsored by: iXsystems, Inc. Differential Revision: https://reviews.freebsd.org/D4534
|
291563 |
01-Dec-2015 |
bdrewery |
META MODE: Update dependencies with 'the-lot' and add missing directories.
This is not properly respecting WITHOUT or ARCH dependencies in target/. Doing so requires a massive effort to rework targets/ to do so. A better approach will be to either include the SUBDIR Makefiles directly and map to DIRDEPS or just dynamically lookup the SUBDIR. These lose the benefit of having a userland/lib, userland/libexec, etc, though and results in a massive package. The current implementation of targets/ is very unmaintainable.
Currently rescue/rescue and sys/modules are still not connected.
Sponsored by: EMC / Isilon Storage Division
|
290824 |
14-Nov-2015 |
trasz |
Add missing error checks.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
290823 |
14-Nov-2015 |
trasz |
Cosmetics; no functional changes.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
290821 |
14-Nov-2015 |
trasz |
Don't try to avoid calling free(3) with NULL argument.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
290615 |
09-Nov-2015 |
mav |
Introduce portal group options in ctl.conf.
While CTL has concept of port options, used at least for iSCSI ports now, before this change it was impossible to set them manually. There still no user-configurable port options now, but I am planning to change that.
|
289677 |
21-Oct-2015 |
eadler |
Fix a ton of speelling errors
arc lint is helpful
Reviewed By: allanjude, wblock, #manpages, chris@bsdjunk.com Differential Revision: https://reviews.freebsd.org/D3337
|
288488 |
02-Oct-2015 |
mav |
Document CD block size of 2048.
|
288486 |
02-Oct-2015 |
mav |
Set default block size for CD to expected 2048 bytes.
|
288310 |
27-Sep-2015 |
mav |
Add to CTL initial support for CDROMs and removable devices.
Relnotes: yes
|
288259 |
26-Sep-2015 |
mav |
Remove some duplicate, legacy, dead and questionable code.
|
288208 |
25-Sep-2015 |
jpaetzel |
Fix typo.
Sponsored by: iXsystems
|
288061 |
21-Sep-2015 |
mav |
Make cltd ignore HA ports.
|
287823 |
15-Sep-2015 |
mav |
Add ctl-lun config option for consistency in HA setups.
|
287766 |
13-Sep-2015 |
mav |
Add negotiation of iSCSIProtocolLevel to 2 (RFC7144).
We may need to pass negotiated value to kernel level, but so far it is not necessary, since it does not use any new features without request.
|
287757 |
13-Sep-2015 |
mav |
When updating port, apply only change of LUN map, not whole.
|
287534 |
07-Sep-2015 |
mav |
Add two new portal group options "tag" and "foreign".
They are going to be useful in clustered setups.
|
287500 |
06-Sep-2015 |
mav |
Allow LUN options modification via CTL_LUNREQ_MODIFY.
Not all changes take effect, but that is a different question.
|
287459 |
04-Sep-2015 |
mav |
Another addition to r287455.
|
287431 |
03-Sep-2015 |
trasz |
Use proper term in the ctld(8) and iscsid(8) man pages.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
287430 |
03-Sep-2015 |
trasz |
Simplify the introductory example in ctl.conf(5) down to absolute basics. The more complicated cases - like how to use physical ports - are explained later, in the "EXAMPLES" section.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
287102 |
24-Aug-2015 |
trasz |
Some random tweaks to ctl.conf(5).
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
286806 |
15-Aug-2015 |
mav |
Drop "internal" CTL frontend.
Its idea was to be a simple initiator and execute several commands from kernel level, but FreeBSD never had consumer for that functionality, while its implementation polluted many unrelated places..
|
286462 |
08-Aug-2015 |
mav |
Refactor early stages of security negotiation.
MFC after: 2 weeks
|
286456 |
08-Aug-2015 |
mav |
Remove some code duplication.
MFC after: 1 week
|
285086 |
03-Jul-2015 |
trasz |
Remove OpenSSL dependency from iscsid(8) and ctld(8).
Differential Revision: https://reviews.freebsd.org/D2866 Submitted by: Tony Morlan <tony at scroner.com> (earlier version) Reviewed by: bapt@, delphij@ MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
284875 |
26-Jun-2015 |
mav |
Plug minor memory leak at r284765.
Reported by: Coverity CID: 1308401 MFC after: 3 days
|
284765 |
24-Jun-2015 |
mav |
Teach ctld about CTL's physical_port and virtual_port fields.
This allows ctld to work with isp(4) virtual ports, specifying them as isp0/1, isp0/2, etc. There are still problems on isp(4) layer with disabling those ports after enabling, but hopefully they can be fixed.
MFC after: 3 days Sponsored by: iXsystems, Inc.
|
284542 |
18-Jun-2015 |
trasz |
Allow '@' in unquoted strings, such as with the "path" statement. Note that one can use any character they like by using double quotes.
PR: 200895 MFC after: 2 weeks Sponsored by: The FreeBSD Foundation
|
283898 |
02-Jun-2015 |
mav |
Fix bug in r278331: unlike some other options MaxRecvDataSegmentLength is not negotiated as minimum, but only reported by sides to each other.
MFC after: 1 week
|
283285 |
22-May-2015 |
trasz |
Advertise ctlstat(8) a little better.
MFC after: 2 weeks Sponsored by: The FreeBSD Foundation
|
282086 |
27-Apr-2015 |
trasz |
Make setproctitle(3) work in Capsicum capability mode. This makes ctld(8) child processes to indicate initiator address and name in their titles, similar to what iscsid(8) child processes do.
PR: 181352 Differential Revision: https://reviews.freebsd.org/D2363 Reviewed by: rwatson@, mjg@ MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
281820 |
21-Apr-2015 |
trasz |
Add hint about "volmode=dev" to ctl.conf(5).
Differential Revision: https://reviews.freebsd.org/D2328 Reviewed by: allanjude@, bcr@ MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
281532 |
14-Apr-2015 |
delphij |
Eliminate unused headers.
|
281163 |
06-Apr-2015 |
mav |
Make ctld to not exit on ECONNABORTED on accept().
That is not really an error for the main process.
MFC after: 1 week Sponsored by: iXsystems, Inc.
|
281084 |
04-Apr-2015 |
jpaetzel |
Fix thinko/copypaste error.
When checking the length of the mutual secret password the variable for the secret password was used by mistake. This resulted in ctld never warning about the length of the mutual secret being wrong even if it was.
MFC after: 3 days Sponsored by: iXsystems
|
279591 |
04-Mar-2015 |
mav |
Add example configuration for FibreChannel ports.
MFC after: 1 week
|
279590 |
04-Mar-2015 |
mav |
If target name starts with "naa.", set it as WWNN for CTL port.
MFC after: 1 week Sponsored by: iXsystems, Inc.
|
279589 |
04-Mar-2015 |
mav |
Fix handling of queued text and logout requests.
While it may have little sense, text and logout requests can be queued. If they are, they consume cmdsn, so we should increment our conn_cmdsn.
MFC after: 1 week
|
279392 |
28-Feb-2015 |
trasz |
Move the "offload" clause from the target section to portal-group section; it makes more sense there.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
279314 |
26-Feb-2015 |
trasz |
Add missing error check.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
279277 |
25-Feb-2015 |
mav |
Fix memory leak on incorrect initiator portal.
Submitted by: Dmitry Luhtionov <dmitryluhtionov@gmail.com> MFC after: 3 days
|
279276 |
25-Feb-2015 |
mav |
Add checks for malloc() failures.
Submitted by: Dmitry Luhtionov <dmitryluhtionov@gmail.com> MFC after: 3 days
|
278622 |
12-Feb-2015 |
trasz |
Remove unused code.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
278594 |
11-Feb-2015 |
mav |
Add missing asprintf() status checks.
MFC after: 2 weeks
|
278585 |
11-Feb-2015 |
trasz |
Fix formatting.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
278354 |
07-Feb-2015 |
mav |
Teach ctld(8) to control non-iSCSI CTL ports.
This change introduces new target option "port", that assigns current target to specified CTL port. On config application ctld(8) will apply LUN mapping according to target configuration to specified port and bring the port up. On shutdown cltd(8) will remove the mapping and put the port down.
This change allows to configure both iSCSI and FibreChannel targets in the same configuration file in alike way.
Kernel side support was added earlier at r278037.
MFC after: 2 weeks Relnotes: yes Sponsored by: iXsystems, Inc.
|
278331 |
06-Feb-2015 |
trasz |
Make it possible to set (via ctl.conf(5)) and query (via ctladm islist -v) target iSCSI offload. Add mechanism to query maximum receive data segment size supported by chosen hardware offload module, and use it in ctld(8) to determine the value to advertise to the other side.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
278322 |
06-Feb-2015 |
mav |
Add support for multiple portal groups per target.
This change allows multiple "portal-group" options to be specified per target. Each of them may include new optional auth-group name parameter to override per-target auth parameters for specific portal group.
Kernel side support was added earlier at r278161.
MFC after: 2 weeks Sponsored by: iXsystems, Inc.
|
278161 |
03-Feb-2015 |
mav |
Bring some more order into iSCSI portal group tags support.
While ctld(8) still does not allow multiple portal groups per target to be configured, kernel should now be able to handle it.
MFC after: 2 weeks Sponsored by: iXsystems, Inc.
|
278037 |
01-Feb-2015 |
mav |
CTL LUN mapping rewrite.
Replace iSCSI-specific LUN mapping mechanism with new one, working for any ports. By default all ports are created without LUN mapping, exposing all CTL LUNs as before. But, if needed, LUN mapping can be manually set on per-port basis via ctladm. For its iSCSI ports ctld does it via ioctl(2). The next step will be to teach ctld to work with FibreChannel ports also.
Respecting additional flexibility of the new mechanism, ctl.conf now allows alternative syntax for LUN definition. LUNs can now be defined in global context, and then referenced from targets by unique name, as needed. It allows same LUN to be exposed several times via multiple targets.
While there, increase limit for LUNs per target in ctld from 256 to 1024. Some initiators do not support LUNs above 255, but that is not our problem.
Discussed with: trasz MFC after: 2 weeks Relnotes: yes Sponsored by: iXsystems, Inc.
|
275925 |
19-Dec-2014 |
mav |
Slightly polish iSCSI parameters negotiation.
MFC after: 1 week
|
275864 |
17-Dec-2014 |
mav |
Make sequence numbers checks more strict.
While we don't support MCS, hole in received sequence numbers may mean only PDU loss. While we don't support lost PDU recovery, terminate the connection to avoid stuck commands.
While there, improve handling of sequence numbers wrap after 2^32 PDUs.
MFC after: 2 weeks
|
275452 |
03-Dec-2014 |
mav |
Do not corrupt the listen string when parsing it.
This fixes problem with ctld reload when it is configured to listen on two portals with same IP, but different ports.
MFC after: 1 week
|
275399 |
02-Dec-2014 |
trasz |
Fix null pointer dereference.
MFC after: 2 weeks Coverity CID: 1256497 Sponsored by: The FreeBSD Foundation
|
275054 |
25-Nov-2014 |
bapt |
Convert usr.sbin to LIBADD Reduce overlinking
|
274954 |
24-Nov-2014 |
trasz |
Remove the pidfile clause from the ctl.conf example. Why do everyone just copy/paste this line into their configs, it's beyond me.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
274853 |
22-Nov-2014 |
mav |
For both iSCSI initiator and target increase socket buffer sizes before establishing connection.
This is a workaround for Chelsio TOE driver, that does not update socket buffer size in hardware after connection established, and unless that is done beforehand, kernel code will stuck, attempting to send/receive full PDU at once.
MFC after: 1 week
|
274804 |
21-Nov-2014 |
mav |
In conf_apply() remove iSCSI ports from kernel before removing LUNs.
Previous order confused initiators with messages about "removed" LUNs during simple ctld restart without any real config change. After this commit initiators only reestablish lost connection, receive "Power on occurred" UNIT ATTENTION status and continue normal operation.
MFC after: 1 month
|
274797 |
21-Nov-2014 |
trasz |
Fix error handling.
MFC after: 1 month Coverity CID: 1249768 Sponsored by: The FreeBSD Foundation
|
274791 |
21-Nov-2014 |
trasz |
Add missing error checking for kernel_port_{add,remove}(). Both can fail for reasons yet unknown; don't make it increment cumulated_error as a kind of temporary workaround.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
274328 |
09-Nov-2014 |
mav |
Make both iSCSI initiator and target support base64 encoded CHAP data.
While all tested initiators and targets use hex-encoded CHAP data, RFC also allows base64 encoding there, and Microsoft certificaition tool uses it.
Reviewed by: trasz (earlier version) MFC after: 2 weeks Sponsored by: iXsystems, Inc.
|
274310 |
09-Nov-2014 |
trasz |
Add HISTORY section to ctld(8).
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
274309 |
09-Nov-2014 |
trasz |
Fix several nits in redirection handling - don't use wrong CSG, and avoid use-after-free.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
274308 |
09-Nov-2014 |
trasz |
Add support for sending redirections to iSCSI target.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
274278 |
08-Nov-2014 |
trasz |
Improve wording in ctl.conf(5).
Differential Revision: https://reviews.freebsd.org/D1020 Reviewed by: bcr@ (earlier version), wblock@ MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
274277 |
08-Nov-2014 |
trasz |
ctld(8) doesn't require -lcam or -lssl; remove those from DPADD and LDADD.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
274248 |
07-Nov-2014 |
mav |
Remove unused assignments, noticed by Clang analyzer.
MFC after: 1 week
|
273822 |
29-Oct-2014 |
trasz |
Fix iscsictl(8) and ctld(8) to correctly handle Windows newlines (CRLF) in iscsi.conf and ctl.conf.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
273821 |
29-Oct-2014 |
trasz |
Keep the token list sorted.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
273820 |
29-Oct-2014 |
trasz |
Make it possible to optionally use semicolon to separate statements. This makes it possible to format stuff like this:
target xxx { lun 0 { path /foo/bar; size 4G; } }
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
273816 |
29-Oct-2014 |
trasz |
Simplify code; no functional changes.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
273813 |
29-Oct-2014 |
trasz |
Add discovery-filter. This makes it possible to restrict which targets are returned during discovery based on initiator portal, name, and CHAP credentials.
Reviewed by: mav@ MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
273793 |
28-Oct-2014 |
trasz |
Document iSNS defaults.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
273770 |
28-Oct-2014 |
trasz |
Fix build after previous commit. While here, improve error messages.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
273768 |
28-Oct-2014 |
trasz |
Remove the distinction between strings and numbers from ctld(8) yacc parser. This fixes problems with passing strings that look like numbers to clauses that expect strings; previously it caused syntax errors and had to be worked by user, using quotes. The workaround introduced in r267833 is no longer neccessary.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
273635 |
25-Oct-2014 |
mav |
Add basic iSNS client to the iSCSI target.
This makes ctld(8) register its iSCSI targets and portals on configured iSNS servers to allow initiators find them without active discovery.
Fetching of allowed initiators from iSNS is not implemented now, so target ACLs still should be configured manually.
Reviewed by: trasz@ MFC after: 1 month Sponsored by: iXsystems, Inc.
|
273585 |
24-Oct-2014 |
trasz |
Improve ctld.conf example.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
273584 |
24-Oct-2014 |
trasz |
Make the initiator-name and initiator-portal checks a little nicer.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
273583 |
24-Oct-2014 |
trasz |
Tidy up the login code; no functional changes.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
273543 |
23-Oct-2014 |
trasz |
Clean up (refactor) discovery a little; no functional changes.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
273470 |
22-Oct-2014 |
trasz |
Fix ctl.conf example to use proper paths to ZVOLs.
Sponsored by: The FreeBSD Foundation
|
273468 |
22-Oct-2014 |
trasz |
Remove misleading statement. Bump date.
MFC after: 1 month Sponsored by: FreeBSD Foundation
|
273467 |
22-Oct-2014 |
trasz |
Comment out parts about iSER; it's not implemented.
Sponsored by: The FreeBSD Foundation
|
273466 |
22-Oct-2014 |
trasz |
Remove spurious empty line.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
273465 |
22-Oct-2014 |
trasz |
Fix ctld(8) to not show the "auth-group <name> not assigned to any target" warning for auth-groups assigned to a portal-group.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
273464 |
22-Oct-2014 |
trasz |
Whitespace fixes.
MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
273459 |
22-Oct-2014 |
trasz |
Untangle iSCSI authentication code by splitting off the CHAP implementation.
Reviewed by: mav@ MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
272911 |
10-Oct-2014 |
mav |
Make ctld start even if some LUNs are unable to open backing storage.
Such LUNs will be visible to initiators, but return "not ready" status on media access commands. If backing storage become available later, `ctladm modify ...` or `service ctld reload` can trigger its reopen.
|
271797 |
18-Sep-2014 |
mav |
Make kernel to update LUN size from the backing storage on configuration reload also if that size was not specified in the new configuration.
Previously it happened only if size was explicitly changed in config.
MFC after: 3 days
|
271571 |
14-Sep-2014 |
trasz |
Fix two small nits in ctl.conf(5).
MFC after: 3 days Sponsored by: The FreeBSD Foundation
|
271560 |
13-Sep-2014 |
allanjude |
Resolve an ambiguity with the definition of a new auth-group Resolve a markup mistake
Reviewed by: trasz Approved by: bcr (mentor), wblock (mentor) Sponsored by: ScaleEngine Inc. CR: https://reviews.freebsd.org/D735
|
271446 |
12-Sep-2014 |
allanjude |
Fix minor syntax error
Submitted by: bjk Approved by: bcr (mentor) Sponsored by: ScaleEngine Inc.
|
271445 |
12-Sep-2014 |
allanjude |
Improve markup and language throughout the ctl.conf man page
Reviewed by: trasz Approved by: bcr (mentor) Sponsored by: ScaleEngine Inc.
|
271320 |
09-Sep-2014 |
trasz |
Use keys_add_int() where appropriate. No functional changes.
Sponsored by: The FreeBSD Foundation
|
271319 |
09-Sep-2014 |
trasz |
Fix ctld(8) to not forget to send TargetPortalGroupTag and TargetAlias when the initiator skips security negotiation. This fixes interoperability with Xtend SAN initiator.
PR: 193021 MFC after: 1 week Sponsored by: The FreeBSD Foundation
|
271187 |
06-Sep-2014 |
trasz |
Avoid ctld(8) crash on getaddrinfo(3) failure.
MFC after: 2 weeks Sponsored by: The FreeBSD Foundation
|
271170 |
05-Sep-2014 |
trasz |
Document initiator-portal netmask support.
MFC after: 2 weeks Sponsored by: The FreeBSD Foundation
|
271169 |
05-Sep-2014 |
trasz |
Turn two errors, which are possible to trigger only by bugs, into assertions.
Discussed with: mav@ MFC after: 2 weeks Sponsored by: The FreeBSD Foundation
|
271157 |
05-Sep-2014 |
trasz |
Fix typo.
MFC after: 2 weeks Sponsored by: The FreeBSD Foundation
|
270279 |
21-Aug-2014 |
trasz |
Make the iSCSI stack use __FBSDID() properly.
MFC after: 2 weeks Sponsored by: The FreeBSD Foundation
|
270179 |
19-Aug-2014 |
ngie |
Add missing libraries to DPADD; sort DPADD so DPADD and LDADD match up
This fixes "make checkdpadd"
Approved by: jmmv (mentor) MFC after: 2 weeks Phabric: D630 PR: 192765
|
269191 |
28-Jul-2014 |
mav |
Fix r269183 build woth GCC.
MFC after: 2 weeks
|
269183 |
28-Jul-2014 |
mav |
Add netmasks support to initiator-portal option.
MFC after: 2 weeks
|
268931 |
20-Jul-2014 |
wblock |
Correct spelling errors in ctld.8. While here, correct similar errors in ctl.conf.5 and fix a couple of contractions.
PR: 191984 Submitted by: olgeni MFC after: 1 week
|
268795 |
17-Jul-2014 |
mav |
Fix ctld crash on startup if target alias is not set.
MFC after: 3 days
|
268328 |
06-Jul-2014 |
mav |
Close race in r268291 between port destruction, delayed by sessions teardown, and new port creation during `service ctld restart`.
Close it by returning iSCSI port internal state, that allows to identify dying ports, which should not be counted as existing, from really alive.
|
268302 |
05-Jul-2014 |
mav |
Pass through iSCSI session ISID from LOGIN request to the CTL frontend.
ISID is an important part of initiator transport ID for iSCSI. It is not used now, but should be to properly implement persistent reservation.
|
268293 |
05-Jul-2014 |
mav |
Burry devid port method, which was a gross hack.
Instead make ports provide wanted port and target IDs, and LUNs provide wanted LUN IDs. After that core Device ID VPD code only had to link all of them together and add relative port and port group numbers.
LUN ID for iSCSI LUNs no longer created by CTL, but by ctld, and passed to CTL as "scsiname" LUN option. This makes LUNs to report the same set of IDs, independently from the port through which it is accessed, as required by SCSI specifications.
|
268291 |
05-Jul-2014 |
mav |
Create separate CTL port for every iSCSI target (and maybe portal group).
Having single port for all iSCSI connections makes problematic implementing some more advanced SCSI functionality in CTL, that require proper ports enumeration and identification.
This change extends CTL iSCSI API, making ctld daemon to control list of iSCSI ports in CTL. When new target is defined in config fine, ctld will create respective port in CTL. When target is removed -- port will be also removed after all active commands through that port properly aborted. This change require ctld to be rebuilt to match the kernel.
As a minor side effect, this allows to have iSCSI targets without LUNs. While that may look odd and not very useful, that is not incorrect.
|
267833 |
24-Jun-2014 |
jpaetzel |
Fix issues in config parser relating to lun serial numbers.
Without this fix some serial numbers needed to be quoted to avoid the config parser bailing out.
Submitted by: delphij Sponsored by: iXsystems
|
267668 |
20-Jun-2014 |
bapt |
use .Mt to mark up email addresses consistently (part2)
PR: 191174 Submitted by: Franco Fichtner <franco@lastsummer.de>
|
267648 |
19-Jun-2014 |
mav |
serial_num and device_id fields are not necessarily null-terminated.
Before this it was impossible to use all 16 bytes of serial number, and client always got serial number NULL-terminated, that is not required.
MFC after: 2 weeks
|
267606 |
18-Jun-2014 |
mav |
On discovery stage add set of TargetAddress keys to reply, reporting to the client all the portal groups addresses and ports.
Reviewed by: trasz@ MFC after: 2 weeks Sponsored by: iXsystems, Inc.
|
264877 |
24-Apr-2014 |
trasz |
Style fixes.
|
264535 |
16-Apr-2014 |
trasz |
English.
Sponsored by: The FreeBSD Foundation
|
264534 |
16-Apr-2014 |
trasz |
If we fail to create LUN, try again on next configuration reload.
Sponsored by: The FreeBSD Foundation
|
264533 |
16-Apr-2014 |
trasz |
Use consistent punctuation.
Sponsored by: The FreeBSD Foundation
|
264532 |
16-Apr-2014 |
trasz |
Use proper terminology in debug messages.
Sponsored by: The FreeBSD Foundation
|
264531 |
16-Apr-2014 |
trasz |
Constify.
Sponsored by: The FreeBSD Foundation
|
264530 |
16-Apr-2014 |
trasz |
Properly pass the initiator address when running in proxy mode.
Sponsored by: The FreeBSD Foundation
|
264529 |
16-Apr-2014 |
trasz |
Use socket address from accept(2) instead of retrieving it via getpeername(2).
Sponsored by: The FreeBSD Foundation
|
264528 |
16-Apr-2014 |
trasz |
Rework the way we enable CTL iSCSI port. Previously conf_apply() needed it to be already enabled, because listening in proxy mode requires it; however, it's conf_apply() that opens pidfiles, so it resulted in port being enabled before pidfile was opened. This was not so bad, but it was also disabled when pidfile couldn't be opened due to ctld already running; this means that starting second ctld instance screwed up the first.
Sponsored by: The FreeBSD Foundation
|
264526 |
16-Apr-2014 |
trasz |
Properly identify target portal when running in proxy mode. While here, remove CTL_ISCSI_CLOSE, it wasn't used or implemented anyway.
Sponsored by: The FreeBSD Foundation
|
264524 |
16-Apr-2014 |
trasz |
Make it possible for the iSCSI target side to operate in both normal and ICL_KERNEL_PROXY mode, and fix some bit rot so the latter actually works again.
Sponsored by: The FreeBSD Foundation
|
264500 |
15-Apr-2014 |
trasz |
Stop treating LUN 0 as mandatory. There is no reason to do that.
Suggested by: mav@ MFC after: 1 month Sponsored by: The FreeBSD Foundation
|
263356 |
19-Mar-2014 |
trasz |
Make the error message more clear.
Sponsored by: The FreeBSD Foundation
|
263234 |
16-Mar-2014 |
rwatson |
Update most userspace consumers of capability.h to use capsicum.h instead.
auditdistd is not updated as I will make the change upstream and then do a vendor import sometime in the next week or two.
MFC after: 3 weeks
|
261768 |
11-Feb-2014 |
trasz |
Make the debug messages during ctld(8) shutdown less confusing.
Sponsored by: The FreeBSD Foundation
|
261767 |
11-Feb-2014 |
trasz |
Show proper configuration file line numbers in error messages.
Sponsored by: The FreeBSD Foundation
|
261766 |
11-Feb-2014 |
trasz |
Make ctld error out on invalid characters in ctl.conf.
Sponsored by: The FreeBSD Foundation
|
261765 |
11-Feb-2014 |
trasz |
Improve error reporting.
Sponsored by: The FreeBSD Foundation
|
261764 |
11-Feb-2014 |
trasz |
So, it turns out SIGCHLD is discarded by default, so we have to set up a dummy handler to make it interrupt an ioctl(2) or select(2).
This makes those short-lived ctld(8) zombies disappear.
Sponsored by: The FreeBSD Foundation
|
261763 |
11-Feb-2014 |
trasz |
Use new auth-type "deny" instead of using "chap" with no chap entries; it's cleaner this way, and gives better feedback to the user.
Sponsored by: The FreeBSD Foundation
|
261762 |
11-Feb-2014 |
trasz |
Use "default" as default discovery-auth-group, instead of "no-access". It doesn't change visible behaviour, as previously auth-group "default" wasn't redefinable, so by default access was always denied.
Sponsored by: The FreeBSD Foundation
|
261761 |
11-Feb-2014 |
trasz |
Update manual page to match recent auth-group "default" changes.
Sponsored by: The FreeBSD Foundation
|
261760 |
11-Feb-2014 |
trasz |
Add a new auth-group "default", defaulting to deny, and make it possible to redefine it. From now on, assigning auth-group to a target is no longer mandatory.
Sponsored by: The FreeBSD Foundation
|
261759 |
11-Feb-2014 |
trasz |
Make it possible to redefine portal-group "default".
Sponsored by: The FreeBSD Foundation
|
261758 |
11-Feb-2014 |
trasz |
Add auth-type.
Sponsored by: The FreeBSD Foundation
|
261757 |
11-Feb-2014 |
trasz |
Rename a variable, no functional changes.
Sponsored by: The FreeBSD Foundation
|
261756 |
11-Feb-2014 |
trasz |
Yacc cleanup; no functional changes.
Sponsored by: The FreeBSD Foundation
|
261755 |
11-Feb-2014 |
trasz |
Make function ordering slightly more logical; no functional changes.
Sponsored by: The FreeBSD Foundation
|
261754 |
11-Feb-2014 |
trasz |
Implement initiator-name and initiator-portal restrictions.
Sponsored by: The FreeBSD Foundation
|
261753 |
11-Feb-2014 |
trasz |
Daemonize after applying configuration, not before, so that any problems - including "daemon already running" - are visible on the terminal and not just in logs.
Sponsored by: The FreeBSD Foundation
|
261752 |
11-Feb-2014 |
trasz |
Empty data segment during Login Phase is rather unlikely, but it's not, by itself, a protocol error. This fixes interoperability with OpenBSD initiator.
Sponsored by: The FreeBSD Foundation
|
261751 |
11-Feb-2014 |
trasz |
The log_whatever() routines don't accept NULL for format strings, so mark them as __printflike instead of__printf0like.
Sponsored by: The FreeBSD Foundation
|
261750 |
11-Feb-2014 |
trasz |
Improve check for duplicated paths. It shows the warning twice for every path (once for each duplicate found), but it should do for now.
Sponsored by: The FreeBSD Foundation
|
261749 |
11-Feb-2014 |
trasz |
Refactor.
Sponsored by: The FreeBSD Foundation
|
259182 |
10-Dec-2013 |
trasz |
Fix handling for empty auth-groups. Without it, ctld child process would either exit on assertion, or, if assertions are not enabled, fail to authenticate the target.
MFC after: 2 days Sponsored by: The FreeBSD Foundation
|
258842 |
02-Dec-2013 |
trasz |
Grammar fix.
MFC after: 3 days Sponsored by: The FreeBSD Foundation
|
258841 |
02-Dec-2013 |
trasz |
Fix typos.
MFC after: 3 days Sponsored by: The FreeBSD Foundation
|
257915 |
10-Nov-2013 |
trasz |
Fix typo.
Submitted by: feld@ MFC after: 3 days Sponsored by: The FreeBSD Foundation
|
256192 |
09-Oct-2013 |
trasz |
Plug memory leaks.
Coverity CID: 1087993, 1087992 Approved by: re (marius) Sponsored by: FreeBSD Foundation
|
256189 |
09-Oct-2013 |
trasz |
Fix off-by-one.
Coverity CID: 1011375 Approved by: re (glebius) Sponsored by: FreeBSD Foundation
|
255678 |
18-Sep-2013 |
trasz |
Fix several problems in the new iSCSI stack; this includes interoperability fix for LIO (Linux target), removing possibility for the target to avoid mutual CHAP by choosing to skip authentication altogether, and fixing truncated error messages in iscsictl(8) output. This also fixes several of the problems found with Coverity.
Note that this change requires world rebuild.
Coverity CID: 1088038, 1087998, 1087990, 1088004, 1088044, 1088041, 1088040 Approved by: re (blanket) Sponsored by: FreeBSD Foundation
|
255665 |
18-Sep-2013 |
trasz |
Make iscsictl(8) automatically try to load the iscsi module. While here, improve module loading in iscsid(8) and ctld(8).
Approved by: re (delphij)
|
255575 |
14-Sep-2013 |
joel |
Minor mdoc fixes.
Approved by: re (blanket)
|
255570 |
14-Sep-2013 |
trasz |
Bring in the new iSCSI target and initiator.
Reviewed by: ken (parts) Approved by: re (delphij) Sponsored by: FreeBSD Foundation
|