On Thu, Feb 14, 2019 at 04:37:43PM -0700, Alex Williamson wrote: > On Thu, 14 Feb 2019 16:21:40 +1100 > Alexey Kardashevskiy <a...@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.
I'd suggest folding it into this series for reconsideration - the
original post's clearly been lost track of.
>
> > 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
>
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
