On 11/10/2014 03:29 PM, Meng Xu wrote:
I'm not sure if I should resend the patch just to change the commit log and add the reason of why doing this. 

I want to first add the reason. If I should resend the patch set, please let me know. 

2014-11-10 7:53 GMT-05:00 George Dunlap <george.dunlap@eu.citrix.com>:
On 10/25/2014 03:16 PM, Meng Xu wrote:
Move call to rt_update_deadline from _alloc to _insert;

The runq queue lock is not grabbed when  ​rt_update_deadline is called in rt_alloc_vdata function, which may cause race condition.

Can you not grab the lock in rt_alloc_vdata?  Or do you think it's just more efficient to do all the work you need all at once?

(I'm not suggesting a change to the code, I'm just trying to clarify the motivation for moving the code rather than adding a lock.)

 -George