After some sleep, I have an idea: Is there any reason we couldn't create a method:
void dev_hold_ref(struct dbg_dev_rfcnt_info* i, netdevice* dev, void* key); This would be virtual identical to the __debug_dev_hold(netdevice* dev, void* key) that I currently have, but it would use the rfcnt_info struct that is passed in instead of mallocing one. We can add a flag to know to free it differently (probably not at all) upon dev_put(). #ifdef magic can allow us to completely ignore the first and last argument when debugging is disabled and use the current non-debug version of dev_put, dev_hold. I believe this would give us the ability to use my patch and it's ability to debug stack-related leaks while also providing a path to remove more and more of the instances where kmalloc is needed. Thanks, Ben -- Ben Greear <[EMAIL PROTECTED]> Candela Technologies Inc http://www.candelatech.com - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
