On Wed, May 03, 2023 at 11:50:51AM +0200, Petr Tesařík wrote: > If anyone ever wants to use this code tagging framework for something > else, they will also have to convert relevant functions to macros, > slowly changing the kernel to a minefield where local identifiers, > struct, union and enum tags, field names and labels must avoid name > conflict with a tagged function. For now, I have to remember that > alloc_pages is forbidden, but the list may grow. Also, since you're not actually a kernel contributor yet... It's not really good decorum to speculate in code review about things that can be answered by just reading the code. If you're going to comment, please do the necessary work to make sure you're saying something that makes sense.