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:	Sun, 8 Mar 2009 19:49:08 -0400 (EDT)
From:	Steven Rostedt <rostedt@...dmis.org>
To:	Sitsofe Wheeler <sitsofe@...oo.com>
cc:	Lai Jiangshan <laijs@...fujitsu.com>,
	Frederic Weisbecker <fweisbec@...il.com>,
	Ingo Molnar <mingo@...e.hu>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [TIP,BISECTED] Negative nice values have become big positive
 numbers


On Sun, 8 Mar 2009, Sitsofe Wheeler wrote:

> (Forgot to cc lkml. Resending...)
> 
> Formally negative nice values have started become very big in positive
> integers in -tip kernels:
> 
>     2 root      15 2147483647     0    0    0 S  0.0  0.0   0:00.00 kthreadd    

Is this the output of top?

> 
> I've just finished bisecting down to this commit:
> 
> commit 1427cdf0592368bdec57276edaf714040ee8744f
> Author: Lai Jiangshan <laijs@...fujitsu.com>
> Date:   Fri Mar 6 17:21:47 2009 +0100

$ git show 1427cdf0592368bdec57276edaf714040ee8744f  | diffstat
 include/linux/ftrace.h       |    3 +
 kernel/trace/Kconfig         |    6 ++
 kernel/trace/Makefile        |    1 
 kernel/trace/trace.c         |   56 +++++++++++++++++++++++++++
 kernel/trace/trace.h         |   12 +++++
 kernel/trace/trace_bprintk.c |   87 
+++++++++++++++++++++++++++++++++++++++++++
 kernel/trace/trace_output.c  |   75 +++++++++++++++++++++++++++++++++++++
 7 files changed, 240 insertions(+)

I find it hard to believe that this would cause normal nice values to be 
messed up. The only file that could could come close to messing with nice 
values in top is ftrace.h:


diff --git a/include/linux/ftrace.h b/include/linux/ftrace.h
index 4987694..1c9cdca 100644
--- a/include/linux/ftrace.h
+++ b/include/linux/ftrace.h
@@ -223,6 +223,9 @@ extern int ftrace_make_nop(struct module *mod,
  */
 extern int ftrace_make_call(struct dyn_ftrace *rec, unsigned long addr);
 
+#ifdef CONFIG_TRACE_BPRINTK
+extern int trace_vbprintk(unsigned long ip, const char *fmt, va_list args);
+#endif
 
 /* May be defined in arch */
 extern int ftrace_arch_read_dyn_info(char *buf, int size);



And that too would be hard to affect nice values. Do you have a config 
that I could test with?

Thanks,

-- Steve

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

Powered by Openwall GNU/*/Linux Powered by OpenVZ