On Thu, Feb 04, 2021 at 07:20:00PM -0400, Jason Gunthorpe wrote:
> > +static inline bool is_cow_mapping(vm_flags_t flags)
> > +{
>
> It feels a bit more logical to pass in a struct vm_area_struct *' here?Agree, but only if I'm adding this as a new function. Though it's a code movement from internal.h to mm.h, so I avoided touching all the users of this function that existed. Thanks, -- Peter Xu

