Re: [Qemu-devel] [PATCH 3/6] virtio for QEMU

[Anthony Liguori]

Dor Laor wrote:
> > +static uint32_t vring_desc_len(VirtQueue *vq, unsigned int i)
> > +{
>
> Below there were place you did use offsetof(vq->vring.desc[i], len) so
> we better be consistent + its nicer
>
> > +    return ldl_phys(vq->vring.desc + i * sizeof(VRingDesc) + 
> > +		    offsetof(VRingDesc, len));
> > +}
> > +

Oh, this bit is different because you need to do offsetof(Type, member) 
and vq->vring.desc[i] is not a type.  It only works when you're doing an 
array with member[X].

Regards,

Anthony Liguori