Re: [PATCH 1/3] virtio: Basic implementation of virtio pstore driver

From	Cornelia Huck <cornelia.huck@de.ibm.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 1/3] virtio: Basic implementation of virtio pstore driver
Date	2016-07-18 11:10 +0200
Message-ID	<rWcIN-4ho-23@gated-at.bofh.it> (permalink)
References	<rW8vv-1wg-1@gated-at.bofh.it> <rW8vv-1wg-5@gated-at.bofh.it> <rWbD4-3lv-9@gated-at.bofh.it> <rWcfL-3Nz-7@gated-at.bofh.it>
Organization	IBM Deutschland Research & Development GmbH Vorsitzende des Aufsichtsrats: Martina Koederitz Geschäftsführung: Dirk Wittkopp Sitz der Gesellschaft: Böblingen Registergericht: Amtsgericht Stuttgart, HRB 243294

Show all headers | View raw

On Mon, 18 Jul 2016 17:29:55 +0900
Namhyung Kim <namhyung@kernel.org> wrote:

> On Mon, Jul 18, 2016 at 09:54:39AM +0200, Cornelia Huck wrote:
> > On Mon, 18 Jul 2016 13:37:39 +0900
> > Namhyung Kim <namhyung@kernel.org> wrote:

> > > +#define VIRT_PSTORE_ORDER    2
> > > +#define VIRT_PSTORE_BUFSIZE  (4096 << VIRT_PSTORE_ORDER)
> > 
> > It may make sense to make the size of the buffer configurable through
> > the config space.
> 
> Right.  I'm considering it too, but it needs a buffer larger than
> kmsg_bytes (= 10K) to work properly in the current implementation.  As
> this version is just to verify the idea is sane and useful, I used a
> fixed size buffer.  Will change in the next version.

Sure, that makes sense for a prototype. We can guard any config space
entry with a feature bit, but this one makes sense to add from the
beginning.

> > Speaking of the standard: I think it makes sense to at least reserve a
> > device id for pstore, as the idea is sound. Maybe prepare a patch to
> > the standard as well if you have time?
> 
> I'd love to.  As I mentioned earlier, I don't have enough knowledge in
> this area.  Could you please provide some links about how can I do that?

See the virtio page at OASIS
(https://www.oasis-open.org/committees/tc_home.php?wg_abbrev=virtio)
for a link to our subversion (yes...) repository. Just do two patches:
one to reserve a device id, and one that specifies how device and
driver work. (For examples, look at the proposed device types that have
been posted to the virtualization lists, e.g. virtio-crypto or
virtio-sdm). You just need to be patient, we're currently a bit
stalled...

Thread

[RFC/PATCHSET 0/3] virtio-pstore: Implement virtio pstore device Namhyung Kim <namhyung@kernel.org> - 2016-07-18 06:40 +0200
  [PATCH 1/3] virtio: Basic implementation of virtio pstore driver Namhyung Kim <namhyung@kernel.org> - 2016-07-18 06:40 +0200
    Re: [PATCH 1/3] virtio: Basic implementation of virtio pstore driver Kees Cook <keescook@chromium.org> - 2016-07-18 07:20 +0200
      Re: [PATCH 1/3] virtio: Basic implementation of virtio pstore driver Namhyung Kim <namhyung@kernel.org> - 2016-07-18 08:00 +0200
        Re: [PATCH 1/3] virtio: Basic implementation of virtio pstore driver Kees Cook <keescook@chromium.org> - 2016-07-18 20:00 +0200
          Re: [PATCH 1/3] virtio: Basic implementation of virtio pstore driver Namhyung Kim <namhyung@kernel.org> - 2016-07-19 15:50 +0200
            Re: [PATCH 1/3] virtio: Basic implementation of virtio pstore driver Namhyung Kim <namhyung@kernel.org> - 2016-07-19 17:40 +0200
            Re: [PATCH 1/3] virtio: Basic implementation of virtio pstore driver Namhyung Kim <namhyung@kernel.org> - 2016-07-20 15:00 +0200
    Re: [PATCH 1/3] virtio: Basic implementation of virtio pstore  driver Cornelia Huck <cornelia.huck@de.ibm.com> - 2016-07-18 10:00 +0200
      Re: [PATCH 1/3] virtio: Basic implementation of virtio pstore driver Namhyung Kim <namhyung@kernel.org> - 2016-07-18 10:40 +0200
        Re: [PATCH 1/3] virtio: Basic implementation of virtio pstore  driver Cornelia Huck <cornelia.huck@de.ibm.com> - 2016-07-18 11:10 +0200
  [PATCH 2/3] qemu: Implement virtio-pstore device Namhyung Kim <namhyung@kernel.org> - 2016-07-18 06:40 +0200
    Re: [PATCH 2/3] qemu: Implement virtio-pstore device Christian Borntraeger <borntraeger@de.ibm.com> - 2016-07-18 09:30 +0200
      Re: [PATCH 2/3] qemu: Implement virtio-pstore device Namhyung Kim <namhyung@kernel.org> - 2016-07-18 10:40 +0200
    Re: [PATCH 2/3] qemu: Implement virtio-pstore device Stefan Hajnoczi <stefanha@gmail.com> - 2016-07-18 12:10 +0200
      Re: [PATCH 2/3] qemu: Implement virtio-pstore device Namhyung Kim <namhyung@kernel.org> - 2016-07-18 16:30 +0200
        Re: [PATCH 2/3] qemu: Implement virtio-pstore device Stefan Hajnoczi <stefanha@gmail.com> - 2016-07-20 13:40 +0200
          Re: [PATCH 2/3] qemu: Implement virtio-pstore device Namhyung Kim <namhyung@kernel.org> - 2016-07-20 14:50 +0200
      Re: [PATCH 2/3] qemu: Implement virtio-pstore device Namhyung Kim <namhyung@kernel.org> - 2016-07-19 17:50 +0200
        Re: [PATCH 2/3] qemu: Implement virtio-pstore device Stefan Hajnoczi <stefanha@gmail.com> - 2016-07-20 13:40 +0200
          Re: [PATCH 2/3] qemu: Implement virtio-pstore device Namhyung Kim <namhyung@kernel.org> - 2016-07-20 14:40 +0200
  [PATCH 3/3] kvmtool: Implement virtio-pstore device Namhyung Kim <namhyung@kernel.org> - 2016-07-18 06:40 +0200

csiph-web