From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752867AbYLSQZX (ORCPT ); Fri, 19 Dec 2008 11:25:23 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751251AbYLSQZJ (ORCPT ); Fri, 19 Dec 2008 11:25:09 -0500 Received: from qw-out-2122.google.com ([74.125.92.25]:48339 "EHLO qw-out-2122.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751337AbYLSQZH (ORCPT ); Fri, 19 Dec 2008 11:25:07 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:cc:in-reply-to:mime-version :content-type:content-transfer-encoding:content-disposition :references; b=xlGfh55m9fn5lX7T3f78zPLBkFtXMDsbU5Hq4npXmGkytjcSePK3n+p5WfWDNMC3IG 3N4iPsfI5j2LH8ZMLgmXmmmr+GH62tKb+o0J7KZk2GaaC8+YP5Vwdmqmpl1PvTjtNOeL QdQh9do0ziut9V4HpxTBPnx/zpppi4cHPXHK4= Message-ID: Date: Fri, 19 Dec 2008 17:25:05 +0100 From: "=?ISO-8859-1?Q?Fr=E9d=E9ric_Weisbecker?=" To: "Thomas Gleixner" Subject: Re: [patch 1/7] ftrace: fix task state printout Cc: LKML , "Ingo Molnar" , "Steven Rostedt" , "Peter Zijlstra" , "Clark Williams" , "Gregory Haskins" , Linux-rt In-Reply-To: <20081219155555.919550408@linutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <20081219155504.735865178@linutronix.de> <20081219155555.919550408@linutronix.de> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Thomas, 2008/12/19 Thomas Gleixner : > Impact: tracer task state decoding is wrong, size check is buggy > > The tracing code has interesting varieties of printing out task state. > Unfortunalely only one of the instances is correct as it copies the > code from sched.c:sched_show_task(). The others are plain wrong as > they treatthe bitfield as an integer offset into the character > array. Also the size check of the character array is wrong as it > includes the trailing \0. > > Use a common state decoder inline which does the Right Thing. > > Signed-off-by: Thomas Gleixner You've already sent it out. It has been applied on -tip :-)