From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-15.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 987DBC433E6 for ; Wed, 27 Jan 2021 22:10:29 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 6861564DC4 for ; Wed, 27 Jan 2021 22:10:29 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236667AbhA0WKX (ORCPT ); Wed, 27 Jan 2021 17:10:23 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58910 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236589AbhA0WKU (ORCPT ); Wed, 27 Jan 2021 17:10:20 -0500 Received: from galois.linutronix.de (Galois.linutronix.de [IPv6:2a0a:51c0:0:12e:550::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4878EC061573 for ; Wed, 27 Jan 2021 14:09:40 -0800 (PST) From: Thomas Gleixner DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020; t=1611785378; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=cRHi3vNzIfr8TCLFZ6ldJ5CafgzcNszokGa318PJPEk=; b=4m3utNItU5lTkzKPDwpu6Ls28uBgchIegkVXHvHZmHpy7XJ3B86OXr5O4NICMLQdsx8dYG EsCqOXQY4fxWUbsJ7FTYPJGK3QffKdgwPS/8RmYu6G/e8GC14u169liwvUtPvWxFkkwxe8 1/zl5iLa6U+qd4Gx5epmenev0dGWql84q2MjGwAosMzVGIB0EBnlxj2SJP9GHZBzkbqDK9 VmCzuynlMlAqlZknS9txUfuKPrg4jBVV2HOWCQtbIJu1zebO9HVL+6odSHw9Qm8+sBawtp AmEeyfzkLlqH4Z2MWeDXoMzCilWT/wP7Zfbigd8y8WFDLPXI/QWF7RQy2fbbrA== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020e; t=1611785378; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=cRHi3vNzIfr8TCLFZ6ldJ5CafgzcNszokGa318PJPEk=; b=K7y1cVn38ujc2CSQ/vRCbO/1Z+SLHEzcmRUGEhlEL3sxstdfMi74aKB5ujNk6jWgf6ZIBW Iksf60JMloEO1NCQ== To: Fenghua Yu , Borislav Petkov , Ingo Molnar , Peter Zijlstra , Tony Luck , Randy Dunlap , Xiaoyao Li , Ravi V Shankar Cc: linux-kernel , x86 , Fenghua Yu Subject: Re: [PATCH v4 4/4] Documentation/admin-guide: Change doc for split_lock_detect parameter In-Reply-To: <20201124205245.4164633-5-fenghua.yu@intel.com> References: <20201124205245.4164633-1-fenghua.yu@intel.com> <20201124205245.4164633-5-fenghua.yu@intel.com> Date: Wed, 27 Jan 2021 23:09:38 +0100 Message-ID: <87o8hadobh.fsf@nanos.tec.linutronix.de> MIME-Version: 1.0 Content-Type: text/plain Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Nov 24 2020 at 20:52, Fenghua Yu wrote: > Since #DB for bus lock detect changes the split_lock_detect parameter, > update the documentation for the changes. Why is this seperate and an all in one thing? patch 2/4 changes the parameter first and 3/4 adds a new option.... > > Signed-off-by: Fenghua Yu > Reviewed-by: Tony Luck > Acked-by: Randy Dunlap > --- > Change Log: > v4: > - Fix a ratelimit wording issue in the doc (Randy). > - Patch 4 is acked by Randy (Randy). > > v3: > - Enable Bus Lock Detection when fatal to handle bus lock from non-WB > (PeterZ). > > v1: > - Fix a few wording issues (Randy). > > RFC v2: > - Simplify the documentation (Randy). > > .../admin-guide/kernel-parameters.txt | 30 +++++++++++++++---- > 1 file changed, 24 insertions(+), 6 deletions(-) > > diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt > index 44fde25bb221..28228539b02a 100644 > --- a/Documentation/admin-guide/kernel-parameters.txt > +++ b/Documentation/admin-guide/kernel-parameters.txt > @@ -5051,27 +5051,45 @@ > spia_peddr= > + > + ratelimit:N - > + Set rate limit to N bus locks per second > + for bus lock detection. 0 < N <= HZ/2. Oh well. So if I have ratelimit:500 on the kernel command line and then this works for CONFIG_HZ=1000, but after rebuilding the kernel with HZ=250 it fails. What? If I allow 500 hits per second then CONFIG_HZ has absolutely nothing to do with it. A second stays a second independent of CONFIG_HZ. So what's the purpose of this HZ business? Thanks, tglx