[vlc-devel] [vlc-commits] picture_pool: fix race condition

Thomas Guillem thomas at gllm.fr
Fri Oct 31 15:23:18 CET 2014



On Fri, Oct 31, 2014, at 11:47, Rémi Denis-Courmont wrote:
> Le 2014-10-31 13:40, Thomas Guillem a écrit :
> > I don't understand well that part of the code, so I trust you and 
> > your
> > code.
> 
> But there is nothing to understand... The semantics of the function are 
> unchanged!
> 
> Literally, the change replaces this pseudocode:
> 
> if (refs > 0) /* atomic_load */
>      continue;
> refs += 1; /* picture_Hold() */
> 
> with equivalent but actually atomic pseudocode:
> 
> if (refs != 0)
>      continue;
> refs = 1;

Ok, I thought picture_Hold was doing more stuff than just a ref++.

When I revert this patch, I got a correct behavior, but finally I think
it's unrelated. I'm on it.


> 
> -- 
> Rémi Denis-Courmont
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel



More information about the vlc-devel mailing list