[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v8 00/20] virtio endian-ambivalent target
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v8 00/20] virtio endian-ambivalent target |
Date: |
Wed, 18 Jun 2014 18:38:14 +0800 |
User-agent: |
Mutt/1.5.23 (2014-03-12) |
On Tue, Jun 17, 2014 at 09:40:19AM +0200, Alexander Graf wrote:
>
> On 17.06.14 09:36, Stefan Hajnoczi wrote:
> >On Fri, Jun 13, 2014 at 01:18:00PM +0200, Greg Kurz wrote:
> >>This version merges the changes requested during the v7 review, remarks from
> >>ppc64 dump support review (yes, we talked about virtio there) and the work
> >>on
> >>virtio subsections migration. Also two new patches have been added:
> >>- patch #1 is a preliminary fix for virtio-serial posted by Alexander Graf
> >>- patch #9 prepares the work on the virtio_is_big_endian() helper
> >>
> >>The most significant changes are:
> >>- introduction of a new CPU method for virtio
> >>- endianness is taken from CPU that resets the device
> >>- fastpath virtio memory accessors for fixed endian targets
> >>- VMState based virtio subsections (compatibility friendly)
> >I'm surprised it's not enough for the virtio device to have an
> >endianness field (big/little). It seems these patches make endianness
> >depend on the CPUState through which the device is being accessed.
> >
> >Can you explain why it's necessary to check the CPUState?
>
> They only check CPUState at the point in time of reset, as that's the only
> case where we can derive the implicit endian configuration from :).
What bothers me is that real hardware can't do this. Given that VIRTIO
1.0 is always little-endian I guess this is just a temporary hack for
ppc little-endian. Would be nice to add a comment so it's clear why
this approach is being taken instead of a cleaner solution.
Stefan
pgpbMONSDn9vd.pgp
Description: PGP signature
- [Qemu-devel] [PATCH v8 16/20] virtio-blk: use virtio wrappers to access headers, (continued)
- [Qemu-devel] [PATCH v8 16/20] virtio-blk: use virtio wrappers to access headers, Greg Kurz, 2014/06/13
- [Qemu-devel] [PATCH v8 17/20] virtio-scsi: use virtio wrappers to access headers, Greg Kurz, 2014/06/13
- [Qemu-devel] [PATCH v8 18/20] virtio-serial-bus: use virtio wrappers to access headers, Greg Kurz, 2014/06/13
- [Qemu-devel] [PATCH v8 19/20] virtio-9p: use virtio wrappers to access headers, Greg Kurz, 2014/06/13
- [Qemu-devel] [PATCH v8 20/20] target-ppc: enable virtio endian ambivalent support, Greg Kurz, 2014/06/13
- Re: [Qemu-devel] [PATCH v8 00/20] virtio endian-ambivalent target, Alexander Graf, 2014/06/13
- Re: [Qemu-devel] [PATCH v8 00/20] virtio endian-ambivalent target, Stefan Hajnoczi, 2014/06/17
- Re: [Qemu-devel] [PATCH v8 00/20] virtio endian-ambivalent target, Stefan Hajnoczi, 2014/06/19
- Re: [Qemu-devel] [PATCH v8 00/20] virtio endian-ambivalent target, Peter Maydell, 2014/06/18
- Re: [Qemu-devel] [PATCH v8 00/20] virtio endian-ambivalent target, Michael S. Tsirkin, 2014/06/18
- Re: [Qemu-devel] [PATCH v8 00/20] virtio endian-ambivalent target, Greg Kurz, 2014/06/18
- Re: [Qemu-devel] [PATCH v8 00/20] virtio endian-ambivalent target, Michael S. Tsirkin, 2014/06/18
- Re: [Qemu-devel] [PATCH v8 00/20] virtio endian-ambivalent target, Peter Maydell, 2014/06/18