Re: irq_set_chained_handler() called too early for hwirq to be initialized

2012-10-28 Thread Thomas Gleixner
On Sun, 28 Oct 2012, Roland Stigge wrote: On 28/10/12 18:34, Thomas Gleixner wrote: On Sun, 28 Oct 2012, Roland Stigge wrote: consider arch/arm/mach-lpc32xx/irq.c: irq_set_chained_handler() is called at a point where it accesses irq_to_desc(IRQ_LPC32XX_SUB2IRQ)-irq_data.hwirq but which

[ANNOUNCE] 3.6.3-rt9

2012-10-28 Thread Thomas Gleixner
Dear RT Folks, I'm pleased to announce the 3.6.3-rt9 release. Changes since 3.6.3-rt8: * hrtimer fix for interrupt stall case (Watanabe) * Disable RCU_FAST_NO_HZ. Uses a timer_list timer in the guts of the irq disable idle code. Needs some thought * Netfilter RT workaround *

[ANNOUNCE] 3.6.4-rt10

2012-10-29 Thread Thomas Gleixner
Dear RT Folks, I'm pleased to announce the 3.6.4-rt10 release. This is just an update to 3.6.4 with no RT related changes The RT patch against 3.6.4 can be found here: http://www.kernel.org/pub/linux/kernel/projects/rt/3.6/patch-3.6.4-rt10.patch.xz The split quilt queue is available at:

[GIT pull] Futex fix for 3.7

2012-11-12 Thread Thomas Gleixner
rules. Thanks, tglx -- Thomas Gleixner (1): futex: Handle futex_pi OWNER_DIED take over correctly kernel/futex.c | 41 ++--- 1 files changed, 22 insertions(+), 19 deletions(-) diff --git a/kernel/futex.c b/kernel/futex.c index

Re: [REGRESSION] 3.7-rc3+git hard lockup on CPU after inserting/removing USB stick

2012-11-12 Thread Thomas Gleixner
On Mon, 12 Nov 2012, Martin Steigerwald wrote: Am Sonntag, 11. November 2012 schrieb Liu, Chuansheng: The first bad commit is: commit 73d4066055e0e2830533041f4b91df8e6e5976ff Author: Chuansheng Liu chuansheng@intel.com Date: Tue Sep 11 16:00:30 2012 +0800

Re: [RFC PATCH v1 02/31] ARC: irqflags

2012-11-12 Thread Thomas Gleixner
On Wed, 7 Nov 2012, Vineet Gupta wrote: + ** + * Inline ASM macros to read/write AUX Regs + * Essentially invocation of lr/sr insns from C + */ + +#if 1 Leftover ??? +#define read_aux_reg(reg)

Re: [RFC PATCH v1 12/31] ARC: Interrupt Handling

