Re: [dpdk-dev] [PATCH] libs/power: fix the resource leaking issue

2019-01-17 Thread Thomas Monjalon
15/01/2019 11:01, David Hunt: > From: Liang Ma > > Fixes: e6c6dc0f96c8 ("power: add p-state driver compatibility") > Coverity issue: 328528 > > v2: focus just on the resource leak. Remove additional code around > Turbo Boost frequency handling. > > Signed-off-by: Liang Ma > Reviewed-by: Le

Re: [dpdk-dev] [PATCH] libs/power: fix the resource leaking issue

2019-01-15 Thread Thomas Monjalon
15/01/2019 11:07, Hunt, David: > > On 14/1/2019 10:35 PM, Thomas Monjalon wrote: > > 28/12/2018 12:32, Liang Ma: > >> Fixes: e6c6dc0f96c8 ("power: add p-state driver compatibility") > >> Coverity issue: 328528 > >> > >> Also add the missing functionality of enable/disable turbo > > If you are chan

Re: [dpdk-dev] [PATCH] libs/power: fix the resource leaking issue

2019-01-15 Thread Hunt, David
On 14/1/2019 10:35 PM, Thomas Monjalon wrote: 28/12/2018 12:32, Liang Ma: Fixes: e6c6dc0f96c8 ("power: add p-state driver compatibility") Coverity issue: 328528 Also add the missing functionality of enable/disable turbo If you are changing two things, you probably need to split in two patche

Re: [dpdk-dev] [PATCH] libs/power: fix the resource leaking issue

2019-01-14 Thread Thomas Monjalon
28/12/2018 12:32, Liang Ma: > Fixes: e6c6dc0f96c8 ("power: add p-state driver compatibility") > Coverity issue: 328528 > > Also add the missing functionality of enable/disable turbo If you are changing two things, you probably need to split in two patches. In 19.02 timeframe, we will get only the

Re: [dpdk-dev] [PATCH] libs/power: fix the resource leaking issue

2019-01-01 Thread Yao, Lei A
> -Original Message- > From: Ma, Liang J > Sent: Friday, December 28, 2018 7:33 PM > To: Hunt, David > Cc: dev@dpdk.org; Burakov, Anatoly ; Yao, Lei > A ; Ma, Liang J > Subject: [PATCH] libs/power: fix the resource leaking issue > > Fixes: e6c6dc0f96c8 ("power: add p-state driver comp