lkml.org 
[lkml]   [2017]   [Jul]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH 3/3] rtmutex: remove unnecessary adjust prio
    From
    Date


    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

    \
     
     \ /
      Last update: 2017-07-13 08:12    [W:3.532 / U:0.040 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site