On Tue, 18 Feb 2014 14:08:35 +0100 Cornelia Huck <[email protected]> wrote: > On Tue, 18 Feb 2014 13:38:54 +0100 > Greg Kurz <[email protected]> wrote: > > > From: Rusty Russell <[email protected]> > > > > 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. > > > > [ ldq_phys() API change, Greg Kurz <[email protected]> ] > > Signed-off-by: Rusty Russell <[email protected]> > > Signed-off-by: Greg Kurz <[email protected]> > > --- > > hw/virtio/virtio.c | 32 ++++++++++++++++---------------- > > 1 file changed, 16 insertions(+), 16 deletions(-) > > > > > static inline void vring_used_flags_unset_bit(VirtQueue *vq, int mask) > > { > > hwaddr pa; > > pa = vq->vring.used + offsetof(VRingUsed, flags); > > - stw_phys(&address_space_memory, > > - pa, lduw_phys(&address_space_memory, pa) & ~mask); > > + virtio_stw_phys(&address_space_memory, > > + pa, lduw_phys(&address_space_memory, pa) & ~mask); > > } > > This needs to be virtio_lduw_phys(), no?
Oops yes it should be... my mistake. :) -- Gregory Kurz [email protected] [email protected] Software Engineer @ IBM/Meiosys http://www.ibm.com Tel +33 (0)562 165 496 "Anarchy is about taking complete responsibility for yourself." Alan Moore.
