From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from [140.186.70.92] (port=34730 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1PNwtu-0002qf-55 for qemu-devel@nongnu.org; Wed, 01 Dec 2010 19:19:55 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1PNwts-0000xy-WC for qemu-devel@nongnu.org; Wed, 01 Dec 2010 19:19:54 -0500 Received: from mail.linux-iscsi.org ([67.23.28.174]:42439 helo=linux-iscsi.org) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1PNwts-0000xp-Qu for qemu-devel@nongnu.org; Wed, 01 Dec 2010 19:19:52 -0500 Subject: Re: [Qemu-devel] [PATCH] Megasas HBA emulation and SCSI update v.2 From: "Nicholas A. Bellinger" In-Reply-To: <4CF66DEC.7010901@suse.de> References: <20101122101535.C3B7FF90B3@ochil.suse.de> <4CEA473F.5030806@suse.de> <1290591664.30138.216.camel@haakon2.linux-iscsi.org> <4CF6593A.1060109@suse.de> <4CF66DEC.7010901@suse.de> Content-Type: text/plain Date: Wed, 01 Dec 2010 16:14:13 -0800 Message-Id: <1291248853.17194.230.camel@haakon2.linux-iscsi.org> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: linux-iscsi-target-dev@googlegroups.com Cc: Kevin Wolf , stefanha@gmail.com, Paolo Bonzini , qemu-devel@nongnu.org, kraxel@redhat.com On Wed, 2010-12-01 at 16:46 +0100, Hannes Reinecke wrote: > On 12/01/2010 03:18 PM, Hannes Reinecke wrote: > > Hey Nic, > > > > On 11/24/2010 10:41 AM, Nicholas A. Bellinger wrote: > >> On Mon, 2010-11-22 at 11:34 +0100, Hannes Reinecke wrote: > > [ .. ] > >> > >> Hey Hannes, > >> > >> Just a heads up, I noticed that the latest v2 megasas w/ scsi-generic -> > >> TCM_loop appears to be broken on a Windows7 (Build 7600) guest, which > >> hangs during boot -> LUN scan with the following: > >> > >> truelife:/usr/src/qemu-kvm.git# ./x86_64-softmmu/qemu-system-x86_64 -m 512 -boot c ~/windows7.img \ > >> -drive if=none,id=mydisk1,file=/dev/sg4 -device megasas,id=raid > >> -device scsi-generic,bus=raid.0,scsi-id=1,drive=mydisk1 > >> > >> megasas: Using 80 sges, 1000 cmds, raid mode > >> megasas: Reset > >> megasas: Mapping MMIO region 0 at f2040000 > >> megasas: Mapping IO region 2 at 0000c200 > >> megasas: Mapping QUEUE region 3 at f2080000 > >> megasas: Mapping MMIO region 0 at f2040000 > >> megasas: Mapping IO region 2 at 0000c200 > >> megasas: Mapping QUEUE region 3 at f2080000 > >> megasas: Mapping MMIO region 0 at f2040000 > >> megasas: Mapping IO region 2 at 0000c200 > >> megasas: Mapping QUEUE region 3 at f2080000 > >> megasas: readl mmio 0xb0 > >> megasas: writel mmio 20: 7 > >> megasas: Reset > >> megasas: readl mmio 0x20 > >> megasas: writel mmio 40: 1ff9c041 > >> megasas: Received frame addr 1ff9c000 count 32 > >> megasas: MFI cmd 0 context 0 count 32 > >> megasas: Return new frame 0 cmd 0x7f7711654330 > >> megasas: Enqueue frame 0 count 32 context 0 tail 0 busy 1 > >> megasas: MFI init firmware: xfer len 0 pa 0 > >> megasas: MFI init firmware: queue at f000ff53f000e2c3 len -268370093 head f000ff53f000ff53 tail f000ff53f000ff53 > >> megasas: Complete frame context 0 > >> > > Okay, it looks as if I've fixed it up. > > Win7 32bit works now with my megasas.v3 tree. > > Curiously, Win7 64bit fails; it crashes at relative address > > 28F4, wherever that's supposed to be. > > And, of course, Windows Vista with newest driver from LSI fails, too > > :-(. > > Guess I need to do some more debugging here. > > > Hmpf. Using a new vista x86 image (build 6002) with SP2 preloaded > megasas works, too. > Dodgy build I had, apparently. > Thanks for the update.. After testing the lastest megasas.v3 HEAD at commit: * megasas.v3 978e61e megasas: Fixup PD query return value it appears that the same Win7 64-bit Build 7600 that is functioning with v0.12.5 windows7-megasas-working will now BSOD the guest. After further checking it appears that this is not megasas HBA specific, and is due to your tree being slightly more out of date than mine. ;) But the good news is that WinXP SP2 is now working via scsi-generic -> TCM_Loop in megasas.v3, and even w/o the original sync ioctl patch we required in v0.12.5 megasas code. Very excellent work Hannes! So, I will be merging the latest changes from megasas.v3 -> megasas-upstream-v1 shortly and retesting with 64-bit Build 7600. Best, --nab