On Wed, Feb 22, 2023 at 4:58 PM Axel Rasmussen axelrasmussen@google.com wrote:
Many userfaultfd ioctl functions take both a 'mode' and a 'wp_copy' argument. In future commits we plan to plumb the flags through to more places, so we'd be proliferating the very long argument list even further.
Let's take the time to simplify the argument list. Combine the two arguments into one - and generalize, so when we add more flags in the future, it doesn't imply more function arguments.
Since the modes (copy, zeropage, continue) are mutually exclusive, store them as an integer value (0, 1, 2) in the low bits. Place combine-able flag bits in the high bits.
Signed-off-by: Axel Rasmussen axelrasmussen@google.com
Acked-by: James Houghton jthoughton@google.com
I don't see anything wrong with this patch. Thanks, Axel.