anchao commented on code in PR #17018: URL: https://github.com/apache/nuttx/pull/17018#discussion_r2383887127
########## include/syslog.h: ########## @@ -121,6 +121,8 @@ #define LOG_INFO 6 /* Informational message */ #define LOG_DEBUG 7 /* Debug-level message */ +#define LOG_LOWOUT 8 /* Log to console straightly for debugging */ Review Comment: > > @wangchdo please consider using LOG_EMERG instead of LOG_LOWOUT > > Isn't this what @xiaoxiang781216 was talking about in the last PR though? That the implementation of `LOG_EMERG` was changed from this time of "early print" to something else for a specific reason? [#16977 (comment)](https://github.com/apache/nuttx/pull/16977#issuecomment-3270622862) > > I think maybe `LOG_LOWOUT` would need to be like `(24 << 3)` to not conflict with any implementation so far. Emergstream was originally designed to be used in interrupt context. It does not require complex backend implementation and is particularly suitable for use in system startup, exception, and interrupt context. OS provided similar functionality in the early days, but after it was removed, it did not provide similar capabilities to developers, which is the problem. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
