Re: [Qemu-devel] [PATCH v3] intel_iommu: allow dynamic switch of IOMMU region

2017-01-04 Thread Tian, Kevin
> From: Alex Williamson [mailto:alex.william...@redhat.com] > Sent: Wednesday, January 04, 2017 5:21 AM > > On Mon, 26 Dec 2016 10:45:55 +0800 > Jason Wang wrote: > > > On 2016年12月23日 11:26, Peter Xu wrote: > > > On Thu, Dec 22, 2016 at 07:34:10PM +0800, Jason Wang wrote: > > >> > > >> On 2016年1

Re: [Qemu-devel] [PATCH v3] intel_iommu: allow dynamic switch of IOMMU region

2017-01-04 Thread Tian, Kevin
> From: Peter Xu [mailto:pet...@redhat.com] > Sent: Thursday, January 05, 2017 11:53 AM > > > > /* Handle Translation Enable/Disable */ > > > static void vtd_handle_gcmd_te(IntelIOMMUState *s, bool en) > > > { > > > +if (s->dmar_enabled == en) { > > > +return; > > > +} > > > + >

Re: [Qemu-devel] [PATCH v3] intel_iommu: allow dynamic switch of IOMMU region

2017-01-04 Thread Peter Xu
Hi, Kevin, On Thu, Jan 05, 2017 at 03:30:28AM +, Tian, Kevin wrote: > > From: Peter Xu [mailto:pet...@redhat.com] > > Sent: Thursday, December 22, 2016 5:48 PM > > > > This is preparation work to finally enabled dynamic switching ON/OFF for > > VT-d protection. The old VT-d codes is using sta

Re: [Qemu-devel] [PATCH v3] intel_iommu: allow dynamic switch of IOMMU region

2017-01-04 Thread Tian, Kevin
> From: Peter Xu [mailto:pet...@redhat.com] > Sent: Thursday, December 22, 2016 5:48 PM > > This is preparation work to finally enabled dynamic switching ON/OFF for > VT-d protection. The old VT-d codes is using static IOMMU region, and > that won't satisfy vfio-pci device listeners. Is "IOMMU ad

Re: [Qemu-devel] [PATCH v3] intel_iommu: allow dynamic switch of IOMMU region

2017-01-03 Thread Alex Williamson
On Mon, 26 Dec 2016 10:45:55 +0800 Jason Wang wrote: > On 2016年12月23日 11:26, Peter Xu wrote: > > On Thu, Dec 22, 2016 at 07:34:10PM +0800, Jason Wang wrote: > >> > >> On 2016年12月22日 19:04, Peter Xu wrote: > >>> On Thu, Dec 22, 2016 at 05:52:58PM +0800, Jason Wang wrote: > On 2016年12月22

Re: [Qemu-devel] [PATCH v3] intel_iommu: allow dynamic switch of IOMMU region

2016-12-25 Thread Jason Wang
On 2016年12月23日 11:26, Peter Xu wrote: On Thu, Dec 22, 2016 at 07:34:10PM +0800, Jason Wang wrote: On 2016年12月22日 19:04, Peter Xu wrote: On Thu, Dec 22, 2016 at 05:52:58PM +0800, Jason Wang wrote: On 2016年12月22日 17:48, Peter Xu wrote: /* Handle Translation Enable/Disable */ static void

Re: [Qemu-devel] [PATCH v3] intel_iommu: allow dynamic switch of IOMMU region

2016-12-22 Thread Peter Xu
On Thu, Dec 22, 2016 at 07:34:10PM +0800, Jason Wang wrote: > > > On 2016年12月22日 19:04, Peter Xu wrote: > >On Thu, Dec 22, 2016 at 05:52:58PM +0800, Jason Wang wrote: > >> > >>On 2016年12月22日 17:48, Peter Xu wrote: > >>> /* Handle Translation Enable/Disable */ > >>> static void vtd_handle_gcmd_t

Re: [Qemu-devel] [PATCH v3] intel_iommu: allow dynamic switch of IOMMU region

2016-12-22 Thread Jason Wang
On 2016年12月22日 19:04, Peter Xu wrote: On Thu, Dec 22, 2016 at 05:52:58PM +0800, Jason Wang wrote: On 2016年12月22日 17:48, Peter Xu wrote: /* Handle Translation Enable/Disable */ static void vtd_handle_gcmd_te(IntelIOMMUState *s, bool en) { +if (s->dmar_enabled == en) { +retur

Re: [Qemu-devel] [PATCH v3] intel_iommu: allow dynamic switch of IOMMU region

2016-12-22 Thread Peter Xu
On Thu, Dec 22, 2016 at 05:52:58PM +0800, Jason Wang wrote: > > > On 2016年12月22日 17:48, Peter Xu wrote: > > /* Handle Translation Enable/Disable */ > > static void vtd_handle_gcmd_te(IntelIOMMUState *s, bool en) > > { > >+if (s->dmar_enabled == en) { > >+return; > >+} > >+ > >

Re: [Qemu-devel] [PATCH v3] intel_iommu: allow dynamic switch of IOMMU region

2016-12-22 Thread Jason Wang
On 2016年12月22日 17:48, Peter Xu wrote: /* Handle Translation Enable/Disable */ static void vtd_handle_gcmd_te(IntelIOMMUState *s, bool en) { +if (s->dmar_enabled == en) { +return; +} + VTD_DPRINTF(CSR, "Translation Enable %s", (en ? "on" : "off")); if (en) {