linux-kernel - Re: [PATCH v1] change likeliness accounting

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Fri, 14 Mar 2008 12:53:31 -0700
From:	Daniel Walker <dwalker@...sta.com>
To:	Roel Kluin <12o3l@...cali.nl>
Cc:	lkml <linux-kernel@...r.kernel.org>, Hua Zhong <hzhong@...il.com>,
	akpm@...ux-foundation.org
Subject: Re: [PATCH v1] change likeliness accounting

On Fri, 2008-03-14 at 00:06 +0100, Roel Kluin wrote:

> -	seq_printf(out, "|%9u|%9u\t%s()@:%s@%d\n", true, false,
> -			entry->func, entry->file, entry->line);
> +	sprint_symbol(function, entry->caller);
> +	seq_printf(out, "|%9u|%9u\t%s()@:%u\n", pos, neg, function,
> +					entry->label >> LP_LINE_SHIFT);

Looks like we're missing a "|" after the second %9u above .. You could
add that as long as your modifying this area.. Also your tabbing is a
little too heavy ..

I wonder if you think the change to saving the caller EIP is better than
saving the file name? The size saving in the likeliness struct is nice,
but since this is for debugging is not at all required ..

Overall it looks like a nice update/cleanup ..

(CC'd Andrew and Hua..)

Daniel

--
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/