<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Ok, I will implement my own TaskScheduler class, but operations that
    read-modify-write memory are not atomic with multiple
    processors/cores. I guess the large portion of users these days will
    be using multicore x86 processors.<br>
    <br>
    I asked this question on stackoverflow and the responses support the
    research I have done myself.<br>
    <br>
<a class="moz-txt-link-freetext" href="http://stackoverflow.com/questions/29099094/is-the-c-operator-atomic-with-a-multicore-processor">http://stackoverflow.com/questions/29099094/is-the-c-operator-atomic-with-a-multicore-processor</a><br>
    <br>
    My updated  application with a protected triggerEvent() method
    didn't cure the problem, one (out of six) of the streams stopped
    after an hour. I then modified BasicTaskScheduler to protect all
    reads/writes of the fTriggersAwaitingHandling and it has been
    running for over 24 hours now. Time will tell but it looks
    promising.<br>
    <br>
    If I'm to speculate why this hasn't been a problem before I would
    guess that the large majority of users are not using the triggers at
    runtime, e.g, they are using the file sources. And for those who are
    using a live source, if they only have one stream then the task
    schedulers' SingleStep() method is probably waiting in the select()
    call when the triggerEvent() method is called making it unlikely
    that they conflict.<br>
    <br>
    Also each thread in my application has it's own event trigger id.<br>
    <br>
    I'm not even suggesting that you change the implementation but at
    least be aware of the problem and document it for future users.<br>
    <br>
    <br>
    <div class="moz-cite-prefix">On 03/16/2015 05:50 PM, Ross Finlayson
      wrote:<br>
    </div>
    <blockquote
      cite="mid:34BBA6A9-C923-4210-9330-6E4A622B40B6@live555.com"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html;
        charset=ISO-8859-1">
      <div>
        <blockquote type="cite" class="">
          <div class="">
            <div bgcolor="#FFFFFF" text="#000000" class="">Having
              thought about it some more, I don't think it's enough to
              just protect calls to triggerEvent() because
              'fTriggersAwaitingHandling' is still modified by
              BasicTaskScheduler::SingleStep().<br class="">
            </div>
          </div>
        </blockquote>
        <div><br class="">
        </div>
        The intention of the code that implements “BasicTaskScheduler”
        was that:</div>
      <div>1/ The event loop code (which is always just a single thread,
        remember) does nothing with “fTriggersAwaitingHandling” except
        clear (using “&=~”) a single bit that was already set, and</div>
      <div>2/ The other thread(s) that can trigger event(s) do nothing
        with “fTriggersAwaitingHandling” except set (using “|=“) bit(s)
        that were not already set.</div>
      <div><br class="">
      </div>
      <div>If these two conditions are met, *and* if the ‘clear a bit’
        and ‘set a bit’ operations are atomic, then the code will be
        thread-safe.</div>
      <div><br class="">
      </div>
      <div>(If, OTOH, you’re worried that the ‘clear a bit’ and ‘set a
        bit’ operations might not be atomic on your system, then you’d
        need to write your own “TaskScheduler” subclass instead.)</div>
      <div><br class="">
      </div>
      <div>Looking at the code again just now, however, I realized that
        the event loop code does not always satisfy condition 1/.   Line
        183 of “BasicTaskScheduler.cpp” should be changed from:</div>
      <div><span class="Apple-tab-span" style="white-space:pre"> </span>fTriggersAwaitingHandling
        = 0;</div>
      <div>to</div>
      <div><span class="Apple-tab-span" style="white-space:pre"> </span>fTriggersAwaitingHandling
        &=~ fLastUsedTriggerMask;</div>
      <div><br class="">
      </div>
      <div>I have just released a new version (2015.03.16) of the code
        that fixes this.  You should upgrade to this version.</div>
      <div><br class="">
      </div>
      <div><br class="">
      </div>
      <div>You should also make sure that your own code satisfies
        condition 2/.  I.e., you should make sure that your code is
        never calling “triggerEvent()” more than once with the same
        ‘event trigger’ value, unless you’re sure that the first event
        (for that event trigger) has already been handled.  (In
        particular, you should never call “triggerEvent()” with the same
        ‘event trigger’ value from more than one thread.)</div>
      <br class="">
      <div apple-content-edited="true" class="">
        <span class="Apple-style-span" style="border-collapse: separate;
          color: rgb(0, 0, 0); font-family: Helvetica; font-style:
          normal; font-variant: normal; font-weight: normal;
          letter-spacing: normal; line-height: normal; orphans: 2;
          text-align: -webkit-auto; text-indent: 0px; text-transform:
          none; white-space: normal; widows: 2; word-spacing: 0px;
          -webkit-border-horizontal-spacing: 0px;
          -webkit-border-vertical-spacing: 0px;
          -webkit-text-decorations-in-effect: none;
          -webkit-text-size-adjust: auto; -webkit-text-stroke-width:
          0px; "><span class="Apple-style-span" style="border-collapse:
            separate; color: rgb(0, 0, 0); font-family: Helvetica;
            font-style: normal; font-variant: normal; font-weight:
            normal; letter-spacing: normal; line-height: normal;
            orphans: 2; text-align: -webkit-auto; text-indent: 0px;
            text-transform: none; white-space: normal; widows: 2;
            word-spacing: 0px; -webkit-border-horizontal-spacing: 0px;
            -webkit-border-vertical-spacing: 0px;
            -webkit-text-decorations-in-effect: none;
            -webkit-text-size-adjust: auto; -webkit-text-stroke-width:
            0px; ">Ross Finlayson<br class="">
            Live Networks, Inc.<br class="">
            <a moz-do-not-send="true" href="http://www.live555.com/"
              class="">http://www.live555.com/</a></span></span>
      </div>
      <div class=""><br class="webkit-block-placeholder">
      </div>
      <br class="">
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">_______________________________________________
live-devel mailing list
<a class="moz-txt-link-abbreviated" href="mailto:live-devel@lists.live555.com">live-devel@lists.live555.com</a>
<a class="moz-txt-link-freetext" href="http://lists.live555.com/mailman/listinfo/live-devel">http://lists.live555.com/mailman/listinfo/live-devel</a>
</pre>
    </blockquote>
    <br>
  </body>
</html>