* Ian Pilcher ([EMAIL PROTECTED]) wrote: > Chris Wright wrote: > > > >From: Andres Salomon <[EMAIL PROTECTED]> > > > >It seems to me that if in the amd8111e_open() fuction dev->irq isn't > >zero and the irq request succeeds it might not get released anymore. > > Based on the wording above, I can't help wondering if this fixes a > problem that anyone is actually seeing.
I had same hesitation, but it's easy to see how lowmem could trigger this latent bug (so it's not a highly theoretical race condition, for example), and the fix is both quite simple and accepted upstream already. Anyway that's what the review cycle is for, it can be rejected. thanks, -chris -- Linux Security Modules http://lsm.immunix.org http://lsm.bkbits.net - 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/