| Message ID | cover.1584955616.git.hongyxia@amazon.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | use new API for Xen page tables | expand |
On Mon, Mar 23, 2020 at 09:41:37AM +0000, Hongyan Xia wrote: > From: Hongyan Xia <hongyxia@amazon.com> > > This small series is basically just rewriting functions using the new > API to map and unmap PTEs. Each patch is independent. > > Apart from mapping and unmapping page tables, no other functional change > intended. > The code looks correct to me. I do wonder if you should've put your SoB instead of Rb in some of the patches. Wei.
On Sun, 2020-03-29 at 16:06 +0100, Wei Liu wrote: > On Mon, Mar 23, 2020 at 09:41:37AM +0000, Hongyan Xia wrote: > > From: Hongyan Xia <hongyxia@amazon.com> > > > > This small series is basically just rewriting functions using the > > new > > API to map and unmap PTEs. Each patch is independent. > > > > Apart from mapping and unmapping page tables, no other functional > > change > > intended. > > > > The code looks correct to me. > > I do wonder if you should've put your SoB instead of Rb in some of > the > patches. I am not exactly sure what the policy is. For a couple of the patches, I did not touch anything but just cherry-picked from your tree, and what I did was just looking at the code and double checking, so I put an Rb there. Will change to SoB if this is not how we do things. Hongyan
Ping. On Mon, 2020-03-23 at 09:41 +0000, Hongyan Xia wrote: > From: Hongyan Xia <hongyxia@amazon.com> > > This small series is basically just rewriting functions using the new > API to map and unmap PTEs. Each patch is independent. > > Apart from mapping and unmapping page tables, no other functional > change > intended. > > Wei Liu (5): > x86/shim: map and unmap page tables in replace_va_mapping > x86_64/mm: map and unmap page tables in m2p_mapped > x86_64/mm: map and unmap page tables in share_hotadd_m2p_table > x86_64/mm: map and unmap page tables in destroy_compat_m2p_mapping > x86_64/mm: map and unmap page tables in destroy_m2p_mapping > > xen/arch/x86/pv/shim.c | 10 ++++--- > xen/arch/x86/x86_64/mm.c | 55 +++++++++++++++++++++++++----------- > -- > xen/include/asm-x86/page.h | 18 +++++++++++++ > 3 files changed, 62 insertions(+), 21 deletions(-) >
On 06.04.2020 10:27, Hongyan Xia wrote: > Ping. Does this somehow imply you didn't get my replies sent on the 1st? Jan > On Mon, 2020-03-23 at 09:41 +0000, Hongyan Xia wrote: >> From: Hongyan Xia <hongyxia@amazon.com> >> >> This small series is basically just rewriting functions using the new >> API to map and unmap PTEs. Each patch is independent. >> >> Apart from mapping and unmapping page tables, no other functional >> change >> intended. >> >> Wei Liu (5): >> x86/shim: map and unmap page tables in replace_va_mapping >> x86_64/mm: map and unmap page tables in m2p_mapped >> x86_64/mm: map and unmap page tables in share_hotadd_m2p_table >> x86_64/mm: map and unmap page tables in destroy_compat_m2p_mapping >> x86_64/mm: map and unmap page tables in destroy_m2p_mapping >> >> xen/arch/x86/pv/shim.c | 10 ++++--- >> xen/arch/x86/x86_64/mm.c | 55 +++++++++++++++++++++++++----------- >> -- >> xen/include/asm-x86/page.h | 18 +++++++++++++ >> 3 files changed, 62 insertions(+), 21 deletions(-) >> >
On Mon, 2020-04-06 at 13:03 +0200, Jan Beulich wrote: > On 06.04.2020 10:27, Hongyan Xia wrote: > > Ping. > > Does this somehow imply you didn't get my replies sent on the 1st? > > Jan Apologies. Somehow your replies ended up in a separate thread. There is a problem with my email client. Hongyan
From: Hongyan Xia <hongyxia@amazon.com> This small series is basically just rewriting functions using the new API to map and unmap PTEs. Each patch is independent. Apart from mapping and unmapping page tables, no other functional change intended. Wei Liu (5): x86/shim: map and unmap page tables in replace_va_mapping x86_64/mm: map and unmap page tables in m2p_mapped x86_64/mm: map and unmap page tables in share_hotadd_m2p_table x86_64/mm: map and unmap page tables in destroy_compat_m2p_mapping x86_64/mm: map and unmap page tables in destroy_m2p_mapping xen/arch/x86/pv/shim.c | 10 ++++--- xen/arch/x86/x86_64/mm.c | 55 +++++++++++++++++++++++++------------- xen/include/asm-x86/page.h | 18 +++++++++++++ 3 files changed, 62 insertions(+), 21 deletions(-)