On Sun, 2015-11-08 at 18:49 -0500, Bob Copeland wrote:
> On Fri, Nov 06, 2015 at 11:57:23AM +0100, Johannes Berg wrote:
> > Instead of using pointers, use sequentially assigned cookies.
> > This is easier to understand while debugging and also avoids
> > problems when the pointer is reused for the next allocation.
> 
> I tested this out in wmediumd and it didn't go so well...
> 
> > +   info->rate_driver_data[0] = (void *)cookie;
> 
> [...]
> 
> > +           skb_cookie = (u64)(uintptr_t)txi->driver_data[0];
> 
> rate_driver_data vs driver_data?

Yep, that fixes it. I've applied the patch with that fix.

johannes
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to