Re: [PATCH] KVM: PCIPT: VT-d support

2008-07-14 Thread Avi Kivity
Ben-Ami Yassour wrote: On Sun, 2008-07-13 at 10:34 +0300, Avi Kivity wrote: Muli Ben-Yehuda wrote: On the other hand, this means that you will not be able to assign devices unless you specified this when creating the VM; but I think this is fair. It will be nice to suppor

Re: [PATCH] KVM: PCIPT: VT-d support

2008-07-14 Thread Ben-Ami Yassour
On Sun, 2008-07-13 at 10:34 +0300, Avi Kivity wrote: > Muli Ben-Yehuda wrote: > > > >> On the other hand, this means that you will not be able to assign > >> devices unless you specified this when creating the VM; but I think > >> this is fair. > >> > > > > It will be nice to support hot-plugg

Re: [PATCH] KVM: PCIPT: VT-d support

2008-07-13 Thread Avi Kivity
Muli Ben-Yehuda wrote: On the other hand, this means that you will not be able to assign devices unless you specified this when creating the VM; but I think this is fair. It will be nice to support hot-plugged pass-through devices some time in the future. And it should be easy to d

Re: [PATCH] KVM: PCIPT: VT-d support

2008-07-10 Thread Muli Ben-Yehuda
On Thu, Jul 10, 2008 at 05:57:56PM +0300, Avi Kivity wrote: > It makes sense to initialize VT-d before. Since memslots can be > created and destroyed dynamically, with the current implementation > we can see > > create slot > create slot > init VT-d > create slot > > which means we need to su

Re: [PATCH] KVM: PCIPT: VT-d support

2008-07-10 Thread Avi Kivity
Muli Ben-Yehuda wrote: On Thu, Jul 10, 2008 at 06:07:27PM +0800, Yang, Sheng wrote: I think it's may be unnecessary to map pages when device assigned. The table can be set up along with set_memory_region(), it covered all memory slots already, or I miss something here? VT-d is only in

Re: [PATCH] KVM: PCIPT: VT-d support

2008-07-10 Thread Muli Ben-Yehuda
On Thu, Jul 10, 2008 at 06:07:27PM +0800, Yang, Sheng wrote: > I think it's may be unnecessary to map pages when device > assigned. The table can be set up along with set_memory_region(), it > covered all memory slots already, or I miss something here? VT-d is only initialized after the slots are

Re: [PATCH] KVM: PCIPT: VT-d support

2008-07-10 Thread Yang, Sheng
On Thursday 10 July 2008 17:51:53 Ben-Ami Yassour wrote: > On Thu, 2008-07-10 at 17:30 +0800, Yang, Sheng wrote: > > On Thursday 10 July 2008 17:14:42 Ben-Ami Yassour wrote: > > > From: Kay, Allen M <[EMAIL PROTECTED]> > > > > > > This patch includes the functions to support VT-d for > > > passthro

Re: [PATCH] KVM: PCIPT: VT-d support

2008-07-10 Thread Ben-Ami Yassour
On Thu, 2008-07-10 at 17:30 +0800, Yang, Sheng wrote: > On Thursday 10 July 2008 17:14:42 Ben-Ami Yassour wrote: > > From: Kay, Allen M <[EMAIL PROTECTED]> > > > > This patch includes the functions to support VT-d for passthrough > > devices. > > > > [Ben: fixed memory pinning] > > > > Signed-off-b

Re: [PATCH] KVM: PCIPT: VT-d support