2012-11-12 Thread Thomas Gleixner
On Wed, 7 Nov 2012, Vineet Gupta wrote: +void __init init_IRQ(void) +{ + const int irq = TIMER0_IRQ; + + /* + * Each CPU needs to register irq of it's private TIMER0. + * The APIs request_percpu_irq()/enable_percpu_irq() will not be + * functional, if we don't prep

Re: [RFC PATCH v1 15/31] ARC: Process/scheduling/clock/Timers/Delay Management

2012-11-12 Thread Thomas Gleixner
On Wed, 7 Nov 2012, Vineet Gupta wrote: +void cpu_idle(void) +{ + /* Since we SLEEP in idle loop, TIF_POLLING_NRFLAG can't be set */ + + /* endless idle loop with no priority at all */ + while (1) { + tick_nohz_idle_enter(); + + while (!need_resched())

[ANNOUNCE] 3.6.6-rt17

2012-11-12 Thread Thomas Gleixner
Dear RT Folks, I'm pleased to announce the 3.6.6-rt17 release. 3.6.6-rt16 is just a not announced update release to 3.6.6. Changes since 3.6.6-rt16: * Finally make the NOHZ softirq pending detection work with the new softirq scheme. * Remove the WARN_ON from

Re: [STABLE REQUEST] add: e1000: fix lockdep splat in shutdown handler

2012-10-14 Thread Thomas Gleixner
On Thu, 11 Oct 2012, Steven Rostedt wrote: commit 3a3847e007aae732d64d8fd1374126393e9879a3 Author: Jesse Brandeburg jesse.brandeb...@intel.com Date: Wed Jan 4 20:23:33 2012 + e1000: fix lockdep splat in shutdown handler as I discussed with Jesse on IRC, there is another possible

Re: [RFC PATCH] posix timers: allocate timer id per task

2012-10-15 Thread Thomas Gleixner
On Mon, 15 Oct 2012, Stanislav Kinsbursky wrote: This patch is required CRIU project (www.criu.org). To migrate processes with posix timers we have to make sure, that we can restore posix timer with proper id. Currently, this is not true, because timer ids are allocated globally. So, this is

[ANNOUNCE] 3.6.1-rt2

2012-10-16 Thread Thomas Gleixner
Dear RT Folks, I'm pleased to announce the 3.6.1-rt2 release. Changes since 3.6.1-rt1: * Picked up Pauls git friendly quilt queue * Compile fix for !RT_FULL (Paul Gortemaker) * Crypto init order fix * Tiny RCU fix which affects UP and is a long standing bug

Re: [ANNOUNCE] 3.6.1-rt2

2012-10-16 Thread Thomas Gleixner
On Tue, 16 Oct 2012, Javier Sanz wrote: Hello, Testing, and FYI $uname -a Linux darkstar 3.6.1-rt2 #1 SMP PREEMPT RT Tue Oct 16 22:47:06 CEST 2012 i686 i686 i386 GNU/Linux shows all time ... [ 30.543233] fuse init (API version 7.20) [ 33.262077] Crap, ksoftirqd/0 looping

Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound when booted, USB-related WARNING

2007-09-20 Thread Thomas Gleixner
Rafael, On Thu, 2007-09-20 at 22:39 +0200, Rafael J. Wysocki wrote: Works as well. What's the difference between this and the real thing ? The real thing also calls device_power_down(PMSG_FREEZE), which is a counterpart of sysdev_shutdown(), more or less, and I think that's what goes belly

Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound when booted, USB-related WARNING

2007-09-20 Thread Thomas Gleixner
Rafael, On Thu, 2007-09-20 at 23:45 +0200, Rafael J. Wysocki wrote: We disable everything in device_suspend() No, we don't. sysdevs are _not_ suspended in device_suspend(). They are suspended in device_power_down(), which is called _after_ disable_nonboot_cpus() (from swsusp_suspend()).

Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound when booted, USB-related WARNING

2007-09-20 Thread Thomas Gleixner
Rafael, On Thu, 2007-09-20 at 23:54 +0200, Rafael J. Wysocki wrote: Hmm. This is close to the ordering we have in STR too. I have some dim memory of there being some ACPI reason why it had to be done that way. Yes. We're executing _INI from the CPU initialization code and that

Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound when booted, USB-related WARNING

2007-09-20 Thread Thomas Gleixner
Linus, On Thu, 2007-09-20 at 14:55 -0700, Linus Torvalds wrote: And I think that's a damn reasonable thing to agree on: timers (and anything else that CPU shutdown/bringup could *possibly* care about) should be considered core enough that they had better be on the suspend_late/resume_early

Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound when booted, USB-related WARNING

2007-09-21 Thread Thomas Gleixner
On Thu, 2007-09-20 at 19:35 -0400, Len Brown wrote: (Btw, the above commit message points to just my response with a testing patch to the real email: the actual explanation of the INSANE ordering is from Len Brown in

Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound when booted, USB-related WARNING

2007-09-21 Thread Thomas Gleixner
On Fri, 2007-09-21 at 14:51 +1000, Paul Mackerras wrote: Linus Torvalds writes: It would indeed be nice if we could just take CPU's down early (while everything is working), and run the whole suspend code with just one CPU, rather than having to worry about the ordering between CPU and

Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound when booted, USB-related WARNING

2007-09-21 Thread Thomas Gleixner
Rafael, On Fri, 2007-09-21 at 00:30 +0200, Rafael J. Wysocki wrote: -ETOOTIRED led me too a wrong conclusion, but still it is a valuable hint that this change is making things work again. Yes, it is. I need to go down into the details of the swsusp_suspend() code path to figure out,

Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound when booted, USB-related WARNING

2007-09-21 Thread Thomas Gleixner
Rafael, On Fri, 2007-09-21 at 16:20 +0200, Rafael J. Wysocki wrote: If you need any help from me with that, please let me know. I'm zooming in. It seems, that the ACPI idle code plays tricks with us. After debugging the swsusp_suspend() code path I figured out, that we end up in C2 or

Re: 2.6.23-rc6-mm1: failure to boot on HP nx6325, no sound when booted, USB-related WARNING

2007-09-21 Thread Thomas Gleixner
Rafael, On Fri, 2007-09-21 at 21:20 +0200, Rafael J. Wysocki wrote: On Friday, 21 September 2007 18:27, Thomas Gleixner wrote: I simply rmmod'ed the processor module before suspend and the problem is solved as well. The cpuidle patches make this problem more prominent due to the possible

Re: clockevents: fix resume logic

2007-09-22 Thread Thomas Gleixner
On Mon, 2007-09-17 at 18:37 +, Pavel Machek wrote: That's a bit tricky because hitting the keyboard is what unsticks things. And the video is black after resume-from-RAM (has always been thus) and we Ok, can we try to fix the video issue for you? That should make the development

Re: RFC: A revised timerfd API

2007-09-22 Thread Thomas Gleixner
On Sat, 2007-09-22 at 18:07 +0200, Michael Kerrisk wrote: Hello Bernd, Please don't trim the CC list when replying! I nearly did not see your reply, and others will have missed it also. Yup. On 9/22/07, Bernd Eckenfels [EMAIL PROTECTED] wrote: In article [EMAIL PROTECTED] you wrote:

Re: RFC: A revised timerfd API

2007-09-22 Thread Thomas Gleixner
or two earlier comments): Let's dismiss option (a), since it is an unlovely multiplexing interface. Option (b) seems a viable. The most notable concern was from Thomas Gleixner, that we might end up duplicating code from the POSIX timers API within the timerfd API -- some eventual refactoring

[PATCH] usb-gadget-ether: Prevent oops caused by error interrupt race -V2 (comments update)

2007-09-22 Thread Thomas Gleixner
-off-by: Thomas Gleixner [EMAIL PROTECTED] diff --git a/drivers/usb/gadget/ether.c b/drivers/usb/gadget/ether.c index 593e235..f2a7bd5 100644 --- a/drivers/usb/gadget/ether.c +++ b/drivers/usb/gadget/ether.c @@ -1989,8 +1989,21 @@ static int eth_start_xmit (struct sk_buff *skb, struct net_device

Re: [PATCH] [4/50] x86: add cpu codenames for Kconfig.cpu

2007-09-22 Thread Thomas Gleixner
On Sat, 2007-09-22 at 00:32 +0200, Andi Kleen wrote: From: Oliver Pinter [EMAIL PROTECTED] add cpu core name for arch/i386/Kconfig.cpu:Pentium 4 sections help add Pentium D for arch/i386/Kconfig.cpu add Pentium D for arch/x86_64/Kconfig Signed-off-by: Oliver Pinter [EMAIL PROTECTED]

Re: [PATCH] [9/50] i386: validate against ACPI motherboard resources

2007-09-22 Thread Thomas Gleixner
On Sat, 2007-09-22 at 10:28 -0600, Robert Hancock wrote: Yinghai Lu wrote: No! MMCONFIG will not work with acpi=off any more. I don't think this is unreasonable. The ACPI MCFG table is how we are supposed to learn about the area in the first place. If we can't get the table location

Re: [PATCH] [19/50] Experimental: detect if SVM is disabled by BIOS

2007-09-22 Thread Thomas Gleixner
On Sat, 2007-09-22 at 00:32 +0200, Andi Kleen wrote: Also allow to set svm lock. Please use two separate patches. The detection and cpuinfo display is not related to set svm lock. TBD double check, documentation, i386 support Yes, documentation would be useful. See below. Signed-off-by:

Re: [PATCH] [20/50] x86_64: Fix some broken white space in arch/x86_64/mm/init.c

2007-09-22 Thread Thomas Gleixner
On Sat, 2007-09-22 at 00:32 +0200, Andi Kleen wrote: No functional changes Signed-off-by: Andi Kleen [EMAIL PROTECTED] Can we please fix _ALL_ white space and coding style issues in this file while we are at it? Updated patch below. tglx diff --git a/arch/x86_64/mm/init.c

Re: [PATCH] [31/50] x86_64: honor notify_die() returning NOTIFY_STOP

2007-09-22 Thread Thomas Gleixner
On Sat, 2007-09-22 at 00:32 +0200, Andi Kleen wrote: - notify_die(DIE_OOPS, str, regs, err, current-thread.trap_no, SIGSEGV); + if (notify_die(DIE_OOPS, str, regs, err, current-thread.trap_no, SIGSEGV) == NOTIFY_STOP) 80 chars please. tglx - To unsubscribe from this list:

Re: [PATCH] usb-gadget-ether: Prevent oops caused by error interrupt race -V2 (comments update)

2007-09-22 Thread Thomas Gleixner
On Sat, 2007-09-22 at 12:18 -0700, David Brownell wrote: I think you misread my comment. Those requests are **NOT** pending!! So this update has a *MORE* incorrect description of the issue. That's just the freelist ... it's a fairly conventional model whereby there's a pool of free

Re: [PATCH] [35/50] i386: Do cpuid_device_create() in CPU_UP_PREPARE instead of CPU_ONLINE.

2007-09-22 Thread Thomas Gleixner
On Sat, 2007-09-22 at 00:32 +0200, Andi Kleen wrote: From: Akinobu Mita [EMAIL PROTECTED] Do cpuid_device_create() in CPU_UP_PREPARE instead of CPU_ONLINE. Cc: H. Peter Anvin [EMAIL PROTECTED] Signed-off-by: Akinobu Mita [EMAIL PROTECTED] Signed-off-by: Andi Kleen [EMAIL PROTECTED] Cc:

Re: [PATCH] usb-gadget-ether: Prevent oops caused by error interrupt race -V2 (comments update)

2007-09-22 Thread Thomas Gleixner
On Sat, 2007-09-22 at 13:14 -0700, David Brownell wrote: How's this? Note that the queue should already have been stopped, so I removed what should be an extra call (as well as fixing the comments). Yeah, stop queue should be not necessary. - Dave From: Thomas Gleixner [EMAIL

Re: RFC: A revised timerfd API

2007-09-22 Thread Thomas Gleixner
On Sat, 2007-09-22 at 14:07 -0700, Davide Libenzi wrote: On Sat, 22 Sep 2007, Michael Kerrisk wrote: So I'm inclined to implement option (b), unless someone has strong objections. Davide, could I persuade you to help? I guess I better do, otherwise you'll continue to stress me ;) int

[patch 0/2] suspend/resume regression fixes

2007-09-22 Thread Thomas Gleixner
Sorry, it took me quite a while to realize the real root cause of the VAIO - and probably many other machines - suspend/resume regressions, which were unearthed by the dyntick / clockevents patches. We disable a lot of ACPI/BIOS functionality during suspend, but we keep the lower idle C-states

[patch 1/2] ACPI: disable lower idle C-states across suspend/resume

2007-09-22 Thread Thomas Gleixner
. Fall back to the default idle implementation (halt) instead. Signed-off-by: Thomas Gleixner [EMAIL PROTECTED] Tested-by: Andrew Morton [EMAIL PROTECTED] Cc: Len Brown [EMAIL PROTECTED] Cc: Venkatesh Pallipadi [EMAIL PROTECTED] Cc: Rafael J. Wysocki [EMAIL PROTECTED] --- drivers/acpi

[patch 2/2] clockevents: remove the suspend/resume workaround^Wthinko

2007-09-22 Thread Thomas Gleixner
C-State. Signed-off-by: Thomas Gleixner [EMAIL PROTECTED] Tested-by: Andrew Morton [EMAIL PROTECTED] Cc: Len Brown [EMAIL PROTECTED] Cc: Venkatesh Pallipadi [EMAIL PROTECTED] Cc: Rafael J. Wysocki [EMAIL PROTECTED] --- kernel/time/tick-broadcast.c | 17 + 1 file changed, 1

Re: [patch 0/2] suspend/resume regression fixes

2007-09-22 Thread Thomas Gleixner
Linus, On Sat, 2007-09-22 at 15:59 -0700, Linus Torvalds wrote: My final enlightment was, when I removed the ACPI processor module, which controls the lower idle C-states, right before resume; this worked fine all the time even without all the workaround hacks. I really hope that this

Re: [PATCH] usb-gadget-ether: Prevent oops caused by error interrupt race -V2 (comments update)

2007-09-22 Thread Thomas Gleixner
On Sat, 2007-09-22 at 13:53 -0700, David Brownell wrote: Sigh. I need a real deep look inside that code to understand, why tx_reqs is not a requestlist but a freelist. Very intuitive naming :) It *is* a list of requests: free ones -- the only kind this level of driver is allowed to

Re: [PATCH] [35/50] i386: Do cpuid_device_create() in CPU_UP_PREPARE instead of CPU_ONLINE.

2007-09-23 Thread Thomas Gleixner
On Sun, 2007-09-23 at 10:52 +0900, Akinobu Mita wrote: arch/i386/kernel/cpuid.c | 32 +++- 1 file changed, 19 insertions(+), 13 deletions(-) Index: linux/arch/i386/kernel/cpuid.c ===

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

2007-09-23 Thread Thomas Gleixner
On Sun, 2007-09-23 at 12:57 +0200, Rafael J. Wysocki wrote: Hi Thomas, Unfortunately, my observation that the patch series: http://tglx.de/projects/hrtimers/2.6.23-rc4/patch-2.6.23-rc4-hrt1.patches.tar.bz2 worked with 2.6.23-rc4 was wrong. It _sometimes_ works, but usually doesn't

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

2007-09-23 Thread Thomas Gleixner
On Sun, 2007-09-23 at 22:08 +0200, Rafael J. Wysocki wrote: Since the boot fails very early, before any messages reach the (VGA) console, I have no idea what to do next, except for digging in the code. Ok, lets track it down. Is there any difference when you add: nohz=off

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

2007-09-24 Thread Thomas Gleixner
On Sun, 2007-09-23 at 22:52 +0200, Rafael J. Wysocki wrote: Second, noacpitimer added to the command line makes all of the kernels, up to and including 2.6.23-rc6-mm1, boot (this seems to be 100% reproducible). That's valuable information. Can you please provide a boot log of one of

Re: [patch 1/3] new timerfd API - new timerfd API

2007-09-24 Thread Thomas Gleixner
Davide, On Sun, 2007-09-23 at 15:49 -0700, Davide Libenzi wrote: This is the new timerfd API as it is implemented by the following patch: --- fs/compat.c | 32 ++- fs/timerfd.c | 199 ++- include/linux/compat.h

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

2007-09-24 Thread Thomas Gleixner
On Mon, 2007-09-24 at 14:57 +0200, Rafael J. Wysocki wrote: http://tglx.de/projects/hrtimers/2.6.23-rc4/patch-2.6.23-rc4-hrt1.patches.tar.bz2 applied. I also have the 2.6.23-rc6-mm1 dmesg output ready, but there's some -mm-specific noise in it. Please let me know if you want it,

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

2007-09-24 Thread Thomas Gleixner
On Mon, 2007-09-24 at 15:52 +0200, Rafael J. Wysocki wrote: So I really wonder, why noacpitimer on the kernel command line makes any difference. I'm confused. \metoo Well, it was probably read as noacpi. :-) Hmm, ACPI is in the log all over the place. Well, noacpi

Re: [patch 1/3] new timerfd API - new timerfd API

2007-09-24 Thread Thomas Gleixner
On Mon, 2007-09-24 at 08:42 -0700, Davide Libenzi wrote: + ticks += (u64) hrtimer_forward(ctx-tmr, hrtimer_cb_get_time(ctx-tmr), You need to use ctx-tmr.base-get_time() here, otherwise you might read

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

2007-09-24 Thread Thomas Gleixner
On Mon, 2007-09-24 at 17:18 +0200, Rafael J. Wysocki wrote: Well, noacpi seems to be a synonym for pci=noacpi. Anyway, it causes acpi_disable_pci() to be executed, which according to Documentation/kernel-parameters.txt means Do not use ACPI for IRQ routing or for PCI scanning

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

2007-09-24 Thread Thomas Gleixner
On Mon, 2007-09-24 at 21:11 +0200, Rafael J. Wysocki wrote: /me scratches head Retested. We know, that - disabling local apic timers work This works reproducibly accross the board. Ok - local apic timers (which turn off PIT) work. when noacpiFSCKEDPARSING This stopped working,

Re: 2.6.23-rc7-mm1

2007-09-24 Thread Thomas Gleixner
On Mon, 2007-09-24 at 12:34 -0700, Andrew Morton wrote: It prints twice 'System halted' and blinks the keyboard leds, but does not switch off. On all other kernel version I only see one keyboard blink before the power goes out. ok... I compared its dmesg to vanilla-rc7 and -rc4-mm1,

Re: [patch 1/4] new timerfd API v2 - introduce a new hrtimer_forward_now() function

2007-09-24 Thread Thomas Gleixner
] Reviewed-and-Acked-by: Thomas Gleixner [EMAIL PROTECTED] - Davide --- include/linux/hrtimer.h |7 +++ 1 file changed, 7 insertions(+) Index: linux-2.6.mod/include/linux/hrtimer.h === --- linux-2.6.mod.orig

Re: x86-64 sporadic hang in 2.6.23rc7 and 2.6.22

2007-09-24 Thread Thomas Gleixner
On Mon, 2007-09-24 at 23:08 +0200, Helge Hafting wrote: The two kernels mentioned hangs occationally. Typically when I compile something and pass the time by surfing the web. A few minutes and then I notice that the mouse (and everything else in X) stops. kbd LEDs does not react to

Re: 2.6.23-rc7-mm1

2007-09-25 Thread Thomas Gleixner
On Tue, 2007-09-25 at 09:32 +0200, Torsten Kaiser wrote: On 9/24/07, Thomas Gleixner [EMAIL PROTECTED] wrote: Can your check whether 2.6.23-rc7 + http://tglx.de/projects/hrtimers/2.6.23-rc7/patch-2.6.23-rc7-hrt1.patch works for you ? Yes, powers off normally. Ok, so it's probably

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

2007-09-25 Thread Thomas Gleixner
On Tue, 2007-09-25 at 10:14 +0400, Mikhail Kshevetskiy wrote: Hello Thomas, Rafael We know, that - disabling local apic timers work As i can see from the log, you are booting on computer with dualcore AMD processor. Do you have C1E feature enabled? i386 kernel disable lapic on

Re: Why do so many machines need noapic?

2007-09-25 Thread Thomas Gleixner
Chuck, On Thu, 2007-09-13 at 12:38 -0400, Chuck Ebbert wrote: On 09/10/2007 03:44 PM, Andi Kleen wrote: Yes, it has an hpet. And I tried every combination of options I could think of. But, even stranger, x86_64 works (only i386 fails.) x86-64 has quite different time code (at least

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

2007-09-25 Thread Thomas Gleixner
Rafael, On Tue, 2007-09-25 at 10:07 +0200, Thomas Gleixner wrote: On Tue, 2007-09-25 at 10:14 +0400, Mikhail Kshevetskiy wrote: Hello Thomas, Rafael We know, that - disabling local apic timers work As i can see from the log, you are booting on computer with dualcore AMD

Re: 2.6.23-rc8-mm1, -rc7-mm1 kill audio on HP nx6325

2007-09-25 Thread Thomas Gleixner
On Tue, 2007-09-25 at 14:08 +0200, Rafael J. Wysocki wrote: Hi, This patch from Andi: x86_64-mm-cpa-einval.patch makes the hda_intel audio driver stop working on my HP nx6325. The following line appears in dmesg (from 2.6.23-rc7-mm1: ALSA

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

2007-09-25 Thread Thomas Gleixner
On Tue, 2007-09-25 at 14:20 +0200, Rafael J. Wysocki wrote: As i can see from the log, you are booting on computer with dualcore AMD processor. Do you have C1E feature enabled? That's possible, how to check? i386 kernel disable lapic on dualcore AMD with C1E support (see

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

2007-09-25 Thread Thomas Gleixner
On Tue, 2007-09-25 at 15:16 +0200, Rafael J. Wysocki wrote: There seems to be a history effect in the box, to make things more interesting. Did you connect this box to Andrews VAIO during KS ? No, but it's famous for being interestingly broken nevertheless. :) I think the only

Re: 2.6.23-rc8-mm1, -rc7-mm1 kill audio on HP nx6325

2007-09-25 Thread Thomas Gleixner
On Tue, 2007-09-25 at 15:20 +0200, Rafael J. Wysocki wrote: The patch is correct. Instead of returning Success in the case of a failure of lookup_address, it now returns -EINVAL, which in turn makes the ioremap fail. OTOH, the driver ioremap call looks straight forward. Can you apply

Re: 2.6.23-rc8-mm1, -rc7-mm1 kill audio on HP nx6325

2007-09-25 Thread Thomas Gleixner
On Tue, 2007-09-25 at 16:29 +0200, Rafael J. Wysocki wrote: lookup failed for 18446604438082158592 [--snipped some USB messages--] ALSA /home/rafael/src/mm/linux-2.6.23-rc8-mm1/sound/pci/hda/hda_intel.c:1756: hda-intel: ioremap error: 2349334528 16384 Stupid me, hex

[PATCH] x86-64: Disable local APIC timer use on AMD systems with C1E

2007-09-25 Thread Thomas Gleixner
someone pointed us to this. I really had forgotten that we debugged this half a year ago already. Sigh, is it just me or is there something yelling arch/x86 into my ear? Signed-off-by: Thomas Gleixner [EMAIL PROTECTED] diff --git a/arch/x86_64/kernel/setup.c b/arch/x86_64/kernel/setup.c index af838f6

Re: [PATCH] UML - time build fix

2007-09-25 Thread Thomas Gleixner
On Tue, 2007-09-25 at 13:37 -0400, Jeff Dike wrote: Put back an implementation of timeval_to_ns in arch/um/os-Linux/time.c. tglx pointed out in his review of tickless support that there was a perfectly good implementation of it in linux/time.h. The problem is that this is userspace code

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

2007-09-25 Thread Thomas Gleixner
Rafael, On Tue, 2007-09-25 at 22:07 +0200, Rafael J. Wysocki wrote: On Tuesday, 25 September 2007 15:17, Thomas Gleixner wrote: On Tue, 2007-09-25 at 15:16 +0200, Rafael J. Wysocki wrote: [--snip--] I start to get desperate. Below is a patch, which moves the apic timer disable check

Re: [PATCH] x86-64: Disable local APIC timer use on AMD systems with C1E

2007-09-25 Thread Thomas Gleixner
On Tue, 2007-09-25 at 22:55 +0200, Rafael J. Wysocki wrote: I have reworked the patch a bit so that it applies on top of 2.6.23-rc8-mm1 and compiles (my version is attached). With this patch applied, the kernel boots correctly on the nx6325. I know. It's basically enforced noapictimer. But

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

2007-09-25 Thread Thomas Gleixner
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 line. apicmaintimer stops the PIT like we do in -hrt and we just use the

Re: 2.6.23-rc8-mm1: somewhat broken forced HPET on ICH5

2007-09-25 Thread Thomas Gleixner
Alexey, On Wed, 2007-09-26 at 00:50 +0400, Alexey Dobriyan wrote: ich-force-hpet-ich5-quirk-to-force-detect-enable.patch is causing the following on Etch boot: [initscripts as usual] Setting system clock: [nothing happens for several seconds] select to /dev/rtc to

Re: [PATCH] UML - time build fix

2007-09-25 Thread Thomas Gleixner
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 arch/um/os-Linux/time.c. tglx pointed out in his review

Re: 2.6.23-rc8-mm1: somewhat broken forced HPET on ICH5

2007-09-26 Thread Thomas Gleixner
On Wed, 2007-09-26 at 13:14 +0400, Alexey Dobriyan wrote: On Tue, Sep 25, 2007 at 11:45:17PM +0200, Thomas Gleixner wrote: On Wed, 2007-09-26 at 00:50 +0400, Alexey Dobriyan wrote: ich-force-hpet-ich5-quirk-to-force-detect-enable.patch is causing the following on Etch boot

Re: [discuss] 2.6.23-rc8-mm1, -rc7-mm1 kill audio on HP nx6325

2007-09-26 Thread Thomas Gleixner
On Wed, 2007-09-26 at 08:32 +0100, Jan Beulich wrote: ioremap_nocache() does __ioremap(..., _PAGE_PCD);, then __ioremap() does ioremap_page_range(..., _PAGE_PCD | other_stuff) That's one. __ioremap() then does ioremap_change_attr(..., _PAGE_PCD);. That's two. So I _think_ we're setting

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

2007-09-26 Thread Thomas Gleixner
On Wed, 2007-09-26 at 17:25 +0200, Rafael J. Wysocki wrote: There still are some oddities. First, with the x86-64: Disable local APIC timer use on AMD systems with C1E patch and my collection of suspend patches applied, the box doesn't boot (the suspend patches don't even thouch the boot

Re: [REGRESSION from 2.6.23-rc8] (was: Re: 2.6.23-rc4-mm1 and -rc6-mm1: boot failure on HP nx6325, related to clockevents)

2007-09-26 Thread Thomas Gleixner
Rafael, On Wed, 2007-09-26 at 23:00 +0200, Rafael J. Wysocki wrote: First, with the x86-64: Disable local APIC timer use on AMD systems with C1E patch and my collection of suspend patches applied, the box doesn't boot (the suspend patches don't even thouch the boot code, so they

Re: [REGRESSION from 2.6.23-rc8] (was: Re: 2.6.23-rc4-mm1 and -rc6-mm1: boot failure on HP nx6325, related to clockevents)

2007-09-26 Thread Thomas Gleixner
On Wed, 2007-09-26 at 15:22 -0700, Linus Torvalds wrote: On Wed, 26 Sep 2007, Thomas Gleixner wrote: 1) current Linus' tree doesn't boot with any command line (regression) [ Linus, please revert commit e66485d747505e9d960b864fc6c37f8b2afafaf0 Reverted. OK, this explains 2

Re: [REGRESSION from 2.6.23-rc8] (was: Re: 2.6.23-rc4-mm1 and -rc6-mm1: boot failure on HP nx6325, related to clockevents)

2007-09-26 Thread Thomas Gleixner
On Thu, 2007-09-27 at 01:30 +0200, Rafael J. Wysocki wrote: Tested for a couple of times with each kernel, the results seem to be reproducible 100% of the time. Thanks for going through this debug marathon. No big deal. I'm glad that you've found what's up. Well, we still have

Re: [PATCH] Compile handle_percpu_irq even for uniprocessor kernels

2007-09-27 Thread Thomas Gleixner
as well and an alternative UP construct is usually needed - for no good reason. Signed-off-by: Ralf Baechle [EMAIL PROTECTED] Makes sense. Acked-by: Thomas Gleixner [EMAIL PROTECTED] --- This fixes uniprocessor configurations for some MIPS SMP systems. diff --git a/kernel/irq/chip.c b

Re: 2.6.23-rc8-mm2: problems on HP nx6325

2007-09-27 Thread Thomas Gleixner
On Thu, 2007-09-27 at 17:59 +0200, Rafael J. Wysocki wrote: 2) CPU hotplug is busted (onlining of CPU1 kills the kernel), probably due to the same issue that I'm having with the -hrt version of 2.6.23-rc8 (we're debugging it right now) This one is fixed by the following patch:

Re: NO_HZ hangs up AMD MK-36

2007-09-27 Thread Thomas Gleixner
On Thu, 2007-09-27 at 23:28 +0300, Dmitry Tyschenko wrote: I have laptop Asus X50M. Using old Debian Etch from February. Kernel from 2.6.21 doesn't boot, hangs up just in 10seconds - 1minute after GRUB screen. I have tryed different versions of gcc (4.1.1, 4.1.2, 4.2.1) to build 2.6.22.8

Re: NO_HZ hangs up AMD MK-36

2007-09-27 Thread Thomas Gleixner
On Fri, 2007-09-28 at 00:01 +0300, Dmitry Tyschenko wrote: Sorry, I am newbie in linux. Hope you was talking about: /boot/vmlinuz-2.6.22-1-k7 root=/dev/sda5 ro nohz=off Yes. But it doesn't help for Debians 2.6.22-1 (I don't have another prebuiled) still same problems. Can you please add:

[PATCH] clockevents: fix bogus next_event reset for oneshot broadcast devices

2007-09-27 Thread Thomas Gleixner
existing users. Signed-off-by: Thomas Gleixner [EMAIL PROTECTED] Acked-and-tested-by: Ralf Baechle [EMAIL PROTECTED] --- diff --git a/kernel/time/tick-broadcast.c b/kernel/time/tick-broadcast.c index 0962e05..acf15b4 100644 --- a/kernel/time/tick-broadcast.c +++ b/kernel/time/tick-broadcast.c

Re: 2.6.23-rc6-mm1 powerpc - kgdb is broken

2007-09-28 Thread Thomas Gleixner
On Fri, 2007-09-28 at 16:07 +0530, Kamalesh Babulal wrote: The kgdb is also broken with 2.6.23-rc8-mm2 on the powerpc . The below patch disables the kgdb from getting compiled over powerpc platform. Signed-off-by : Kamalesh Babulal [EMAIL PROTECTED] --- ---

Re: [PATCH] spin_lock_unlocked cleanups

2007-09-28 Thread Thomas Gleixner
On Fri, 2007-09-28 at 09:56 +0100, Andy Whitcroft wrote: I think we're ready to wire checkpatch up to a email robot which monitors the mailing lists and sends people nastygrams. I bet that'll be popular ;) We should wire it up to git-commit as well. A lot of that comes in via git

