Re: [PATCH v2] workqueue: Fix double kfree for rescuer

2020-05-25 Thread qzhang2
I see, kfree does nothing with null pointers and direct return. but again kfree is not a good suggestion. On 5/25/20 5:50 PM, Markus Elfring wrote: The duplicate memory release should be deleted from the implementation of the callback function "rcu_free_wq". I tried to help with the selection

Re: [PATCH v2] workqueue: Fix double kfree for rescuer

2020-05-25 Thread Markus Elfring
> * The function call “destroy_workqueue” can be performed there in an if branch > after the statement “wq->rescuer = NULL” was executed. Another correction: * The function call “kfree(rescuer)” can be performed there in an if branch after the statement “wq->rescuer = NULL” was executed. Rega

Re: [PATCH v2] workqueue: Fix double kfree for rescuer

2020-05-25 Thread Markus Elfring
> The duplicate memory release should be deleted from the implementation > of the callback function "rcu_free_wq". I tried to help with the selection of a better commit message. I have taken another look also at the implementation of the function “destroy_workqueue”. * The function call “destroy