On Mon, Jul 5, 2021 at 5:06 PM Jan Beulich wrote: > This is to make it easier to see which parts of p2m.c still aren't HVM- > specific: In one case the conditionals sat in an already guarded region, > while in the other case P2M_AUDIT implies HVM. > I think this would be much more easy to understand what's going on if it was more like this: --- x86/p2m: P2M_AUDIT implies CONFIG_HVM Remove one #endif / #ifdef CONFIG_HVM pair to make all the audit code CONFIG_HVM only. This is to make it easier to see which parts of p2m.c still aren't HVM-specific. While here, remove a redundant set of nested #ifdef CONFIG_HVM. --- Reviewed-by: George Dunlap