Paul E. McKenney wrote: > So if a second CNIC driver attempts to register, it gets -EBUSY or > something, right? There is only one CNIC driver and it will only register once per BNX2 netdevice. > > You are right. We should just unconditionally set up the IRQ > > information without checking for c_ops. The data > structures we set up > > below are owned by us. > > OK. Hmmm.... You cannot even get away with sarcasm these days! ;-) Hmm, not sure what's the sarcasm. The code here is trying to set up the IRQ information so that CNIC driver can see it. Even if the CNIC driver is deregistering or was never registered, it doesn't hurt to set up that information. Anyway, I'll add some comments to the code to explain this better when I respin. > Very good. Could you please add a comment to that effect? Otherwise > people search for what data structure is being freed up. > Will do. Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html