summaryrefslogtreecommitdiffstats
path: root/usr.sbin/ctld
Commit message (Collapse)AuthorAgeFilesLines
* MFC r319072:trasz2017-06-061-2/+5
| | | | | | Make ctld(8) mention cfiscsi(4). Approved by: re (marius)
* MFC r314659,r314676:ngie2017-05-292-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | 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
* MFC r314659,r314676:ngie2017-05-291-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | 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
* MFC r316677: Do not register in CTL portal groups without portals.mav2017-04-242-6/+18
| | | | | | 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.
* MFC r314987-r314989, r314993-r314994, r315046-r315049, r315102araujo2017-04-171-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* MFC r316050,r316051:ngie2017-03-301-5/+4
| | | | | | | | | | | | | | | | 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.
* MFC r316049:ngie2017-03-301-1/+1
| | | | Allow WARNS to be overridden by the end-user
* MFC r310633: Add MAX_LUNS overflow safety checks.mav2017-01-102-4/+27
| | | | | While this MAX_LUNS limitation is too synthetic and should be removed, it is better to enforce it while it is here.
* MFC r309241,309243:delphij2016-12-121-1/+4
| | | | Plug memory leaks.
* MFC r308133, r308134: Fix wrong copy/paste in error message.mav2016-11-141-2/+2
|
* MFC r303541, r303542, r303543, r303545, r303546, r303547, r304225, r304226, ↵araujo2016-10-213-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* MFC r303199:np2016-07-251-1/+1
| | | | | | | | | | | | 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
* MFC r303132:trasz2016-07-242-2/+96
| | | | | | | | | 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
* MFC r302503:trasz2016-07-121-1/+1
| | | | | | | Fix ctld(8) to not exit when reloading configuration with invalid initiator-portal clause. Approved by: re (gjb)
* Don't cap FirstBurstLength to maximum MaxRecvDataSegmentLength claimedtrasz2016-06-092-6/+6
| | | | | | | by the offload driver; there is no reason to do so, and it actually harms performance. MFC after: 1 month
* Report negotiated MaxBurstLength and FirstBurstLength in "iscsictl -v"trasz2016-06-053-4/+3
| | | | | | and "ctladm islist -v" outputs. MFC after: 1 month
* Describe default value for "offload".trasz2016-06-011-0/+2
| | | | MFC after: 1 month
* Fix error message.trasz2016-05-191-2/+2
| | | | | MFC after: 1 month Sponsored by: The FreeBSD Foundation
* pdu_delete(request) frees request, so move the call aftertruckman2016-05-161-1/+1
| | | | | | | login_new_response(request) to avoid a use-after-free error Reported by: Coverity Reviewed by: 1331219, 1331220
* Make the usage() mention the -u option added in r295212.trasz2016-04-121-1/+1
| | | | | MFC after: 1 month Sponsored by: The FreeBSD Foundation
* Use the log_errno value passed to the function instead of the errno globaltrasz2016-03-151-6/+9
| | | | | | | | variable. Suggested by: Daniel Braniss MFC after: 1 month Sponsored by: The FreeBSD Foundation
* Check value returned from ucl_parser_add_file().jceel2016-03-131-2/+1
| | | | | | | Reviewed by: trasz Approved by: trasz (mentor) MFC after: 1 month Sponsored by: iXsystems, Inc.
* DIRDEPS_BUILD: Regenerate without local dependencies.bdrewery2016-02-241-6/+0
| | | | | | | | 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
* DIRDEPS_BUILD: Update dependencies.bdrewery2016-02-161-0/+2
| | | | Sponsored by: EMC / Isilon Storage Division
* Don't check CmdSN for initial login request.mav2016-02-121-3/+4
| | | | | | | | | 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
* Add an additional, libucl-based configuration file parser to ctld.jceel2016-02-035-93/+1023
| | | | | | | | | | | | 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
* META MODE: Update dependencies with 'the-lot' and add missing directories.bdrewery2015-12-011-0/+31
| | | | | | | | | | | | | | 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
* Add missing error checks.trasz2015-11-141-6/+12
| | | | | MFC after: 1 month Sponsored by: The FreeBSD Foundation
* Cosmetics; no functional changes.trasz2015-11-141-1/+2
| | | | | MFC after: 1 month Sponsored by: The FreeBSD Foundation
* Don't try to avoid calling free(3) with NULL argument.trasz2015-11-141-4/+2
| | | | | MFC after: 1 month Sponsored by: The FreeBSD Foundation
* Introduce portal group options in ctl.conf.mav2015-11-095-76/+98
| | | | | | 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.
* Fix a ton of speelling errorseadler2015-10-211-1/+1
| | | | | | | arc lint is helpful Reviewed By: allanjude, wblock, #manpages, chris@bsdjunk.com Differential Revision: https://reviews.freebsd.org/D3337
* Document CD block size of 2048.mav2015-10-021-1/+1
|
* Set default block size for CD to expected 2048 bytes.mav2015-10-022-1/+5
|
* Add to CTL initial support for CDROMs and removable devices.mav2015-09-276-4/+49
| | | | Relnotes: yes
* Remove some duplicate, legacy, dead and questionable code.mav2015-09-261-1/+1
|
* Fix typo.jpaetzel2015-09-251-1/+1
| | | | Sponsored by: iXsystems
* Make cltd ignore HA ports.mav2015-09-211-1/+7
|
* Add ctl-lun config option for consistency in HA setups.mav2015-09-155-3/+36
|
* Add negotiation of iSCSIProtocolLevel to 2 (RFC7144).mav2015-09-131-0/+5
| | | | | 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.
* When updating port, apply only change of LUN map, not whole.mav2015-09-133-3/+11
|
* Add two new portal group options "tag" and "foreign".mav2015-09-075-7/+58
| | | | They are going to be useful in clustered setups.
* Allow LUN options modification via CTL_LUNREQ_MODIFY.mav2015-09-063-14/+35
| | | | Not all changes take effect, but that is a different question.
* Another addition to r287455.mav2015-09-041-1/+0
|
* Use proper term in the ctld(8) and iscsid(8) man pages.trasz2015-09-031-2/+2
| | | | | MFC after: 1 month Sponsored by: The FreeBSD Foundation
* Simplify the introductory example in ctl.conf(5) down to absolutetrasz2015-09-031-7/+1
| | | | | | | | 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
* Some random tweaks to ctl.conf(5).trasz2015-08-241-12/+38
| | | | | MFC after: 1 month Sponsored by: The FreeBSD Foundation
* Drop "internal" CTL frontend.mav2015-08-151-1/+0
| | | | | | 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..
* Refactor early stages of security negotiation.mav2015-08-081-73/+77
| | | | MFC after: 2 weeks
* Remove some code duplication.mav2015-08-081-7/+1
| | | | MFC after: 1 week
OpenPOWER on IntegriCloud