linux-kernel - Re: [PATCH] rcu/tree_plugin: Fix nohz status in stall warning

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <99ea3a0a-31b1-be28-418e-a3a80b02260b@codeaurora.org>
Date:   Sat, 9 Mar 2019 09:12:50 +0530
From:   Neeraj Upadhyay <neeraju@...eaurora.org>
To:     paulmck@...ux.ibm.com
Cc:     josh@...htriplett.org, rostedt@...dmis.org,
        mathieu.desnoyers@...icios.com, jiangshanlai@...il.com,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] rcu/tree_plugin: Fix nohz status in stall warning



On 3/9/19 2:48 AM, Paul E. McKenney wrote:
> On Fri, Mar 08, 2019 at 11:51:49PM +0530, Neeraj Upadhyay wrote:
>> Fix stall warning, to show correct nohz marker.
>>
>> Signed-off-by: Neeraj Upadhyay <neeraju@...eaurora.org>
> 
> Good eyes, thank you!
> 
> I applied and pushed all three with modified commit logs.  Please check
> to make sure that I didn't mess anything up.  If testing and reviews
> are OK, I expect to submit them for the v5.2 merge window.
> 
> I had to hand-apply this one, so in future patches could you please
> generate them against -rcu?  It may be found here:
> 
> 	git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git
> 
> 							Thanx, Paul
> 

Thanks Paul, I reviewed the patches on -rcu; they look good. Thanks for 
applying them! I will move my codebase to  -rcu, for all future work.

Thanks
Neeraj

>> ---
>>   kernel/rcu/tree_plugin.h | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/kernel/rcu/tree_plugin.h b/kernel/rcu/tree_plugin.h
>> index 97dba50..93da32c 100644
>> --- a/kernel/rcu/tree_plugin.h
>> +++ b/kernel/rcu/tree_plugin.h
>> @@ -1659,7 +1659,7 @@ static void print_cpu_stall_fast_no_hz(char *cp, int cpu)
>>   		rdp->last_accelerate & 0xffff, jiffies & 0xffff,
>>   		".l"[rdp->all_lazy],
>>   		".L"[!rcu_segcblist_n_nonlazy_cbs(&rdp->cblist)],
>> -		".D"[!rdp->tick_nohz_enabled_snap]);
>> +		".D"[!!rdp->tick_nohz_enabled_snap]);
>>   }
>>
>>   #else /* #ifdef CONFIG_RCU_FAST_NO_HZ */
>> -- 
>> QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a
>> member of the Code Aurora Forum, hosted by The Linux Foundation
>>
> 

-- 
QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a
member of the Code Aurora Forum, hosted by The Linux Foundation