On 6/14/22 05:02, Kirill A. Shutemov wrote: > Rework try_accept_one() to return accepted size instead of modifying > 'start' inside the helper. It makes 'start' in-only argumaent and ^ argument > streamlines code on the caller side. > > Signed-off-by: Kirill A. Shutemov <kirill.shutemov@xxxxxxxxxxxxxxx> > Suggested-by: Borislav Petkov <bp@xxxxxxxxx> I'm not sure how much it actually streamlines things. The line count looks pretty similar. But, I do generally dislike implicit return values, so with that typo fixed: Reviewed-by: Dave Hansen <dave.hansen@xxxxxxxxxxxxxxx>