[vlc-devel] [PATCH] dvdnav: add Demux submodule

[Luca Barbato]

On 17/03/15 23:06, Rémi Denis-Courmont wrote:
> Le mardi 17 mars 2015, 22:57:20 Luca Barbato a écrit :
>> On 12/03/15 13:52, Rémi Denis-Courmont wrote:
>>> Le 2015-03-12 13:14, Thomas Guillem a écrit :
>>>> It allows to use dvdnav via a VLC access.
>>>
>>> And it locks up if the stream returns no data (something that cannot
>>> happen with regular files).
>>
>> Uhm sounds bad if you use nfs or anything similar, if the streams return
>> an error it is not handled?
>
> I mean it locks up if the stream read does not return (in any reasonable time
> frame), in other words if you have a (long) blocking read. That situation is
> not possible for a regular file. Regular file I/O is never blocking: it either
> completes or fails straight away.
>

And access doesn't have tunable timeouts with poll so the problem gets 
quite substantial.

Maybe net_Read could get a timeout field or it won't help at all? I can 
think of a number of situations in which even a normal demuxer might end 
up locked in the bad way you pointed out =/

lu