2018-12-21 07:32:04

by Greensky, James J

[permalink] [raw]
Subject: Fix "perf tools: Synthesize GROUP_DESC feature in pipe mode" in the LT 4.14 branch

Commit d38d272592737ea88a20 ("perf tools: Synthesize GROUP_DESC feature in pipe mode") broke the LT 4.14 branch when using event groups in pipe-mode.

# perf record -e '{cycles,instructions,branches}' -- sleep 4 | perf report
# To display the perf.data header info, please use --header/--header-only options
#
Oxd7c [0x60]: failed to process type: 80
Error:
Failed to process sample

Commit a2015516c5c0be932a69 ("perf record: Synthesize features before events in pipe mode") is the fix. Can we get this cherry-picked and applied?



2018-12-21 16:31:21

by 王金浦

[permalink] [raw]
Subject: Re: Fix "perf tools: Synthesize GROUP_DESC feature in pipe mode" in the LT 4.14 branch

+cc Greg, stable

Greensky, James J <[email protected]> 于2018年12月21日周五 上午11:48写道:
>
> Commit d38d272592737ea88a20 ("perf tools: Synthesize GROUP_DESC feature in pipe mode") broke the LT 4.14 branch when using event groups in pipe-mode.
>
> # perf record -e '{cycles,instructions,branches}' -- sleep 4 | perf report
> # To display the perf.data header info, please use --header/--header-only options
> #
> Oxd7c [0x60]: failed to process type: 80
> Error:
> Failed to process sample
>
> Commit a2015516c5c0be932a69 ("perf record: Synthesize features before events in pipe mode") is the fix. Can we get this cherry-picked and applied?
>

2018-12-22 18:07:56

by Greensky, James J

[permalink] [raw]
Subject: RE: Fix "perf tools: Synthesize GROUP_DESC feature in pipe mode" in the LT 4.14 branch

This commit does not apply cleanly due to other changes. Is there something I can do to make it easier.

This line:
If (data->is_pipe) {
Must be changed to:
If (file->is_pipe) {

Let me know if there is a more standard way of communicating cherry-picked changes for inclusion.

-----Original Message-----
From: Jinpu Wang [mailto:[email protected]]
Sent: Friday, December 21, 2018 2:58 AM
To: Greensky, James J <[email protected]>; Greg Kroah-Hartman <[email protected]>
Cc: [email protected]; v3.14+, only the raid10 part <[email protected]>
Subject: Re: Fix "perf tools: Synthesize GROUP_DESC feature in pipe mode" in the LT 4.14 branch

+cc Greg, stable

Greensky, James J <[email protected]> 于2018年12月21日周五 上午11:48写道:
>
> Commit d38d272592737ea88a20 ("perf tools: Synthesize GROUP_DESC feature in pipe mode") broke the LT 4.14 branch when using event groups in pipe-mode.
>
> # perf record -e '{cycles,instructions,branches}' -- sleep 4 | perf report
> # To display the perf.data header info, please use --header/--header-only options
> #
> Oxd7c [0x60]: failed to process type: 80
> Error:
> Failed to process sample
>
> Commit a2015516c5c0be932a69 ("perf record: Synthesize features before events in pipe mode") is the fix. Can we get this cherry-picked and applied?
>

2018-12-23 09:43:18

by Sasha Levin

[permalink] [raw]
Subject: Re: Fix "perf tools: Synthesize GROUP_DESC feature in pipe mode" in the LT 4.14 branch

On Fri, Dec 21, 2018 at 11:57:53AM +0100, Jinpu Wang wrote:
>+cc Greg, stable
>
>Greensky, James J <[email protected]> 于2018年12月21日周五 上午11:48写道:
>>
>> Commit d38d272592737ea88a20 ("perf tools: Synthesize GROUP_DESC feature in pipe mode") broke the LT 4.14 branch when using event groups in pipe-mode.
>>
>> # perf record -e '{cycles,instructions,branches}' -- sleep 4 | perf report
>> # To display the perf.data header info, please use --header/--header-only options
>> #
>> Oxd7c [0x60]: failed to process type: 80
>> Error:
>> Failed to process sample
>>
>> Commit a2015516c5c0be932a69 ("perf record: Synthesize features before events in pipe mode") is the fix. Can we get this cherry-picked and applied?

I've queued d38d272592737ea88a20 for 4.14, thank you.

--
Thanks,
Sasha

2018-12-23 16:30:02

by Sasha Levin

[permalink] [raw]
Subject: Re: Fix "perf tools: Synthesize GROUP_DESC feature in pipe mode" in the LT 4.14 branch

On Fri, Dec 21, 2018 at 10:43:09PM +0000, Greensky, James J wrote:
>This commit does not apply cleanly due to other changes. Is there something I can do to make it easier.
>
>This line:
> If (data->is_pipe) {
>Must be changed to:
> If (file->is_pipe) {

Indeed, I've resolved it the same.

>Let me know if there is a more standard way of communicating cherry-picked changes for inclusion.

Sending a mail to [email protected] is pretty much the standard
way.

--
Thanks,
Sasha