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

2015-12-14 Thread Jan Beulich
>>> On 14.12.15 at 09:31, wrote: >> On 14.12.2015 at 4:19pm, wrote: >> >>> On 12.12.15 at 10:03, wrote: >> >> On 11.12.2015 at 6:01pm, wrpte: >> >> >>> On 10.12.15 at 10:33, wrote: >> >> > @@ -167,10 +172,12 @@ static int queue_invalidate_wait(struct iommu >> >> *iommu, >> >> > start

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

2015-12-14 Thread Xu, Quan
> On 14.12.2015 at 4:19pm, wrote: > >>> On 12.12.15 at 10:03, wrote: > >> On 11.12.2015 at 6:01pm, wrpte: > >> >>> On 10.12.15 at 10:33, wrote: > >> > @@ -167,10 +172,12 @@ static int queue_invalidate_wait(struct iommu > >> *iommu, > >> > start_time = NOW(); > >> > while ( pol

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

2015-12-14 Thread Jan Beulich
>>> On 12.12.15 at 10:03, wrote: >> On 11.12.2015 at 6:01pm, wrpte: >> >>> On 10.12.15 at 10:33, wrote: >> > @@ -167,10 +172,12 @@ static int queue_invalidate_wait(struct iommu >> *iommu, >> > start_time = NOW(); >> > while ( poll_slot != QINVAL_STAT_DONE ) >> > { >> >

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

2015-12-12 Thread Xu, Quan
>On 11.12.2015 at 6:01pm, wrpte: > >>> On 10.12.15 at 10:33, wrote: > > @@ -167,10 +172,12 @@ static int queue_invalidate_wait(struct iommu > *iommu, > > start_time = NOW(); > > while ( poll_slot != QINVAL_STAT_DONE ) > > { > > -if ( NOW() > (start_time + DM

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

2015-12-11 Thread Xu, Quan
> On 11.12.2015 at 6:01pm, wrote: > >>> On 10.12.15 at 10:33, wrote: > > @@ -167,10 +172,12 @@ static int queue_invalidate_wait(struct iommu > *iommu, > > start_time = NOW(); > > while ( poll_slot != QINVAL_STAT_DONE ) > > { > > -if ( NOW() > (start_time + D

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

2015-12-11 Thread Jan Beulich
>>> On 10.12.15 at 10:33, wrote: > @@ -167,10 +172,12 @@ static int queue_invalidate_wait(struct iommu *iommu, > start_time = NOW(); > while ( poll_slot != QINVAL_STAT_DONE ) > { > -if ( NOW() > (start_time + DMAR_OPERATION_TIMEOUT) ) > +if ( NOW(

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

2015-12-11 Thread Xu, Quan
On 11.12.2015 at 4:37pm, wrote: > On 11/12/2015 02:09, Xu, Quan wrote: > > On 11.12.2015 at 3:03pm, wrote: > >> On 10/12/15 09:33, Quan Xu wrote: > >>> Signed-off-by: Quan Xu > >> This patch misses a second use of DMAR_OPERATION_TIMEOUT, in > >> IOMMU_WAIT_OP() which in turn is used in a large

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

2015-12-11 Thread Andrew Cooper
On 11/12/2015 02:09, Xu, Quan wrote: > On 11.12.2015 at 3:03pm, wrote: >> On 10/12/15 09:33, Quan Xu wrote: >>> Signed-off-by: Quan Xu >>> --- >>> xen/drivers/passthrough/vtd/qinval.c | 11 +-- >>> 1 file changed, 9 insertions(+), 2 deletions(-) >>> >>> diff --git a/xen/drivers/passthrou

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

2015-12-10 Thread Xu, Quan
On 11.12.2015 at 3:01pm, wrote: > > From: Xu, Quan > > Sent: Friday, December 11, 2015 10:09 AM > > > > On 11.12.2015 at 3:03pm, wrote: > > > On 10/12/15 09:33, Quan Xu wrote: > > > > Signed-off-by: Quan Xu > > > > --- > > > > xen/drivers/passthrough/vtd/qinval.c | 11 +-- > > > > 1 fi

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

2015-12-10 Thread Tian, Kevin
> From: Xu, Quan > Sent: Friday, December 11, 2015 10:09 AM > > On 11.12.2015 at 3:03pm, wrote: > > On 10/12/15 09:33, Quan Xu wrote: > > > Signed-off-by: Quan Xu > > > --- > > > xen/drivers/passthrough/vtd/qinval.c | 11 +-- > > > 1 file changed, 9 insertions(+), 2 deletions(-) > > > >

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

2015-12-10 Thread Xu, Quan
On 11.12.2015 at 3:03pm, wrote: > On 10/12/15 09:33, Quan Xu wrote: > > Signed-off-by: Quan Xu > > --- > > xen/drivers/passthrough/vtd/qinval.c | 11 +-- > > 1 file changed, 9 insertions(+), 2 deletions(-) > > > > diff --git a/xen/drivers/passthrough/vtd/qinval.c > > b/xen/drivers/passth

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

2015-12-10 Thread Andrew Cooper
On 10/12/15 09:33, Quan Xu wrote: > Signed-off-by: Quan Xu > --- > xen/drivers/passthrough/vtd/qinval.c | 11 +-- > 1 file changed, 9 insertions(+), 2 deletions(-) > > diff --git a/xen/drivers/passthrough/vtd/qinval.c > b/xen/drivers/passthrough/vtd/qinval.c > index b81b0bd..990baf2 1006

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

2015-12-10 Thread Quan Xu
Signed-off-by: Quan Xu --- xen/drivers/passthrough/vtd/qinval.c | 11 +-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/xen/drivers/passthrough/vtd/qinval.c b/xen/drivers/passthrough/vtd/qinval.c index b81b0bd..990baf2 100644 --- a/xen/drivers/passthrough/vtd/qinval.c +++