<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<meta http-equiv="Content-Style-Type" content="text/css" />
<meta name="generator" content="pandoc" />
<title></title>
<style type="text/css">code{white-space: pre;}</style>
</head>
<body>
<p>Hi Rémi,</p>
<p>On 2017-03-29 22:10, Rémi Denis-Courmont wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Le keskiviikkona 29. maaliskuuta 2017, 19.39.22 EEST Filip Roséen a écrit :</code></pre>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Hi Rémi,
On 2017-03-29 20:33, Rémi Denis-Courmont wrote:</code></pre>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> Le perjantaina 24. maaliskuuta 2017, 3.28.30 EEST Filip Roséen a écrit :</code></pre>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> + vlc_interrupt_t interrupt;
+ vlc_thread_t thread;
+ atomic_bool active;</code></pre>
</blockquote>
<pre><code> This can´t be right. Either there is a lock and atomicity is not
necessary, or this will race.</code></pre>
</blockquote>
<pre><code> Could you elaborate on the potential race while keeping in mind the
*synchronizes-with* relationship in terms of operations on that
atomic?</code></pre>
</blockquote>
<pre><code> You have a mutex and an atomic for what looks like the same thing.
Either one is too many, or there is a race.</code></pre>
</blockquote>
<p>They do not denote the same thing, if there was no album-cache the lock would not be required.</p>
<p>I would also like to point out that the atomic and the lock lives in different entities, there is no mutex and atomic that would be used to refer/manage synchronization to the same set of data.</p>
</body>
</html>