[haiku-commits] Re: haiku: hrev50749 - src/kits/media

  • From: Dario Casalinuovo <b.vitruvio@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Fri, 9 Dec 2016 13:38:22 +0100

On Fri, Dec 9, 2016 at 3:52 AM, Rene Gollent <anevilyak@xxxxxxxxx> wrote:



On Dec 8, 2016 4:45 PM, "Dario Casalinuovo" <b.vitruvio@xxxxxxxxx> wrote:


In the case of the crash, the node was itself timesource and since the
media_kit has that confused pseudo-locking strategy....it happens from time
to time that the timesource is being updated (PubblishTime()) while the
ControlLoop() is calling RealTimeFor().



That implies that the locking is still broken, and you've simply changed
the timing during which the race condition can occur without actually
fixing it.


No that's not so directly implied. Great parts of the media_kit rely just
on timing of events and there's not any particular enforced mechanism.

As stated so much times in past so that I'm becoming bored of myself, the
BMediaClient API by replacing BMediaEventLooper and adding a graph based
processing scheme would enforce better mechanisms.

Other related posts: