lkml.org 
[lkml]   [2011]   [Sep]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] Janitor: Typo in stack debug code
From
2011/9/13 Linas Vepstas (Code Aurora) <linas@codeaurora.org>:
>
> In the debug routine check_stack_usage(), free stack space is counted
> in units of (unsigned long), not bytes.  Make this clear when printing.
>
> It seems easiest to just change the message, instead of multiplying
> by sizeof(unsigned long).
>
> Signed-off-by: Linas Vepstas <linas@codeaurora.org>
>
> ---
>
> ===================================================================
> --- a/kernel/exit.c
> +++ b/kernel/exit.c
> @@ -888,7 +888,7 @@ static void check_stack_usage(void)
>
>        spin_lock(&low_water_lock);
>        if (free < lowest_to_date) {
> -               printk(KERN_WARNING "%s used greatest stack depth: %lu bytes "
> +               printk(KERN_WARNING "%s used greatest stack depth: %lu words "
>                                "left\n",
>                                current->comm, free);
>                lowest_to_date = free;
>
>
>
>

Thats not true, everything is fine.

dmesg | grep stack
[ 1.564907] kworker/u:0 used greatest stack depth: 5592 bytes left
[ 2.467909] modprobe used greatest stack depth: 5576 bytes left
[ 2.483228] exe used greatest stack depth: 5552 bytes left
[ 2.489430] all_generic_ide used greatest stack depth: 5496 bytes left
[ 2.524731] scsi_id used greatest stack depth: 5440 bytes left
[ 2.643103] blkid used greatest stack depth: 4768 bytes left
[ 4.232051] exe used greatest stack depth: 4320 bytes left
[ 4.761443] stty used greatest stack depth: 3784 bytes left
[ 6.011566] modprobe used greatest stack depth: 3664 bytes left
[ 6.011681] modprobe used greatest stack depth: 3184 bytes left
[ 339.890273] nroff used greatest stack depth: 3032 bytes left
--
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/

\
 
 \ /
  Last update: 2011-09-13 18:33    [W:0.041 / U:0.684 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site