Em Mon, 10 Oct 2016 07:44:53 -0400 Michael Ira Krufky <mkrufky@xxxxxxxxxxx> escreveu: > Antti makes a very good point. If we consider a situation where we > are streaming data while concurrently checking frontend status and > polling for IR codes, some locking will certainly be required in all > of these drivers. > > -Mike Krufky > > On Mon, Oct 10, 2016 at 7:24 AM, Antti Palosaari <crope@xxxxxx> wrote: > > Hello > > If you use usb buffers from the state you will need add lock in order to > > protect concurrent access to buffer. There may have multiple concurrent > > operations from rc-polling/demux/frontend. Lets say you are reading ber and > > it sets data to buffer (state), then context switch to remote controller > > polling => buffer in state is overwritten, then context is changed back to > > ber reading and now there is bad data. Indeed a mutex to protect it is required. I added it and re-submitted the series, with a few extra patches, fixing issues on some drivers that are outside dvb-usb. Feel free to review. Regards, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
