qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Gavin Shan <gwshan@linux.vnet.ibm.com>
To: qemu-ppc@nongnu.org
Cc: agraf@suse.de, aik@ozlabs.ru, qemu-devel@nongnu.org,
	Gavin Shan <gwshan@linux.vnet.ibm.com>,
	alex.williamson@redhat.com
Subject: [Qemu-devel] [PATCH v13 0/3] EEH Support for VFIO Devices
Date: Mon, 15 Dec 2014 11:15:05 +1100	[thread overview]
Message-ID: <1418602508-30845-1-git-send-email-gwshan@linux.vnet.ibm.com> (raw)

The series of patches adds support EEH for VFIO PCI devices on sPAPR platform.
It requires corresponding host kernel support, which was merged during 3.17
merge window. This patchset has been rebased to Alex Graf's QEMU repository:

   git://github.com/agraf/qemu.git (branch: ppc-next)

The implementations notes are below. Please consider for merging!

* RTAS calls are received in spapr_pci.c, sanity check is done there. RTAS
  handlers handle what they can. If there is something it cannot handle and
  sPAPRPHBClass::eeh_handler callback is defined, it is called.
* sPAPRPHBClass::eeh_handler is only implemented for VFIO now. It does ioctl()
  to the IOMMU container fd to complete the call. Error codes from that ioctl()
  are transferred back to the guest.

Changelog
=========
v10 -> v11:
        * Don't explicitly set struct vfio_eeh_pe_op::flags to 0.
        * rtas_handle_eeh_request() checks the validity of sPAPRPHBState and
          sPAPRPHBClass::eeh_handler.
        * The first level functions set the RTAS call return value.
        * Replace "if (...)" with "switch (...)".
v11 -> v12:
        * RTAS call returns RTAS_OUT_NO_ERRORS_FOUND on success because we
          don't support it yet.
        * Add one more patch from Alexey to make find_phb() and find_dev()
          phblic.
        * Rebase to Alex Graf's QEMU repository.
v12 -> v13:
	* Rebase to Alex Graf's QEMU repository ("ppc-next" branch).
        * Drop the patch for header file (vfio.h) changes, which was merged
          to QEMU repository by commit a9fd1654 ("linux-headers: update to 3.17-rc7").
	* Retested on Emulex adapter and EEH errors are recovered successfully.

Alexey Kardashevskiy (1):
  spapr_pci: Make find_phb()/find_dev() public

Gavin Shan (2):
  sPAPR: Implement EEH RTAS calls
  sPAPR: Implement sPAPRPHBClass::eeh_handler

 hw/misc/vfio.c              |   1 +
 hw/ppc/spapr_pci.c          | 268 ++++++++++++++++++++++++++++++++++++++++++--
 hw/ppc/spapr_pci_vfio.c     |  56 +++++++++
 include/hw/pci-host/spapr.h |  11 ++
 include/hw/ppc/spapr.h      |  43 ++++++-
 5 files changed, 366 insertions(+), 13 deletions(-)

-- 
1.8.3.2

             reply	other threads:[~2014-12-15  0:15 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-15  0:15 Gavin Shan [this message]
2014-12-15  0:15 ` [Qemu-devel] [PATCH v13 1/3] spapr_pci: Make find_phb()/find_dev() public Gavin Shan
2014-12-23  4:24   ` David Gibson
2014-12-25  3:18     ` Gavin Shan
2014-12-15  0:15 ` [Qemu-devel] [PATCH v13 2/3] sPAPR: Implement EEH RTAS calls Gavin Shan
2014-12-15 14:52   ` Alexander Graf
2014-12-15 23:08     ` Gavin Shan
2014-12-15 23:13       ` Alexander Graf
2014-12-15 23:29         ` Gavin Shan
2014-12-16  0:08           ` Alexander Graf
2014-12-16  0:31             ` Gavin Shan
2014-12-16  0:54               ` Alexander Graf
2014-12-23  4:22   ` David Gibson
2014-12-25  3:17     ` Gavin Shan
2014-12-15  0:15 ` [Qemu-devel] [PATCH v13 3/3] sPAPR: Implement sPAPRPHBClass::eeh_handler Gavin Shan

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=1418602508-30845-1-git-send-email-gwshan@linux.vnet.ibm.com \
    --to=gwshan@linux.vnet.ibm.com \
    --cc=agraf@suse.de \
    --cc=aik@ozlabs.ru \
    --cc=alex.williamson@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    /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).