mirror of
https://github.com/SCST-project/scst.git
synced 2026-08-28 20:06:33 +00:00
Gleb Chesnokov
9a45498bc7
The iSCSI READMEs, HOWTO and man pages combine current code with procfs, old daemon configuration and fixed runtime output. The result documents invalid paths, the wrong object creation order and unsafe live commands. Commit08e532715d("Remove procfs support") removed the main obsolete interface. Commit70eb596f60("Rework iSCSI aborts to make them more correct and remove recent workarounds (one more post-IET cleanup)") added the SCST abort callback requested by an old ToDo entry. Use iscsi_scst.h, kernel/config.c and usr/config.c for the kernel and daemon contract. Correct sysfs ownership, authentication limits, setup ordering, logging and dynamic output, and remove the completed task. Keep iscsi-scstd.conf and iscsi-scst-adm documented because current code still consumes them, but direct new setups to scst.conf. Replace the removed service command and warn before exposing storage, credentials, portals or sessions.
18 lines
852 B
Plaintext
18 lines
852 B
Plaintext
- Remove the "poor man solution" suspending. Instead, immediate/unsolicited
|
|
data for suspended commands should be received in the dummy_page and
|
|
then, when they are resumed, aborted with BUSY status. All other
|
|
commands should proceed normally.
|
|
|
|
- Separate iSCSI request and response structures. Currently they share the
|
|
same structure. This architecture decision derived from IET and makes
|
|
the code a lot less readable, maintainable and effective as well as more
|
|
errors prone, so it must be corrected.
|
|
|
|
- Code beautifying, i.e. make it be written in the same nice looking style.
|
|
Particularly, all functions and variables names should share the same style.
|
|
|
|
- Fix support of ranges in parameters negotiation. Are there any initiators who
|
|
use ranges and, hence, can be used for testing?
|
|
|
|
- Minor "ToDo"'s spread in the code.
|