Re: [PATCH] SCSI driver for VMware's virtual HBA - V4.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Christoph Hellwig wrote:
On Wed, Sep 09, 2009 at 05:12:26PM -0500, Anthony Liguori wrote:
Alok Kataria wrote:
I see your point, but the ring logic or the ABI that we use to
communicate between the hypervisor and guest is not shared between our
storage and network drivers. As a result, I don't see any benefit of
separating out this ring handling mechanism, on the contrary it might
just add some overhead of translating between various layers for our
SCSI driver.
But if you separate out the ring logic, it allows the scsi logic to be shared by other paravirtual device drivers. This is significant and important from a Linux point of view.

As someone who has been hacking on a virtio scsi prototype I don't think
it's a good idea.  The vmware driver is a horrible design and I don't
think it should be merged.

What are the issues with the design compared to how you're approaching virtio-scsi?

  Besides beeing a ugly driver and ABI we
really should not support this kind of closed protocol development.

I don't see how a VMM that doesn't share the source code for it's backends or doesn't implement standard ABIs is any different than the hundreds of hardware vendors that behave exactly the same way.

We haven't even been successful in getting the Xen folks to present their work on lkml before shipping it to their users. Why would we expect more from VMware if we're willing to merge the Xen stuff?

Regards,

Anthony Liguori


--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux