[dpdk-dev] [PATCH] sched: fix useless call

2016-05-13 Thread Thomas Monjalon
2016-05-11 10:46, Ferruh Yigit: > On 5/10/2016 6:18 PM, Dumitrescu, Cristian wrote: > > As previously discussed on this email list, the rte_bitmap_free() is an API > > function that works as a placeholder for any resource freeing that needs to > > be done for the bitmap. The API function should

[dpdk-dev] [PATCH] sched: fix useless call

2016-05-13 Thread Dumitrescu, Cristian
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Friday, May 13, 2016 11:12 AM > To: Dumitrescu, Cristian > Cc: dev at dpdk.org; Yigit, Ferruh ; Mrzyglod, > DanielX > T > Subject: Re: [dpdk-dev] [PATCH] sched: fix usel

[dpdk-dev] [PATCH] sched: fix useless call

2016-05-11 Thread Ferruh Yigit
On 5/10/2016 6:18 PM, Dumitrescu, Cristian wrote: > > >> -Original Message- >> From: Mrzyglod, DanielX T >> Sent: Tuesday, May 10, 2016 11:11 AM >> To: Dumitrescu, Cristian >> Cc: dev at dpdk.org; Mrzyglod, DanielX T >> Subject: [PATCH] sched: fix useless call >> >> Fix issue reported

[dpdk-dev] [PATCH] sched: fix useless call

2016-05-10 Thread Ferruh Yigit
On 5/10/2016 11:11 AM, Daniel Mrzyglod wrote: > Fix issue reported by Coverity. > Coverity ID 13338 > > A function call that seems to have an intended effect has no actual effect > on the logic of the program. > > In rte_sched_port_free: A function is called that is only useful for its > return

[dpdk-dev] [PATCH] sched: fix useless call

2016-05-10 Thread Dumitrescu, Cristian
> -Original Message- > From: Mrzyglod, DanielX T > Sent: Tuesday, May 10, 2016 11:11 AM > To: Dumitrescu, Cristian > Cc: dev at dpdk.org; Mrzyglod, DanielX T > Subject: [PATCH] sched: fix useless call > > Fix issue reported by Coverity. > Coverity ID 13338 > > A function call that

[dpdk-dev] [PATCH] sched: fix useless call

2016-05-10 Thread Daniel Mrzyglod
Fix issue reported by Coverity. Coverity ID 13338 A function call that seems to have an intended effect has no actual effect on the logic of the program. In rte_sched_port_free: A function is called that is only useful for its return value, and this value is ignored. Fixes: de3cfa2c9823