(resending)

On 17/04/2020 10:52, Hongyan Xia wrote:
From: Wei Liu <[email protected]>

Also fix a weird indentation.

Signed-off-by: Wei Liu <[email protected]>
Signed-off-by: Hongyan Xia <[email protected]>
---
  xen/arch/x86/x86_64/mm.c | 14 +++++++-------
  1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/xen/arch/x86/x86_64/mm.c b/xen/arch/x86/x86_64/mm.c
index e85ef449f3..18210405f4 100644
--- a/xen/arch/x86/x86_64/mm.c
+++ b/xen/arch/x86/x86_64/mm.c
@@ -737,8 +737,8 @@ static void cleanup_frame_table(struct mem_hotadd_info 
*info)
while (sva < eva)
      {
-        l3e = l4e_to_l3e(idle_pg_table[l4_table_offset(sva)])[
-          l3_table_offset(sva)];
+        l3e = l3e_from_l4e(idle_pg_table[l4_table_offset(sva)],
+                           l3_table_offset(sva));

This macro doesn't exist yet in the tree. It would be good to spell out the dependencies in the cover letter so this doesn't get merged before the dependency is merged.

Reviewed-by: Julien Grall <[email protected]>

Cheers,

--
Julien Grall

Reply via email to