Re: [PATCH] disable non-boot CPUs before poweroff

2007-09-28 Thread Thomas Gleixner
-by: Mark Lord [EMAIL PROTECTED] Fixes my new toybox as well. Thanks for tracking it down before I had to dig in. Acked-by: Thomas Gleixner [EMAIL PROTECTED] --- --- linux/kernel/sys.c.orig 2007-09-13 09:49:11.0 -0400 +++ linux/kernel/sys.c2007-09-28 09:48:54.0 -0400

Re: [PATCH] disable non-boot CPUs before poweroff

2007-09-28 Thread Thomas Gleixner
On Fri, 2007-09-28 at 17:05 +0200, Rafael J. Wysocki wrote: if (pm_power_off_prepare) pm_power_off_prepare(); sysdev_shutdown(); + disable_nonboot_cpus(); Before sysdev_shutdown(), please. sysdev_shutdown() may touch things that belong to CPU0. Damn, you're

Re: [REGRESSION from 2.6.23-rc8]

2007-09-28 Thread Thomas Gleixner
On Fri, 2007-09-28 at 11:07 -0400, Chuck Ebbert wrote: On 09/26/2007 06:35 PM, Thomas Gleixner wrote: It's even worse than I thought on the first check: noapictimer on the command line of an SMP box prevents _ONLY_ the boot CPU apic timer from being used. But the secondary CPU is still

