Mike Looijmans <mike.looijmans@xxxxxxxx> writes: > Regarding this commit: > > https://lkml.org/lkml/2014/12/12/709 > > rsi: fix memory leak in rsi_load_ta_instructions() > > Memory allocated by kmemdup() in rsi_load_ta_instructions() is leaked. > But duplication of firmware data here is useless, > so the patch removes kmemdup() at all. > > Found by Linux Driver Verification project (linuxtesting.org). > > Signed-off-by: Alexey Khoroshilov <khoroshilov@xxxxxxxxx> > Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxx> > > We use this driver for the Redpine Wifi chip on our "florida" board, > and after this commit it stopped working. Symptom was that the "wlan0" > device was not created at all. Reverting the commit makes it work > again. > > Apparently, the kmemdup action is needed for something. I suspect the > DMA controller is still copying the firmware data before the method > returned. > > Having no experience with this part of the kernel, I wasn't able to > come up with a more constructive solution than just reverting the > patch. Hmm, rsi doesn't seem to have an entry in MAINTAINERS? Do we have a maintainer for this driver? Adding Fariya as the first rsi commiter. Unless someone has better suggestions I'll just revert the patch. -- Kalle Valo -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
