Re: [Xen-devel] [PATCH v8 1/3] VT-d: Reduce spin timeout to 1ms, which can be boot-time changed

2016-04-01 Thread Xu, Quan
On March 28, 2016 9:32pm, Konrad Rzeszutek Wilk wrote: > On Mon, Mar 28, 2016 at 06:27:21AM +, Xu, Quan wrote: > > On March 26, 2016 4:07am, Konrad Rzeszutek Wilk > wrote: > > > On Thu, Mar 24, 2016 at 01:57:58PM +0800, Quan Xu wrote: > > DX > I would recommend you send out an email to

Re: [Xen-devel] [PATCH v8 1/3] VT-d: Reduce spin timeout to 1ms, which can be boot-time changed

2016-03-28 Thread Konrad Rzeszutek Wilk
On Mon, Mar 28, 2016 at 06:27:21AM +, Xu, Quan wrote: > On March 26, 2016 4:07am, Konrad Rzeszutek Wilk > wrote: > > On Thu, Mar 24, 2016 at 01:57:58PM +0800, Quan Xu wrote: > > > > Hey! > > > > Thanks for the patch! > > > > I see that you have __must_check.. > > > > But if you check the

Re: [Xen-devel] [PATCH v8 1/3] VT-d: Reduce spin timeout to 1ms, which can be boot-time changed

2016-03-27 Thread Xu, Quan
On March 26, 2016 4:07am, Konrad Rzeszutek Wilk wrote: > On Thu, Mar 24, 2016 at 01:57:58PM +0800, Quan Xu wrote: > > Hey! > > Thanks for the patch! > > I see that you have __must_check.. > > But if you check the callchain: > > iommu_flush_iec_[index|global] -> > __iommu_flush_iec->invali

Re: [Xen-devel] [PATCH v8 1/3] VT-d: Reduce spin timeout to 1ms, which can be boot-time changed

2016-03-25 Thread Konrad Rzeszutek Wilk
On Thu, Mar 24, 2016 at 01:57:58PM +0800, Quan Xu wrote: Hey! Thanks for the patch! I see that you have __must_check.. But if you check the callchain: iommu_flush_iec_[index|global] -> __iommu_flush_iec->invalidate_sync-> queue_invalidate_wait you will see that the callers of iommu_flush

Re: [Xen-devel] [PATCH v8 1/3] VT-d: Reduce spin timeout to 1ms, which can be boot-time changed

2016-03-24 Thread Xu, Quan
On March 24, 2016 7:04pm, wrote: > On Thu, 2016-03-24 at 13:57 +0800, Quan Xu wrote: > > > > --- a/docs/misc/xen-command-line.markdown > > +++ b/docs/misc/xen-command-line.markdown > > @@ -1532,6 +1532,13 @@ Note that if **watchdog** option is also > > specified vpmu will be turned off. > >  As th

Re: [Xen-devel] [PATCH v8 1/3] VT-d: Reduce spin timeout to 1ms, which can be boot-time changed

2016-03-24 Thread Dario Faggioli
On Thu, 2016-03-24 at 13:57 +0800, Quan Xu wrote: >  > --- a/docs/misc/xen-command-line.markdown > +++ b/docs/misc/xen-command-line.markdown > @@ -1532,6 +1532,13 @@ Note that if **watchdog** option is also > specified vpmu will be turned off. >  As the virtualisation is not 100% safe, don't use th

[Xen-devel] [PATCH v8 1/3] VT-d: Reduce spin timeout to 1ms, which can be boot-time changed

2016-03-23 Thread Quan Xu
Signed-off-by: Quan Xu --- docs/misc/xen-command-line.markdown | 7 +++ xen/drivers/passthrough/vtd/qinval.c | 17 - 2 files changed, 19 insertions(+), 5 deletions(-) diff --git a/docs/misc/xen-command-line.markdown b/docs/misc/xen-command-line.markdown index ca77e3b..384d