From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932833AbcL0At4 (ORCPT ); Mon, 26 Dec 2016 19:49:56 -0500 Received: from mailx.hitachi.co.jp ([133.145.228.49]:41225 "EHLO mailx.hitachi.co.jp" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755511AbcL0Aty (ORCPT ); Mon, 26 Dec 2016 19:49:54 -0500 From: =?utf-8?B?5bKp5p2+5L+h5rSLIC8gSVdBTUFUU1XvvIxOT0JVSElSTw==?= To: =?utf-8?B?5bKp5p2+5L+h5rSLIC8gSVdBTUFUU1XvvIxOT0JVSElSTw==?= , Kees Cook CC: Anton Vorontsov , Colin Cross , "Tony Luck" , LKML , "cti.systems-productivity-manager.ts@hitachi.com" Subject: RE: [PATCH v3 0/8] pstore: ramoops: support multiple pmsg instances Thread-Topic: [PATCH v3 0/8] pstore: ramoops: support multiple pmsg instances Thread-Index: AQHSKQ8+A7ZEBe2YYUCRhJOgl2aZ7KDT656AgCT0MCCAIoNcUA== Date: Tue, 27 Dec 2016 00:48:41 +0000 Deferred-Delivery: Tue, 27 Dec 2016 00:48:00 +0000 Message-ID: References: <1476774814-6840-1-git-send-email-nobuhiro.iwamatsu.kw@hitachi.com> In-Reply-To: Accept-Language: ja-JP, en-US Content-Language: ja-JP X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.232.28.229] Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 X-TM-AS-GCONF: 00 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from base64 to 8bit by mail.home.local id uBR0o4hb012311 Ping?_ > -----Original Message----- > From: linux-kernel-owner@vger.kernel.org > [mailto:linux-kernel-owner@vger.kernel.org] On Behalf Of 岩松信洋 / > IWAMATSU，NOBUHIRO > Sent: Monday, December 05, 2016 10:47 AM > To: Kees Cook > Cc: Anton Vorontsov; Colin Cross; Tony Luck; LKML; > cti.systems-productivity-manager.ts@hitachi.com > Subject: RE: [PATCH v3 0/8] pstore: ramoops: support multiple pmsg instances > > Hi, Kees. > > > -----Original Message----- > > From: keescook@google.com [mailto:keescook@google.com] On Behalf Of > > Kees Cook > > Sent: Saturday, November 12, 2016 7:24 AM > > To: 岩松信洋 / IWAMATSU，NOBUHIRO > > Cc: Anton Vorontsov; Colin Cross; Tony Luck; LKML; > > cti.systems-productivity-manager.ts@hitachi.com > > Subject: Re: [PATCH v3 0/8] pstore: ramoops: support multiple pmsg > > instances > > > > On Tue, Oct 18, 2016 at 12:13 AM, Nobuhiro Iwamatsu > > wrote: > > > The following series implements multiple pmsg. This feature allows > > > userspace program to control individual content aging or priority. > > > > > > If a pstore backend module(e.g. ramoops) requires the multiple pmsg > > > instances when registering itself to pstore, multiple /dev/pmsg[ID] > > > are created. Writes to each /dev/pmsg[ID] are isolated each other. > > > After reboot, the contents are available in > > /sys/fs/pstore/pmsg-[backend]-[ID]. > > > > > > In addition, we add multiple pmsg support for ramoops. We can > > > specify multiple pmsg area size by its module parameter as follows. > > > > > > pmsg_size=0x1000,0x2000,... > > > > > > I did check the operation of this feature on CycloneV (socfpga) > > > Helio > > board. > > > > > > v3: > > > Rebase to v4.8. > > > Split patch. > > > merged device_create(). > > > Remove Blank lines. > > > Update documentiation of DT binding. > > > Update parsing function of ramoops_pmsg_size, add NULL termination. > > > Update module parameters for pmsg_size list. > > > > Thanks for this v3! Sorry for the delay, I should be able to review > > this shortly. > > Thank you. > I will wait for your review. > > > > > -Kees > > > > Best regards, > Nobuhiro > > > > > > > Hiraku Toyooka (2): > > > pstore: support multiple pmsg instances > > > selftests/pstore: add testcases for multiple pmsg instances > > > > > > Nobuhiro Iwamatsu (6): > > > pstore: Replace four kzalloc() calls by kcalloc() in > > ramoops_init_przs() > > > pstore: Change parameter of ramoops_free_przs() > > > ramoops: Add __ramoops_init_prz() as generic function > > > pstore: Rename 'przs' to 'dprzs' in struct ramoops_context > > > ramoops: Rename ramoops_init_prz() to ramoops_init_dprzs() > > > ramoops: support multiple pmsg instances > > > > > > .../bindings/reserved-memory/ramoops.txt | 6 +- > > > Documentation/ramoops.txt | 22 ++ > > > fs/pstore/pmsg.c | 23 +- > > > fs/pstore/ram.c | 344 > > ++++++++++++++++----- > > > include/linux/pstore.h | 1 + > > > include/linux/pstore_ram.h | 8 +- > > > tools/testing/selftests/pstore/common_tests | 21 +- > > > .../selftests/pstore/pstore_post_reboot_tests | 27 +- > > > tools/testing/selftests/pstore/pstore_tests | 14 +- > > > 9 files changed, 360 insertions(+), 106 deletions(-) > > > > > > -- > > > 2.9.3 > > > > > > > > > > > > > > -- > > Kees Cook > > Nexus Security