On Wed, Jan 27, 2016 at 5:44 AM, Lucas Stach <[email protected]> wrote: > Am Montag, den 25.01.2016, 18:44 +0900 schrieb Alexandre Courbot: >> nvkm_device_tegra_new initializes the irq member of the Tegra device >> to -1 in order to signal that it is uninitialized. However, >> nvkm_device_tegra_fini tests it against 0 to check whether an IRQ has >> been allocated or not. This leads to free_irq being called on -1 >> during >> device initialization. >> > The convention in other parts of the Linux kernel is that IRQ number 0 > means unallocated/invalid IRQ. So I think it is the initialization to > -1 that should be fixed instead.
Uh, that sounds unsafe. Thanks for spotting this, will resend. _______________________________________________ Nouveau mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/nouveau
