2008/9/24 Bob Copeland <[EMAIL PROTECTED]>: >> 2008/9/24 Bob Copeland <[EMAIL PROTECTED]>: >> > Shouldn't this be something like: >> > >> > ah->ah_ai_state = kzalloc()... >> > >> > Otherwise ath5k_hw_ani_set_min_immunity segfaults immediately. >> >> Already tested that and it doesn't segfault, we set ah_ani_state before >> that... >> struct ath5k_ani_state *ani_state = ah->ah_ani_state; > > Hmm, well I got a null pointer deref right at > ath5k_hw_ani_set_min_immunity+0x9. The line of code you posted does the > opposite of setting ah->ah_ani_state, right? >
Well it's a pointer, it should work, it worked for me at least. Might be my gcc or something, i'll look into it... > After I made my change to set the kzalloc-ed pointers into > ah->ah_ani_state/stats, it loaded (except, as you said, rx didn't seem to > work). I thought maybe you just reformatted the code before posting. > >> Yup we 'll make a label err: or something, that's not what i'm after >> right now, we must find out the initial values we should use and make >> it work. > > Yup, I know :) The code makes sense from reading the patent, though > I haven't gone through it too closely, yet. > ACK, hope you find something because i haven't so far ;-) -- GPG ID: 0xD21DB2DB As you read this post global entropy rises. Have Fun ;-) Nick _______________________________________________ ath5k-devel mailing list ath5k-devel@lists.ath5k.org https://lists.ath5k.org/mailman/listinfo/ath5k-devel
