Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752210AbbHTR43 (ORCPT ); Thu, 20 Aug 2015 13:56:29 -0400 Received: from mx1.redhat.com ([209.132.183.28]:60616 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751798AbbHTR41 (ORCPT ); Thu, 20 Aug 2015 13:56:27 -0400 Subject: Re: [PATCH v3] tools lib traceevent: add checks for returned EVENT_ERROR type To: Steven Rostedt References: <20150820151632.13927.13791.email-sent-by-dnelson@teal> <20150820130558.4f3a2e35@grimm.local.home> Cc: a.p.zijlstra@chello.org, mingo@redhat.com, acme@kernel.org, namhyung@kernel.org, linux-kernel@vger.kernel.org, jolsa@kernel.org From: Dean Nelson Message-ID: <55D614C9.8080503@redhat.com> Date: Thu, 20 Aug 2015 12:56:25 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.1.0 MIME-Version: 1.0 In-Reply-To: <20150820130558.4f3a2e35@grimm.local.home> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2451 Lines: 70 On 08/20/2015 12:05 PM, Steven Rostedt wrote: > On Thu, 20 Aug 2015 11:16:32 -0400 > Dean Nelson wrote: > >> Running the following perf-stat command on an arm64 system produces the >> following result... >> >> [root@aarch64 ~]# perf stat -e kmem:mm_page_alloc -a sleep 1 >> Warning: [kmem:mm_page_alloc] function sizeof not defined >> Warning: Error: expected type 4 but read 0 >> Segmentation fault >> [root@aarch64 ~]# >> >> The second warning message and SIGSEGV stem from the issue expressed in the >> first warning message, and are the result of ignoring the EVENT_ERROR type >> returned back through the call chain. >> >> Dealing with the first warning message is beyond the scope of this patch. But >> the second warning is addressed by this patch's first hunk. And the SIGSEGV is >> eliminated by its second hunk. > > Patch looks fine, but this change log is lacking. I don't think you > need to resend though. But Arnaldo, can you add more to this change log > to describe the following, and that's only if I got it right ;-) If I > didn't get it right, then the change log definitely needs to be > explained better. No you definitely got it right. I thought that was what I was saying by the paragraph beginning with "The second warning...", with the notion that the 2nd warning and SIGSEGV "stem from" the 1st warning. And that the latter two issues "are the result of ignoring the EVENT_ERROR" encountered by the 1st warning's issue. At least that is what that paragraph was intended to be all about. Obviously I failed to communicate. Yours is clear to me. So why not just replace my poorly done paragraph with your good paragraph... > ==== > The second warning was a result of the first warning not stopping > processing after it detected the issue. That is, code that found the > issue reported the first problem, but because it did not exit out of > the functions smoothly, it caused the other warning to appear and not > only that, it later caused the SIGSEGV. > ==== Thanks for the review. > > -- Steve > > Other than that... > > Acked-by: Steven Rostedt > > >> >> Signed-off-by: Dean Nelson -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/