Re: [PATCH 3/3] rtmutex: remove unnecessary adjust prio

From: Alex Shi
Date: Thu Jul 13 2017 - 02:12:20 EST




On 07/12/2017 10:14 PM, Steven Rostedt wrote:
> On Tue, 11 Jul 2017 22:39:24 +0800
> Alex Shi <alex.shi@xxxxxxxxxx> 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@xxxxxxxxxxx>

Thanks Steven!

Regards
Alex