Messages in this thread | | | Subject | Re: [PATCH 3/3] rtmutex: remove unnecessary adjust prio | From | Alex Shi <> | Date | Thu, 13 Jul 2017 14:12:03 +0800 |
| |
On 07/12/2017 10:14 PM, Steven Rostedt wrote: > On Tue, 11 Jul 2017 22:39:24 +0800 > Alex Shi <alex.shi@linaro.org> wrote: > >> Any comments for this little change? It's passed on 0day testing. > > I think the problem was that this was a third patch after two > documentation patches. Where, people put documentation review at the > bottom of their priority list. > > This should have been sent as separate patch on its own. >
Got it. I will resend it.
<snip>...
> > Interesting, I did some git mining and this was added with the original > entry of the rtmutex.c (23f78d4a0). Looking at even that version, I > don't see the purpose of adjusting the task prio here. It is done > before anything changes in the task. > > Reviewed-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
Thanks Steven!
Regards Alex
| |