From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [Qemu-devel] [PATCH][v7] megasas: LSI Megaraid SAS emulation Date: Tue, 19 Jul 2011 18:08:43 +0200 Message-ID: <20110719160843.GA3414@lst.de> References: <1311074364-3489-1-git-send-email-hare@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: qemu-devel@nongnu.org, Paolo Bonzini , Stefan Haynoczi , kvm@vger.kernel.org, Alexander Graf To: Hannes Reinecke Return-path: Received: from verein.lst.de ([213.95.11.211]:48535 "EHLO newverein.lst.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751871Ab1GSQIo (ORCPT ); Tue, 19 Jul 2011 12:08:44 -0400 Content-Disposition: inline In-Reply-To: <1311074364-3489-1-git-send-email-hare@suse.de> Sender: kvm-owner@vger.kernel.org List-ID: I've mentioned this a few times before, but just to make sure it's not lost: This is a really bad idea for adding to qemu. It's not a controller that actually speaks a plain SCSI protocol to disks, but a RAID controller, that has it's own command set for data plan operation, and minimal support for a few SCSI CDBs around it. I also supports passthrough channels for CDROMs and other periphals, but using those with disks is not recommended by LSI, thus explicitly disabled in most drivers and most certainly not tested. So it fits the qemu model, especially if using scsi-generic undeneath pretty badly and I don't think it helps us in practice.