From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934779AbdADGVO (ORCPT ); Wed, 4 Jan 2017 01:21:14 -0500 Received: from mail-io0-f179.google.com ([209.85.223.179]:33924 "EHLO mail-io0-f179.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933305AbdADGVJ (ORCPT ); Wed, 4 Jan 2017 01:21:09 -0500 MIME-Version: 1.0 In-Reply-To: References: <1476774814-6840-1-git-send-email-nobuhiro.iwamatsu.kw@hitachi.com> From: Kees Cook Date: Tue, 3 Jan 2017 22:20:13 -0800 X-Google-Sender-Auth: D2ls5r3aYqmPf0xg1WOAEJaLlr8 Message-ID: Subject: Re: [PATCH v3 0/8] pstore: ramoops: support multiple pmsg instances To: =?UTF-8?B?5bKp5p2+5L+h5rSLIC8gSVdBTUFUU1XvvIxOT0JVSElSTw==?= Cc: Anton Vorontsov , Colin Cross , Tony Luck , LKML , "cti.systems-productivity-manager.ts@hitachi.com" Content-Type: text/plain; charset=UTF-8 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 quoted-printable to 8bit by mail.home.local id v046LIWQ021103 On Mon, Dec 26, 2016 at 4:48 PM, 岩松信洋 / IWAMATSU，NOBUHIRO wrote: > 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. Now that the big changes have landed in Linus's tree, are you able to rebase your series on those? -Kees -- Kees Cook Nexus Security