Re: 2.6.23-rc4-mm1 and -rc6-mm1: boot failure on HP nx6325, related to clockevents

2007-09-26 Thread Rafael J. Wysocki
Thomas, On Tuesday, 25 September 2007 23:24, Thomas Gleixner wrote: Rafael, On Tue, 2007-09-25 at 23:28 +0200, Rafael J. Wysocki wrote: I'm a bit confused by your earlier confirmation, that mainline w/o the -hrt patches boots fine, when you add apicmaintimer to the kernel command

Re: [RFC Patch] Trace - Samples

2007-09-26 Thread David Wilder
Randy Dunlap wrote: Mathieu Desnoyers wrote: * Randy Dunlap ([EMAIL PROTECTED]) wrote: On Tue, 25 Sep 2007 13:35:47 -0700 David J. Wilder wrote: This patch Moves the trace example into the new samples/ infrastructure Requires: [patch 3/5] Add samples subdir

Re: sys_chroot+sys_fchdir Fix

2007-09-26 Thread Chris Adams
Once upon a time, Alan Cox [EMAIL PROTECTED] said: Well if citing standards documents at people is rudeness so be it. I hate to get involved in this, but actually chroot() is no longer part of SuS as of version 3. For other Unix versions, both Tru64 (5.1B) and Solaris (9) chroot(2) man pages

Re: [PATCH 0/4] allow drivers to flush in-flight DMA

2007-09-26 Thread Jesse Barnes
On Tuesday, September 25, 2007 11:49:50 pm Grant Grundler wrote: Upon reading the 2) Platforms that permit DMA reordering, I think I have been confusing coherency with ordering. I think I have because DMA is leaving the PCI domain, crossing an unordered domain (NUMA, interconnect), and then

Re: [PATCH 03/25] Unionfs: display informational messages only if debug is on

