Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751239AbbFYED6 (ORCPT ); Thu, 25 Jun 2015 00:03:58 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:51048 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750810AbbFYEDt (ORCPT ); Thu, 25 Jun 2015 00:03:49 -0400 Message-ID: <558B7D76.4020705@oracle.com> Date: Thu, 25 Jun 2015 00:03:02 -0400 From: Sasha Levin User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Steven Rostedt , Linus Torvalds CC: LKML , Ingo Molnar , Peter Zijlstra , Vince Weaver , Arnaldo Carvalho de Melo Subject: Re: [GIT PULL] tracing: Have filter check for balanced ops References: <20150617083638.20304e44@gandalf.local.home> In-Reply-To: <20150617083638.20304e44@gandalf.local.home> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Source-IP: userv0021.oracle.com [156.151.31.71] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2017 Lines: 40 On 06/17/2015 08:36 AM, Steven Rostedt wrote: > Linus, > > Vince Weaver reported a warning when he added perf event filters > into his fuzzer tests. There's a missing check of balanced > operations when parenthesis are used, and this triggers a WARN_ON() > and when reading the failure, the filter reports no failure occurred. Hey Steven, My fuzzings are hitting the warning added by this patch: [2175114.187536] WARNING: CPU: 16 PID: 10388 at kernel/trace/trace_events_filter.c:1388 replace_preds+0x814/0x2140() [2175114.190213] Modules linked in: [2175114.191111] CPU: 16 PID: 10388 Comm: trinity-c48 Not tainted 4.1.0-next-20150623-sasha-00039-ga1eb83a-dirty #2280 [2175114.194463] ffff880a23350000 000000006a8e22d4 ffff880a2335f878 ffffffffabc8cfa3 [2175114.196547] 0000000000000000 0000000000000000 ffff880a2335f8c8 ffffffffa21ebd36 [2175114.198604] ffff880e60fe09e0 ffffffffa24608f4 ffff880e61b14830 ffff880e60fe09d8 [2175114.200666] Call Trace: [2175114.201377] [] dump_stack+0x4f/0x7b [2175114.202793] [] warn_slowpath_common+0xc6/0x120 [2175114.206235] [] warn_slowpath_null+0x1a/0x20 [2175114.207819] [] replace_preds+0x814/0x2140 [2175114.216433] [] create_filter+0x15a/0x210 [2175114.231529] [] apply_event_filter+0x28b/0x780 [2175114.241196] [] event_filter_write+0x106/0x1c0 [2175114.242823] [] do_loop_readv_writev+0x128/0x1e0 [2175114.248901] [] do_readv_writev+0x5ae/0x6c0 [2175114.256760] [] vfs_writev+0x72/0xb0 [2175114.258134] [] SyS_pwritev+0x1b4/0x220 [2175114.261291] [] tracesys_phase2+0x88/0x8d Thanks, Sasha -- 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/