On 19.05.14 14:45, Greg Kurz wrote:
On Mon, 19 May 2014 14:02:39 +0200
Alexander Graf <ag...@suse.de> wrote:

On 19.05.14 10:38, Greg Kurz wrote:
Hi,

This patch set tries to address comments from the initial
review. For this round, I have focused on two changes:
- as suggested by Andreas, we now call the device specific
    code from the generic code to ease the implementation of
    future devices. This is achieved with the addition of
    load/save methods to VirtioDeviceClass.
- virtio subsections now implement a "needed" concept with
    the same semantics as in the VMState code.

I haven't looked at compat mode issues yet, but it is
on my TODO list.
If you fix up the comments to be either

    /*
     * foo
     */

or

    /* foo */

style, not

    /* foo
     */

then you get my


Acked-by: Alexander Graf <ag...@suse.de>


Alex

I'll certainly do that then ! :)

BTW, the faulty comments are in patch 8/8 that I was
sending for informational purpose only as it is part
of the bi-endian virtio serie. Should I merge the
two patch sets at some point ?


Either way works for me, let's wait for virtio and migration specialists to comment first ;)


Alex



Reply via email to