Hi Mikel, On Tue, Jul 24, 2012 at 6:52 PM, Mikel Astiz <mikel.astiz.oss@xxxxxxxxx> wrote: > From: Mikel Astiz <mikel.astiz@xxxxxxxxxxxx> > > This patch series adds a flag to the Acquire method in the Media API, in order to fix the race condition described in patch v1 3/4. > > v1 tries to follow the proposal from Luiz by avoiding changes in the internal callbacks, and instead using the state information available in struct media_transport. > > Patch v1 4/4 needs careful review since the replacement of the in_use flag is not trivial. > > From original patch: > > This patch reopens the discussion started by the thread "when is acquire > ok to call". The race condition seems to be real (even thought difficult > to reproduce), and I couldn't think of any approach to solve this > without altering the Media API. Btw, I guess it is important to clarify that this doesn't break the API as the new flag is ignored in previous versions. The other important thing to mention is that this flag is not specific to one role, otherwise we would have done this without adding a new API, because there could be some situations where the transport must be connected regardless of the role. -- Luiz Augusto von Dentz -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
