On 08.04.2020 15:36, Hongyan Xia wrote: > @@ -290,26 +291,30 @@ static void destroy_m2p_mapping(struct mem_hotadd_info > *info) > continue; > } > > - l2_ro_mpt = l3e_to_l2e(l3_ro_mpt[l3_table_offset(va)]); > - if (!(l2e_get_flags(l2_ro_mpt[l2_table_offset(va)]) & _PAGE_PRESENT)) > + l2_ro_mpt = map_l2t_from_l3e(l3_ro_mpt[l3_table_offset(va)]) + > + l2_table_offset(va);
Either the name of the variable should be changed or you shouldn't add in l2_table_offset(va) here. Personally I'd go with renaming to just pl2e. Jan