MFC r261711:

Yacc cleanup; no functional changes.

Sponsored by:	The FreeBSD Foundation
MFC r261712:

Remove unused variable.

Sponsored by:	The FreeBSD Foundation
MFC r261713:

Simplify.

Sponsored by:	The FreeBSD Foundation
MFC r261714:

Make iscsictl(8) properly handle (optional) semicolons in iscsi.conf,
instead of ignoring them and echoing them to stdout.

Sponsored by:	The FreeBSD Foundation
MFC r262247:

Make it clear that there are two ways to add a session using iscsictl(8),
and some options require configuration file.

Sponsored by:	The FreeBSD Foundation
MFC r262525:

There is no need to prevent iscsictl from adding iSER session when there
is no iSER support in ctld and/or kernel; should the user make that mistake,
the output from "iscsictl -L" is enough to determine what the problem is.

Sponsored by:	The FreeBSD Foundation
MFC r261715:

The log_whatever() routines don't accept NULL for format strings,
so mark them as __printflike instead of __printf0like.

Sponsored by:	The FreeBSD Foundation
MFC r261747:

Empty data segment during Login Phase is rather unlikely, but it's not
a protocol error.

Sponsored by:	The FreeBSD Foundation
MFC r261748:

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 iscsid(8) zombies disappear.

Sponsored by:	The FreeBSD Foundation