diff options
| author | trasz <trasz@FreeBSD.org> | 2014-04-15 14:55:56 +0000 |
|---|---|---|
| committer | trasz <trasz@FreeBSD.org> | 2014-04-15 14:55:56 +0000 |
| commit | 16b4ecc28d8ca79860c8906ca8f6daafab00f23f (patch) | |
| tree | 87b1bb87ef2107f109da8e58d3989b2523c8c97a /usr.sbin | |
| parent | 4d8489ab7f537ca898792f864f738389603158c3 (diff) | |
| download | FreeBSD-src-16b4ecc28d8ca79860c8906ca8f6daafab00f23f.zip FreeBSD-src-16b4ecc28d8ca79860c8906ca8f6daafab00f23f.tar.gz | |
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
Diffstat (limited to 'usr.sbin')
| -rw-r--r-- | usr.sbin/ctld/ctld.c | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/usr.sbin/ctld/ctld.c b/usr.sbin/ctld/ctld.c index 56092db..1090a52 100644 --- a/usr.sbin/ctld/ctld.c +++ b/usr.sbin/ctld/ctld.c @@ -1099,7 +1099,7 @@ conf_verify(struct conf *conf) struct portal_group *pg; struct target *targ; struct lun *lun; - bool found_lun0; + bool found_lun; int error; if (conf->conf_pidfile_path == NULL) @@ -1116,17 +1116,16 @@ conf_verify(struct conf *conf) "default"); assert(targ->t_portal_group != NULL); } - found_lun0 = false; + found_lun = false; TAILQ_FOREACH(lun, &targ->t_luns, l_next) { error = conf_verify_lun(lun); if (error != 0) return (error); - if (lun->l_lun == 0) - found_lun0 = true; + found_lun = true; } - if (!found_lun0) { - log_warnx("mandatory LUN 0 not configured " - "for target \"%s\"", targ->t_name); + if (!found_lun) { + log_warnx("no LUNs defined for target \"%s\"", + targ->t_name); return (1); } } |