Re: [PATCH] spin_lock_unlocked cleanups

2007-09-28 Thread Thomas Gleixner
On Fri, 2007-09-28 at 01:26 -0700, Andrew Morton wrote: On Fri, 28 Sep 2007 10:17:30 +0200 Thomas Gleixner [EMAIL PROTECTED] wrote: can we please add this to checkpatch.pl ? -spinlock_t bpci_lock = SPIN_LOCK_UNLOCKED; +DEFINE_SPINLOCK(bpci_lock); That check is already

Re: [PATCH] spin_lock_unlocked cleanups

2007-09-28 Thread Thomas Gleixner
On Thu, 2007-09-27 at 23:36 +0200, roel wrote: Replace some SPIN_LOCK_UNLOCKED with DEFINE_SPINLOCK Signed-off-by: Roel Kluin [EMAIL PROTECTED] Acked-by: Thomas Gleixner [EMAIL PROTECTED] Andy, Randy, can we please add this to checkpatch.pl ? -spinlock_t bpci_lock = SPIN_LOCK_UNLOCKED

Re: [patch 0/2] suspend/resume regression fixes

2007-09-28 Thread Thomas Gleixner
On Fri, 2007-09-28 at 16:27 -0400, Mark Lord wrote: Linus Torvalds wrote: On Sat, 22 Sep 2007, Thomas Gleixner wrote: My final enlightment was, when I removed the ACPI processor module, which controls the lower idle C-states, right before resume; this worked fine all the time even

