On Sat, 2008-11-29 at 13:15 +0100, Christian Lamparter wrote: > > Unless we want to defer RX packet processing to a workqueue rather than > > a tasklet? > Hmm, but if we put the RX packet processing into a workqueue, we have to > put the tx status report processing in one too (see comment about rx/tx race > in main.c line 400)? Yeah, probably. I don't know all the details all the time, so I write such comments ;) > Or do you mean I should only put ap_sta_ps_end (& ap_sta_ps_start) > resending stuff into a workqueue? (Yeah, this makes a lot more sense, > and might actually work!, well let's prepare another patch) No idea! The only thing I was thinking is that it's not a good plan to have a callback that has different locking requirements depending on a parameter. johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
