* [PATCH] x86/PoD: use clear_domain_page()
@ 2015-09-14 10:26 Jan Beulich
2015-09-14 10:27 ` Andrew Cooper
2015-09-14 11:18 ` George Dunlap
0 siblings, 2 replies; 3+ messages in thread
From: Jan Beulich @ 2015-09-14 10:26 UTC (permalink / raw)
To: xen-devel; +Cc: George Dunlap, Andrew Cooper, Keir Fraser
[-- Attachment #1: Type: text/plain, Size: 606 bytes --]
Signed-off-by: Jan Beulich <jbeulich@suse.com>
--- a/xen/arch/x86/mm/p2m-pod.c
+++ b/xen/arch/x86/mm/p2m-pod.c
@@ -107,11 +107,7 @@ p2m_pod_cache_add(struct p2m_domain *p2m
* promise to provide zero pages. So we scrub pages before using.
*/
for ( i = 0; i < (1 << order); i++ )
- {
- char *b = map_domain_page(_mfn(mfn_x(page_to_mfn(page)) + i));
- clear_page(b);
- unmap_domain_page(b);
- }
+ clear_domain_page(_mfn(mfn_x(page_to_mfn(page)) + i));
/* First, take all pages off the domain list */
lock_page_alloc(p2m);
[-- Attachment #2: x86-p2m-pod-clear-domain-page.patch --]
[-- Type: text/plain, Size: 636 bytes --]
x86/PoD: use clear_domain_page()
Signed-off-by: Jan Beulich <jbeulich@suse.com>
--- a/xen/arch/x86/mm/p2m-pod.c
+++ b/xen/arch/x86/mm/p2m-pod.c
@@ -107,11 +107,7 @@ p2m_pod_cache_add(struct p2m_domain *p2m
* promise to provide zero pages. So we scrub pages before using.
*/
for ( i = 0; i < (1 << order); i++ )
- {
- char *b = map_domain_page(_mfn(mfn_x(page_to_mfn(page)) + i));
- clear_page(b);
- unmap_domain_page(b);
- }
+ clear_domain_page(_mfn(mfn_x(page_to_mfn(page)) + i));
/* First, take all pages off the domain list */
lock_page_alloc(p2m);
[-- 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] x86/PoD: use clear_domain_page()
2015-09-14 10:26 [PATCH] x86/PoD: use clear_domain_page() Jan Beulich
@ 2015-09-14 10:27 ` Andrew Cooper
2015-09-14 11:18 ` George Dunlap
1 sibling, 0 replies; 3+ messages in thread
From: Andrew Cooper @ 2015-09-14 10:27 UTC (permalink / raw)
To: Jan Beulich, xen-devel; +Cc: George Dunlap, Keir Fraser
On 14/09/15 11:26, Jan Beulich wrote:
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] x86/PoD: use clear_domain_page()
2015-09-14 10:26 [PATCH] x86/PoD: use clear_domain_page() Jan Beulich
2015-09-14 10:27 ` Andrew Cooper
@ 2015-09-14 11:18 ` George Dunlap
1 sibling, 0 replies; 3+ messages in thread
From: George Dunlap @ 2015-09-14 11:18 UTC (permalink / raw)
To: Jan Beulich; +Cc: xen-devel, Keir Fraser, Andrew Cooper
On Mon, Sep 14, 2015 at 11:26 AM, Jan Beulich <JBeulich@suse.com> wrote:
> Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: George Dunlap <george.dunlap@citrix.com>
>
> --- a/xen/arch/x86/mm/p2m-pod.c
> +++ b/xen/arch/x86/mm/p2m-pod.c
> @@ -107,11 +107,7 @@ p2m_pod_cache_add(struct p2m_domain *p2m
> * promise to provide zero pages. So we scrub pages before using.
> */
> for ( i = 0; i < (1 << order); i++ )
> - {
> - char *b = map_domain_page(_mfn(mfn_x(page_to_mfn(page)) + i));
> - clear_page(b);
> - unmap_domain_page(b);
> - }
> + clear_domain_page(_mfn(mfn_x(page_to_mfn(page)) + i));
>
> /* First, take all pages off the domain list */
> lock_page_alloc(p2m);
>
>
>
>
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2015-09-14 11:18 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-14 10:26 [PATCH] x86/PoD: use clear_domain_page() Jan Beulich
2015-09-14 10:27 ` Andrew Cooper
2015-09-14 11:18 ` George Dunlap
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).