On Sunday 19 Dec 2010, Theodore Kilgore wrote: > Finally, one concern that I have in the back of my mind is the question of > control settings for a camera which streams in bulk mode and requires the > setup of a workqueue. The owner of the camera says that he has > "encountered no problems" with running the two controls mentioned above. > Clearly, that is not a complete answer which overcomes all possible > objections. Rather, things are OK if and only if we can ensure that these > controls can be run only while the workqueue that does the streaming is > inactive. Somehow, I suspect that the fact that a sensible user would only > run such commands at camera setup is an insufficient guarantee that no > problems will ever be encountered. > > So, as I said, the question of interaction of a control and a workqueue is > another problem interesting little problem. Your thoughts on this > interesting little problem would be appreciated. I don't think you can assume a user won't try to adjust such controls while streaming - if I had one I'd certainly want to try swapping the control while streaming to see if I could see any affect on the output. Even though sq905.c doesn't have any controls on the camera it still ended up needing the locking that would make this safe. See the header comment on sq905_dostream Adam -- 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
