On 24/04/2020 10:21, Hongyan Xia wrote:
Hi Julien,
On Fri, 2020-04-24 at 09:59 +0100, Julien Grall wrote:
(resending)
On 17/04/2020 10:52, Hongyan Xia wrote:
From: Wei Liu <wei.l...@citrix.com>
Also fix a weird indentation.
Signed-off-by: Wei Liu <wei.l...@citrix.com>
Signed-off-by: Hongyan Xia <hongy...@amazon.com>
---
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.
I believe the introduction of the new macros has been merged in staging
as 6c8afe5aadb33761431b24157d99b25eac15fc7e.
Hmmmm you are right. I must have been blind. Sorry for the noise.
Cheers,
--
Julien Grall
