From: Paolo Bonzini <pbonzini@redhat.com>
To: Stefan Hajnoczi <stefanha@gmail.com>
Cc: qemu-devel@nongnu.org, Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
Subject: Re: [Qemu-devel] [PATCH 11/15] virtio-scsi: Add virtio-scsi stub device
Date: Fri, 10 Feb 2012 13:46:20 +0100 [thread overview]
Message-ID: <4F35119C.3030404@redhat.com> (raw)
In-Reply-To: <CAJSP0QX-t7Jy-oijnqNv5-RmwicMBmKEYmVFYrehbLHZR+TAsw@mail.gmail.com>
On 02/10/2012 01:41 PM, Stefan Hajnoczi wrote:
> At this early stage we still have the option of designing the device
> to be little- or big-endian. I'm in favour of a fixed endianness even
> though existing virtio devices tend to use guest-endian.
>
> What do you think?
Interesting idea, but I'd say no for three reasons:
1) The virtqueue would remain in guest-endianness.
2) Even if the result is better, it is still different from every other
device, so right now it would be a one-off exception (and it would
likely remain so; it's not like there are new virtio devices every week :)).
3) I'm afraid of breaking something else, for example pSeries virtio is
doing complicated things that I don't really understand to split the I/O
BAR in two parts, the first in PCI endianness and the second in native
endianness.
Paolo
next prev parent reply other threads:[~2012-02-10 12:46 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-16 16:30 [Qemu-devel] [PATCH v2 00/15] SCSI s/g + SCSI migration + virtio-scsi Paolo Bonzini
2012-01-16 16:30 ` [Qemu-devel] [PATCH 01/15] dma-helpers: make QEMUSGList target independent Paolo Bonzini
2012-01-16 16:30 ` [Qemu-devel] [PATCH 02/15] dma-helpers: add dma_buf_read and dma_buf_write Paolo Bonzini
2012-01-16 16:30 ` [Qemu-devel] [PATCH 03/15] dma-helpers: add accounting wrappers Paolo Bonzini
2012-01-16 16:30 ` [Qemu-devel] [PATCH 04/15] ahci: use new DMA helpers Paolo Bonzini
2012-01-16 16:30 ` [Qemu-devel] [PATCH 05/15] scsi: pass residual amount to command_complete Paolo Bonzini
2012-02-10 11:44 ` Stefan Hajnoczi
2012-02-10 11:58 ` Paolo Bonzini
2012-01-16 16:30 ` [Qemu-devel] [PATCH 06/15] scsi: add scatter/gather functionality Paolo Bonzini
2012-01-16 16:30 ` [Qemu-devel] [PATCH 07/15] scsi-disk: enable " Paolo Bonzini
2012-01-16 16:30 ` [Qemu-devel] [PATCH 08/15] scsi: add SCSIDevice vmstate definitions Paolo Bonzini
2012-01-16 16:30 ` [Qemu-devel] [PATCH 09/15] scsi-generic: add migration support Paolo Bonzini
2012-01-16 16:30 ` [Qemu-devel] [PATCH 10/15] scsi-disk: " Paolo Bonzini
2012-01-16 16:30 ` [Qemu-devel] [PATCH 11/15] virtio-scsi: Add virtio-scsi stub device Paolo Bonzini
2012-02-10 12:41 ` Stefan Hajnoczi
2012-02-10 12:46 ` Paolo Bonzini [this message]
2012-01-16 16:30 ` [Qemu-devel] [PATCH 12/15] virtio-scsi: Add basic request processing infrastructure Paolo Bonzini
2012-01-16 16:30 ` [Qemu-devel] [PATCH 13/15] virtio-scsi: add basic SCSI bus operation Paolo Bonzini
2012-01-16 16:30 ` [Qemu-devel] [PATCH 14/15] virtio-scsi: process control queue requests Paolo Bonzini
2012-01-16 16:30 ` [Qemu-devel] [PATCH 15/15] virtio-scsi: add migration support Paolo Bonzini
2012-01-18 7:39 ` [Qemu-devel] [PATCH v2 00/15] SCSI s/g + SCSI migration + virtio-scsi Hu Tao
2012-01-19 15:53 ` Paolo Bonzini
2012-01-23 13:36 ` Paolo Bonzini
2012-01-30 9:33 ` Hu Tao
2012-01-31 12:36 ` Paolo Bonzini
2012-02-09 5:46 ` Hu Tao
2012-02-09 7:13 ` Paolo Bonzini
2012-02-09 8:08 ` Hu Tao
2012-02-10 14:01 ` Stefan Hajnoczi
2012-02-10 14:15 ` Paolo Bonzini
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4F35119C.3030404@redhat.com \
--to=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@gmail.com \
--cc=stefanha@linux.vnet.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.