From: Hannes Reinecke <hare@suse.de>
To: qemu-devel@nongnu.org
Cc: Paolo Bonzini <pbonzini@redhat.com>,
Hannes Reinecke <hare@suse.de>,
Stefan Haynoczi <stefanha@linux.vnet.ibm.com>,
kvm@vger.kernel.org, Alexander Graf <agraf@suse.de>
Subject: [Qemu-devel] [PATCH 0/5][v6] Megasas HBA emulation
Date: Tue, 5 Jul 2011 13:03:30 +0200 [thread overview]
Message-ID: <1309863815-28236-1-git-send-email-hare@suse.de> (raw)
Hi all,
as Alex Graf reminded me the driver needed some more bugfixing
to be done. I've found some issues and also moved the megasas
emulation over to the new trace infrastructure.
Driver works for me now and a full installation of
openSUSE-12.1 works perfectly.
I've also included the fixes suggested by Stefan Hajnoczi.
And during debugging I've found two minor issues in scsi_disk.c
Changes since v5:
- scsi-disk: Fixup debugging statement
A debugging statement wasn't converted. Do so now.
- scsi-disk: Mask out serial number EVPD
The 'serial' parameter to scsi-disk is optional. So if it's
not set we should mask it out in the list of supported EVPD
pages and not return '0' here.
- megasas: Use tracing infrastructure instead of DPRINTF
- megasas: Use new PCI infrastructure
- megasas: Check for iovec mapping failure
cpu_map_physical_memory() might fail, so we need to check for
it when mapping iovecs.
- megasas: Trace scsi buffer overflow
The transfer length as specified in the SCSI command might
disagree with the length of the iovec. We should be tracing
these issues.
- megasas: Reset frames after init firmware
When receiving an INIT FIRMWARE command we need reset all
frames, otherwise some frames might point to invalid memory.
Chances since v4:
- iov: Update parameter usage in iov_(to|from)_buf()
Updated description for the first patch and clarified the usage
Renamed arguments for io_XXX for clarification
- scsi: Add 'hba_private' to SCSIRequest
Kept 'tag' for tracing and just add 'hba_private' as an
additional field as per request from Paolo
- megasas: checkpatch.pl fixes and update to work with the
changed interface in scsi_req_new(). Also included the
suggested fixes from Alex.
Hannes Reinecke (5):
iov: Update parameter usage in iov_(to|from)_buf()
scsi: Add 'hba_private' to SCSIRequest
scsi-disk: Fixup debugging statement
scsi-disk: Mask out serial number EVPD
megasas: LSI Megaraid SAS emulation
Makefile.objs | 1 +
default-configs/pci.mak | 1 +
hw/esp.c | 2 +-
hw/lsi53c895a.c | 22 +-
hw/megasas.c | 1950 +++++++++++++++++++++++++++++++++++++++++++++++
hw/mfi.h | 1197 +++++++++++++++++++++++++++++
hw/pci_ids.h | 3 +-
hw/scsi-bus.c | 9 +-
hw/scsi-disk.c | 21 +-
hw/scsi-generic.c | 5 +-
hw/scsi.h | 10 +-
hw/spapr_vscsi.c | 29 +-
hw/usb-msd.c | 9 +-
hw/virtio-net.c | 2 +-
hw/virtio-serial-bus.c | 2 +-
iov.c | 49 +-
iov.h | 10 +-
trace-events | 71 ++
18 files changed, 3306 insertions(+), 87 deletions(-)
create mode 100644 hw/megasas.c
create mode 100644 hw/mfi.h
--
1.7.3.4
next reply other threads:[~2011-07-05 11:04 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-05 11:03 Hannes Reinecke [this message]
2011-07-05 11:03 ` [Qemu-devel] [PATCH 1/5] iov: Update parameter usage in iov_(to|from)_buf() Hannes Reinecke
2011-07-05 11:03 ` [Qemu-devel] [PATCH 2/5] scsi: Add 'hba_private' to SCSIRequest Hannes Reinecke
2011-07-05 11:03 ` [Qemu-devel] [PATCH 3/5] scsi-disk: Fixup debugging statement Hannes Reinecke
2011-07-05 11:03 ` [Qemu-devel] [PATCH 4/5] scsi-disk: Mask out serial number EVPD Hannes Reinecke
2011-07-05 11:03 ` [Qemu-devel] [PATCH 5/5] megasas: LSI Megaraid SAS emulation Hannes Reinecke
2011-07-05 13:06 ` Alexander Graf
2011-07-05 13:38 ` Alexander Graf
2011-07-05 13:59 ` Paolo Bonzini
2011-07-05 14:05 ` Alexander Graf
2011-07-05 15:21 ` Stefan Hajnoczi
2011-07-06 6:20 ` Hannes Reinecke
2011-07-06 8:39 ` Paolo Bonzini
2011-07-05 11:06 ` [Qemu-devel] [PATCH 4/5] scsi-disk: Mask out serial number EVPD Paolo Bonzini
2011-07-05 11:05 ` [Qemu-devel] [PATCH 3/5] scsi-disk: Fixup debugging statement Paolo Bonzini
2011-07-05 13:01 ` [Qemu-devel] [PATCH 0/5][v6] Megasas HBA emulation Alexander Graf
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=1309863815-28236-1-git-send-email-hare@suse.de \
--to=hare@suse.de \
--cc=agraf@suse.de \
--cc=kvm@vger.kernel.org \
--cc=pbonzini@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@linux.vnet.ibm.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).