On Mon, Aug 26, 2024 at 04:30:59PM +1000, NeilBrown wrote: > wake_up_bit() currently allows a "void *". While this isn't strictly a > problem as the address is never dereferenced, it is inconsistent with > the corresponding wait_var_event() which requires "unsigned long *" and > does dereference the pointer. I'm having trouble parsing this. The way I read it, you're contradicting yourself. Where does wait_var_event() require 'unsigned long *' ? > And code that needs to wait for a change in something other than an > unsigned long would be better served by wake_up_var(). This, afaict the whole var thing is size invariant. It only cares about the address.
