All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] IOMMU: skip domains without page tables when dumping
@ 2015-08-27 14:04 Jan Beulich
  2015-08-27 14:27 ` Roger Pau Monné
  2015-08-27 14:29 ` Wei Liu
  0 siblings, 2 replies; 3+ messages in thread
From: Jan Beulich @ 2015-08-27 14:04 UTC (permalink / raw)
  To: xen-devel; +Cc: Wei Liu, Roger Pau Monne

[-- Attachment #1: Type: text/plain, Size: 473 bytes --]

Reported-by: Roger Pau Monné <roger.pau@citrix.com>
Signed-off-by: Jan Beulich <jbeulich@suse.com>

--- a/xen/drivers/passthrough/iommu.c
+++ b/xen/drivers/passthrough/iommu.c
@@ -411,7 +411,7 @@ static void iommu_dump_p2m_table(unsigne
     ops = iommu_get_ops();
     for_each_domain(d)
     {
-        if ( is_hardware_domain(d) )
+        if ( is_hardware_domain(d) || need_iommu(d) <= 0 )
             continue;

         if ( iommu_use_hap_pt(d) )



[-- Attachment #2: IOMMU-dont-dump-innocent.patch --]
[-- Type: text/plain, Size: 526 bytes --]

IOMMU: skip domains without page tables when dumping

Reported-by: Roger Pau Monné <roger.pau@citrix.com>
Signed-off-by: Jan Beulich <jbeulich@suse.com>

--- a/xen/drivers/passthrough/iommu.c
+++ b/xen/drivers/passthrough/iommu.c
@@ -411,7 +411,7 @@ static void iommu_dump_p2m_table(unsigne
     ops = iommu_get_ops();
     for_each_domain(d)
     {
-        if ( is_hardware_domain(d) )
+        if ( is_hardware_domain(d) || need_iommu(d) <= 0 )
             continue;
 
         if ( iommu_use_hap_pt(d) )

[-- Attachment #3: Type: text/plain, Size: 126 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] IOMMU: skip domains without page tables when dumping
  2015-08-27 14:04 [PATCH] IOMMU: skip domains without page tables when dumping Jan Beulich
@ 2015-08-27 14:27 ` Roger Pau Monné
  2015-08-27 14:29 ` Wei Liu
  1 sibling, 0 replies; 3+ messages in thread
From: Roger Pau Monné @ 2015-08-27 14:27 UTC (permalink / raw)
  To: Jan Beulich, xen-devel; +Cc: Wei Liu

Thanks, this solves the problem.

El 27/08/15 a les 16.04, Jan Beulich ha escrit:
> Reported-by: Roger Pau Monné <roger.pau@citrix.com>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>

Tested-by: Roger Pau Monné <roger.pau@citrix.com>
Acked-by: Roger Pau Monné <roger.pau@citrix.com>

> --- a/xen/drivers/passthrough/iommu.c
> +++ b/xen/drivers/passthrough/iommu.c
> @@ -411,7 +411,7 @@ static void iommu_dump_p2m_table(unsigne
>      ops = iommu_get_ops();
>      for_each_domain(d)
>      {
> -        if ( is_hardware_domain(d) )
> +        if ( is_hardware_domain(d) || need_iommu(d) <= 0 )
>              continue;
>  
>          if ( iommu_use_hap_pt(d) )
> 
> 


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] IOMMU: skip domains without page tables when dumping
  2015-08-27 14:04 [PATCH] IOMMU: skip domains without page tables when dumping Jan Beulich
  2015-08-27 14:27 ` Roger Pau Monné
@ 2015-08-27 14:29 ` Wei Liu
  1 sibling, 0 replies; 3+ messages in thread
From: Wei Liu @ 2015-08-27 14:29 UTC (permalink / raw)
  To: Jan Beulich; +Cc: xen-devel, Wei Liu, Roger Pau Monne

On Thu, Aug 27, 2015 at 08:04:19AM -0600, Jan Beulich wrote:
> Reported-by: Roger Pau Monné <roger.pau@citrix.com>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> 

Release-acked-by: Wei Liu <wei.liu2@citrix.com>

> --- a/xen/drivers/passthrough/iommu.c
> +++ b/xen/drivers/passthrough/iommu.c
> @@ -411,7 +411,7 @@ static void iommu_dump_p2m_table(unsigne
>      ops = iommu_get_ops();
>      for_each_domain(d)
>      {
> -        if ( is_hardware_domain(d) )
> +        if ( is_hardware_domain(d) || need_iommu(d) <= 0 )
>              continue;
>  
>          if ( iommu_use_hap_pt(d) )
> 
> 

> IOMMU: skip domains without page tables when dumping
> 
> Reported-by: Roger Pau Monn?? <roger.pau@citrix.com>
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
> 
> --- a/xen/drivers/passthrough/iommu.c
> +++ b/xen/drivers/passthrough/iommu.c
> @@ -411,7 +411,7 @@ static void iommu_dump_p2m_table(unsigne
>      ops = iommu_get_ops();
>      for_each_domain(d)
>      {
> -        if ( is_hardware_domain(d) )
> +        if ( is_hardware_domain(d) || need_iommu(d) <= 0 )
>              continue;
>  
>          if ( iommu_use_hap_pt(d) )

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2015-08-27 14:30 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-27 14:04 [PATCH] IOMMU: skip domains without page tables when dumping Jan Beulich
2015-08-27 14:27 ` Roger Pau Monné
2015-08-27 14:29 ` Wei Liu

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.