All of lore.kernel.org
 help / color / mirror / Atom feed
From: Hannes Reinecke <hare@suse.de>
To: "Nicholas A. Bellinger" <nab@linux-iscsi.org>
Cc: Paul Brook <paul@codesourcery.com>,
	Jan Kiszka <jan.kiszka@siemens.com>,
	Kevin Wolf <kwolf@redhat.com>, Gerd Hoffmann <kraxel@redhat.com>,
	FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>,
	Matthew Wilcox <willy@linux.intel.com>,
	kvm-devel <kvm@vger.kernel.org>,
	qemu-devel <qemu-devel@nongnu.org>
Subject: Re: [PATCH] lsi53c895a: Add support for OS/2 Warp SYM8XX.ADD driver
Date: Thu, 30 Sep 2010 15:23:11 +0200	[thread overview]
Message-ID: <4CA48F3F.7070509@suse.de> (raw)
In-Reply-To: <1285823236-6661-1-git-send-email-nab@linux-iscsi.org>

Nicholas A. Bellinger wrote:
> From: Nicholas Bellinger <nab@linux-iscsi.org>
> 
> Greetings Paul, Jan, Kevin and co,
> 
> This series is against my v0.12.5 qemu-kvm.git that contains QEMU SCSI layer
> SGL passthrough from Gerd Hoffman, 8708EM2 MegaSas emulation from Dr. Hannes
> Reinecke, and well as my own hw/scsi-bsg.c support.  This tree is located here:
> 
> http://git.kernel.org/?p=virt/kvm/nab/qemu-kvm.git;a=summary
> 
> This first patch adds a missing qdev->reset() NOP caller in hw/scsi-generic.c that
> is now expected by lsi53c895a.c in >= v0.12.5 code.  You will want to apply this to
> all >= v0.12.5 QEMU trees so scsi-generic does not segfault with lsi53c895a expecting
> a valid qdev->reset().
> 
> The second item is a bit more exotic.. 8-)  So I have been thinking about how to get
> qemu-kvm.git scsi-generic <-> TCM_Loop to function with OS/2 Warp v4 (SP15) in guest
> for a while now, and I am happy to report that after sending some time in the last weeks
> getting OS/2 setup (hey, it has been +13 years) and finding a functioning sym53c895a
> driver, and finally finding a working SYM8XX.ADD and being able to fill in missing
> informational registers and adding a workaround to fix a bogus Destination ID register
> WRITE from the now +10 year old SYM8XX.ADD driver code.
> 
Hey, and while you're at it:
The lsi53c895a emulation is missing support for 'abort' and 'device
reset' TMFs; newer Linux kernel have the habit of sending them
accordingly. Should be fairly straightforward, only I gave up on it
after reading the first 10 odd pages of the manual ...

Cheers,

Hannes
-- 
Dr. Hannes Reinecke		      zSeries & Storage
hare@suse.de			      +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Markus Rex, HRB 16746 (AG Nürnberg)

WARNING: multiple messages have this Message-ID (diff)
From: Hannes Reinecke <hare@suse.de>
To: "Nicholas A. Bellinger" <nab@linux-iscsi.org>
Cc: Kevin Wolf <kwolf@redhat.com>, kvm-devel <kvm@vger.kernel.org>,
	Jan Kiszka <jan.kiszka@siemens.com>,
	qemu-devel <qemu-devel@nongnu.org>,
	FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>,
	Paul Brook <paul@codesourcery.com>,
	Matthew Wilcox <willy@linux.intel.com>,
	Gerd Hoffmann <kraxel@redhat.com>
Subject: [Qemu-devel] Re: [PATCH] lsi53c895a: Add support for OS/2 Warp SYM8XX.ADD driver
Date: Thu, 30 Sep 2010 15:23:11 +0200	[thread overview]
Message-ID: <4CA48F3F.7070509@suse.de> (raw)
In-Reply-To: <1285823236-6661-1-git-send-email-nab@linux-iscsi.org>

Nicholas A. Bellinger wrote:
> From: Nicholas Bellinger <nab@linux-iscsi.org>
> 
> Greetings Paul, Jan, Kevin and co,
> 
> This series is against my v0.12.5 qemu-kvm.git that contains QEMU SCSI layer
> SGL passthrough from Gerd Hoffman, 8708EM2 MegaSas emulation from Dr. Hannes
> Reinecke, and well as my own hw/scsi-bsg.c support.  This tree is located here:
> 
> http://git.kernel.org/?p=virt/kvm/nab/qemu-kvm.git;a=summary
> 
> This first patch adds a missing qdev->reset() NOP caller in hw/scsi-generic.c that
> is now expected by lsi53c895a.c in >= v0.12.5 code.  You will want to apply this to
> all >= v0.12.5 QEMU trees so scsi-generic does not segfault with lsi53c895a expecting
> a valid qdev->reset().
> 
> The second item is a bit more exotic.. 8-)  So I have been thinking about how to get
> qemu-kvm.git scsi-generic <-> TCM_Loop to function with OS/2 Warp v4 (SP15) in guest
> for a while now, and I am happy to report that after sending some time in the last weeks
> getting OS/2 setup (hey, it has been +13 years) and finding a functioning sym53c895a
> driver, and finally finding a working SYM8XX.ADD and being able to fill in missing
> informational registers and adding a workaround to fix a bogus Destination ID register
> WRITE from the now +10 year old SYM8XX.ADD driver code.
> 
Hey, and while you're at it:
The lsi53c895a emulation is missing support for 'abort' and 'device
reset' TMFs; newer Linux kernel have the habit of sending them
accordingly. Should be fairly straightforward, only I gave up on it
after reading the first 10 odd pages of the manual ...

Cheers,

Hannes
-- 
Dr. Hannes Reinecke		      zSeries & Storage
hare@suse.de			      +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: Markus Rex, HRB 16746 (AG Nürnberg)

  reply	other threads:[~2010-09-30 13:23 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-30  5:07 [PATCH] lsi53c895a: Add support for OS/2 Warp SYM8XX.ADD driver Nicholas A. Bellinger
2010-09-30  5:07 ` [Qemu-devel] " Nicholas A. Bellinger
2010-09-30 13:23 ` Hannes Reinecke [this message]
2010-09-30 13:23   ` [Qemu-devel] " Hannes Reinecke
2010-09-30 21:44   ` Nicholas A. Bellinger
2010-09-30 21:44     ` [Qemu-devel] " Nicholas A. Bellinger
2010-10-07 10:59 ` Kevin Wolf
2010-10-07 10:59   ` [Qemu-devel] " Kevin Wolf

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=4CA48F3F.7070509@suse.de \
    --to=hare@suse.de \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=jan.kiszka@siemens.com \
    --cc=kraxel@redhat.com \
    --cc=kvm@vger.kernel.org \
    --cc=kwolf@redhat.com \
    --cc=nab@linux-iscsi.org \
    --cc=paul@codesourcery.com \
    --cc=qemu-devel@nongnu.org \
    --cc=willy@linux.intel.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.