<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><DIV>Oh, I think we need mask, because it is really ready task , the execute id get from it<BR></DIV>
<DIV>At 2013-09-17 21:43:02,"Derek Buitenhuis" <derek.buitenhuis@gmail.com> wrote:<BR>>On 9/17/2013 2:11 PM, Min Chen wrote:<BR>>> # HG changeset patch<BR>>> # User Min Chen <chenm003@163.com><BR>>> # Date 1379423463 -28800<BR>>> # Node ID c5c645aa14bfdbf482186730c59a000fdae10e07<BR>>> # Parent  0d33ff236f68bc2238138a7213301b2efc0e6426<BR>>> Resolve some patching issues for previous patch (deadlock).<BR>><BR>>Can you add a short description of how it is fixed to<BR>>the commit message?<BR>><BR>>Much of the patch is confusing without context.<BR>><BR>>>          &nbsp
 ;   uint64_t oldval = m_queuedBitmap[w];<BR>>> -            if (oldval == 0) // race condition<BR>>> +            uint64_t mask = m_queuedBitmap[w] & m_enableBitmap[w];<BR>>> +            if (mask == 0) // race condition<BR>>>                  break;<BR>><BR>>if (!(oldval & m_enableBitmap[w]))<BR>>    break;<BR>><BR>>No need for a new stack var. Just a matter of preference; feel<BR>>free to ignore.<BR>><BR>>- Derek<BR>>_____
 __________________________________________<BR>>x265-devel mailing list<BR>>x265-devel@videolan.org<BR>>https://mailman.videolan.org/listinfo/x265-devel<BR></DIV></div>