From: Alex Williamson <alex.williamson@redhat.com>
To: Alexey Kardashevskiy <aik@ozlabs.ru>
Cc: qemu-devel@nongnu.org, qemu-ppc@nongnu.org,
David Gibson <david@gibson.dropbear.id.au>,
Reza Arbab <arbab@linux.ibm.com>,
Piotr Jaroszynski <pjaroszynski@nvidia.com>,
Jose Ricardo Ziviani <joserz@linux.ibm.com>,
Daniel Henrique Barboza <danielhb413@gmail.com>
Subject: Re: [Qemu-devel] [PATCH qemu v2 0/4] spapr_pci, vfio: NVIDIA V100 + POWER9 passthrough
Date: Thu, 14 Feb 2019 16:37:43 -0700 [thread overview]
Message-ID: <20190214163743.7162ad53@w520.home> (raw)
In-Reply-To: <20190214052144.59541-1-aik@ozlabs.ru>
On Thu, 14 Feb 2019 16:21:40 +1100
Alexey Kardashevskiy <aik@ozlabs.ru> wrote:
> This is for passing through NVIDIA V100 GPUs on POWER9 systems.
>
> This implements a subdriver for NVIDIA V100 GPU with coherent memory and
> NPU/ATS support available in the POWER9 CPU.
>
> 1/4 is a preparation for bigger DMA windows.
> 2/4 is a small cleanup.
>
> Here is the kernel driver:
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/vfio/pci?h=v5.0-rc6&id=7f92891778dff62303c070ac81de7b7d80de331a
>
> SLOF changes already went in.
>
> This depends on "pci: Move NVIDIA vendor id to the rest of ids" (posted separately).
TBH, I'm not sure it was the best idea to let it live or die on it's
own when it now creates a build dependency for this series.
> This is based on sha1
> 1ea6057 Mark Cave-Ayland "mac_newworld: change default NIC to sungem for mac99 machine".
Perhaps this is why it doesn't apply cleanly against qemu.git. Are
there dependencies we need to wait for in the ppc tree as well?
> Please comment. Thanks.
Besides the build dependency on PCI_VENDOR_ID_NVIDIA, I also get this:
.../qemu.git/hw/vfio/spapr.c: In function ‘vfio_spapr_create_window’:
.../qemu.git/hw/vfio/spapr.c:212:9: error: ‘ret’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
error_report("Failed to create a window, ret = %d (%m)", ret);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors
Is this series expected to go in through the ppc branch given the bulk
of the changes are there? Thanks,
Alex
next prev parent reply other threads:[~2019-02-14 23:37 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-02-14 5:21 [Qemu-devel] [PATCH qemu v2 0/4] spapr_pci, vfio: NVIDIA V100 + POWER9 passthrough Alexey Kardashevskiy
2019-02-14 5:21 ` [Qemu-devel] [PATCH qemu v2 1/4] vfio/spapr: Fix indirect levels calculation Alexey Kardashevskiy
2019-02-15 1:03 ` David Gibson
2019-02-14 5:21 ` [Qemu-devel] [PATCH qemu v2 2/4] vfio/spapr: Rename local systempagesize variable Alexey Kardashevskiy
2019-02-15 3:32 ` David Gibson
2019-02-15 3:37 ` Alexey Kardashevskiy
2019-02-15 3:57 ` David Gibson
2019-02-15 4:47 ` Alexey Kardashevskiy
2019-02-14 5:21 ` [Qemu-devel] [PATCH qemu v2 3/4] vfio: Make vfio_get_region_info_cap public Alexey Kardashevskiy
2019-02-14 5:21 ` [Qemu-devel] [PATCH qemu v2 4/4] spapr: Support NVIDIA V100 GPU with NVLink2 Alexey Kardashevskiy
2019-02-15 3:22 ` David Gibson
2019-02-15 4:42 ` Alexey Kardashevskiy
2019-02-15 5:30 ` David Gibson
2019-02-19 0:52 ` Alexey Kardashevskiy
2019-02-14 23:37 ` Alex Williamson [this message]
2019-02-15 0:35 ` [Qemu-devel] [PATCH qemu v2 0/4] spapr_pci, vfio: NVIDIA V100 + POWER9 passthrough Alexey Kardashevskiy
2019-02-15 3:24 ` David Gibson
2019-02-15 3:32 ` Alexey Kardashevskiy
2019-02-15 3:54 ` David Gibson
2019-02-15 4:34 ` Alexey Kardashevskiy
2019-02-15 5:21 ` David Gibson
2019-02-25 6:39 ` Alexey Kardashevskiy
2019-02-28 2:21 ` David Gibson
2019-02-15 3:22 ` David Gibson
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=20190214163743.7162ad53@w520.home \
--to=alex.williamson@redhat.com \
--cc=aik@ozlabs.ru \
--cc=arbab@linux.ibm.com \
--cc=danielhb413@gmail.com \
--cc=david@gibson.dropbear.id.au \
--cc=joserz@linux.ibm.com \
--cc=pjaroszynski@nvidia.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).