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=-8.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, UNPARSEABLE_RELAY,USER_AGENT_SANE_1 autolearn=unavailable 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 297CEC3F68F for ; Wed, 22 Jan 2020 15:01:28 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 090D22465B for ; Wed, 22 Jan 2020 15:01:28 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726061AbgAVPB1 (ORCPT ); Wed, 22 Jan 2020 10:01:27 -0500 Received: from smtp2207-205.mail.aliyun.com ([121.197.207.205]:48844 "EHLO smtp2207-205.mail.aliyun.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725802AbgAVPB1 (ORCPT ); Wed, 22 Jan 2020 10:01:27 -0500 X-Alimail-AntiSpam: AC=CONTINUE;BC=0.07484593|-1;CH=green;DM=CONTINUE|CONTINUE|true|0.243332-0.015942-0.740726;DS=CONTINUE|ham_system_inform|0.377455-0.00372375-0.618821;FP=0|0|0|0|0|-1|-1|-1;HT=e01l07440;MF=liaoweixiong@allwinnertech.com;NM=1;PH=DS;RN=17;RT=17;SR=0;TI=SMTPD_---.GgEOwdx_1579705279; Received: from 192.168.43.221(mailfrom:liaoweixiong@allwinnertech.com fp:SMTPD_---.GgEOwdx_1579705279) by smtp.aliyun-inc.com(10.147.44.118); Wed, 22 Jan 2020 23:01:21 +0800 Subject: Re: [PATCH v1 06/11] Documentation: pstore/blk: blkoops: create document for pstore_blk To: Randy Dunlap , Kees Cook , Anton Vorontsov , Colin Cross , Tony Luck , Jonathan Corbet , Miquel Raynal , Richard Weinberger , Vignesh Raghavendra , Mauro Carvalho Chehab , "David S. Miller" , Rob Herring , Greg Kroah-Hartman , Jonathan Cameron Cc: linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org References: <1579482233-2672-1-git-send-email-liaoweixiong@allwinnertech.com> <1579482233-2672-7-git-send-email-liaoweixiong@allwinnertech.com> <40d7f57a-119e-e51f-99a5-63e85ab5ab91@infradead.org> <3337f687-a668-c058-178b-a1438641c519@allwinnertech.com> <597e2b49-667a-490e-91b6-641ca25401d8@infradead.org> From: liaoweixiong Message-ID: <6d94b9d5-abef-db5e-1c80-00ea8c1b0003@allwinnertech.com> Date: Wed, 22 Jan 2020 23:01:19 +0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.4.1 MIME-Version: 1.0 In-Reply-To: <597e2b49-667a-490e-91b6-641ca25401d8@infradead.org> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2020/1/21 下午11:34, Randy Dunlap wrote: > On 1/21/20 12:19 AM, liaoweixiong wrote: >> hi Randy Dunlap, >> >> On 2020/1/21 2:36 PM, Randy Dunlap wrote: >>> On 1/20/20 9:23 PM, liaoweixiong wrote: >>>> hi Randy Dunlap, >>>> >>>> On 2020/1/21 PM12:13, Randy Dunlap wrote: >>>>> Hi, >>>>> >>>>> I have some documentation comments for you: >>>>> >>>>> >>>>> On 1/19/20 5:03 PM, WeiXiong Liao wrote: >>>>>> The document, at Documentation/admin-guide/pstore-block.rst, tells us >>>>>> how to use pstore/blk and blkoops. >>>>>> >>>>>> Signed-off-by: WeiXiong Liao >>>>>> --- >>>>>>   Documentation/admin-guide/pstore-block.rst | 278 +++++++++++++++++++++++++++++ >>>>>>   MAINTAINERS                                |   1 + >>>>>>   fs/pstore/Kconfig                          |   2 + >>>>>>   3 files changed, 281 insertions(+) >>>>>>   create mode 100644 Documentation/admin-guide/pstore-block.rst >>>>>> >>>>>> diff --git a/Documentation/admin-guide/pstore-block.rst b/Documentation/admin-guide/pstore-block.rst >>>>>> new file mode 100644 >>>>>> index 000000000000..58418d429c55 >>>>>> --- /dev/null >>>>>> +++ b/Documentation/admin-guide/pstore-block.rst >>>>>> + >>>>>> + >>>>>> +dmesg_size >>>>>> +~~~~~~~~~~ >>>>>> + >>>>>> +The chunk size in bytes for dmesg(oops/panic). It **MUST** be a multiple of >>>>>> +4096. If you don't need it, safely set it 0 or ignore it. >>>>> >>>>>                                        set it to 0 or ignore it. >>>>> >>>> >>>> I will fix it, thank you. >>>> >>>>> The example above is:  blkoops.dmesg_size=64 >>>>> where 64 is not a multiple of 4096. (?) >>>>> >>>> >>>> The module parameter dmesg_size is in unit KB. >>> >>> I didn't see that documented anywhere. >>> >> >> Oh, sorry, that is my oversight. It seems that not only the other size introductions but also introductions on Kconfig should be corrected. Thank you very much and is the following modification OK? >> >> The chunk size in KB for dmesg(oops/panic). It **MUST** be a multiple of 4. > > OK. > > >>>>>> +Compression and header >>>>>> +---------------------- >>>>>> + >>>>>> +Block device is large enough for uncompressed dmesg data. Actually we do not >>>>>> +recommend data compression because pstore/blk will insert some information into >>>>>> +the first line of dmesg data. For example:: >>>>>> + >>>>>> +        Panic: Total 16 times >>>>>> + >>>>>> +It means that it's the 16th times panic log since the first booting. Sometimes >>>>> >>>>>                                 time of a panic log since ... >>>>> >>>> >>>> Should it be like this? >>>> It means the time of a panic log since the first booting. >>> >>> That sounds like clock time, not the number of instances or occurrences. >>> >> >> It is an oops/panic counter too. How about this? >> >> It means that it's OOPS/PANIC for the 16th time since the first booting. > > since the last booting {or boot}. > Not the last booting but the first booting. This is the number of triggers since the first time the system was installed. > thanks. >