| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20080226041349.GA17772@kryten> Date: Mon, 25 Feb 2008 22:13:49 -0600 From: Anton Blanchard <anton@...ba.org> To: Ingo Molnar <mingo@...e.hu> Cc: Arjan van de Ven <arjan@...radead.org>, linux-kernel@...r.kernel.org, sandmann@...hat.com, tglx@...x.de, hpa@...or.com Subject: Re: [PATCH] x86: add the debugfs interface for the sysprof tool > > From: Soren Sandmann <sandmann@...hat.com> > > Subject: [PATCH] x86: add the debugfs interface for the sysprof tool > > > > The sysprof tool is a very easy to use GUI tool to find out where > > userspace is spending CPU time. See > > http://www.daimi.au.dk/~sandmann/sysprof/ for more information and > > screenshots on this tool. > > > > Sysprof needs a 200 line kernel module to do it's work, this module > > puts some simple profiling data into debugfs. > > thanks, looks good to me - applied. Woah slow down guys. Did I miss the review? Yes it's a 200 line patch, but it's a 200 line x86 patch. Surely we should apply some of the same rigour we did when we merged the oprofile patch? Is it biarch safe? Will it run on powerpc, arm etc? I'm still struggling to understand why we need this functionality at all. Lets argue the ABI and not cloud it with a discussion about userspace eye candy. What does this give you that is an improvement over the oprofile kernel-user ABI? Anton -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists