Re: + delay-accounting-taskstats-interface-send-tgid-once.patch added to -mm tree




CPU 0 CPU 1
(in irq) (in the code above)

actually CPU 0 doesn't even have to be in irq context; any context will
do

stats_lock is taken
tasklist_lock is taken
stats_lock_is taken <spin>
<interrupt happens>
tasklist_lock is taken

which now forms an AB-BA deadlock!


this happens at least in copy_process which can call taskstats_tgid_free
without first disabling interrupts (via cleanup_signal). There may be
many other cases, I've not checked deeper yet.

Solution should be to make these functions use irqsave variant... any
comments from the authors of this patch ?


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



Relevant Pages

  • Re: NIU - Sun Neptune 10g - Transmit timed out reset (2.6.24)
    ... Empty" interrupt by not relying on the TX interrupt at all. ... In such a case we could batch the TX queue refill, ... More majordomo info at http://vger.kernel.org/majordomo-info.html ... Please read the FAQ at http://www.tux.org/lkml/ ...
    (Linux-Kernel)
  • [PATCH] [TRIVIAL] prism54/islpci_eth.c: dev_kfree_skb in irq context
    ... dev_kfree_skb shouldn't be used in an IRQ context. ... More majordomo info at http://vger.kernel.org/majordomo-info.html ... Please read the FAQ at http://www.tux.org/lkml/ ... Prev by Date: ...
    (Linux-Kernel)
  • Re: rcu_do_batch: rcu_data->qlen is not irq safe
    ... from interrupt. ... So, is it worth fixing? ... More majordomo info at http://vger.kernel.org/majordomo-info.html ... Please read the FAQ at http://www.tux.org/lkml/ ...
    (Linux-Kernel)
  • rcu_do_batch: rcu_data->qlen is not irq safe
    ... from interrupt. ... So, is it worth fixing? ... More majordomo info at http://vger.kernel.org/majordomo-info.html ... Please read the FAQ at http://www.tux.org/lkml/ ...
    (Linux-Kernel)
  • Re: + via-pata-controller-xfer-fixes.patch added to -mm tree
    ... This appears to be a _device_ that sends its interrupt early. ... If that is the case, the device may appear on any controller, not just VIA, and we would have to handle it globally via a device special-case in libata-core. ... More majordomo info at http://vger.kernel.org/majordomo-info.html ... Please read the FAQ at http://www.tux.org/lkml/ ...
    (Linux-Kernel)