Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-28 Thread Jike Song
On 10/26/2016 10:45 PM, Paolo Bonzini wrote: > On 26/10/2016 15:44, Jike Song wrote: >> On 10/21/2016 01:06 AM, Paolo Bonzini wrote: >>> On 20/10/2016 03:48, Xiao Guangrong wrote: I understood that KVM side is safe, however, vfio side is independent with kvm and the user of usrdata can

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-26 Thread Paolo Bonzini
On 26/10/2016 15:44, Jike Song wrote: > On 10/21/2016 01:06 AM, Paolo Bonzini wrote: >> On 20/10/2016 03:48, Xiao Guangrong wrote: >>> I understood that KVM side is safe, however, vfio side is independent with >>> kvm and the user of usrdata can fetch kvm struct at any time, consider >>> this

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-26 Thread Jike Song
On 10/21/2016 01:06 AM, Paolo Bonzini wrote: > On 20/10/2016 03:48, Xiao Guangrong wrote: >> I understood that KVM side is safe, however, vfio side is independent with >> kvm and the user of usrdata can fetch kvm struct at any time, consider >> this scenario: >> >> CPU 0

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-24 Thread Jike Song
On 10/19/2016 09:56 PM, Eric Blake wrote: > 17 levels of quoting is rather over-the-top. It is OKAY (and in fact > DESIRABLE) to trim your emails to relevant portions, when posting to a > high-volume list. Readers shouldn't have to scroll through pages of > deeply-nested quoting... Hi Eric,

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-20 Thread Jike Song
On 10/21/2016 01:19 AM, Xiao, Guangrong wrote: >> On 10/19/2016 10:14 PM, Paolo Bonzini wrote: >>> On 19/10/2016 15:39, Xiao Guangrong wrote: >>> >>> >>> I understood that KVM side is safe, however, vfio side is independent >>> with kvm and the user of usrdata can fetch kvm struct at any time,

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-20 Thread Xiao, Guangrong
;; Alex Williamson <alex.william...@redhat.com> Cc: Tian, Kevin <kevin.t...@intel.com>; Neo Jia <c...@nvidia.com>; k...@vger.kernel.org; qemu-devel <qemu-devel@nongnu.org>; Chen, Xiaoguang <xiaoguang.c...@intel.com>; Kirti Wankhede <kwankh...@nvidia.com> Subj

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-20 Thread Paolo Bonzini
On 20/10/2016 03:48, Xiao Guangrong wrote: > > > On 10/19/2016 10:14 PM, Paolo Bonzini wrote: >> >> >> On 19/10/2016 15:39, Xiao Guangrong wrote: >>> >>> >>> On 10/19/2016 07:56 PM, Paolo Bonzini wrote: On 19/10/2016 07:45, Xiao Guangrong wrote: > > > On 10/19/2016

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-19 Thread Xiao Guangrong
On 10/19/2016 10:14 PM, Paolo Bonzini wrote: On 19/10/2016 15:39, Xiao Guangrong wrote: On 10/19/2016 07:56 PM, Paolo Bonzini wrote: On 19/10/2016 07:45, Xiao Guangrong wrote: On 10/19/2016 10:32 AM, Jike Song wrote: +EXPORT_SYMBOL_GPL(vfio_group_set_usrdata); + +void

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-19 Thread Paolo Bonzini
On 19/10/2016 15:39, Xiao Guangrong wrote: > > > On 10/19/2016 07:56 PM, Paolo Bonzini wrote: >> >> >> On 19/10/2016 07:45, Xiao Guangrong wrote: >>> >>> >>> On 10/19/2016 10:32 AM, Jike Song wrote: >>> +EXPORT_SYMBOL_GPL(vfio_group_set_usrdata); >> + >> +void

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-19 Thread Eric Blake
[meta-comment] On 10/18/2016 09:32 PM, Jike Song wrote: > On 10/18/2016 10:59 PM, Alex Williamson wrote: ... >>> On 10/10/2016 20:01, Neo Jia wrote: > Hi Neo, 17 levels of quoting is rather over-the-top. It is OKAY (and in fact DESIRABLE) to trim your

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-19 Thread Xiao Guangrong
On 10/19/2016 07:56 PM, Paolo Bonzini wrote: On 19/10/2016 07:45, Xiao Guangrong wrote: On 10/19/2016 10:32 AM, Jike Song wrote: +EXPORT_SYMBOL_GPL(vfio_group_set_usrdata); + +void *vfio_group_get_usrdata(struct vfio_group *group) +{ +return group->usrdata; +}

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-19 Thread Xiao Guangrong
On 10/19/2016 10:32 AM, Jike Song wrote: +EXPORT_SYMBOL_GPL(vfio_group_set_usrdata); + +void *vfio_group_get_usrdata(struct vfio_group *group) +{ + return group->usrdata; +} +EXPORT_SYMBOL_GPL(vfio_group_get_usrdata); + +void *vfio_group_get_usrdata_by_device(struct device *dev) +{ +

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-19 Thread Paolo Bonzini
On 19/10/2016 07:45, Xiao Guangrong wrote: > > > On 10/19/2016 10:32 AM, Jike Song wrote: > +EXPORT_SYMBOL_GPL(vfio_group_set_usrdata); + +void *vfio_group_get_usrdata(struct vfio_group *group) +{ +return group->usrdata; +}

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-18 Thread Jike Song
On 10/18/2016 10:59 PM, Alex Williamson wrote: > On Tue, 18 Oct 2016 20:38:21 +0800 > Jike Song wrote: >> On 10/18/2016 12:02 AM, Alex Williamson wrote: >>> On Fri, 14 Oct 2016 15:19:01 -0700 >>> Neo Jia wrote: >>> On Fri, Oct 14, 2016 at 10:51:24AM

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-18 Thread Alex Williamson
On Tue, 18 Oct 2016 20:38:21 +0800 Jike Song wrote: > On 10/18/2016 12:02 AM, Alex Williamson wrote: > > On Fri, 14 Oct 2016 15:19:01 -0700 > > Neo Jia wrote: > > > >> On Fri, Oct 14, 2016 at 10:51:24AM -0600, Alex Williamson wrote: > >>> On Fri, 14

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-18 Thread Jike Song
On 10/18/2016 12:02 AM, Alex Williamson wrote: > On Fri, 14 Oct 2016 15:19:01 -0700 > Neo Jia wrote: > >> On Fri, Oct 14, 2016 at 10:51:24AM -0600, Alex Williamson wrote: >>> On Fri, 14 Oct 2016 09:35:45 -0700 >>> Neo Jia wrote: >>> On Fri, Oct 14, 2016

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-17 Thread Alex Williamson
On Fri, 14 Oct 2016 15:19:01 -0700 Neo Jia wrote: > On Fri, Oct 14, 2016 at 10:51:24AM -0600, Alex Williamson wrote: > > On Fri, 14 Oct 2016 09:35:45 -0700 > > Neo Jia wrote: > > > > > On Fri, Oct 14, 2016 at 08:46:01AM -0600, Alex Williamson wrote: > > >

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-14 Thread Neo Jia
On Fri, Oct 14, 2016 at 10:51:24AM -0600, Alex Williamson wrote: > On Fri, 14 Oct 2016 09:35:45 -0700 > Neo Jia wrote: > > > On Fri, Oct 14, 2016 at 08:46:01AM -0600, Alex Williamson wrote: > > > On Fri, 14 Oct 2016 08:41:58 -0600 > > > Alex Williamson

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-14 Thread Alex Williamson
On Fri, 14 Oct 2016 09:35:45 -0700 Neo Jia wrote: > On Fri, Oct 14, 2016 at 08:46:01AM -0600, Alex Williamson wrote: > > On Fri, 14 Oct 2016 08:41:58 -0600 > > Alex Williamson wrote: > > > > > On Fri, 14 Oct 2016 18:37:45 +0800 > > > Jike Song

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-14 Thread Neo Jia
On Fri, Oct 14, 2016 at 08:46:01AM -0600, Alex Williamson wrote: > On Fri, 14 Oct 2016 08:41:58 -0600 > Alex Williamson wrote: > > > On Fri, 14 Oct 2016 18:37:45 +0800 > > Jike Song wrote: > > > > > On 10/11/2016 05:47 PM, Paolo Bonzini wrote:

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-14 Thread Alex Williamson
On Fri, 14 Oct 2016 08:41:58 -0600 Alex Williamson wrote: > On Fri, 14 Oct 2016 18:37:45 +0800 > Jike Song wrote: > > > On 10/11/2016 05:47 PM, Paolo Bonzini wrote: > > > > > > > > > On 11/10/2016 11:21, Xiao Guangrong wrote: > > >> > >

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-14 Thread Alex Williamson
On Fri, 14 Oct 2016 18:37:45 +0800 Jike Song wrote: > On 10/11/2016 05:47 PM, Paolo Bonzini wrote: > > > > > > On 11/10/2016 11:21, Xiao Guangrong wrote: > >> > >> > >> On 10/11/2016 04:54 PM, Paolo Bonzini wrote: > >>> > >>> > >>> On 11/10/2016 04:39, Xiao Guangrong

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-14 Thread Jike Song
On 10/14/2016 06:43 PM, Paolo Bonzini wrote: > > > On 14/10/2016 12:37, Jike Song wrote: >> Hi Paolo & Alex, >> >> IIUC, passing file descriptors means touching QEMU and the UAPI between >> QEMU and VFIO. Would you guys have a look at below draft patch? If it's >> on the correct direction, I'll

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-14 Thread Paolo Bonzini
On 14/10/2016 12:37, Jike Song wrote: > Hi Paolo & Alex, > > IIUC, passing file descriptors means touching QEMU and the UAPI between > QEMU and VFIO. Would you guys have a look at below draft patch? If it's > on the correct direction, I'll send the split ones. Thanks! > > -- > Thanks, > Jike >

Re: [Qemu-devel] [PATCH 1/2] KVM: page track: add a new notifier type: track_flush_slot

2016-10-14 Thread Jike Song
On 10/11/2016 05:47 PM, Paolo Bonzini wrote: > > > On 11/10/2016 11:21, Xiao Guangrong wrote: >> >> >> On 10/11/2016 04:54 PM, Paolo Bonzini wrote: >>> >>> >>> On 11/10/2016 04:39, Xiao Guangrong wrote: On 10/11/2016 02:32 AM, Paolo Bonzini wrote: > > > On 10/10/2016