On Mon, Jan 06, 2025 at 08:50:27AM -0800, Dave Hansen wrote:
> There's some precedent for defining them all up front, like we did for
> invpcid_flush_*().
>
> For INVPCID, there are four variants and two of them got used up front.
> But I get that it's a balancing act between having untested code that
> might bitrot and introducing helpers at a time when someone (Rik) is
> very likely to get all the variants coded up correctly.
That's just silly. We don't add all possible hw interface bits, defines etc,
when they're not going to be used. If we did, the kernel would be an unwieldy
mess of MSRs and their bits, unused insn opcodes and the like.
If someone wants to use them, someone can add them *when* they're needed - not
preemptively, in anticipation that *someone* *might* use them in the future.
Guys, I can't believe I'm actually arguing for something so obvious.
:-\
--
Regards/Gruss,
Boris.
https://people.kernel.org/tglx/notes-about-netiquette
