On 02/12/14 17:07, Taras Kondratiuk wrote:
On 12/02/2014 06:42 PM, Zoltan Kiss wrote:
Hi,
At the moment, the careless developer look at "enum odp_log_level" and
starts using ODP_LOG(ODP_LOG_DBG, ...) instead of ODP_DBG.
I think it would be good to provide either ODP_LOG(loglevel, ...) or
ODP_[loglevel](...), but not both.
These are internal functions, so it is up to a platform
developer/maintainer to decide how he want to use them.
platform/linux-generic/include/api/odp_debug.h can be used by
applications, as my recent patch showed, the OVS interface does that for
example. And this problem applies to ODP_LOG_ABORT as well: ODP_ABORT
will call abort(), while ODP_LOG(ODP_LOG_ABORT, ...) doesn't.
I think you can just drop the wholw ODP_LOG macro completely, as far as
I see nothing uses it anymore.
Zoli
_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp
