Re: [PATCH 0/2] extend page_track for external usage

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2016.11.04 11:02:05 +0100, Paolo Bonzini wrote:
> >> On 25/10/2016 09:50, Jike Song wrote:
> >>> page_track is currently used only by KVM internally to protect guest
> >>> page tables. To be used by external user like KVMGT, the callbacks
> >>> should carry more information other than KVM/VCPU, and the symbols
> >>> should be exported.
> >>>
> >>> Jike Song (2):
> >>>   kvm/page_track: call notifiers with kvm_page_track_notifier_node
> >>>   kvm/page_track: export symbols for external usage
> >>>
> >>>  arch/x86/include/asm/kvm_page_track.h | 7 +++++--
> >>>  arch/x86/kvm/mmu.c                    | 6 ++++--
> >>>  arch/x86/kvm/page_track.c             | 8 ++++++--
> >>>  3 files changed, 15 insertions(+), 6 deletions(-)
> >>
> >> Thanks, applied to kvm/queue.
> >>
> >> Paolo
> >>
> > 
> > Hi Paolo,
> > 
> > I just synced kvm repo and it seemd there was a forced-update on
> > kvm/queue, and the commits are dropped? 
> 
> I temporarily reverted everything after the APICv patches.  They'll be
> back soon (and in kvm/next).
> 

Paolo, for this case, do you think it's feasible we pick them through
drm/i915 merge path? As currently initial KVMGT patch sets require these
exported symbols, that's why I ask how we should handle this dependency.

Current KVMGT initial patch set is on https://github.com/01org/gvt-linux/tree/topic/gvt-next-kvmgt
I'm planning to send pull request for Daniel next week.

Also cc Daniel, this is first case we need kvm dependency, in near
future there would also be VFIO/mdev and other kvm small changes.
So might better to sync about way to handle this.

Thanks

-- 
Open Source Technology Center, Intel ltd.

$gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux