[dm-devel] [PATCH] multipathd: avoid crash in uevent_cleanup()

lixiaokeng lixiaokeng at huawei.com
Tue Mar 2 15:49:57 UTC 2021



On 2021/3/2 19:07, Martin Wilck wrote:
> On Tue, 2021-03-02 at 16:41 +0800, lixiaokeng wrote:
>>
>>>
>>> Hi Martin:
>>>    Here I add condlog(2, "start funcname"),
>>> pthread_cleanup_push(func_print, NULL)
>>> in every pthread_create func. When these two core happened, "exit
>>> tur_thread"
>>> are less than "start tur_thread". So the trouble may be in
>>> tur_thread.
>>>
>>
>> If I made no mistake, these coredumps happened when the
>> tur_thread(which is
>> libcheck_thread in latest code) was not finished but checker(tur)was
>> cleared by cleanup_checkers. But I can't understand this, because the
>> ctx
>> in tur_thread wasn't freed by cleanup_checkers.
>>
>> Please give me some helps, thanks.
> 
> So you think the crashing thread is libcheck_thread? Do you have a core
> dump from this situation?
> 
> Does the code you use include commit 38ffd89 ("libmultipath: prevent
> DSO unloading with astray checker threads")?
> 
> Martin
> 

Hi Martin,

   I am sorry. I test code doesn't include commit 38ffd89. I confuse
the master and queue branches. I just test 0.7.7 and 0.8.5 master.
Please ignore libcheck_thread and focus on last email with thread
info.

Regards,
Lixiaokeng





More information about the dm-devel mailing list