b02, so if we save+restore with nested_run_pending, we end up creating vmcb02 on the destination from what we put in vmcb02 in the source, not vmcb12, which may or may not be the right thing to do. This is probably a heavier lift than we think it is, or maybe it's simpler once I start coding it :) Honestly, I'd rather keep the existing patch for stable@. It's not that complicated, and downstream trees that take it don't have to live with the FIXME code. The heavier lift to clean this up can be done separately, or I can send a new version with the first 2 patches in the beginning for stable@ and the cleanups on top, depends on how we decide to implement this. Generally agree, but in this case I am referencing the calls right above and right below, and it's probably clearer to mention the ordering constraint directly with their names. That being said, if you feel strongly I can probably do sth like your suggestion above: /* * Only sync fields from vmcb02 to cache after completing * interrupts, as NextRIP may be updated (e.g. when re-injecting a * soft interrupt). */[PATCH 1/4] KVM: nSVM: Sync next_rip to cached vmcb12 after VMRUN of L2Yosry Ahmed undefinedSean Christopherson undefined undefined undefined undefined undefinedP4