WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-changelog

[Xen-changelog] [xen-unstable] Simplify + fix previous cset (remove_all_

To: xen-changelog@xxxxxxxxxxxxxxxxxxx
Subject: [Xen-changelog] [xen-unstable] Simplify + fix previous cset (remove_all_shadows() is indeed required :-)
From: Xen patchbot-unstable <patchbot-unstable@xxxxxxxxxxxxxxxxxxx>
Date: Wed, 20 Sep 2006 18:02:34 +0000
Delivery-date: Wed, 20 Sep 2006 11:03:14 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxx
List-help: <mailto:xen-changelog-request@lists.xensource.com?subject=help>
List-id: BK change log <xen-changelog.lists.xensource.com>
List-post: <mailto:xen-changelog@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-changelog>, <mailto:xen-changelog-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-changelog>, <mailto:xen-changelog-request@lists.xensource.com?subject=unsubscribe>
Reply-to: xen-devel@xxxxxxxxxxxxxxxxxxx
Sender: xen-changelog-bounces@xxxxxxxxxxxxxxxxxxx
# HG changeset patch
# User Steven Hand <steven@xxxxxxxxxxxxx>
# Node ID 596b0e4fbef4069bfb3ee9807f2ccfdd65052c46
# Parent  a151311fa9c79c61b20797246690fb16a3a26651
Simplify + fix previous cset (remove_all_shadows() is indeed required :-)

Signed-off-by: Steven Hand <steven@xxxxxxxxxxxxx>
---
 xen/arch/x86/mm.c |   11 +++++++----
 1 files changed, 7 insertions(+), 4 deletions(-)

diff -r a151311fa9c7 -r 596b0e4fbef4 xen/arch/x86/mm.c
--- a/xen/arch/x86/mm.c Wed Sep 20 16:22:06 2006 +0100
+++ b/xen/arch/x86/mm.c Wed Sep 20 16:52:15 2006 +0100
@@ -1622,8 +1622,7 @@ void put_page_type(struct page_info *pag
             if (shadow_lock_is_acquired(owner))  /* this is a shadow page */
                 return;
 
-            if (!shadow_mode_translate(owner))
-                mark_dirty(owner, page_to_mfn(page));
+            mark_dirty(owner, page_to_mfn(page));
         }
     }
 }
@@ -1985,8 +1984,7 @@ int do_mmuext_op(
                 break;
             }
 
-            if ( shadow_mode_enabled(d) )
-                mark_dirty(d, mfn);
+            mark_dirty(d, mfn);
            
             break;
 
@@ -2005,7 +2003,12 @@ int do_mmuext_op(
                 put_page_and_type(page);
                 put_page(page);
                 if ( shadow_mode_enabled(d) )
+                {
+                    shadow_lock(d);
+                    shadow_remove_all_shadows(v, _mfn(mfn));
+                    shadow_unlock(d);
                     mark_dirty(d, mfn);
+                }
             }
             else
             {

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog

<Prev in Thread] Current Thread [Next in Thread>
  • [Xen-changelog] [xen-unstable] Simplify + fix previous cset (remove_all_shadows() is indeed required :-), Xen patchbot-unstable <=