Re: [dpdk-dev] [PATCH v2] log: support custom log function

2021-02-11 Thread Feng Li
On Tue, Feb 9, 2021 at 6:40 AM Dmitry Kozlyuk wrote: > > On Mon, 8 Feb 2021 14:58:29 +0800, Feng Li wrote: > > > > +/** > > > > + * Usage function typedef used by the application usage function. > > > > > > It's unrelated to the following typedef purpose, is it? > > It's borrowed from the front ty

Re: [dpdk-dev] [PATCH v2] log: support custom log function

2021-02-08 Thread Dmitry Kozlyuk
On Mon, 8 Feb 2021 14:58:29 +0800, Feng Li wrote: > > > +/** > > > + * Usage function typedef used by the application usage function. > > > > It's unrelated to the following typedef purpose, is it? > It's borrowed from the front typedef sentence. Doesn't make much sense here anyway. > > > + *

Re: [dpdk-dev] [PATCH v2] log: support custom log function

2021-02-08 Thread Feng Li
Thanks for your comments. Dmitry Kozlyuk 于2021年2月6日周六 上午3:32写道: > > On Sat, 6 Feb 2021 01:42:04 +0800, Li Feng wrote: > > Currently, the dpdk log is out to stdout/stderr and syslog. > > The rte_openlog_stream could set an external FILE* stream, but it asks the > > consumer to give it a FILE* poi

Re: [dpdk-dev] [PATCH v2] log: support custom log function

2021-02-07 Thread Feng Li
Li Feng 于2021年2月6日周六 上午1:42写道: > > Currently, the dpdk log is out to stdout/stderr and syslog. > The rte_openlog_stream could set an external FILE* stream, but it asks the > consumer to give it a FILE* pointer. > For C++ or other languages, it's hard to get a libc FILE*. > > Support to set a hook

Re: [dpdk-dev] [PATCH v2] log: support custom log function

2021-02-05 Thread Dmitry Kozlyuk
On Sat, 6 Feb 2021 01:42:04 +0800, Li Feng wrote: > Currently, the dpdk log is out to stdout/stderr and syslog. > The rte_openlog_stream could set an external FILE* stream, but it asks the > consumer to give it a FILE* pointer. > For C++ or other languages, it's hard to get a libc FILE*. > > Supp

[dpdk-dev] [PATCH v2] log: support custom log function

2021-02-05 Thread Li Feng
Currently, the dpdk log is out to stdout/stderr and syslog. The rte_openlog_stream could set an external FILE* stream, but it asks the consumer to give it a FILE* pointer. For C++ or other languages, it's hard to get a libc FILE*. Support to set a hook method is another choice for this scenario.