Hi,
On 11/03/2016 18:29, Jan Beulich wrote:
On 10.03.16 at 23:00, <shank...@codeaurora.org> wrote:
--- a/xen/arch/arm/mm.c
+++ b/xen/arch/arm/mm.c
@@ -730,6 +730,7 @@ void __init setup_xenheap_mappings(unsigned long base_mfn,
else
{
unsigned long first_mfn = alloc_boot_pages(1, 1);
+ memset(mfn_to_virt(first_mfn), 0, PAGE_SIZE);
If I was maintainer of this code, I would demand use of clear_page()
and ask for insertion of the missing blank line here (separating
declaration and statements).
+1
@@ -771,6 +772,7 @@ void __init setup_frametable_mappings(paddr_t ps, paddr_t
pe)
nr_second = frametable_size >> SECOND_SHIFT;
second_base = alloc_boot_pages(nr_second, 1);
second = mfn_to_virt(second_base);
+ memset(second, 0, nr_second * PAGE_SIZE);
for ( i = 0; i < nr_second; i++ )
{
pte = mfn_to_xen_entry(second_base + i, WRITEALLOC);
Along those lines here - use clear_page(), presumably by moving it
into the loop.
+1
Regards,
--
Julien Grall
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel