[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage
From: |
Rusty Russell |
Subject: |
[Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage |
Date: |
Wed, 30 Sep 2009 21:30:32 +0930 |
User-agent: |
KMail/1.11.2 (Linux/2.6.28-15-generic; KDE/4.2.2; i686; ; ) |
On Wed, 30 Sep 2009 11:47:00 am Anthony Liguori wrote:
> Rusty Russell wrote:
> > On Wed, 30 Sep 2009 06:25:39 am Anthony Liguori wrote:
> >
> >> virtio-blk isn't an ATA device so why pretend like it is?
...
> That's an argument to implement the HDIO_GET_IDENTITY ioctl in the Linux
> virtio-blk driver. That's a totally reasonable thing to do.
Your second sentence contradicts your first here. If it's reasonable to
imitate ATA in Linux, it's reasonable to imitate ATA in virtio.
Using an existing standard should always be the default; one should need a
compelling reason *not* to do so. In practice, (1) that code already exists
in kvm, and (2) it makes the Linux implementation trivial.
So, what are the real arguments?
(1) Some fields are redundant (eg. geometry) with existing fields already.
(2) Future fields will have to decide whether to use ATA IDENTIFY or normal
features,
(3) We don't know enough about ATA IDENTIFY to do a decent job of filling
it in.
I can buy (1), but I still have to deal with (2) and (3) if they're inside
the virtio_blk driver anyway.
Anything else?
Rusty.
- [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage, (continued)
- [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage, Rusty Russell, 2009/09/29
- [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage, Anthony Liguori, 2009/09/29
- [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage, Rusty Russell, 2009/09/29
- Re: [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage, Jamie Lokier, 2009/09/29
- [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage, john cooper, 2009/09/29
- [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage, Anthony Liguori, 2009/09/29
- [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage, Rusty Russell, 2009/09/29
- [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage, Anthony Liguori, 2009/09/29
- [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage,
Rusty Russell <=
- Re: [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage, Jamie Lokier, 2009/09/30
- Re: [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage, Paul Brook, 2009/09/30
- [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage, Anthony Liguori, 2009/09/29
- [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage, Avi Kivity, 2009/09/29
- [Qemu-devel] Re: [PATCH 0/2] fix virtio_blk serial pci config breakage, Anthony Liguori, 2009/09/29
- [Qemu-devel] [PATCH 1/2] fix virtio_blk serial pci config breakage, john cooper, 2009/09/29
- [Qemu-devel] Re: [PATCH 1/2] fix virtio_blk serial pci config breakage, Michael S. Tsirkin, 2009/09/29
- [Qemu-devel] [PATCH 2/2] fix virtio_blk serial pci config breakage, john cooper, 2009/09/29
- [Qemu-devel] Re: [PATCH 2/2] fix virtio_blk serial pci config breakage, Michael S. Tsirkin, 2009/09/29
- [Qemu-devel] Re: [PATCH 2/2] fix virtio_blk serial pci config breakage, Rusty Russell, 2009/09/29