Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-11-15 Thread Michael S. Tsirkin
On Tue, Nov 15, 2016 at 03:23:36PM +0900, Namhyung Kim wrote: > On Fri, Nov 11, 2016 at 12:50:03AM +0200, Michael S. Tsirkin wrote: > > On Fri, Sep 16, 2016 at 07:05:47PM +0900, Namhyung Kim wrote: > > > On Tue, Sep 13, 2016 at 06:57:10PM +0300, Michael S. Tsirkin wrote: > > > > On Sat, Aug 20, 201

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-11-14 Thread Namhyung Kim
On Fri, Nov 11, 2016 at 12:50:03AM +0200, Michael S. Tsirkin wrote: > On Fri, Sep 16, 2016 at 07:05:47PM +0900, Namhyung Kim wrote: > > On Tue, Sep 13, 2016 at 06:57:10PM +0300, Michael S. Tsirkin wrote: > > > On Sat, Aug 20, 2016 at 05:07:43PM +0900, Namhyung Kim wrote: > > > > + > > > > +/* the i

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-11-10 Thread Michael S. Tsirkin
On Fri, Sep 16, 2016 at 07:05:47PM +0900, Namhyung Kim wrote: > On Tue, Sep 13, 2016 at 06:57:10PM +0300, Michael S. Tsirkin wrote: > > On Sat, Aug 20, 2016 at 05:07:43PM +0900, Namhyung Kim wrote: > > > Add virtio pstore device to allow kernel log files saved on the host. > > > It will save the lo

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-09-22 Thread Namhyung Kim
On Thu, Sep 22, 2016 at 01:23:16PM +0100, Stefan Hajnoczi wrote: > On Sun, Sep 04, 2016 at 11:38:59PM +0900, Namhyung Kim wrote: > > +static void virtio_pstore_handle_io(VirtIODevice *vdev, VirtQueue *vq) > > +{ > > +VirtIOPstore *s = VIRTIO_PSTORE(vdev); > > +VirtQueueElement *elem; > > +

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-09-22 Thread Stefan Hajnoczi
On Sun, Sep 04, 2016 at 11:38:59PM +0900, Namhyung Kim wrote: > +static void virtio_pstore_handle_io(VirtIODevice *vdev, VirtQueue *vq) > +{ > +VirtIOPstore *s = VIRTIO_PSTORE(vdev); > +VirtQueueElement *elem; > +struct virtio_pstore_req req; > +struct virtio_pstore_res res; > +

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-09-16 Thread Namhyung Kim
On Tue, Sep 13, 2016 at 06:57:10PM +0300, Michael S. Tsirkin wrote: > On Sat, Aug 20, 2016 at 05:07:43PM +0900, Namhyung Kim wrote: > > Add virtio pstore device to allow kernel log files saved on the host. > > It will save the log files on the directory given by pstore device > > option. > > > >

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-09-13 Thread Michael S. Tsirkin
On Sat, Aug 20, 2016 at 05:07:43PM +0900, Namhyung Kim wrote: > Add virtio pstore device to allow kernel log files saved on the host. > It will save the log files on the directory given by pstore device > option. > > $ qemu-system-x86_64 -device virtio-pstore,directory=dir-xx ... > > (guest)

[PATCH 2/3] qemu: Implement virtio-pstore device

2016-09-04 Thread Namhyung Kim
From: Namhyung Kim Add virtio pstore device to allow kernel log files saved on the host. It will save the log files on the directory given by pstore device option. $ qemu-system-x86_64 -device virtio-pstore,directory=dir-xx ... (guest) # echo c > /proc/sysrq-trigger $ ls dir-xx dmesg-1

[PATCH 2/3] qemu: Implement virtio-pstore device

2016-08-31 Thread Namhyung Kim
From: Namhyung Kim Add virtio pstore device to allow kernel log files saved on the host. It will save the log files on the directory given by pstore device option. $ qemu-system-x86_64 -device virtio-pstore,directory=dir-xx ... (guest) # echo c > /proc/sysrq-trigger $ ls dir-xx dmesg-1

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-08-26 Thread Daniel P. Berrange
On Fri, Aug 26, 2016 at 01:48:40PM +0900, Namhyung Kim wrote: > Hi Daniel, > > On Wed, Aug 24, 2016 at 06:00:51PM -0400, Daniel P. Berrange wrote: > > > +fd = open(filename, O_RDONLY); > > > +if (fd < 0) { > > > +error_report("cannot open %s", filename); > > > +goto out; >

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-08-25 Thread Namhyung Kim
Hi Daniel, On Wed, Aug 24, 2016 at 06:00:51PM -0400, Daniel P. Berrange wrote: > > > diff --git a/hw/virtio/virtio-pstore.c b/hw/virtio/virtio-pstore.c > > new file mode 100644 > > index 000..b8fb4be > > --- /dev/null > > +++ b/hw/virtio/virtio-pstore.c > > @@ -0,0 +1,699 @@ > > +/* > > + * V

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-08-24 Thread Daniel P. Berrange
> diff --git a/hw/virtio/virtio-pstore.c b/hw/virtio/virtio-pstore.c > new file mode 100644 > index 000..b8fb4be > --- /dev/null > +++ b/hw/virtio/virtio-pstore.c > @@ -0,0 +1,699 @@ > +/* > + * Virtio Pstore Device > + * > + * Copyright (C) 2016 LG Electronics > + * > + * Authors: > + * Nam

[PATCH 2/3] qemu: Implement virtio-pstore device

2016-08-20 Thread Namhyung Kim
Add virtio pstore device to allow kernel log files saved on the host. It will save the log files on the directory given by pstore device option. $ qemu-system-x86_64 -device virtio-pstore,directory=dir-xx ... (guest) # echo c > /proc/sysrq-trigger $ ls dir-xx dmesg-1.enc.z dmesg-2.enc.z

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-07-20 Thread Namhyung Kim
On Wed, Jul 20, 2016 at 09:29:06AM +0100, Stefan Hajnoczi wrote: > On Mon, Jul 18, 2016 at 11:21:18PM +0900, Namhyung Kim wrote: > > On Mon, Jul 18, 2016 at 11:03:53AM +0100, Stefan Hajnoczi wrote: > > > On Mon, Jul 18, 2016 at 01:37:40PM +0900, Namhyung Kim wrote: > > > > From: Namhyung Kim > > >

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-07-20 Thread Namhyung Kim
On Wed, Jul 20, 2016 at 09:21:08AM +0100, Stefan Hajnoczi wrote: > On Wed, Jul 20, 2016 at 12:48:39AM +0900, Namhyung Kim wrote: > > Hello, > > > > On Mon, Jul 18, 2016 at 11:03:53AM +0100, Stefan Hajnoczi wrote: > > > On Mon, Jul 18, 2016 at 01:37:40PM +0900, Namhyung Kim wrote: > > > > +static v

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-07-20 Thread Stefan Hajnoczi
On Mon, Jul 18, 2016 at 11:21:18PM +0900, Namhyung Kim wrote: > On Mon, Jul 18, 2016 at 11:03:53AM +0100, Stefan Hajnoczi wrote: > > On Mon, Jul 18, 2016 at 01:37:40PM +0900, Namhyung Kim wrote: > > > From: Namhyung Kim > > > > > > Add virtio pstore device to allow kernel log files saved on the h

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-07-20 Thread Stefan Hajnoczi
On Wed, Jul 20, 2016 at 12:48:39AM +0900, Namhyung Kim wrote: > Hello, > > On Mon, Jul 18, 2016 at 11:03:53AM +0100, Stefan Hajnoczi wrote: > > On Mon, Jul 18, 2016 at 01:37:40PM +0900, Namhyung Kim wrote: > > > +static void virtio_pstore_handle_io(VirtIODevice *vdev, VirtQueue *vq) > > > +{ > > >

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-07-19 Thread Namhyung Kim
Hello, On Mon, Jul 18, 2016 at 11:03:53AM +0100, Stefan Hajnoczi wrote: > On Mon, Jul 18, 2016 at 01:37:40PM +0900, Namhyung Kim wrote: > > +static void virtio_pstore_handle_io(VirtIODevice *vdev, VirtQueue *vq) > > +{ > > +VirtIOPstore *s = VIRTIO_PSTORE(vdev); > > +VirtQueueElement *elem

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-07-18 Thread Namhyung Kim
Hello, On Mon, Jul 18, 2016 at 11:03:53AM +0100, Stefan Hajnoczi wrote: > On Mon, Jul 18, 2016 at 01:37:40PM +0900, Namhyung Kim wrote: > > From: Namhyung Kim > > > > Add virtio pstore device to allow kernel log files saved on the host. > > It will save the log files on the directory given by ps

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-07-18 Thread Namhyung Kim
Hello, On Mon, Jul 18, 2016 at 09:28:42AM +0200, Christian Borntraeger wrote: > On 07/18/2016 06:37 AM, Namhyung Kim wrote: > > Can you do the virtio-mmio and virtio-ccw plumbing as well, or > do you need help with that? Any help would be greatly appreciated! Thanks, Namhyung > > [...] > > d

[PATCH 2/3] qemu: Implement virtio-pstore device

2016-07-18 Thread Namhyung Kim
From: Namhyung Kim Add virtio pstore device to allow kernel log files saved on the host. It will save the log files on the directory given by pstore device option. $ qemu-system-x86_64 -device virtio-pstore,directory=dir-xx ... (guest) # echo c > /proc/sysrq-trigger $ ls dir-xx dmesg-0

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-07-18 Thread Stefan Hajnoczi
On Mon, Jul 18, 2016 at 01:37:40PM +0900, Namhyung Kim wrote: > From: Namhyung Kim > > Add virtio pstore device to allow kernel log files saved on the host. > It will save the log files on the directory given by pstore device > option. > > $ qemu-system-x86_64 -device virtio-pstore,directory=d

Re: [PATCH 2/3] qemu: Implement virtio-pstore device

2016-07-18 Thread Christian Borntraeger
On 07/18/2016 06:37 AM, Namhyung Kim wrote: Can you do the virtio-mmio and virtio-ccw plumbing as well, or do you need help with that? [...] > diff --git a/hw/virtio/virtio-pci.c b/hw/virtio/virtio-pci.c > index 2b34b43..8281b80 100644 > --- a/hw/virtio/virtio-pci.c > +++ b/hw/virtio/virtio-pci.c