[dpdk-dev] [PATCH] power: fix argument cannot be negative

2016-05-16 Thread Thomas Monjalon
2016-05-16 14:39 GMT+02:00 Mcnamara, John : > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon >> The next statement is probably a useless copy paste of the coverity >> report. >> >> > In send_msg: Negative value used as argument to a function expecting a >> > positive value

[dpdk-dev] [PATCH] power: fix argument cannot be negative

2016-05-16 Thread Thomas Monjalon
The title do not convey the real issue. We should be more concerned by an issue of "wrong error message" rather than an "argument" which "cannot be negative". 2016-04-20 16:39, Daniel Mrzyglod: > Fix issue reported by Coverity. > Coverity ID 13269 & 13266: It is better to put these references bel

[dpdk-dev] [PATCH] power: fix argument cannot be negative

2016-05-16 Thread Mcnamara, John
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Monday, May 16, 2016 2:00 PM > ... > > A question on this point. Is it just that the Coverity message is > > useless in this case or in general? For other error/warning fixes we > > include the messag

[dpdk-dev] [PATCH] power: fix argument cannot be negative

2016-05-16 Thread Mcnamara, John
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Thomas Monjalon > Sent: Monday, May 16, 2016 1:17 PM > To: Mrzyglod, DanielX T > Cc: dev at dpdk.org; Carew, Alan > Subject: Re: [dpdk-dev] [PATCH] power: fix argument cannot be nega

[dpdk-dev] [PATCH] power: fix argument cannot be negative

2016-04-20 Thread Daniel Mrzyglod
Fix issue reported by Coverity. Coverity ID 13269 & 13266: Function strerror(errno) has built strings only for non-negative errno values. for negative values of errno it describe error as "Unknown error -errno" to be more descriptive i put string "channel not found" taken from header. The negativ