> > What I don't understand - and you haven't really described - is how > > the changes fix it? Could you resend with a paragraph added that > > explains that? > > "Since downgrading unicast traffic to vif->txq must not be done > there's no txq to put a frame on if sta->uploaded is false. Therefore > the code is made to fall back to regular tx() op path if the > described condition is hit. " -- is this sufficient? Makes sense. > > Also, you're adding a test: > > > > > if (sta && !sta->uploaded) > > > > but couldn't do move that into the existing "if (sta)" block? > > Everything before that only ever returns NULL anyway. > > Good point. It makes more sense to put the sta->uploaded check in if > (sta) block. I'll move it. > Ok, thanks. johannes
