On Mon, 31 Mar 2014 18:26:54 +0200
Andreas Färber <afaer...@suse.de> wrote:
> Am 31.03.2014 18:24, schrieb Alexander Graf:
> > On 03/28/2014 11:57 AM, Greg Kurz wrote:
> >> From: Rusty Russell <ru...@rustcorp.com.au>
> >>
> >> This is based on a simpler patch by Anthony Liguouri, which only handled
> >> the vring accesses. We also need some drivers to access these helpers,
> >> eg. for data which contains headers.
> >>
> >> Signed-off-by: Rusty Russell <ru...@rustcorp.com.au>
> >> [ ldq_phys() API change,
> >> use per-device needs_byteswap flag,
> >> Greg Kurz <gk...@linux.vnet.ibm.com> ]
> >> Signed-off-by: Greg Kurz <gk...@linux.vnet.ibm.com>
> >> ---
> >> hw/virtio/virtio.c | 93
> >> ++++++++++++++++++++++++++++++----------------------
> >> 1 file changed, 53 insertions(+), 40 deletions(-)
> >>
> >> diff --git a/hw/virtio/virtio.c b/hw/virtio/virtio.c
> >> index 24b565f..1877b46 100644
> >> --- a/hw/virtio/virtio.c
> >> +++ b/hw/virtio/virtio.c
> >> @@ -102,53 +102,57 @@ static void virtqueue_init(VirtQueue *vq)
> >> vq->vring.align);
> >> }
> >> -static inline uint64_t vring_desc_addr(hwaddr desc_pa, int i)
> >> +static inline uint64_t vring_desc_addr(hwaddr desc_pa, int i,
> >> + struct VirtIODevice *vdev)
> >
> > The logical ordering for helper is usually to have the device as the
> > first parameter (it's basically the self object). Could you please order
> > it accordingly?
>
> Also please note that QEMU Coding Style requests the use of typedefs,
> i.e. drop the "struct".
>
> Regards,
> Andreas
>
>
Oops my bad... I did not know about the HACKING file until your mail. :)
I will fixe that.
Thanks.
--
Gregory Kurz kurzg...@fr.ibm.com
gk...@linux.vnet.ibm.com
Software Engineer @ IBM/Meiosys http://www.ibm.com
Tel +33 (0)562 165 496
"Anarchy is about taking complete responsibility for yourself."
Alan Moore.