2007-09-26 Thread Jan Engelhardt
On Sep 26 2007 10:01, Erez Zadok wrote: On Sep 25 2007 23:09, Erez Zadok wrote: --- a/fs/unionfs/commonfops.c +++ b/fs/unionfs/commonfops.c @@ -394,8 +394,8 @@ int unionfs_file_revalidate(struct file *file, bool willwrite) if (willwrite IS_WRITE_FLAG(file-f_flags)

Re: [PATCH 10/25] Unionfs: add un/likely conditionals on copyup ops

2007-09-26 Thread Kyle Moffett
On Sep 26, 2007, at 09:40:20, Erez Zadok wrote: In message [EMAIL PROTECTED], Kok, Auke writes: I've been told several times that adding these is almost always bogus - either it messes up the CPU branch prediction or the compiler/CPU just does a lot better at finding the right way without

Re: subsystem_unregister() breakage in -mm

2007-09-26 Thread Greg KH
On Wed, Sep 26, 2007 at 10:17:45AM +0200, Miklos Szeredi wrote: BUG: failure at mm/slab.c:591/page_get_cache()! Kernel panic - not syncing: BUG! Ok, here's a patch that fixes this for me. It should go on top of the -mm tree. Let me know of this works or not. Yes, it works for me

Re: subsystem_unregister() breakage in -mm

2007-09-26 Thread Greg KH
On Wed, Sep 26, 2007 at 09:21:56AM +0200, Cornelia Huck wrote: On Tue, 25 Sep 2007 19:41:21 -0700, Greg KH [EMAIL PROTECTED] wrote: Ok, here's a patch that fixes this for me. It should go on top of the -mm tree. Let me know of this works or not. Looks sane to me. In the end, we

Re: [PATCHSET 1/4] sysfs: misc updates

2007-09-26 Thread Greg KH
On Wed, Sep 26, 2007 at 01:04:24PM +0200, Cornelia Huck wrote: On Thu, 20 Sep 2007 16:05:09 +0900, Tejun Heo [EMAIL PROTECTED] wrote: 0001-sysfs-kill-SYSFS_FLAG_REMOVED.patch 0002-sysfs-fix-comments-of-sysfs_add-remove_one.patch

Re: kswapd high CPU usage with no swap

2007-09-26 Thread Rik van Riel
On Wed, 26 Sep 2007 09:27:36 +0200 Jan Kundrát [EMAIL PROTECTED] wrote: Rik van Riel wrote: Could you try out the attached patch, too? Sorry, I wasn't able to apply it against 2.6.22-gentoo-r3 and vanilla 2.6.22.7; I don't have the order member in the struct scan_control and also the bit

RE: Regression in 2.6.23-pre Was: Problems with 2.6.23-rc6 on AMD Geode LX800

2007-09-26 Thread Joerg Pommnitz
There is something very fishy. The only documentation you've given us so far is a screen shot which contained a message (BIOS data check successful) which doesn't occur in the kernel. The loader string doesn't look all that familiar either; it looks like an extremely old version

Re: Linux Kernel Markers - performance characterization with large IO load on large-ish system

2007-09-26 Thread Alan D. Brunelle
Mathieu Desnoyers wrote: * Alan D. Brunelle ([EMAIL PROTECTED]) wrote: Taking Linux 2.6.23-rc6 + 2.6.23-rc6-mm1 as a basis, I took some sample runs of the following on both it and after applying Mathieu Desnoyers 11-patch sequence (19 September 2007). * 32-way IA64 + 132GiB + 10 FC

Re: [PATCH 03/25] Unionfs: display informational messages only if debug is on

2007-09-26 Thread Erez Zadok
In message [EMAIL PROTECTED], Jan Engelhardt writes: [...] Surprise, pr_debug() is just that all nicely wrapped up. Want debug? Do it like this. #ifdef CONFIG_UNIONFS_DEBUG # define DEBUG 1 #endif and pr_debug() works magic. [...] Sounds good. I'll do that. Thanks, Erez. - To

Re: [PATCH] Module use count must be updated as bridges are created/destroyed

2007-09-26 Thread Stephen Hemminger
On Wed, 26 Sep 2007 08:53:27 +0100 Jan Beulich [EMAIL PROTECTED] wrote: Otherwise 'modprobe -r' on a module having a dependency on bridge will implicitly unload bridge, bringing down all connectivity that was using bridges. Signed-off-by: Jan Beulich [EMAIL PROTECTED] No, network devices

2.6.23-rc8-mm1: ata3: soft resetting link after STR

2007-09-26 Thread Alexey Dobriyan
Frequently get these messages after resume from STR, (subjectively, first STR is always OK). CONFIG_ATA_PIIX=y is used. CONFIG_IDE=n. This happened with 2.6.23-rc7-mm1, 2.6.23-rc8-mm1, and 2.6.23-rc7-hrt1.patch ata3.00: exception Emask 0x0 SAct 0x0 SErr 0x0 action 0x2 ata3.00: BMDMA stat 0x6

Re: Network slowdown due to CFS

2007-09-26 Thread Stephen Hemminger
On Wed, 26 Sep 2007 15:31:38 +0200 Ingo Molnar [EMAIL PROTECTED] wrote: * David Schwartz [EMAIL PROTECTED] wrote: I think the real fix would be for iperf to use blocking network IO though, or maybe to use a POSIX mutex or POSIX semaphores. So it's definitely not a bug in the

Re: Regression in 2.6.23-pre Was: Problems with 2.6.23-rc6 on AMD Geode LX800

2007-09-26 Thread Jordan Crouse
On 26/09/07 07:10 -0700, H. Peter Anvin wrote: Joerg Pommnitz wrote: Hello all, this is what git bisect told me about the problem: [EMAIL PROTECTED]:~/linux-2.6$ git bisect good 4fd06960f120e02e9abc802a09f9511c400042a5 is first bad commit commit

Re: [uml-devel] [PATCH] UML - Fix irqstack crash

2007-09-26 Thread Paolo Giarrusso
On lunedì 24 settembre 2007, Jeff Dike wrote: On Thu, Sep 20, 2007 at 05:57:49PM +0200, Paolo Giarrusso wrote: Yes, indeed - or sign extension on 64bit machines would set to 1 the whole high-word. But using long for that mask makes no difference; either int or long long (or better,

Re: ACPI power off regression in 2.6.23-rc8 (NOT in rc7)

2007-09-26 Thread Tim Post
Bah, its too damn stable. Break it and do it again. From 2.6.20.3 : Boot time cut in half. My PC no longer 'wakes up' angrily. My wife does that, I'm going to start sleeping with the P4, its more agreeable now. P4 HT with generic Intel chipset. What fun is this when nothing breaks? Thank you

Re: [PATCH 10/25] Unionfs: add un/likely conditionals on copyup ops

2007-09-26 Thread Erez Zadok
In message [EMAIL PROTECTED], Kyle Moffett writes: On Sep 26, 2007, at 09:40:20, Erez Zadok wrote: [...] Recently we've done a full audit of the entire code, and added un/ likely where we felt that the chance of succeeding is 95% or better (e.g., error conditions that should rarely

Re: [uml-devel] [PATCH] UML - time build fix

2007-09-26 Thread Paolo Giarrusso
On mercoledì 26 settembre 2007, Thomas Gleixner wrote: Jeff, On Tue, 2007-09-25 at 17:56 -0400, Jeff Dike wrote: On Tue, Sep 25, 2007 at 09:54:15PM +0200, Thomas Gleixner wrote: On Tue, 2007-09-25 at 13:37 -0400, Jeff Dike wrote: Put back an implementation of timeval_to_ns in

Re: Network slowdown due to CFS

2007-09-26 Thread Stephen Hemminger
Here is the combined fixes from iperf-users list. Begin forwarded message: Date: Thu, 30 Aug 2007 15:55:22 -0400 From: Andrew Gallatin [EMAIL PROTECTED] To: [EMAIL PROTECTED] Subject: [PATCH] performance fixes for non-linux Hi, I've attached a patch which gives iperf similar performance to

[PATCH 0/5] A config option to compile out some namespaces code

2007-09-26 Thread Pavel Emelyanov
There were some questions like do I need this on my cellphone in reply to different namespaces patches. Indeed, the namespaces are not useful for most of the embedded systems, but the code creating and releasing them weights a lot. So I propose to add a config option which will help embedded

[PATCH 2/5] Move the UST namespace under the option

2007-09-26 Thread Pavel Emelyanov
Currently all the namespace management code is in the kernel/utsname.c file, so just compile it out and make stub in .h file. The init namespace itself is in init/version.c and is left in the kernel. Signed-off-by: Pavel Emelyanov [EMAIL PROTECTED] --- diff --git a/include/linux/utsname.h

Re: 2.6.23-rc8-mm1: ata3: soft resetting link after STR

2007-09-26 Thread Alan Cox
On Wed, 26 Sep 2007 19:39:01 +0400 Alexey Dobriyan [EMAIL PROTECTED] wrote: Frequently get these messages after resume from STR, (subjectively, first STR is always OK). Does this occur if you have the acpi support enabled (libata.noacpi=0) - To unsubscribe from this list: send the line

[PATCH 1/5] The config option itself

2007-09-26 Thread Pavel Emelyanov
The option is called NAMESPACES. It can be selectable only if EMBEDDED is chosen (this was Eric's requisition). When the EMBEDDED is off namespaces will be on automatically. One more option (NAMESPACES_EXPERIMENTAL) was added by Serge's request to move there all the namespaces that are not

[PATCH 4/5] Move the user namespace under the option

2007-09-26 Thread Pavel Emelyanov
We currently have a CONFIG_USER_NS option. Just rename it into CONFIG_NAMESPACES_EXPERIMANTAL and move the init_user_ns into user.c file to make the kernel compile and work without the namespaces support. Signed-off-by: Pavel Emelyanov [EMAIL PROTECTED] --- diff --git

Re: [PATCH] ahci: enable GHC.AE bit before set GHC.HR

2007-09-26 Thread Alan Cox
On Wed, 26 Sep 2007 10:33:28 -0400 Jeff Garzik [EMAIL PROTECTED] wrote: Alan Cox wrote: NAK - mmio is an iomap so writel and readl are the wrong things to use The patch is consistent with the rest of the driver. You are welcome to submit a patch to convert ahci to using ioremap. You

[PATCH 5/5] Move the PID namespace under the option

2007-09-26 Thread Pavel Emelyanov
For the same reasons as with the IPC namespaces, all the prototypes and stuns go to the pid_namespace.h file. The namespace management code itself is moved to the pid_namespace.c file. The pid_namespace cache is created inside an initcall, i.e. a bit later than the pid hash is initialized. This

[PATCH 3/5] Move the IPC namespace under the option

2007-09-26 Thread Pavel Emelyanov
Currently all the IPC namespace management code is in ipc/util.c. I moved this code into ipc/namespace.c file which is compiled out when needed. The linux/ipc_namespace.h file is used to store the prototypes of the functions in namespace.c and the stubs for NAMESPACES=n case. This is done so,

Re: KPROBES: Instrumenting a function's call site

2007-09-26 Thread Avishay Traeger
On Wed, 2007-09-26 at 14:33 +0530, Ananth N Mavinakayanahalli wrote: What happens when the call is singlestepped is that the instruction pointer is moved to the call target. That explains the lower latency you are seeing. You'll need to do something along the lines I suggested in the earlier

Re: missing mnt_drop_write() on open error

2007-09-26 Thread Dave Hansen
On Wed, 2007-09-26 at 10:38 +0200, Miklos Szeredi wrote: On Wed, 2007-09-26 at 01:14 +0200, Miklos Szeredi wrote: I get this at umount, if there was a failed open(): WARNING: at fs/namespace.c:586 __mntput() I think the problem is that may_open() calls mnt_want_write(), but if

Re: [patch 1/7] Extended crashkernel command line

2007-09-26 Thread Bernhard Walle
* Oleg Verych [EMAIL PROTECTED] [2007-09-25 22:53]: + * + * The function returns 0 on success and -EINVAL on failure. + */ +static int __init parse_crashkernel_mem(char *cmdline, + unsigned long long system_ram, +

Re: missing mnt_drop_write() on open error

2007-09-26 Thread Dave Hansen
On Wed, 2007-09-26 at 10:38 +0200, Miklos Szeredi wrote: In __dentry_open() there's still a few places where fput() won't be called, notably when -open fails, which is what I'm triggering I think. Also even more horrible things can happen because of the nd-intent.open.file thing. For

Re: [PATCH] ahci: enable GHC.AE bit before set GHC.HR

2007-09-26 Thread Jeff Garzik
Alan Cox wrote: On Wed, 26 Sep 2007 10:33:28 -0400 Jeff Garzik [EMAIL PROTECTED] wrote: Alan Cox wrote: NAK - mmio is an iomap so writel and readl are the wrong things to use The patch is consistent with the rest of the driver. You are welcome to submit a patch to convert ahci to using

Re: [PATCH 1/5] The config option itself

2007-09-26 Thread Randy Dunlap
On Wed, 26 Sep 2007 19:43:28 +0400 Pavel Emelyanov wrote: The option is called NAMESPACES. It can be selectable only if EMBEDDED is chosen (this was Eric's requisition). When the EMBEDDED is off namespaces will be on automatically. and when EMBEDDED is on, namespaces will be off

Re: [PATCH 1/5] The config option itself

2007-09-26 Thread Serge E. Hallyn
Quoting Pavel Emelyanov ([EMAIL PROTECTED]): The option is called NAMESPACES. It can be selectable only if EMBEDDED is chosen (this was Eric's requisition). When the EMBEDDED is off namespaces will be on automatically. One more option (NAMESPACES_EXPERIMENTAL) was added by Serge's request

[PATCH 2/8] taskstats: split the basic accounting fields

2007-09-26 Thread Guillaume Chazarain
Split the basic accounting taskstats fields into the threadgroup specific ones and the thread specific ones. This should have no effect on the execution. Signed-off-by: Guillaume Chazarain [EMAIL PROTECTED] Cc: Balbir Singh [EMAIL PROTECTED] Cc: Jay Lan [EMAIL PROTECTED] Cc: Jonathan Lim [EMAIL

Re: KPROBES: Instrumenting a function's call site

2007-09-26 Thread Abhishek Sagar
On 9/26/07, Avishay Traeger [EMAIL PROTECTED] wrote: So to measure the latency of foo(), I basically want kprobes to do this: pre_handler(); foo(); post_handler(); The problem is that the latencies that I am getting are consistently low (~10,000 cycles). When I manually instrument the

[PATCH 4/8] taskstats: separate PID/TGID stats producers to complete the TGID ones

2007-09-26 Thread Guillaume Chazarain
TASKSTATS_CMD_ATTR_TGID used to return only the delay accounting stats, not the basic and extended accounting. With this patch, TASKSTATS_CMD_ATTR_TGID also aggregates the accounting info for all threads of a thread group. TASKSTATS_CMD_ATTR_PID output should be unchanged TASKSTATS_CMD_ATTR_TGID

[PATCH 6/8] taskstats: tell fill_threadgroup_stats() if it replies with PID or TGID stats

2007-09-26 Thread Guillaume Chazarain
fill_threadgroup_stats() may want to know if it is filling TASKSTATS_CMD_ATTR_TGID or TASKSTATS_CMD_ATTR_PID stats, so give it this information in the tg_stats boolean. Signed-off-by: Guillaume Chazarain [EMAIL PROTECTED] Cc: Balbir Singh [EMAIL PROTECTED] Cc: Jay Lan [EMAIL PROTECTED] Cc:

[PATCH 3/8] taskstats: split the extended accounting fields

2007-09-26 Thread Guillaume Chazarain
Split the extended accounting taskstats fields into the threadgroup specific ones and the thread specific ones. This should have no effect on the execution. Signed-off-by: Guillaume Chazarain [EMAIL PROTECTED] Cc: Balbir Singh [EMAIL PROTECTED] Cc: Jay Lan [EMAIL PROTECTED] Cc: Jonathan Lim

[PATCH 5/8] taskstats: factor out version and context switch accounting

2007-09-26 Thread Guillaume Chazarain
All the specific taskstats fields should only be manipulated in {add_tsk,fill_threadgroup}_stats(). Signed-off-by: Guillaume Chazarain [EMAIL PROTECTED] Cc: Balbir Singh [EMAIL PROTECTED] Cc: Jay Lan [EMAIL PROTECTED] Cc: Jonathan Lim [EMAIL PROTECTED] Cc: Oleg Nesterov [EMAIL PROTECTED] ---

[PATCH 1/8] taskstats: fix indentation of long argument lists

2007-09-26 Thread Guillaume Chazarain
Align with the opening parenthesis. Changelog since V1 (http://lkml.org/lkml/2007/9/21/527): - renamed fill_threadgroup() and add_tsk() to respectively fill_threadgroup_stats() and add_tsk_stats() as suggested by Balbir Singh. - added braces around do/while. - added patch to unbreak binary

[PATCH 7/8] taskstats: fix stats-ac_exitcode to work on threads and use group_exit_code

2007-09-26 Thread Guillaume Chazarain
Threads also have an exit code on their own, so report it in TASKSTATS_CMD_ATTR_PID. For TASKSTATS_CMD_ATTR_TGID, instead of relying only on the exit code of the leader, we use task-signal-group_exit_code if not null as suggested by Oleg Nesterov. Also, document that as of this patch,

[PATCH 8/8] taskstats: avoid breaking binary compatibility between taskstats v6 and before

2007-09-26 Thread Guillaume Chazarain
Place fields added in v6 at the end of the struct. Signed-off-by: Guillaume Chazarain [EMAIL PROTECTED] Cc: Michael Neuling [EMAIL PROTECTED] Cc: Balbir Singh [EMAIL PROTECTED] Cc: Jay Lan [EMAIL PROTECTED] Cc: Jonathan Lim [EMAIL PROTECTED] Cc: Oleg Nesterov [EMAIL PROTECTED] ---

Re: [PATCH 10/25] Unionfs: add un/likely conditionals on copyup ops

2007-09-26 Thread Jan Engelhardt
On Sep 26 2007 11:43, Erez Zadok wrote: *That's* the information I was looking for, Kyle: what's the estimated probability I should be using as my guideline. I used 95% (20/1 ratio), and ;-) 19:1 = 95:5 = 95% = ratio=0.95 != 20.0 (=20/1) you're telling me I should use 99% (100/1 ratio).

commit 6dccd16b7c2703e8bbf8bca62b5cf248332afbe2 kills r8169 send performance

2007-09-26 Thread Timo Jantunen
Heip! (commit: r8169: merge with version 6.001.00 of Realtek's r8169 driver) In current 2.6.23-rc8 snapshot r8169 send performance is bad, around 32MB/s. In 2.6.22 it was around 83MB/s. Interestingly, the receive performance has increased from around 85MB/s to 96MB/s at the same time! Git

Re: [PATCH 10/25] Unionfs: add un/likely conditionals on copyup ops

2007-09-26 Thread Erez Zadok
In message [EMAIL PROTECTED], Jan Engelhardt writes: On Sep 26 2007 11:43, Erez Zadok wrote: *That's* the information I was looking for, Kyle: what's the estimated probability I should be using as my guideline. I used 95% (20/1 ratio), and ;-) 19:1 = 95:5 = 95% = ratio=0.95 != 20.0

Re: KPROBES: Instrumenting a function's call site

2007-09-26 Thread Abhishek Sagar
On 9/26/07, Ananth N Mavinakayanahalli [EMAIL PROTECTED] wrote: PS: There was a thought of providing a facility to run a handler at function entry even when just a kretprobe is used. Maybe we need to relook at that; it'd have been useful in this case. That would be really useful. I was

Re: sys_chroot+sys_fchdir Fix

2007-09-26 Thread David Newall
Alan, Alan Cox wrote: therefore it must be right. You present no reasoning to explain why the behavior is correct; instead you use insults. I've exhausted my tolerance for rudeness. Well if citing standards documents at people is rudeness so be it. Did you just tell a porky? Did

Re: sys_chroot+sys_fchdir Fix

2007-09-26 Thread Alan Cox
You quoted the standard, I merely pointed out you forgot to read it properly. Thats your problem not mine. Alan - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: Regression in 2.6.23-pre Was: Problems with 2.6.23-rc6 on AMD Geode LX800

2007-09-26 Thread H. Peter Anvin
Jordan Crouse wrote: As background, I'm using syslinux 3.36 as my loader here - I've used this exact same version for a very long time, so I don't blame it in the least. Something is getting confused in the early kernel, and whatever that something is, a still unknown change in a newer

Re: __kernel_vsyscall () hangs in SIGCHLD handler

2007-09-26 Thread Ulrich Drepper
On 9/26/07, John Z. Bohach [EMAIL PROTECTED] wrote: Is there some reason that syslog() sleeps in __kernel_vsyscall() when invoked from a signal handler? Only very few functions are allowed to be called from signal handlers. This is clearly spelled out in the POSIX spec. Section XSH 2.4.3

Re: [PATCH 1/5] The config option itself

2007-09-26 Thread Serge E. Hallyn
Quoting Serge E. Hallyn ([EMAIL PROTECTED]): Quoting Pavel Emelyanov ([EMAIL PROTECTED]): The option is called NAMESPACES. It can be selectable only if EMBEDDED is chosen (this was Eric's requisition). When the EMBEDDED is off namespaces will be on automatically. One more option

Re: NinjaSCSI PCMCIA card

2007-09-26 Thread Alan Cox
On Wed, 26 Sep 2007 11:39:12 +0400 Pavel Fedin [EMAIL PROTECTED] wrote: Hello! Can i get this card working? Hotplug doesn't recognise it, however i know that Linux supports it. The kernel supports the NinjaSCSI-3 and UltraNinja-16 (you couldn't make these names up could you) but not

[PATCH 1/8] taskstats: fix indentation of long argument lists

2007-09-26 Thread Guillaume Chazarain
Align with the opening parenthesis. Changelog since V1 (http://lkml.org/lkml/2007/9/21/527): - renamed fill_threadgroup() and add_tsk() to respectively fill_threadgroup_stats() and add_tsk_stats() as suggested by Balbir Singh. - added braces around do/while. - added patch to unbreak binary

[PATCH 2/8] taskstats: split the basic accounting fields

2007-09-26 Thread Guillaume Chazarain
Split the basic accounting taskstats fields into the threadgroup specific ones and the thread specific ones. This should have no effect on the execution. Signed-off-by: Guillaume Chazarain [EMAIL PROTECTED] Cc: Balbir Singh [EMAIL PROTECTED] Cc: Jay Lan [EMAIL PROTECTED] Cc: Jonathan Lim [EMAIL

[PATCH 3/8] taskstats: split the extended accounting fields

2007-09-26 Thread Guillaume Chazarain
Split the extended accounting taskstats fields into the threadgroup specific ones and the thread specific ones. This should have no effect on the execution. Signed-off-by: Guillaume Chazarain [EMAIL PROTECTED] Cc: Balbir Singh [EMAIL PROTECTED] Cc: Jay Lan [EMAIL PROTECTED] Cc: Jonathan Lim

[PATCH 4/8] taskstats: separate PID/TGID stats producers to complete the TGID ones

2007-09-26 Thread Guillaume Chazarain
TASKSTATS_CMD_ATTR_TGID used to return only the delay accounting stats, not the basic and extended accounting. With this patch, TASKSTATS_CMD_ATTR_TGID also aggregates the accounting info for all threads of a thread group. TASKSTATS_CMD_ATTR_PID output should be unchanged TASKSTATS_CMD_ATTR_TGID

[PATCH 5/8] taskstats: factor out version and context switch accounting

2007-09-26 Thread Guillaume Chazarain
All the specific taskstats fields should only be manipulated in {add_tsk,fill_threadgroup}_stats(). Signed-off-by: Guillaume Chazarain [EMAIL PROTECTED] Cc: Balbir Singh [EMAIL PROTECTED] Cc: Jay Lan [EMAIL PROTECTED] Cc: Jonathan Lim [EMAIL PROTECTED] Cc: Oleg Nesterov [EMAIL PROTECTED] ---

[PATCH 6/8] taskstats: tell fill_threadgroup_stats() if it replies with PID or TGID stats

2007-09-26 Thread Guillaume Chazarain
fill_threadgroup_stats() may want to know if it is filling TASKSTATS_CMD_ATTR_TGID or TASKSTATS_CMD_ATTR_PID stats, so give it this information in the tg_stats boolean. Signed-off-by: Guillaume Chazarain [EMAIL PROTECTED] Cc: Balbir Singh [EMAIL PROTECTED] Cc: Jay Lan [EMAIL PROTECTED] Cc:

[PATCH 7/8] taskstats: fix stats-ac_exitcode to work on threads and use group_exit_code

2007-09-26 Thread Guillaume Chazarain
Threads also have an exit code on their own, so report it in TASKSTATS_CMD_ATTR_PID. For TASKSTATS_CMD_ATTR_TGID, instead of relying only on the exit code of the leader, we use task-signal-group_exit_code if not null as suggested by Oleg Nesterov. Also, document that as of this patch,

[PATCH 8/8] taskstats: avoid breaking binary compatibility between taskstats v6 and before

2007-09-26 Thread Guillaume Chazarain
Place fields added in v6 at the end of the struct. Signed-off-by: Guillaume Chazarain [EMAIL PROTECTED] Cc: Michael Neuling [EMAIL PROTECTED] Cc: Balbir Singh [EMAIL PROTECTED] Cc: Jay Lan [EMAIL PROTECTED] Cc: Jonathan Lim [EMAIL PROTECTED] Cc: Oleg Nesterov [EMAIL PROTECTED] ---

Re: commit 6dccd16b7c2703e8bbf8bca62b5cf248332afbe2 kills r8169 send performance

2007-09-26 Thread Francois Romieu
Timo Jantunen [EMAIL PROTECTED] : [...] Git bisect gave commit 6dccd16b7c2703e8bbf8bca62b5cf248332afbe2 as the culprit, and reverting it from the 2.6.23-rc8 increases send speed back to .22 level. (It didn't revert cleanly and the file needed some cleaning up by hand.) You are welcome but

Re: [PATCH 1/8] taskstats: fix indentation of long argument lists

2007-09-26 Thread Guillaume Chazarain
Sorry for the resend, this is the same version as few minutes ago, but stgit 0.13 correctly handles the mail --auto flag (unlike stgit 0.12.1 :-( ) so all recipients are actually Cced. -- Guillaume - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message

Re: sys_chroot+sys_fchdir Fix

2007-09-26 Thread David Newall
Alan Cox wrote: You quoted the standard, I merely pointed out you forgot to read it properly. Thats your problem not mine. How bizarre. Last email you claimed to quote the standards (but you never did.) Your becoming an embarrassment. You were rude, and multiple times. Please just

Re: sys_chroot+sys_fchdir Fix

2007-09-26 Thread Alan Cox
** Plonk ** Welcome to my killfile. - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: KPROBES: Instrumenting a function's call site

2007-09-26 Thread Ananth N Mavinakayanahalli
On Wed, Sep 26, 2007 at 12:09:35PM -0400, Avishay Traeger wrote: On Wed, 2007-09-26 at 14:33 +0530, Ananth N Mavinakayanahalli wrote: What happens when the call is singlestepped is that the instruction pointer is moved to the call target. That explains the lower latency you are seeing.

Re: sys_chroot+sys_fchdir Fix

2007-09-26 Thread David Newall
Alan Cox wrote: ** Plonk ** Welcome to my killfile. Well that's a relief. - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at

Re: KPROBES: Instrumenting a function's call site

2007-09-26 Thread Keshavamurthy, Anil S
On Wed, Sep 26, 2007 at 10:09:33AM +0530, Ananth N Mavinakayanahalli wrote: On Tue, Sep 25, 2007 at 06:12:38PM -0400, Avishay Traeger wrote: Hello, I am trying to use kprobes to measure the latency of a function by instrumenting its call site. Basically, I find the call instruction, and

Re: [PATCH 1/3] taskstats: separate PID/TGID stats producers to complete the TGID ones

2007-09-26 Thread Guillaume Chazarain
Le Sat, 22 Sep 2007 23:36:29 +0530, Balbir Singh [EMAIL PROTECTED] a écrit : [reordered] How about calling this one fill_threadgroup_stats()? How about we call function add_tsk_stats()? I still prefer braces around do -- while, I think the code is easier to read with them. Could we further

Re: NinjaSCSI PCMCIA card

2007-09-26 Thread Jan Engelhardt
On Sep 26 2007 18:13, Alan Cox wrote: Hello! Can i get this card working? Hotplug doesn't recognise it, however i know that Linux supports it. The kernel supports the NinjaSCSI-3 and UltraNinja-16 (you couldn't make these names up could you) but not afaik the card you have a dump of

Re: KPROBES: Instrumenting a function's call site

2007-09-26 Thread Avishay Traeger
On Wed, 2007-09-26 at 22:57 +0530, Ananth N Mavinakayanahalli wrote: On Wed, Sep 26, 2007 at 12:09:35PM -0400, Avishay Traeger wrote: On Wed, 2007-09-26 at 14:33 +0530, Ananth N Mavinakayanahalli wrote: What happens when the call is singlestepped is that the instruction pointer is moved

Re: KPROBES: Instrumenting a function's call site

2007-09-26 Thread Avishay Traeger
On Wed, 2007-09-26 at 10:28 -0700, Keshavamurthy, Anil S wrote: On Wed, Sep 26, 2007 at 10:09:33AM +0530, Ananth N Mavinakayanahalli wrote: On Tue, Sep 25, 2007 at 06:12:38PM -0400, Avishay Traeger wrote: Hello, I am trying to use kprobes to measure the latency of a function by

Re: missing mnt_drop_write() on open error

2007-09-26 Thread Miklos Szeredi
On Wed, 2007-09-26 at 10:38 +0200, Miklos Szeredi wrote: In __dentry_open() there's still a few places where fput() won't be called, notably when -open fails, which is what I'm triggering I think. Also even more horrible things can happen because of the nd-intent.open.file thing.

Re: [patch 1/7] Extended crashkernel command line

2007-09-26 Thread Oleg Verych
Wed, Sep 26, 2007 at 06:16:02PM +0200, Bernhard Walle (part two, see bottom): memparse(), as a wrapper for somple_strtoll(), always have a return value (zero by default). http://article.gmane.org/gmane.linux.kernel/583426 Sorry for my typos, i should write `simple_strtoull()'. This function

Re: Man page for revised timerfd API

2007-09-26 Thread Davide Libenzi
Michael, SCB ... On Wed, 26 Sep 2007, Michael Kerrisk wrote: .TH TIMERFD_CREATE 2 2007-09-26 Linux Linux Programmer's Manual .SH NAME timerfd_create, timerfd_settime, timer_gettime \- timers that notify via file descriptors .SH SYNOPSIS .\ FIXME . This header file may well change .\

Re: missing mnt_drop_write() on open error

2007-09-26 Thread Dave Hansen
On Wed, 2007-09-26 at 19:50 +0200, Miklos Szeredi wrote: Maybe. Can we do the mnt_want_write() from __dentry_open(), instead of may_open()? That would be a lot cleaner. I'll explore that. It may make very good sense. Btw, may_open() doesn't do mnt_want_write() around the truncation if

Re: [PATCH 01/24] CRED: Introduce a COW credentials record

2007-09-26 Thread Al Viro
On Wed, Sep 26, 2007 at 03:21:05PM +0100, David Howells wrote: To alter the credentials record, a copy must be made. This copy may then be altered and then the pointer in the task_struct redirected to it. From that point on the new record should be considered immutable. Umm... Perhaps a

[PATCH] fs/ocfs2/: removed unneeded initial value and function's return value

2007-09-26 Thread Denis Cheng
Signed-off-by: Denis Cheng [EMAIL PROTECTED] --- fs/ocfs2/super.c | 17 - 1 files changed, 8 insertions(+), 9 deletions(-) diff --git a/fs/ocfs2/super.c b/fs/ocfs2/super.c index c034b51..b98ec12 100644 --- a/fs/ocfs2/super.c +++ b/fs/ocfs2/super.c @@ -105,7 +105,7 @@ static int

Linux 2.6.22.9

2007-09-26 Thread Greg Kroah-Hartman
We (the -stable team) are announcing the release of the 2.6.22.9 kernel. It fixes a number of reported bugs, and any user of the 2.6.22 series is encouraged to upgrade. I'll also be replying to this message with a copy of the patch between 2.6.22.8 and 2.6.22.9 The updated 2.6.22.y git tree can

Re: Linux 2.6.22.9

2007-09-26 Thread Greg Kroah-Hartman
diff --git a/Documentation/dvb/get_dvb_firmware b/Documentation/dvb/get_dvb_firmware index 4820366..6cb3080 100644 --- a/Documentation/dvb/get_dvb_firmware +++ b/Documentation/dvb/get_dvb_firmware @@ -56,7 +56,7 @@ syntax(); sub sp8870 { my $sourcefile = tt_Premium_217g.zip; -my $url

Re: [patch 1/7] Extended crashkernel command line

2007-09-26 Thread Bernhard Walle
* Oleg Verych [EMAIL PROTECTED] [2007-09-26 20:18]: I was thinking about errors in YaST or typos in bootloader config, that may appear sometimes. And kernel must tolerate this kind of userspace input to be more reliable. But you know better, i just am waving hands. Of course the kernel must

[PATCH] just rename call_rcu_bh instead of making it a macro

2007-09-26 Thread Steven Rostedt
Seems that I found a box that has a config that passes call_rcu_bh as a function pointer (see net/sctp/sm_make_chunk.c), so declaring the call_rcu_bh has a macro function isn't good enough. This patch makes it just another name of call_rcu for rcupreempt. Signed-off-by: Steven Rostedt [EMAIL

[PATCH 0/3] A kernel tracing interface - (updated)

2007-09-26 Thread David J. Wilder
These patches provide a kernel tracing interface called trace. (update) Moved the sample code to the new samples\ subdir The motivation for trace is to: - Provide a simple set of tracing primitives that will utilize the high- performance and low-overhead of relayfs for passing traces data from

[patch 1/3] Trace code and documentation

2007-09-26 Thread David J. Wilder
Trace - Provides tracing primitives Signed-off-by: Tom Zanussi [EMAIL PROTECTED] Signed-off-by: Martin Hunt [EMAIL PROTECTED] Signed-off-by: David Wilder [EMAIL PROTECTED] --- Documentation/trace.txt | 160 ++ include/linux/trace.h | 99 + lib/Kconfig |9

[patch 2/3] Relay Reset Consumed

2007-09-26 Thread David J. Wilder
This patch allows relay channels to be reset i.e. unconsumed. Basically allows a 'rewind' function for flight-recorder tracing. Signed-off-by: Tom Zanussi [EMAIL PROTECTED] Signed-off-by: David Wilder [EMAIL PROTECTED] --- Documentation/filesystems/relay.txt | 11 ++ include/linux/relay.h

[patch 3/3] Trace sample

2007-09-26 Thread David J. Wilder
Trace example - Adds the trace example to samples/ Signed-off-by: David Wilder [EMAIL PROTECTED] --- samples/Kconfig|6 ++ samples/Makefile |1 + samples/trace/Makefile |4 + samples/trace/fork_trace.c | 132 4

nmi_watchdog fix for x86_64 to be more like i386

2007-09-26 Thread David Bahi
Thanks to tglx and ghaskins for all the help in tracking down a very early nmi_watchdog crash on certain x86_64 machines. This modifies nmi_watchdog_tick behavior for x86_64 arch to consider both timer and hpet IRQs just as the i386 arch does. Signed-off-by: David Bahi [EMAIL PROTECTED] ---

Re: [PATCH 3/3] CRED: Move the effective capabilities into the cred struct

2007-09-26 Thread Al Viro
On Wed, Sep 19, 2007 at 09:11:26PM -0700, Andrew Morgan wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 David Howells wrote: Move the effective capabilities mask from the task struct into the credentials record. Note that the effective capabilities mask in the cred struct

Proposed 2.6 Patch for AMD MIPS Alchemy au1550 I2C interface I2C interface

2007-09-26 Thread Chris David
Hello, Please CC me on replies. I have made a trivial patch to fix a problem on the AMD MIPS Alchemy au1550 I2C interface. The PSC (programmable serial controller) seem to 'hang' when I sent only an 'address' byte on the I2C bus. The patch essentially uses the PSC_SMBSTAT register's TE

Re: [PATCH] Since we have counters in __u64 format we have to print them with %llu macros.

2007-09-26 Thread H. Peter Anvin
Balbir Singh wrote: Andreas Schwab wrote: Maxim Uvarov [EMAIL PROTECTED] writes: diff --git a/Documentation/accounting/getdelays.c b/Documentation/accounting/getdelays.c index cbee3a2..73924df 100644 --- a/Documentation/accounting/getdelays.c +++ b/Documentation/accounting/getdelays.c @@

Re: [PATCH 10/25] Unionfs: add un/likely conditionals on copyup ops

2007-09-26 Thread Adrian Bunk
On Wed, Sep 26, 2007 at 09:40:20AM -0400, Erez Zadok wrote: ... Also, Auke, if indeed compilers are [sic] likely to do better than programmers adding un/likely wrappers, then why do we still support that in the kernel? (Working for a company tat produces high-quality compilers, you may know

Re: [PATCH 2/6] LBS: fix uninitialized swapper_space

2007-09-26 Thread Hugh Dickins
On Mon, 24 Sep 2007, Christoph Lameter wrote: On Fri, 21 Sep 2007, Hugh Dickins wrote: Swapping crashed immediately: must initialize new fields of swapper_space. Thanks for finding that. It may be better though to use the new mapping_setup() function instead? That way there is no #ifdef.

forcedeth question

2007-09-26 Thread roel
in file ./drivers/net/forcedeth.c line 2142 of current git I have a for (i=0;i=np-register_size;i+= 32) { ^ shouldn't this be a '' In the same file on line 4015: for (i = 0;i = np-register_size/sizeof(u32); i++) shouldn't the = be a ''? Roel - To unsubscribe from this list: send the

Re: sys_chroot+sys_fchdir Fix

2007-09-26 Thread Al Viro
On Wed, Sep 26, 2007 at 08:04:14PM +0930, David Newall wrote: Al Viro wrote: Oh, for fsck sake... Folks, it's standard-required behaviour. Ability to chroot() implies the ability to break out of it. Could we please add that (along with reference to SuS) to l-k FAQ and be done with that

Re: [patch 1/3] Trace code and documentation

2007-09-26 Thread Randy Dunlap
On Wed, 26 Sep 2007 11:22:35 -0700 David J. Wilder wrote: diff --git a/Documentation/trace.txt b/Documentation/trace.txt new file mode 100644 index 000..0e42fb8 --- /dev/null +++ b/Documentation/trace.txt @@ -0,0 +1,160 @@ +Trace User Interface +=== +When a trace

Re: [PATCH] Patches for tiny 386 kernels, again. Linux kernel 2.6.22.7

2007-09-26 Thread Jonathan Campbell
Here is the DMI patch again, written against linux-2.6.23-rc8, with some of the #ifdef CONFIG_DMI's removed and moved to include/linux/dmi.h. Putting them there in the way I've done ensures that you don't have to put #ifdef CONFIG_DMI around each dmi_check_machine() and that you don't have to

Re: [patch 3/3] Trace sample

2007-09-26 Thread Randy Dunlap
On Wed, 26 Sep 2007 11:22:43 -0700 David J. Wilder wrote: Trace example - Adds the trace example to samples/ Signed-off-by: David Wilder [EMAIL PROTECTED] Acked-by: Randy Dunlap [EMAIL PROTECTED] --- samples/Kconfig|6 ++ samples/Makefile |1 +

<    2   3   4   5   6   7   8   9   >