Re: [PATCH v2] KVM: selftests: Check result in hyperv_features.c test only for successful hypercalls

From: Sean Christopherson
Date: Thu Sep 22 2022 - 16:25:43 EST


On Thu, Sep 22, 2022, Sean Christopherson wrote:
> The bug Vitaly encountered is exactly why it's pre

Premature send :-)

What I was going to say...

The bug Vitaly encountered is exactly why upstream process _strongly_ prefers
splitting patches by logical changes, even when the changes are related or tiny.
Bundling the fix for the egregious bug with the enhancement makes it unnecessarily
difficult to grab _just_ the fix. In this case, Vitaly was on top of things and
there was minimal fallout, but imagine if the fix was for KVM proper and needed to
be backported. Some unsuspecting user would grab the "fix", apply it to their
kernel, and suddenly be presented with previously unseen failures.