[lkml]   [2006]   [Jun]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] i386: Fix softirq accounting with 4K stacks
    On 2006.06.25 19:43:17 +0200, Arjan van de Ven wrote:
    > On Sun, 2006-06-25 at 19:44 +0200, Mike Galbraith wrote:
    > > On Sun, 2006-06-25 at 16:24 +0200, Björn Steinbrink wrote:
    > >
    > > > Still no idea why your "fix" works, but the following patch also fixes
    > > > the problem and is at least a little more like the RightThing.
    > >
    > > Yeah. I don't know about you, but I fully intend to blatantly ignore
    > > that 'why' ;-)
    > the why is relatively easy ;)
    > since the "is a softirq executing" bit is on the stack, and each context
    > (user, soft and hard irq) has their own stack, it's not automatic that
    > the hardirq stack gets the softirq-executing flag... which your patch
    > fixes.

    That's mine, not Mike's. Mike's patch removed the #ifdef CONFIG_SMP
    around update_process_times() in smp_local_timer_interrupt().

    > NMI's and apic irqs generally don't go via the normal irq path and thus
    > don't do a stack switch... so they don't lose the bit (for accounting
    > purposes)

    Hm, doesn't that mean that mean that hardirq accounting is still broken?
    APIC irq comes in, increases hardirq count, then the timer irq fires,
    switches the stack and looses the hardirq count that was incremented by
    the APIC irq.

    I just booted with both patches applied, mine and Mike's, and that
    actually makes a difference in hardirq cpu time accounting. With my
    patch only, hi is 0 in top while the box gets a ping flood. With both
    patches, I get about 1% hi. Mike's patch causes update_process_times()
    to be called twice on UP, but that alone shouldn't change the
    percentages, right?
    OTOH top shows "hi" as zero with 8K stacks as well unless Mike's patch
    is applied, so the results with Mike's patch are bogus (if so, why?) or
    hardirq accounting is broken in general.

    Btw, which path do apic irqs go? I stumbled across the nmi stuff, but
    didn't see anything special for the apic irqs.

    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2006-06-25 20:45    [W:0.026 / U:4.896 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site