Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <1488160439-7140-1-git-send-email-nobuhiro.iwamatsu.kw@hitachi.com>
 <CAGXu5jLEU2n8jUjXhCiOAqfonMKov6rTv5-3KWSt4EmOEUn5Bg@mail.gmail.com>
In-Reply-To: <CAGXu5jLEU2n8jUjXhCiOAqfonMKov6rTv5-3KWSt4EmOEUn5Bg@mail.gmail.com>
From:   Nobuhiro Iwamatsu <iwamatsu@nigauri.org>
Date:   Mon, 3 Dec 2018 12:37:11 +0900
Message-ID: <CABMQnVLh1_KZXr2nHrijGoOVk36-=D2w7Hq9hmbHFnu2Oa1u2g@mail.gmail.com>
Subject: Re: [PATCH v5 0/5] pstore: ramoops: support multiple pmsg instances
To:     keescook@chromium.org
Cc:     anton@enomsg.org, ccross@android.com, tony.luck@intel.com,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

# Remove nobuhiro.iwamatsu.kw@hitachi.com from To.

Hi,

I am already leaving Hitachi.
However, I would like to merge these patches into the kernel. I will
resume work.

Best regards,
  Nobuhiro

2018=E5=B9=B411=E6=9C=8830=E6=97=A5(=E9=87=91) 8:33 Kees Cook <keescook@chr=
omium.org>:
>
> On Sun, Feb 26, 2017 at 5:54 PM Nobuhiro Iwamatsu
> <nobuhiro.iwamatsu.kw@hitachi.com> wrote:
> > The following series implements multiple pmsg. This feature allows user=
space
> > program to control individual content aging or priority.
>
> I'd like to take a look at this series again, if you're still
> interested in upstreaming it. :) There were a lot of things that
> needed refactoring in pstore to sanely deal with this, and I think
> we're there now. Is this still desired?
>
> Thanks,
>
> -Kees
>
> >
> > If a pstore backend module(e.g. ramoops) requires the multiple pmsg ins=
tances
> > when registering itself to pstore, multiple /dev/pmsg[ID] are created. =
Writes
> > to each /dev/pmsg[ID] are isolated each other. After reboot, the conten=
ts are
> > available in /sys/fs/pstore/pmsg-[backend]-[ID].
> >
> > In addition, we add multiple pmsg support for ramoops. We can specify m=
ultiple
> > pmsg area size by its module parameter as follows.
> >
> >  pmsg_size=3D0x1000,0x2000,...
> >
> > I did check the operation of this feature on CycloneV (socfpga) Helio b=
oard.
> >
> > v5:
> >   Add commit: "pstore: Change parameter of ramoops_free_przs()"
> >     - I forgot addition to previous patch series.
> >   Update commit: "pstore: support multiple pmsg instances"
> >     - Fix comment.
> >     - Fix initialization of num_pmsg.
> >
> > v4:
> >   Rebase to 4.10-rc5
> >   The following patches have been removed from this series as similar f=
unctions
> >   were modified by other commit.
> >      - pstore: Replace four kzalloc() calls by kcalloc() in ramoops_ini=
t_przs()
> >      - pstore: Change parameter of ramoops_free_przs()
> >      - pstore: Rename 'przs' to 'dprzs' in struct ramoops_context
> >      - ramoops: Rename ramoops_init_prz() to ramoops_init_dprzs()
> >
> > 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.
> >
> > Hiraku Toyooka (2):
> >   pstore: support multiple pmsg instances
> >   selftests/pstore: add testcases for multiple pmsg instances
> >
> > Nobuhiro Iwamatsu (3):
> >   pstore: Change parameter of ramoops_free_przs()
> >   ramoops: Add __ramoops_init_prz() as generic function
> >   ramoops: support multiple pmsg instances
> >
> >  Documentation/admin-guide/ramoops.rst              |  22 ++
> >  .../bindings/reserved-memory/ramoops.txt           |   6 +-
> >  fs/pstore/pmsg.c                                   |  23 +-
> >  fs/pstore/ram.c                                    | 317 +++++++++++++=
+++-----
> >  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, 342 insertions(+), 97 deletions(-)
> >
> > --
> > 2.11.0
> >
>
>
> --
> Kees Cook


--=20
Nobuhiro Iwamatsu
   iwamatsu at {nigauri.org / debian.org}
   GPG ID: 40AD1FA6