Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1751144AbdGMGML (ORCPT <rfc822;w@1wt.eu>);
        Thu, 13 Jul 2017 02:12:11 -0400
Received: from mail-pg0-f48.google.com ([74.125.83.48]:35557 "EHLO
        mail-pg0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1750947AbdGMGMK (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Thu, 13 Jul 2017 02:12:10 -0400
Subject: Re: [PATCH 3/3] rtmutex: remove unnecessary adjust prio
To: Steven Rostedt <rostedt@goodmis.org>
Cc: peterz@infradead.org, mingo@redhat.com, corbet@lwn.net,
        "open list:LOCKING PRIMITIVES" <linux-kernel@vger.kernel.org>,
        "open list:DOCUMENTATION" <linux-doc@vger.kernel.org>,
        Sebastian Siewior <bigeasy@linutronix.de>,
        Mathieu Poirier <mathieu.poirier@linaro.org>,
        Juri Lelli <juri.lelli@arm.com>, Thomas Gleixner <tglx@linutronix.de>
References: <1499395922-542-1-git-send-email-alex.shi@linaro.org>
 <1499395922-542-3-git-send-email-alex.shi@linaro.org>
 <c97943ea-0972-49b5-49fd-3bf939d7c7ad@linaro.org>
 <20170712101449.26b47984@gandalf.local.home>
From: Alex Shi <alex.shi@linaro.org>
Message-ID: <a8c14aab-5d90-8c9f-7bc9-bfe26f9d839e@linaro.org>
Date: Thu, 13 Jul 2017 14:12:03 +0800
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101
 Thunderbird/52.1.1
MIME-Version: 1.0
In-Reply-To: <20170712101449.26b47984@gandalf.local.home>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 820
Lines: 31



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