libosmocore[master]: vty: fix "everything" logging option

Holger Freyther gerrit-no-reply at
Sat Feb 11 14:56:05 UTC 2017

Patch Set 4: Code-Review-1


I think given we have configs in the wild that include everything.. where it might or might not be intended.. we should have a new name.. "all" maybe?

git grep everything -- openbsc/doc/ | wc -l
File include/osmocom/vty/logging.h:

Line 7: void logging_vty_add_cmds(const struct log_info *unused_info);
then remove it... as long as we don't write (void) the compiler should not complain..
File src/logging.c:

Line 142: 	"Log everything (use with extreme care)",
What about existing configs that write "everything"? Maybe introduce "all" and have everything as an alias for "XYZ???"

Line 356: static inline bool log_to_target(struct log_target *tar, int subsys, int level)
But it doesn't look. it is a rather long predicate function.. should_log_to_target?

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib51987fb2f64a70fca130f3799dc8fd71cc7085c
Gerrit-PatchSet: 4
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Max <msuraev at>
Gerrit-Reviewer: Harald Welte <laforge at>
Gerrit-Reviewer: Holger Freyther <holger at>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <msuraev at>
Gerrit-HasComments: Yes

More information about the gerrit-log mailing list