Re: [REGRESSION from 2.6.23-rc8] (was: Re: 2.6.23-rc4-mm1 and -rc6-mm1: boot failure on HP nx6325, related to clockevents)

2007-09-30 Thread Thomas Gleixner
On Sun, 30 Sep 2007, Andi Kleen wrote: OK, this explains 2) and 3). I just looked into the code and the logic vs. noapictimer on SMP is completely broken. noapictimer really doesn't make any sense on non SMP imho with the old timer architecture. That is why I never bothered to implement it.

Re: [PATCH] robust futex thread exit race

2007-09-30 Thread Thomas Gleixner
the bug for me. good catch! A quick preliminary review of your patch indicates it's fine - and it might be v2.6.23 material. Acked-by: Ingo Molnar [EMAIL PROTECTED] Acked-by: Thomas Gleixner [EMAIL PROTECTED] Calling handle_futex_death in exit_robust_list for the different robust

Re: x86-64 sporadic hang in 2.6.23rc7 and 2.6.22

2007-09-30 Thread Thomas Gleixner
On Sat, 29 Sep 2007, Helge Hafting wrote: Thomas Gleixner wrote: I have gone back to 2.6.22rc4, which seems to work. This is a single opteron, although on a dual-slot board. Can you switch to serial console, so we can get some information out of that box? Sysrq-B

