Re: [PATCH v2] Adjustments: lock/unlock task in context_switch

2017-12-19 Thread Peter Zijlstra
On Tue, Dec 19, 2017 at 12:23:57PM -0200, Rodrigo Siqueira wrote: > In the original patch, I called this function as release_lock_task, > because the release_task was already declared as extern in > include/linux/sched/task.h. I believe there is a function name conflict > here, is that correct? Ba

Re: [PATCH v2] Adjustments: lock/unlock task in context_switch

2017-12-19 Thread Rodrigo Siqueira
Thanks for the review :) Below I just have a small comment in the changed version of the patch > Thanks; I've slightly changed it, find below. I'll queue it for the next > merge window. > > --- > Subject: sched: Rework / clarify prepare_lock_switch() > From: rodrigosiqueira > Date: Fri, 15 Dec

Re: [PATCH v2] Adjustments: lock/unlock task in context_switch

2017-12-18 Thread Peter Zijlstra
Thanks; I've slightly changed it, find below. I'll queue it for the next merge window. --- Subject: sched: Rework / clarify prepare_lock_switch() From: rodrigosiqueira Date: Fri, 15 Dec 2017 12:06:03 -0200 The function prepare_lock_switch has an unused parameter, and also the function name was

[PATCH v2] Adjustments: lock/unlock task in context_switch

2017-12-15 Thread rodrigosiqueira
Function prepare_lock_switch have an unused parameter, and also the function name was not descriptive. To improve the readability and remove the extra parameter, the following changes were made: * Moved prepare_lock_switch from kernel/sched/sched.h to kernel/sched/core.c, renamed it to acquire_l