On Tue, Apr 09, 2024 at 11:01:26AM GMT, Steven Rostedt <rost...@goodmis.org> wrote: > > - tpid = pid & (PID_MAX_DEFAULT - 1); > > + tpid = pid % PID_MAP_SIZE; > > Does that compile to the same? This is a fast path.
I didn't check. If fast is the intetion, I would change it to something like BUILD_BUG_ON(!(PID_MAP_SIZE % 2)) and keep the bit operation without reliance on compiler optimizations. Thanks for the response (I may not follow up on this single commit though). Michal
signature.asc
Description: PGP signature