Re: [PATCH] kernel/smp.c: free related resources when failure occurs in hotplug_cfd()

2013-07-08 Thread Chen Gang
On 07/09/2013 08:28 AM, Wang YanQing wrote: > On Mon, Jul 08, 2013 at 04:50:24PM +0800, Chen Gang wrote: >> > When failure occurs in hotplug_cfd(), need release related resources, >> > or will cause memory leak. >> > >> > Also beautify the related code. >> > >> > Signed-off-by: Chen Gang >> > --

Re: [PATCH] kernel/smp.c: free related resources when failure occurs in hotplug_cfd()

2013-07-08 Thread Wang YanQing
On Mon, Jul 08, 2013 at 04:50:24PM +0800, Chen Gang wrote: > When failure occurs in hotplug_cfd(), need release related resources, > or will cause memory leak. > > Also beautify the related code. > > Signed-off-by: Chen Gang > --- > kernel/smp.c | 13 + > 1 files changed, 9 insert

Re: [PATCH] kernel/smp.c: free related resources when failure occurs in hotplug_cfd()

2013-07-08 Thread Chen Gang F T
On 07/08/2013 10:26 PM, Paul Gortmaker wrote: > [[PATCH] kernel/smp.c: free related resources when failure occurs in > hotplug_cfd()] On 08/07/2013 (Mon 16:50) Chen Gang wrote: > >> > When failure occurs in hotplug_cfd(), need release related resources, >> >

Re: [PATCH] kernel/smp.c: free related resources when failure occurs in hotplug_cfd()

2013-07-08 Thread Paul Gortmaker
[[PATCH] kernel/smp.c: free related resources when failure occurs in hotplug_cfd()] On 08/07/2013 (Mon 16:50) Chen Gang wrote: > When failure occurs in hotplug_cfd(), need release related resources, > or will cause memory leak. > > Also beautify the related code. No. Please do

[PATCH] kernel/smp.c: free related resources when failure occurs in hotplug_cfd()

2013-07-08 Thread Chen Gang
When failure occurs in hotplug_cfd(), need release related resources, or will cause memory leak. Also beautify the related code. Signed-off-by: Chen Gang --- kernel/smp.c | 13 + 1 files changed, 9 insertions(+), 4 deletions(-) diff --git a/kernel/smp.c b/kernel/smp.c index 02a88