Re: Fwd: x86_64 and AMD with C1E

2007-10-01 Thread Thomas Gleixner
On Mon, 1 Oct 2007, Mikhail Kshevetskiy wrote: No, it boot and work normally. The only thing i bother, is the additional 260 timer interrupts per seconds. Here is short result: c1e enabled: -- power consumption about 23 watts -- there is only C1 power state enabled -- there are

Re: Fwd: x86_64 and AMD with C1E

2007-10-01 Thread Thomas Gleixner
On Mon, 1 Oct 2007, Andi Kleen wrote: There is work in progress on a patch, which allows to utilize the hpet timers as per cpu timers. This should solve the problem. Be patient. Given that e.g. ICH8 only has 3 HPET timers that seems doubtful except for the special case of single-socket non

Re: Fwd: x86_64 and AMD with C1E

2007-10-01 Thread Thomas Gleixner
On Mon, 1 Oct 2007, Andi Kleen wrote: So if the number of hpet channels is greater/equal to the number of possible CPUs it's perfectly fine and does not need IPI at all. That is only a stop gap then. I don't see this being generally true in the future. e.g. Intel announced SMT will be

Re: nmi_watchdog fix for x86_64 to be more like i386

2007-10-01 Thread Thomas Gleixner
On Mon, 1 Oct 2007, Andi Kleen wrote: On Wednesday 26 September 2007 20:03:12 David Bahi wrote: Thanks to tglx and ghaskins for all the help in tracking down a very early nmi_watchdog crash on certain x86_64 machines. The patch is totally bogus. irq 0 doesn't say anything about whether

