On 13/08/19 15:53, Vitaly Kuznetsov wrote: > Changes since v3 [Sean Christopherson]: > - add Reviewed-by tag to PATCH5 > - __skip_emulated_instruction()/skip_emulated_instruction() split, > 'unlikely(r != EMULATE_DONE)' in PATCH2 > - Make nested_svm_vmrun() return an int in PATCH6 (moved from PATCH7) > - Avoid weird-looking 'if (rc) return ret' in PATCH7 > > Original description: > > Jim rightfully complains that hardcoding instuctions lengths is not always > correct: additional (redundant) prefixes can be used. Luckily, the ugliness > is mostly harmless: modern AMD CPUs support NRIP_SAVE feature but I'd like > to clean things up and sacrifice speed in favor of correctness. Queued, thanks. Paolo