From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Google-Smtp-Source: AH8x227anLtAfS2zInPs6MTfOSgOP1eeCf6CSXemw15+Q2nLnG0eOS+tZcbZRDYVKNmfSWp4PwvN ARC-Seal: i=1; a=rsa-sha256; t=1516611236; cv=none; d=google.com; s=arc-20160816; b=XlpfO76S+Vz6mUkFiHsJBkYXN7oJlhAxClGDWOWqMJzrjyMuFkOMxmsOhVz2Odpj8X zfaXBPtjNAnbAzTSoOyespC7SXgCtbNL/mhjM35THe7t7xse9P51vjbMrWDBJFApCiaa z2GVKJc46lGCsu95H9NS/QGACcXBoPpsO2KBXudWz0B+wU55/aSoOjhHVWm1N1JYo4R/ 8oLbZwrJ4Fcx8OewvEGlm5mKSRH/sFLOli+rrZvIJWYm9yA0LSSU6jpX11UonvqNlYWX og1SxvHE8UU+ljJRwrQDg/wdBh8cnwPs3TCQBojD09PrCer0vXay+04ubGTnGjOK9sp1 aMWg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=mime-version:user-agent:references:in-reply-to:message-id:date :subject:cc:to:from:arc-authentication-results; bh=cQaYMkHZh3w8sotnzj6//LVaHbsvKiyyzJ56yRqpdEE=; b=i4TFqrWef5u4awyeezAi7jnMuCpCkFknFECElHvD6lMt3Xcx8QcDNPesBubWIJQhqd JE0gE6tcj6OzMFot6TKM6IT5eETucKAEdQ3Fs+mtx0uQ8NZgWuNA3VD+c4DvCiyneifk n43Fvda6niemTi/5+eDl4EvWZOSNlXDTqNAO1SH1WcMSkIyIBqcmDk0byjev2nhsNttz 9FmyQ6sUAkoW0w1wtwXLzItHlJS6MK7uKHu7dKDHSI8o339axkOTjCcxrVsVXLiVUbJ7 leRi42c7Xfh/qrUveez2EXw2h+sy9BNE6JWghAv+2F0JFE8/MUYw3tSp48bEbD3MGbc/ 8dlQ== ARC-Authentication-Results: i=1; mx.google.com; spf=softfail (google.com: domain of transitioning gregkh@linuxfoundation.org does not designate 90.92.71.90 as permitted sender) smtp.mailfrom=gregkh@linuxfoundation.org Authentication-Results: mx.google.com; spf=softfail (google.com: domain of transitioning gregkh@linuxfoundation.org does not designate 90.92.71.90 as permitted sender) smtp.mailfrom=gregkh@linuxfoundation.org From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Chuck Lever , "Steven Rostedt (VMware)" Subject: [PATCH 4.14 55/89] tracing: Fix converting enums from the map in trace_event_eval_update() Date: Mon, 22 Jan 2018 09:45:35 +0100 Message-Id: <20180122084000.204291831@linuxfoundation.org> X-Mailer: git-send-email 2.16.0 In-Reply-To: <20180122083954.683903493@linuxfoundation.org> References: <20180122083954.683903493@linuxfoundation.org> User-Agent: quilt/0.65 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-LABELS: =?utf-8?b?IlxcU2VudCI=?= X-GMAIL-THRID: =?utf-8?q?1590281473840055393?= X-GMAIL-MSGID: =?utf-8?q?1590282144123472104?= X-Mailing-List: linux-kernel@vger.kernel.org List-ID: 4.14-stable review patch. If anyone has any objections, please let me know. ------------------ From: Steven Rostedt (VMware) commit 1ebe1eaf2f02784921759992ae1fde1a9bec8fd0 upstream. Since enums do not get converted by the TRACE_EVENT macro into their values, the event format displaces the enum name and not the value. This breaks tools like perf and trace-cmd that need to interpret the raw binary data. To solve this, an enum map was created to convert these enums into their actual numbers on boot up. This is done by TRACE_EVENTS() adding a TRACE_DEFINE_ENUM() macro. Some enums were not being converted. This was caused by an optization that had a bug in it. All calls get checked against this enum map to see if it should be converted or not, and it compares the call's system to the system that the enum map was created under. If they match, then they call is processed. To cut down on the number of iterations needed to find the maps with a matching system, since calls and maps are grouped by system, when a match is made, the index into the map array is saved, so that the next call, if it belongs to the same system as the previous call, could start right at that array index and not have to scan all the previous arrays. The problem was, the saved index was used as the variable to know if this is a call in a new system or not. If the index was zero, it was assumed that the call is in a new system and would keep incrementing the saved index until it found a matching system. The issue arises when the first matching system was at index zero. The next map, if it belonged to the same system, would then think it was the first match and increment the index to one. If the next call belong to the same system, it would begin its search of the maps off by one, and miss the first enum that should be converted. This left a single enum not converted properly. Also add a comment to describe exactly what that index was for. It took me a bit too long to figure out what I was thinking when debugging this issue. Link: http://lkml.kernel.org/r/717BE572-2070-4C1E-9902-9F2E0FEDA4F8@oracle.com Fixes: 0c564a538aa93 ("tracing: Add TRACE_DEFINE_ENUM() macro to map enums to their values") Reported-by: Chuck Lever Teste-by: Chuck Lever Signed-off-by: Steven Rostedt (VMware) Signed-off-by: Greg Kroah-Hartman --- kernel/trace/trace_events.c | 16 +++++++++++++++- 1 file changed, 15 insertions(+), 1 deletion(-) --- a/kernel/trace/trace_events.c +++ b/kernel/trace/trace_events.c @@ -2213,6 +2213,7 @@ void trace_event_eval_update(struct trac { struct trace_event_call *call, *p; const char *last_system = NULL; + bool first = false; int last_i; int i; @@ -2220,15 +2221,28 @@ void trace_event_eval_update(struct trac list_for_each_entry_safe(call, p, &ftrace_events, list) { /* events are usually grouped together with systems */ if (!last_system || call->class->system != last_system) { + first = true; last_i = 0; last_system = call->class->system; } + /* + * Since calls are grouped by systems, the likelyhood that the + * next call in the iteration belongs to the same system as the + * previous call is high. As an optimization, we skip seaching + * for a map[] that matches the call's system if the last call + * was from the same system. That's what last_i is for. If the + * call has the same system as the previous call, then last_i + * will be the index of the first map[] that has a matching + * system. + */ for (i = last_i; i < len; i++) { if (call->class->system == map[i]->system) { /* Save the first system if need be */ - if (!last_i) + if (first) { last_i = i; + first = false; + } update_event_printk(call, map[i]); } }