Hi,

On 06/14/2013 04:34 PM, Nathan Hjelm wrote:
> Sorry for the mistake. Don't know why I didn't catch that earlier.

Well my patch is wrong too, besides adding the pthread_join, I
instinctively swapped the location of the pthread_cancel and
the udev_monitor_unref(udev_monitor); call, thinking that the fd
should be closed before the cancel, but that is wrong because
that means the udev_monitor can be unreffed while in use by
the thread.

So I'll go and commit a fix to my fix now ...

Regards,

Hans


>
> -Nathan
>
> On Jun 14, 2013, at 07:48 AM, Hans de Goede <hdego...@redhat.com> wrote:
>
>> Hi,
>>
>> On 06/14/2013 02:43 PM, Xiaofan Chen wrote:
>> > On Fri, Jun 14, 2013 at 5:59 PM, Hans de Goede <hdego...@redhat.com 
>> > <mailto:hdego...@redhat.com>> wrote:
>> >> Hmm, ok, error 11 is EAGAIN, so indeed you're running out of some 
>> >> resource.
>> >>
>> >> Can you give the attached patch a try please ?
>> >>
>> >
>> > Great. Your patch fixed the issues I have under Ubuntu Linux.
>>
>> Good to hear, I've pushed the patch to master now.
>>
>> Regards,
>>
>> Hans
>>
>> ------------------------------------------------------------------------------
>> This SF.net <http://SF.net> email is sponsored by Windows:
>>
>> Build for Windows Store.
>>
>> http://p.sf.net/sfu/windows-dev2dev
>> _______________________________________________
>> libusbx-devel mailing list
>> libusbx-devel@lists.sourceforge.net 
>> <mailto:libusbx-devel@lists.sourceforge.net>
>> https://lists.sourceforge.net/lists/listinfo/libusbx-devel
>
>
> ------------------------------------------------------------------------------
> This SF.net email is sponsored by Windows:
>
> Build for Windows Store.
>
> http://p.sf.net/sfu/windows-dev2dev
>
>
>
> _______________________________________________
> libusbx-devel mailing list
> libusbx-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/libusbx-devel
>

------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:

Build for Windows Store.

http://p.sf.net/sfu/windows-dev2dev
_______________________________________________
libusbx-devel mailing list
libusbx-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/libusbx-devel

Reply via email to