Re: [PATCH] cpumask: fix cpumask leak in partition_sched_domains

2013-08-15 Thread Peter Zijlstra
On Tue, Aug 06, 2013 at 08:06:42PM +0800, Xiaotian Feng wrote: > If doms_new is NULL, partition_sched_domains() will reset ndoms_cur > to 0, and free old sched domains with free_sched_domains(doms_cur, ndoms_cur). > As ndoms_cur is 0, the cpumask will not be freed. > > Signed-off-by: Xiaotian

Re: [PATCH] cpumask: fix cpumask leak in partition_sched_domains

2013-08-15 Thread Peter Zijlstra
On Tue, Aug 06, 2013 at 08:06:42PM +0800, Xiaotian Feng wrote: If doms_new is NULL, partition_sched_domains() will reset ndoms_cur to 0, and free old sched domains with free_sched_domains(doms_cur, ndoms_cur). As ndoms_cur is 0, the cpumask will not be freed. Signed-off-by: Xiaotian Feng

Re: [PATCH] cpumask: fix cpumask leak in partition_sched_domains

2013-08-14 Thread Xiaotian Feng
On Tue, Aug 6, 2013 at 8:06 PM, Xiaotian Feng wrote: > If doms_new is NULL, partition_sched_domains() will reset ndoms_cur > to 0, and free old sched domains with free_sched_domains(doms_cur, ndoms_cur). > As ndoms_cur is 0, the cpumask will not be freed. > > Signed-off-by: Xiaotian Feng > Cc:

Re: [PATCH] cpumask: fix cpumask leak in partition_sched_domains

2013-08-14 Thread Xiaotian Feng
On Tue, Aug 6, 2013 at 8:06 PM, Xiaotian Feng xtf...@gmail.com wrote: If doms_new is NULL, partition_sched_domains() will reset ndoms_cur to 0, and free old sched domains with free_sched_domains(doms_cur, ndoms_cur). As ndoms_cur is 0, the cpumask will not be freed. Signed-off-by: Xiaotian

[PATCH] cpumask: fix cpumask leak in partition_sched_domains

2013-08-06 Thread Xiaotian Feng
If doms_new is NULL, partition_sched_domains() will reset ndoms_cur to 0, and free old sched domains with free_sched_domains(doms_cur, ndoms_cur). As ndoms_cur is 0, the cpumask will not be freed. Signed-off-by: Xiaotian Feng Cc: Ingo Molnar Cc: Peter Zijlstra Cc: Rusty Russell Cc: Thomas

[PATCH] cpumask: fix cpumask leak in partition_sched_domains

2013-08-06 Thread Xiaotian Feng
If doms_new is NULL, partition_sched_domains() will reset ndoms_cur to 0, and free old sched domains with free_sched_domains(doms_cur, ndoms_cur). As ndoms_cur is 0, the cpumask will not be freed. Signed-off-by: Xiaotian Feng xtf...@gmail.com Cc: Ingo Molnar mi...@redhat.com Cc: Peter Zijlstra

Re: [PATCH] cpumask: fix cpumask leak in partition_sched_domains

2013-08-05 Thread Xiaotian Feng
On Tue, Aug 6, 2013 at 12:37 PM, Rusty Russell wrote: > Xiaotian Feng writes: >> On Sat, Jul 27, 2013 at 3:26 PM, Xiaotian Feng wrote: >>> If doms_new is NULL, partition_sched_domains() will reset ndoms_cur >>> to 0, and free old sched domains with free_sched_domains(doms_cur, >>> ndoms_cur).

Re: [PATCH] cpumask: fix cpumask leak in partition_sched_domains

2013-08-05 Thread Rusty Russell
Xiaotian Feng writes: > On Sat, Jul 27, 2013 at 3:26 PM, Xiaotian Feng wrote: >> If doms_new is NULL, partition_sched_domains() will reset ndoms_cur >> to 0, and free old sched domains with free_sched_domains(doms_cur, >> ndoms_cur). >> As ndoms_cur is 0, the cpumask will not be freed. >> >>

Re: [PATCH] cpumask: fix cpumask leak in partition_sched_domains

2013-08-05 Thread Xiaotian Feng
On Sat, Jul 27, 2013 at 3:26 PM, Xiaotian Feng wrote: > If doms_new is NULL, partition_sched_domains() will reset ndoms_cur > to 0, and free old sched domains with free_sched_domains(doms_cur, ndoms_cur). > As ndoms_cur is 0, the cpumask will not be freed. > > Signed-off-by: Xiaotian Feng > Cc:

Re: [PATCH] cpumask: fix cpumask leak in partition_sched_domains

2013-08-05 Thread Xiaotian Feng
On Sat, Jul 27, 2013 at 3:26 PM, Xiaotian Feng xtf...@gmail.com wrote: If doms_new is NULL, partition_sched_domains() will reset ndoms_cur to 0, and free old sched domains with free_sched_domains(doms_cur, ndoms_cur). As ndoms_cur is 0, the cpumask will not be freed. Signed-off-by: Xiaotian

Re: [PATCH] cpumask: fix cpumask leak in partition_sched_domains

2013-08-05 Thread Rusty Russell
Xiaotian Feng xtf...@gmail.com writes: On Sat, Jul 27, 2013 at 3:26 PM, Xiaotian Feng xtf...@gmail.com wrote: If doms_new is NULL, partition_sched_domains() will reset ndoms_cur to 0, and free old sched domains with free_sched_domains(doms_cur, ndoms_cur). As ndoms_cur is 0, the cpumask will

Re: [PATCH] cpumask: fix cpumask leak in partition_sched_domains

2013-08-05 Thread Xiaotian Feng
On Tue, Aug 6, 2013 at 12:37 PM, Rusty Russell ru...@rustcorp.com.au wrote: Xiaotian Feng xtf...@gmail.com writes: On Sat, Jul 27, 2013 at 3:26 PM, Xiaotian Feng xtf...@gmail.com wrote: If doms_new is NULL, partition_sched_domains() will reset ndoms_cur to 0, and free old sched domains with

[PATCH] cpumask: fix cpumask leak in partition_sched_domains

2013-07-27 Thread Xiaotian Feng
If doms_new is NULL, partition_sched_domains() will reset ndoms_cur to 0, and free old sched domains with free_sched_domains(doms_cur, ndoms_cur). As ndoms_cur is 0, the cpumask will not be freed. Signed-off-by: Xiaotian Feng Cc: Ingo Molnar Cc: Peter Zijlstra Cc: linux-kernel@vger.kernel.org

[PATCH] cpumask: fix cpumask leak in partition_sched_domains

2013-07-27 Thread Xiaotian Feng
If doms_new is NULL, partition_sched_domains() will reset ndoms_cur to 0, and free old sched domains with free_sched_domains(doms_cur, ndoms_cur). As ndoms_cur is 0, the cpumask will not be freed. Signed-off-by: Xiaotian Feng xtf...@gmail.com Cc: Ingo Molnar mi...@redhat.com Cc: Peter Zijlstra