[vlc-devel] [RFC PATCH 7/8] dsm/sd: discover in a separate thread
remi at remlab.net
Fri Nov 28 13:03:32 CET 2014
Le 2014-11-28 12:59, Thomas Guillem a écrit :
> On Fri, Nov 28, 2014, at 10:52, Rémi Denis-Courmont wrote:
>> Le 2014-11-27 14:00, Thomas Guillem a écrit :
>> > @@ -105,7 +116,8 @@ void bdsm_SdClose (vlc_object_t *p_this)
>> > if( p_sys == NULL )
>> > return;
>> > - if( p_sys->ns != NULL )
>> > - netbios_ns_destroy( p_sys->ns );
>> > + vlc_join( p_sys->thread, NULL );
>> If netbios_ns_discover() is fast (which the current code seems to
>> assume), running it in a thread seems rather pointless. If it is
>> the code is broken and moving it into an uninterruptible thread
>> fix it.
> netbios_ns_discover is not fast. Without that commit, the UI thread
> blocked while you discover (tested width vlc desktop).
So it is slow for the sake of my previous email and your patch does not
really solve the problem. Try to unload the SD (or exit the player)...
> Should I add a vlc_cancel call ?
I am not familiar with libDSM, so I don't know.
More information about the vlc-devel