> I do think it's missing some Documentation, and the cover letter is bit sparse. It > would be nice to see some high-level information about things like, for instance, > why there needs to be FRED refactoring for NMI/#PF/#DB/#MC specifically, but > not other exceptions. We do have some comments in the commit messages or around the code changes. However a high level document in the Documentation/x86/ directory probably works better, I can do that. > > There also aren't any new selftests. I faintly recall some tweak to the selftests > recently that was FRED-oriented, but I'd still expect all the selftests that poke at > the entry code to be perturbed by this a bit. Because FRED code majorly replaces the IDT entry/dispatch/return code, and makes few changes to the event handlers, our focus was more of to check if all the event handlers are properly called and returned, which is very well covered by the existing IDT selftests. One area we need to add selftests to is the FRED event dispatch framework, to make sure we cover all the possible dispatch paths. > Basically, if anyone else has been procrastinating on reviewing this set, now is > probably the time to dig in. (I'll include myself in that category, btw) I really appreciate it! Thanks! Xin
