diff options
| author | Andy Grover <agrover@redhat.com> | 2012-01-19 13:39:23 -0800 |
|---|---|---|
| committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2012-02-07 06:41:04 +0000 |
| commit | 1edcdb497ef418122cd4f98e157660cf594b345a (patch) | |
| tree | bf41780935b8996f85a0ae090897495961deb6d7 /.mailmap | |
| parent | 95fe1ee41e23fa271416da67483594dde74bc6ca (diff) | |
| download | linux-stable-1edcdb497ef418122cd4f98e157660cf594b345a.tar.gz linux-stable-1edcdb497ef418122cd4f98e157660cf594b345a.tar.bz2 linux-stable-1edcdb497ef418122cd4f98e157660cf594b345a.zip | |
target: Change target_submit_cmd() to return void
Retval not very useful, and may even be harmful. Once submitted, fabrics
should expect a sense error if anything goes wrong. All fabrics checking
of this retval are useless or broken:
fc checks it just to emit more debug output.
ib_srpt trickles retval up, then it is ignored.
qla2xxx trickles it up, which then causes a bug because the abort goto
in qla_target.c thinks cmd hasn't been sent to target.
Just returning nothing is best.
Signed-off-by: Andy Grover <agrover@redhat.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Diffstat (limited to '.mailmap')
0 files changed, 0 insertions, 0 deletions