A quite quick review, which I'll complete later, but unfortunately only after a while since I'm about to be almost completely AFK for ~4 weeks.

Please merge this into D44456.

Please see D56546's herald comment.

You should probably merge this revision with D56547. I have reviewed the merged result of this one, that one and D57939 (Koine Yuusuke's). Inline comments have been put in the relevant reviews as much as possible, but general comments are all listed below.

In D58336#1338546, @rlibby wrote:

Nothing planned. I was only trying to clean this up related to the build break, which is now solved. If you'll be cleaning this up soon anyway, I'll be happy to either push this now as-is or drop it entirely and leave it to you. LMK, or, if you don't have a preference, I'll commit tomorrow to clear it off my patch stack.

Please integrate this into D58012, having this separate does not make sense, and D58012 does not compile without it.

I like the idea presented here, it goes further than some initial thoughts I had. It looks like it could be made to work. However, the implications of this new feature appear not to have been explored thoroughly, and more work remains for the change here to be consistent.

In D57829#1330701, @minsoochoo0122_proton.me wrote:

Forget what I said above, we cannot do that under current design. I think clamping is fine if 1) it only happens to a few lowest priorities and 2) this situation doesn't happen often.

Fine, though please note I have a lot of rework of hwpstate_intel(4) incoming, in particular dropping this hybrid core detection code, so if you plan more/significant work in this area please liaise with me.

Yes, that makes sense.