Re: [dpdk-dev] [PATCH] ethdev: decrease log level for successful API

2018-08-02 Thread Kevin Traynor
subject prefix not updated - will resend as [PATCH v2] On 08/02/2018 07:33 PM, Kevin Traynor wrote: > Change log level of messages from ERR to INFO where > the post condition of the API is success, but no action > was actually needed as the condition already existed. > e.g. calling rte_eth_dev_sta

[dpdk-dev] [PATCH] ethdev: decrease log level for successful API

2018-08-02 Thread Kevin Traynor
Change log level of messages from ERR to INFO where the post condition of the API is success, but no action was actually needed as the condition already existed. e.g. calling rte_eth_dev_start() for a device that is already started. Fixes: bea1e0c70cfc ("ethdev: convert static log type usage to dy

Re: [dpdk-dev] [PATCH] ethdev: decrease log level for successful API

2018-08-02 Thread Kevin Traynor
On 08/02/2018 07:01 PM, Andrew Rybchenko wrote: > On 02.08.2018 19:14, Thomas Monjalon wrote: >> 02/08/2018 16:52, Kevin Traynor: >>> On 08/02/2018 03:41 PM, Andrew Rybchenko wrote: On 02.08.2018 17:09, Kevin Traynor wrote: > On 08/02/2018 02:52 PM, Andrew Rybchenko wrote: >> On 02.08.

Re: [dpdk-dev] [PATCH] ethdev: decrease log level for successful API

2018-08-02 Thread Andrew Rybchenko
On 02.08.2018 19:14, Thomas Monjalon wrote: 02/08/2018 16:52, Kevin Traynor: On 08/02/2018 03:41 PM, Andrew Rybchenko wrote: On 02.08.2018 17:09, Kevin Traynor wrote: On 08/02/2018 02:52 PM, Andrew Rybchenko wrote: On 02.08.2018 16:35, Kevin Traynor wrote: On 08/02/2018 01:59 PM, Andrew Rybc

Re: [dpdk-dev] [PATCH] ethdev: decrease log level for successful API

2018-08-02 Thread Thomas Monjalon
02/08/2018 16:52, Kevin Traynor: > On 08/02/2018 03:41 PM, Andrew Rybchenko wrote: > > On 02.08.2018 17:09, Kevin Traynor wrote: > >> On 08/02/2018 02:52 PM, Andrew Rybchenko wrote: > >>> On 02.08.2018 16:35, Kevin Traynor wrote: > On 08/02/2018 01:59 PM, Andrew Rybchenko wrote: > > On 02.

Re: [dpdk-dev] [PATCH] ethdev: decrease log level for successful API

2018-08-02 Thread Stephen Hemminger
On Thu, 2 Aug 2018 13:33:43 +0100 Kevin Traynor wrote: > Change log level of messages from ERR back to DEBUG where > existing API indicates that the condition is a success. > > This means applications using the API in its current form will > not get new ERR logs. > > Fixes: bea1e0c70cfc ("ethd

Re: [dpdk-dev] [PATCH] ethdev: decrease log level for successful API

2018-08-02 Thread Kevin Traynor
On 08/02/2018 03:41 PM, Andrew Rybchenko wrote: > On 02.08.2018 17:09, Kevin Traynor wrote: >> On 08/02/2018 02:52 PM, Andrew Rybchenko wrote: >>> On 02.08.2018 16:35, Kevin Traynor wrote: On 08/02/2018 01:59 PM, Andrew Rybchenko wrote: > On 02.08.2018 15:33, Kevin Traynor wrote: >> Ch

Re: [dpdk-dev] [PATCH] ethdev: decrease log level for successful API

2018-08-02 Thread Andrew Rybchenko
On 02.08.2018 17:09, Kevin Traynor wrote: On 08/02/2018 02:52 PM, Andrew Rybchenko wrote: On 02.08.2018 16:35, Kevin Traynor wrote: On 08/02/2018 01:59 PM, Andrew Rybchenko wrote: On 02.08.2018 15:33, Kevin Traynor wrote: Change log level of messages from ERR back to DEBUG where existing API

Re: [dpdk-dev] [PATCH] ethdev: decrease log level for successful API

2018-08-02 Thread Kevin Traynor
On 08/02/2018 02:52 PM, Andrew Rybchenko wrote: > On 02.08.2018 16:35, Kevin Traynor wrote: >> On 08/02/2018 01:59 PM, Andrew Rybchenko wrote: >>> On 02.08.2018 15:33, Kevin Traynor wrote: Change log level of messages from ERR back to DEBUG where existing API indicates that the condition

Re: [dpdk-dev] [PATCH] ethdev: decrease log level for successful API

2018-08-02 Thread Andrew Rybchenko
On 02.08.2018 16:35, Kevin Traynor wrote: On 08/02/2018 01:59 PM, Andrew Rybchenko wrote: On 02.08.2018 15:33, Kevin Traynor wrote: Change log level of messages from ERR back to DEBUG where existing API indicates that the condition is a success. This means applications using the API in its cur

Re: [dpdk-dev] [PATCH] ethdev: decrease log level for successful API

2018-08-02 Thread Kevin Traynor
On 08/02/2018 01:59 PM, Andrew Rybchenko wrote: > On 02.08.2018 15:33, Kevin Traynor wrote: >> Change log level of messages from ERR back to DEBUG where >> existing API indicates that the condition is a success. >> >> This means applications using the API in its current form will >> not get new ERR

Re: [dpdk-dev] [PATCH] ethdev: decrease log level for successful API

2018-08-02 Thread Andrew Rybchenko
On 02.08.2018 15:33, Kevin Traynor wrote: Change log level of messages from ERR back to DEBUG where existing API indicates that the condition is a success. This means applications using the API in its current form will not get new ERR logs. Fixes: bea1e0c70cfc ("ethdev: convert static log type

Re: [dpdk-dev] [PATCH] ethdev: decrease log level for successful API

2018-08-02 Thread Andrew Rybchenko
On 02.08.2018 15:33, Kevin Traynor wrote: Change log level of messages from ERR back to DEBUG where existing API indicates that the condition is a success. This means applications using the API in its current form will not get new ERR logs. Fixes: bea1e0c70cfc ("ethdev: convert static log type

[dpdk-dev] [PATCH] ethdev: decrease log level for successful API

2018-08-02 Thread Kevin Traynor
Change log level of messages from ERR back to DEBUG where existing API indicates that the condition is a success. This means applications using the API in its current form will not get new ERR logs. Fixes: bea1e0c70cfc ("ethdev: convert static log type usage to dynamic") Signed-off-by: Kevin Tra