Re: [PATCHv7 13/14] x86/tdx: Refactor try_accept_one()

From: Dave Hansen
Date: Thu Jun 23 2022 - 14:30:00 EST


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>