2008-07-10 Thread Yang, Sheng
On Thursday 10 July 2008 17:14:42 Ben-Ami Yassour wrote: > From: Kay, Allen M <[EMAIL PROTECTED]> > > This patch includes the functions to support VT-d for passthrough > devices. > > [Ben: fixed memory pinning] > > Signed-off-by: Kay, Allen M <[EMAIL PROTECTED]> > Signed-off-by: Weidong Han <[EMAIL

Re: [PATCH] KVM: PCIPT: VT-d support

2008-07-10 Thread Ben-Ami Yassour
On Wed, 2008-07-09 at 10:49 -0500, Anthony Liguori wrote: > Ben-Ami Yassour wrote: > > From: Kay, Allen M <[EMAIL PROTECTED]> > > > > This patch includes the functions to support VT-d for passthrough > > devices. > > > > [Ben: fixed memory pinning] > > > > Signed-off-by: Kay, Allen M <[EMAIL PROTEC

[PATCH] KVM: PCIPT: VT-d support

2008-07-10 Thread Ben-Ami Yassour
From: Kay, Allen M <[EMAIL PROTECTED]> This patch includes the functions to support VT-d for passthrough devices. [Ben: fixed memory pinning] Signed-off-by: Kay, Allen M <[EMAIL PROTECTED]> Signed-off-by: Weidong Han <[EMAIL PROTECTED]> Signed-off-by: Ben-Ami Yassour <[EMAIL PROTECTED]> --- arc

Re: [PATCH] KVM: PCIPT: VT-d support

2008-07-09 Thread Anthony Liguori
Ben-Ami Yassour wrote: From: Kay, Allen M <[EMAIL PROTECTED]> This patch includes the functions to support VT-d for passthrough devices. [Ben: fixed memory pinning] Signed-off-by: Kay, Allen M <[EMAIL PROTECTED]> Signed-off-by: Weidong Han <[EMAIL PROTECTED]> Signed-off-by: Ben-Ami Yassour <[E

RE: [PATCH] KVM: PCIPT: VT-d support

2008-07-06 Thread Ben-Ami Yassour
On Thu, 2008-07-03 at 12:55 +0800, Han, Weidong wrote: > Ben-Ami Yassour wrote: > > From: Kay, Allen M <[EMAIL PROTECTED]> > > > > This patch includes the functions to support VT-d for passthrough > > devices. > > > > [Ben: fixed memory pinning] > > > > Signed-off-by: Kay, Allen M <[EMAIL PROTEC

[PATCH] KVM: PCIPT: VT-d support

2008-07-06 Thread Ben-Ami Yassour
From: Kay, Allen M <[EMAIL PROTECTED]> This patch includes the functions to support VT-d for passthrough devices. [Ben: fixed memory pinning] Signed-off-by: Kay, Allen M <[EMAIL PROTECTED]> Signed-off-by: Weidong Han <[EMAIL PROTECTED]> Signed-off-by: Ben-Ami Yassour <[EMAIL PROTECTED]> --- arc

RE: [PATCH] KVM: PCIPT: VT-d support

2008-07-03 Thread Ben-Ami Yassour
On Thu, 2008-07-03 at 14:16 +0800, Han, Weidong wrote: > Han, Weidong wrote: > > Ben-Ami Yassour wrote: > >> From: Kay, Allen M <[EMAIL PROTECTED]> > >> > >> This patch includes the functions to support VT-d for passthrough > >> devices. > >> > >> [Ben: fixed memory pinning] > >> > >> Signed-of

RE: [PATCH] KVM: PCIPT: VT-d support

2008-07-03 Thread Han, Weidong
Han, Weidong wrote: > Ben-Ami Yassour wrote: >> From: Kay, Allen M <[EMAIL PROTECTED]> >> >> This patch includes the functions to support VT-d for passthrough >> devices. >> >> [Ben: fixed memory pinning] >> >> Signed-off-by: Kay, Allen M <[EMAIL PROTECTED]> >> Signed-off-by: Weidong Han <[EMAI

RE: [PATCH] KVM: PCIPT: VT-d support

2008-07-03 Thread Han, Weidong
Ben-Ami Yassour wrote: > From: Kay, Allen M <[EMAIL PROTECTED]> > > This patch includes the functions to support VT-d for passthrough > devices. > > [Ben: fixed memory pinning] > > Signed-off-by: Kay, Allen M <[EMAIL PROTECTED]> > Signed-off-by: Weidong Han <[EMAIL PROTECTED]> > Signed-off-by: B

[PATCH] KVM: PCIPT: VT-d support

2008-07-02 Thread Ben-Ami Yassour
From: Kay, Allen M <[EMAIL PROTECTED]> This patch includes the functions to support VT-d for passthrough devices. [Ben: fixed memory pinning] Signed-off-by: Kay, Allen M <[EMAIL PROTECTED]> Signed-off-by: Weidong Han <[EMAIL PROTECTED]> Signed-off-by: Ben-Ami Yassour <[EMAIL PROTECTED]> --- arc