Re: [PATCH v2 02/11] xen/bitops: Switch from __pure to attr_const

2024-08-30 Thread Andrew Cooper
On 29/08/2024 3:07 pm, Jan Beulich wrote: > On 29.08.2024 00:03, Andrew Cooper wrote: >> All of the ffs()/fls() infrastructure is in fact (attr) const, because it >> doesn't even read global state. This allows the compiler even more >> flexibility to optimise. >> >> No functional change. >> >> Rep

Re: [PATCH v2 02/11] xen/bitops: Switch from __pure to attr_const

2024-08-29 Thread Jan Beulich
On 29.08.2024 00:03, Andrew Cooper wrote: > All of the ffs()/fls() infrastructure is in fact (attr) const, because it > doesn't even read global state. This allows the compiler even more > flexibility to optimise. > > No functional change. > > Reported-by: Jan Beulich > Signed-off-by: Andrew Co

[PATCH v2 02/11] xen/bitops: Switch from __pure to attr_const

2024-08-28 Thread Andrew Cooper
All of the ffs()/fls() infrastructure is in fact (attr) const, because it doesn't even read global state. This allows the compiler even more flexibility to optimise. No functional change. Reported-by: Jan Beulich Signed-off-by: Andrew Cooper --- CC: Jan Beulich CC: Roger Pau Monné CC: Stefan