Re: nmi_watchdog fix for x86_64 to be more like i386

2007-10-01 Thread Thomas Gleixner
On Mon, 1 Oct 2007, Andi Kleen wrote: On Monday 01 October 2007 20:54:21 Thomas Gleixner wrote: On Mon, 1 Oct 2007, Andi Kleen wrote: On Wednesday 26 September 2007 20:03:12 David Bahi wrote: Thanks to tglx and ghaskins for all the help in tracking down a very early nmi_watchdog

Re: nmi_watchdog fix for x86_64 to be more like i386

2007-10-01 Thread Thomas Gleixner
and invocation in the common code and allows x86 to handle the nmi watchdog accounting correctly. Signed-off-by: Thomas Gleixner [EMAIL PROTECTED] diff --git a/arch/i386/kernel/apic.c b/arch/i386/kernel/apic.c index 3d67ae1..180dde8 100644 --- a/arch/i386/kernel/apic.c +++ b/arch/i386/kernel

Re: nmi_watchdog fix for x86_64 to be more like i386

2007-10-01 Thread Thomas Gleixner
On Mon, 1 Oct 2007, Andi Kleen wrote: IRQ_NOBALANCING is not preventing cpu unplug. It moves the affinity to the next CPU, but the check in NMI watchdog for CPU == 0 would not longer work. That cannot happen right now because cpu_disable() on both i386/x86-64 reject CPU #0. So just

Re: nmi_watchdog fix for x86_64 to be more like i386

2007-10-01 Thread Thomas Gleixner
On Tue, 2 Oct 2007, Andi Kleen wrote: OTOH, the accounting hook would allow us to remove the IRQ#0 - CPU#0 restriction. Not sure whether it's worth the trouble. Some SIS chipsets hang the machine when you migrate irq 0 to another CPU. It's better to keep that Also I wouldn't be surprised

Re: nmi_watchdog fix for x86_64 to be more like i386

2007-10-02 Thread Thomas Gleixner
On Tue, 2 Oct 2007, Andi Kleen wrote: Agreed. I just got a x8664-hrt report, where I found the following oddity: 0: 1197 172881 IO-APIC-edge timer That's one of those infamous AMD C1E boxen. Strange, all my systems have IRQ#0 on CPU#0 and nowhere else. Any

<    1   2   3   4   5   6   7   8   9   10   >