stable: please queue commit 0231bb5 ("perf: Fix event group context move")

stable: please queue commit 0231bb5 ("perf: Fix event group context move")

[Li Zefan]

Plese consider adding commit 0231bb5336758426b44ccd798ccd3c5419c95d58
to stable tree. It fixes a real bug, which was explained here:

	http://thread.gmane.org/gmane.linux.kernel.perf.user/1144

and I've managed to reproduce the bug in 3.4 kernel with the test
program attached in the above email, and I've confirmed the bug fix.

The effect of the bug is, if we open a group of mixed events(sw/hw)
and the "disabled" flag is not set and sw events come first, then
some of the perf events won't be started after the call to perf_event_open().

The bug fix should be applicable to kernels >= 3.0 && <= 3.7.

--
Li Zefan

Re: stable: please queue commit 0231bb5 ("perf: Fix event group context move")

[Ingo Molnar]

* Li Zefan <lizefan@huawei.com> wrote:

> Plese consider adding commit 0231bb5336758426b44ccd798ccd3c5419c95d58
> to stable tree. It fixes a real bug, which was explained here:
> 
> 	http://thread.gmane.org/gmane.linux.kernel.perf.user/1144
> 
> and I've managed to reproduce the bug in 3.4 kernel with the test
> program attached in the above email, and I've confirmed the bug fix.
> 
> The effect of the bug is, if we open a group of mixed events(sw/hw)
> and the "disabled" flag is not set and sw events come first, then
> some of the perf events won't be started after the call to perf_event_open().
> 
> The bug fix should be applicable to kernels >= 3.0 && <= 3.7.

Acked-by: Ingo Molnar <mingo@kernel.org>

Thanks,

	Ingo

Re: stable: please queue commit 0231bb5 ("perf: Fix event group context move")

[Ben Hutchings]

[-- Attachment #1: Type: text/plain, Size: 960 bytes --]

On Mon, 2013-08-05 at 10:21 +0200, Ingo Molnar wrote:
> * Li Zefan <lizefan@huawei.com> wrote:
> 
> > Plese consider adding commit 0231bb5336758426b44ccd798ccd3c5419c95d58
> > to stable tree. It fixes a real bug, which was explained here:
> > 
> > 	http://thread.gmane.org/gmane.linux.kernel.perf.user/1144
> > 
> > and I've managed to reproduce the bug in 3.4 kernel with the test
> > program attached in the above email, and I've confirmed the bug fix.
> > 
> > The effect of the bug is, if we open a group of mixed events(sw/hw)
> > and the "disabled" flag is not set and sw events come first, then
> > some of the perf events won't be started after the call to perf_event_open().
> > 
> > The bug fix should be applicable to kernels >= 3.0 && <= 3.7.
> 
> Acked-by: Ingo Molnar <mingo@kernel.org>

Queued up for 3.2, thanks.

Ben.

-- 
Ben Hutchings
If God had intended Man to program,
we'd have been born with serial I/O ports.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 828 bytes --]

Re: stable: please queue commit 0231bb5 ("perf: Fix event group context move")

[Luis Henriques]

Li Zefan <lizefan@huawei.com> writes:

> Plese consider adding commit 0231bb5336758426b44ccd798ccd3c5419c95d58
> to stable tree. It fixes a real bug, which was explained here:
>
> 	http://thread.gmane.org/gmane.linux.kernel.perf.user/1144
>
> and I've managed to reproduce the bug in 3.4 kernel with the test
> program attached in the above email, and I've confirmed the bug fix.
>
> The effect of the bug is, if we open a group of mixed events(sw/hw)
> and the "disabled" flag is not set and sw events come first, then
> some of the perf events won't be started after the call to perf_event_open().
>
> The bug fix should be applicable to kernels >= 3.0 && <= 3.7.
>
> --
> Li Zefan
> --
> To unsubscribe from this list: send the line "unsubscribe stable" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Thanks, I'm queuing it for the 3.5 kernel.

Cheers,
-- 
Luis