Hi Andi,

On 2026-05-08 at 11:41:14 +0200, Andi Shyti wrote:
> Hi Krzysztof,
> 
> ...
> 
> > +static struct mm_struct *
> > +get_mm(int u_pid_nr)
> 
> This fits in one line, no need to break it.
> 
> I'm not a big fan of the name "get_mm()". It's too generic and
> sounds like a library function.
> 
> With the two above changed:
> 
> Reviewed-by: Andi Shyti <[email protected]>
> 
> On the other hand I don't want to ask for v11: if it's OK with
> you I would bring the lines together and change the function to
> get_selftest_mm() before merging.
> 
> Let me know,
> Andi
Yeah, that would be wonderful, go for it :)

-- 
Best Regards,
Krzysztof

Reply via email to