All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Gibson <david@gibson.dropbear.id.au>
To: Alexey Kardashevskiy <aik@ozlabs.ru>
Cc: Alex Williamson <alex.williamson@redhat.com>,
	linuxppc-dev@lists.ozlabs.org, Paul Mackerras <paulus@samba.org>,
	kvm-ppc@vger.kernel.org, kvm@vger.kernel.org
Subject: Re: [PATCH kernel v8 00/10] powerpc/kvm/vfio: Enable in-kernel acceleration
Date: Tue, 14 Mar 2017 11:55:33 +1100	[thread overview]
Message-ID: <20170314005533.GC12564@umbus.fritz.box> (raw)
In-Reply-To: <0c355414-253f-2ceb-b45e-90d8c01fafdf@ozlabs.ru>

[-- Attachment #1: Type: text/plain, Size: 4002 bytes --]

On Tue, Mar 14, 2017 at 11:54:03AM +1100, Alexey Kardashevskiy wrote:
> On 10/03/17 15:48, David Gibson wrote:
> > On Fri, Mar 10, 2017 at 02:53:27PM +1100, Alexey Kardashevskiy wrote:
> >> This is my current queue of patches to add acceleration of TCE
> >> updates in KVM.
> >>
> >> This is based on Linus'es tree sha1 c1aa905a304e.
> > 
> > I think we're finally there - I've now sent an R-b for all patches.
> 
> Thanks for the patience.
> 
> 
> I supposed in order to proceed now I need an ack from Alex, correct?

That, or simply for him to merge it.

> 
> 
> > 
> > 
> >>
> >> Please comment. Thanks.
> >>
> >> Changes:
> >> v8:
> >> * kept fixing oddities with error handling in 10/10
> >>
> >> v7:
> >> * added realmode's WARN_ON_ONCE_RM in arch/powerpc/kvm/book3s_64_vio_hv.c
> >>
> >> v6:
> >> * reworked the last patch in terms of error handling and parameters checking
> >>
> >> v5:
> >> * replaced "KVM: PPC: Separate TCE validation from update" with
> >> "KVM: PPC: iommu: Unify TCE checking"
> >> * changed already reviewed "powerpc/iommu/vfio_spapr_tce: Cleanup iommu_table disposal"
> >> * reworked "KVM: PPC: VFIO: Add in-kernel acceleration for VFIO"
> >> * more details in individual commit logs
> >>
> >> v4:
> >> * addressed comments from v3
> >> * updated subject lines with correct component names
> >> * regrouped the patchset in order:
> >> 	- powerpc fixes;
> >> 	- vfio_spapr_tce driver fixes;
> >> 	- KVM/PPC fixes;
> >> 	- KVM+PPC+VFIO;
> >> * everything except last 2 patches have "Reviewed-By: David"
> >>
> >> v3:
> >> * there was no full repost, only last patch was posted
> >>
> >> v2:
> >> * 11/11 reworked to use new notifiers, it is rather RFC as it still has
> >> a issue;
> >> * got 09/11, 10/11 to use notifiers in 11/11;
> >> * added rb: David to most of patches and added a comment in 05/11.
> >>
> >> Alexey Kardashevskiy (10):
> >>   powerpc/mmu: Add real mode support for IOMMU preregistered memory
> >>   powerpc/powernv/iommu: Add real mode version of
> >>     iommu_table_ops::exchange()
> >>   powerpc/iommu/vfio_spapr_tce: Cleanup iommu_table disposal
> >>   powerpc/vfio_spapr_tce: Add reference counting to iommu_table
> >>   KVM: PPC: Reserve KVM_CAP_SPAPR_TCE_VFIO capability number
> >>   KVM: PPC: Enable IOMMU_API for KVM_BOOK3S_64 permanently
> >>   KVM: PPC: Pass kvm* to kvmppc_find_table()
> >>   KVM: PPC: Use preregistered memory API to access TCE list
> >>   KVM: PPC: iommu: Unify TCE checking
> >>   KVM: PPC: VFIO: Add in-kernel acceleration for VFIO
> >>
> >>  Documentation/virtual/kvm/devices/vfio.txt |  22 +-
> >>  arch/powerpc/include/asm/iommu.h           |  32 ++-
> >>  arch/powerpc/include/asm/kvm_host.h        |   8 +
> >>  arch/powerpc/include/asm/kvm_ppc.h         |  12 +-
> >>  arch/powerpc/include/asm/mmu_context.h     |   4 +
> >>  include/uapi/linux/kvm.h                   |   9 +
> >>  arch/powerpc/kernel/iommu.c                |  86 +++++---
> >>  arch/powerpc/kvm/book3s_64_vio.c           | 330 ++++++++++++++++++++++++++++-
> >>  arch/powerpc/kvm/book3s_64_vio_hv.c        | 303 ++++++++++++++++++++++----
> >>  arch/powerpc/kvm/powerpc.c                 |   2 +
> >>  arch/powerpc/mm/mmu_context_iommu.c        |  39 ++++
> >>  arch/powerpc/platforms/powernv/pci-ioda.c  |  46 ++--
> >>  arch/powerpc/platforms/powernv/pci.c       |   1 +
> >>  arch/powerpc/platforms/pseries/iommu.c     |   3 +-
> >>  arch/powerpc/platforms/pseries/vio.c       |   2 +-
> >>  drivers/vfio/vfio_iommu_spapr_tce.c        |   2 +-
> >>  virt/kvm/vfio.c                            |  60 ++++++
> >>  arch/powerpc/kvm/Kconfig                   |   1 +
> >>  18 files changed, 855 insertions(+), 107 deletions(-)
> >>
> > 
> 
> 




-- 
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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: David Gibson <david@gibson.dropbear.id.au>
To: Alexey Kardashevskiy <aik@ozlabs.ru>
Cc: Alex Williamson <alex.williamson@redhat.com>,
	linuxppc-dev@lists.ozlabs.org, Paul Mackerras <paulus@samba.org>,
	kvm-ppc@vger.kernel.org, kvm@vger.kernel.org
Subject: Re: [PATCH kernel v8 00/10] powerpc/kvm/vfio: Enable in-kernel acceleration
Date: Tue, 14 Mar 2017 00:55:33 +0000	[thread overview]
Message-ID: <20170314005533.GC12564@umbus.fritz.box> (raw)
In-Reply-To: <0c355414-253f-2ceb-b45e-90d8c01fafdf@ozlabs.ru>

[-- Attachment #1: Type: text/plain, Size: 4002 bytes --]

On Tue, Mar 14, 2017 at 11:54:03AM +1100, Alexey Kardashevskiy wrote:
> On 10/03/17 15:48, David Gibson wrote:
> > On Fri, Mar 10, 2017 at 02:53:27PM +1100, Alexey Kardashevskiy wrote:
> >> This is my current queue of patches to add acceleration of TCE
> >> updates in KVM.
> >>
> >> This is based on Linus'es tree sha1 c1aa905a304e.
> > 
> > I think we're finally there - I've now sent an R-b for all patches.
> 
> Thanks for the patience.
> 
> 
> I supposed in order to proceed now I need an ack from Alex, correct?

That, or simply for him to merge it.

> 
> 
> > 
> > 
> >>
> >> Please comment. Thanks.
> >>
> >> Changes:
> >> v8:
> >> * kept fixing oddities with error handling in 10/10
> >>
> >> v7:
> >> * added realmode's WARN_ON_ONCE_RM in arch/powerpc/kvm/book3s_64_vio_hv.c
> >>
> >> v6:
> >> * reworked the last patch in terms of error handling and parameters checking
> >>
> >> v5:
> >> * replaced "KVM: PPC: Separate TCE validation from update" with
> >> "KVM: PPC: iommu: Unify TCE checking"
> >> * changed already reviewed "powerpc/iommu/vfio_spapr_tce: Cleanup iommu_table disposal"
> >> * reworked "KVM: PPC: VFIO: Add in-kernel acceleration for VFIO"
> >> * more details in individual commit logs
> >>
> >> v4:
> >> * addressed comments from v3
> >> * updated subject lines with correct component names
> >> * regrouped the patchset in order:
> >> 	- powerpc fixes;
> >> 	- vfio_spapr_tce driver fixes;
> >> 	- KVM/PPC fixes;
> >> 	- KVM+PPC+VFIO;
> >> * everything except last 2 patches have "Reviewed-By: David"
> >>
> >> v3:
> >> * there was no full repost, only last patch was posted
> >>
> >> v2:
> >> * 11/11 reworked to use new notifiers, it is rather RFC as it still has
> >> a issue;
> >> * got 09/11, 10/11 to use notifiers in 11/11;
> >> * added rb: David to most of patches and added a comment in 05/11.
> >>
> >> Alexey Kardashevskiy (10):
> >>   powerpc/mmu: Add real mode support for IOMMU preregistered memory
> >>   powerpc/powernv/iommu: Add real mode version of
> >>     iommu_table_ops::exchange()
> >>   powerpc/iommu/vfio_spapr_tce: Cleanup iommu_table disposal
> >>   powerpc/vfio_spapr_tce: Add reference counting to iommu_table
> >>   KVM: PPC: Reserve KVM_CAP_SPAPR_TCE_VFIO capability number
> >>   KVM: PPC: Enable IOMMU_API for KVM_BOOK3S_64 permanently
> >>   KVM: PPC: Pass kvm* to kvmppc_find_table()
> >>   KVM: PPC: Use preregistered memory API to access TCE list
> >>   KVM: PPC: iommu: Unify TCE checking
> >>   KVM: PPC: VFIO: Add in-kernel acceleration for VFIO
> >>
> >>  Documentation/virtual/kvm/devices/vfio.txt |  22 +-
> >>  arch/powerpc/include/asm/iommu.h           |  32 ++-
> >>  arch/powerpc/include/asm/kvm_host.h        |   8 +
> >>  arch/powerpc/include/asm/kvm_ppc.h         |  12 +-
> >>  arch/powerpc/include/asm/mmu_context.h     |   4 +
> >>  include/uapi/linux/kvm.h                   |   9 +
> >>  arch/powerpc/kernel/iommu.c                |  86 +++++---
> >>  arch/powerpc/kvm/book3s_64_vio.c           | 330 ++++++++++++++++++++++++++++-
> >>  arch/powerpc/kvm/book3s_64_vio_hv.c        | 303 ++++++++++++++++++++++----
> >>  arch/powerpc/kvm/powerpc.c                 |   2 +
> >>  arch/powerpc/mm/mmu_context_iommu.c        |  39 ++++
> >>  arch/powerpc/platforms/powernv/pci-ioda.c  |  46 ++--
> >>  arch/powerpc/platforms/powernv/pci.c       |   1 +
> >>  arch/powerpc/platforms/pseries/iommu.c     |   3 +-
> >>  arch/powerpc/platforms/pseries/vio.c       |   2 +-
> >>  drivers/vfio/vfio_iommu_spapr_tce.c        |   2 +-
> >>  virt/kvm/vfio.c                            |  60 ++++++
> >>  arch/powerpc/kvm/Kconfig                   |   1 +
> >>  18 files changed, 855 insertions(+), 107 deletions(-)
> >>
> > 
> 
> 




-- 
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

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

  reply	other threads:[~2017-03-14  0:55 UTC|newest]

Thread overview: 53+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-10  3:53 [PATCH kernel v8 00/10] powerpc/kvm/vfio: Enable in-kernel acceleration Alexey Kardashevskiy
2017-03-10  3:53 ` Alexey Kardashevskiy
2017-03-10  3:53 ` [PATCH kernel v8 01/10] powerpc/mmu: Add real mode support for IOMMU preregistered memory Alexey Kardashevskiy
2017-03-10  3:53   ` Alexey Kardashevskiy
2017-03-10  3:53 ` [PATCH kernel v8 02/10] powerpc/powernv/iommu: Add real mode version of iommu_table_ops::exchange() Alexey Kardashevskiy
2017-03-10  3:53   ` Alexey Kardashevskiy
2017-03-10  3:53 ` [PATCH kernel v8 03/10] powerpc/iommu/vfio_spapr_tce: Cleanup iommu_table disposal Alexey Kardashevskiy
2017-03-10  3:53   ` Alexey Kardashevskiy
2017-03-10  3:53   ` Alexey Kardashevskiy
2017-03-14 18:21   ` Alex Williamson
2017-03-14 18:21     ` Alex Williamson
2017-03-10  3:53 ` [PATCH kernel v8 04/10] powerpc/vfio_spapr_tce: Add reference counting to iommu_table Alexey Kardashevskiy
2017-03-10  3:53   ` Alexey Kardashevskiy
2017-03-14 19:58   ` Alex Williamson
2017-03-14 19:58     ` Alex Williamson
2017-03-10  3:53 ` [PATCH kernel v8 05/10] KVM: PPC: Reserve KVM_CAP_SPAPR_TCE_VFIO capability number Alexey Kardashevskiy
2017-03-10  3:53   ` Alexey Kardashevskiy
2017-03-10  3:53 ` [PATCH kernel v8 06/10] KVM: PPC: Enable IOMMU_API for KVM_BOOK3S_64 permanently Alexey Kardashevskiy
2017-03-10  3:53   ` Alexey Kardashevskiy
2017-03-10  3:53 ` [PATCH kernel v8 07/10] KVM: PPC: Pass kvm* to kvmppc_find_table() Alexey Kardashevskiy
2017-03-10  3:53   ` Alexey Kardashevskiy
2017-03-10  3:53 ` [PATCH kernel v8 08/10] KVM: PPC: Use preregistered memory API to access TCE list Alexey Kardashevskiy
2017-03-10  3:53   ` Alexey Kardashevskiy
2017-03-10  3:53 ` [PATCH kernel v8 09/10] KVM: PPC: iommu: Unify TCE checking Alexey Kardashevskiy
2017-03-10  3:53   ` Alexey Kardashevskiy
2017-03-10  3:53 ` [PATCH kernel v8 10/10] KVM: PPC: VFIO: Add in-kernel acceleration for VFIO Alexey Kardashevskiy
2017-03-10  3:53   ` Alexey Kardashevskiy
2017-03-10  4:47   ` David Gibson
2017-03-10  4:47     ` David Gibson
2017-03-14 21:05   ` Alex Williamson
2017-03-14 21:05     ` Alex Williamson
2017-03-15  4:40     ` David Gibson
2017-03-15  4:40       ` David Gibson
2017-03-15 16:18       ` Alex Williamson
2017-03-15 16:18         ` Alex Williamson
2017-03-15 16:18         ` Alex Williamson
2017-03-16  3:42         ` David Gibson
2017-03-16  3:42           ` David Gibson
2017-03-15 13:21     ` Alexey Kardashevskiy
2017-03-15 13:21       ` Alexey Kardashevskiy
2017-03-15 16:39       ` Alex Williamson
2017-03-15 16:39         ` Alex Williamson
2017-03-15 16:39         ` Alex Williamson
2017-03-15 23:39         ` Alexey Kardashevskiy
2017-03-15 23:39           ` Alexey Kardashevskiy
2017-03-10  4:48 ` [PATCH kernel v8 00/10] powerpc/kvm/vfio: Enable in-kernel acceleration David Gibson
2017-03-10  4:48   ` David Gibson
2017-03-14  0:54   ` Alexey Kardashevskiy
2017-03-14  0:54     ` Alexey Kardashevskiy
2017-03-14  0:55     ` David Gibson [this message]
2017-03-14  0:55       ` David Gibson
2017-03-14 17:59       ` Alex Williamson
2017-03-14 17:59         ` Alex Williamson

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=20170314005533.GC12564@umbus.fritz.box \
    --to=david@gibson.dropbear.id.au \
    --cc=aik@ozlabs.ru \
    --cc=alex.williamson@redhat.com \
    --cc=kvm-ppc@vger.kernel.org \
    --cc=kvm@vger.kernel.org \
    --cc=linuxppc-dev@lists.ozlabs.org \
    --cc=paulus@samba.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 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.