[stable] Patch Upstream: oprofile, ARM: Release resources on failure

[stable] Patch Upstream: oprofile, ARM: Release resources on failure

[Greg KH]

This patch doesn't apply to the .35 or .36 stable tree
If someone wants it applied there, please email the backport
to stable at kernel.org

thanks,

greg k-h

> commit: 81771974ae49bf79aab60c42eac7a6d730a9ef2b
> From: Robert Richter <robert.richter@amd.com>
> Date: Wed, 29 Sep 2010 16:52:25 +0200
> Subject: [PATCH] oprofile, ARM: Release resources on failure
> 
> This patch fixes a resource leak on failure, where the
> oprofilefs and some counters may not released properly.
> 
> Signed-off-by: Robert Richter <robert.richter@amd.com>
> Acked-by: Will Deacon <will.deacon@arm.com>
> Cc: linux-arm-kernel at lists.infradead.org
> Cc: <stable@kernel.org> # .35.x
> LKML-Reference: <20100929145225.GJ13563@erda.amd.com>
> Signed-off-by: Ingo Molnar <mingo@elte.hu>
> ---
>  drivers/oprofile/oprofile_perf.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/oprofile/oprofile_perf.c b/drivers/oprofile/oprofile_perf.c
> index ebb40cb..f3d3df2 100644
> --- a/drivers/oprofile/oprofile_perf.c
> +++ b/drivers/oprofile/oprofile_perf.c
> @@ -84,6 +84,7 @@ static int op_create_counter(int cpu, int event)
>  	if (IS_ERR(pevent)) {
>  		ret = PTR_ERR(pevent);
>  	} else if (pevent->state != PERF_EVENT_STATE_ACTIVE) {
> +		perf_event_release_kernel(pevent);
>  		pr_warning("oprofile: failed to enable event %d "
>  				"on CPU %d\n", event, cpu);
>  		ret = -EBUSY;
> 
> _______________________________________________
> stable mailing list
> stable at linux.kernel.org
> http://linux.kernel.org/mailman/listinfo/stable

[stable] Patch Upstream: oprofile, ARM: Release resources on failure

[Robert Richter]

On 29.10.10 16:58:29, Greg KH wrote:
> 
> This patch doesn't apply to the .35 or .36 stable tree
> If someone wants it applied there, please email the backport
> to stable at kernel.org
> 
> thanks,
> 
> greg k-h
> 
> > commit: 81771974ae49bf79aab60c42eac7a6d730a9ef2b
> > From: Robert Richter <robert.richter@amd.com>
> > Date: Wed, 29 Sep 2010 16:52:25 +0200
> > Subject: [PATCH] oprofile, ARM: Release resources on failure

This patch was accidentally marked as stable. There are already fixes
in .35 and .36:

 98d943b02f6f1b57787ff1aa6f34d019a407e3ee upstream in .36
 1b6740a13ca9774810edacafc5e12514092651e2 in .35 stable of 98d943b

-Robert

-- 
Advanced Micro Devices, Inc.
Operating System Research Center

[stable] Patch Upstream: oprofile, ARM: Release resources on failure

[Greg KH]

On Mon, Nov 22, 2010 at 03:22:03PM +0100, Robert Richter wrote:
> On 29.10.10 16:58:29, Greg KH wrote:
> > 
> > This patch doesn't apply to the .35 or .36 stable tree
> > If someone wants it applied there, please email the backport
> > to stable at kernel.org
> > 
> > thanks,
> > 
> > greg k-h
> > 
> > > commit: 81771974ae49bf79aab60c42eac7a6d730a9ef2b
> > > From: Robert Richter <robert.richter@amd.com>
> > > Date: Wed, 29 Sep 2010 16:52:25 +0200
> > > Subject: [PATCH] oprofile, ARM: Release resources on failure
> 
> This patch was accidentally marked as stable. There are already fixes
> in .35 and .36:
> 
>  98d943b02f6f1b57787ff1aa6f34d019a407e3ee upstream in .36
>  1b6740a13ca9774810edacafc5e12514092651e2 in .35 stable of 98d943b

Ok, thanks.

greg k-h