Subject: Re: [PATCH net v2] docs: netdev: document guidance on cleanup.h

From: Simon Horman <[email-masked]>
Date: Fri, 30 Aug 2024 21:31:09 +0100

On Fri, Aug 30, 2024 at 10:14:42AM -0700, Jakub Kicinski wrote:
> Document what was discussed multiple times on list and various
> virtual / in-person conversations. guard() being okay in functions
> <= 20 LoC is a bit of my own invention. If the function is trivial
> it should be fine, but feel free to disagree :)
> 
> We'll obviously revisit this guidance as time passes and we and other
> subsystems get more experience.
> 
> Reviewed-by: Eric Dumazet <[email-masked]>
> Reviewed-by: Nikolay Aleksandrov <[email-masked]>
> Reviewed-by: Andrew Lunn <[email-masked]>
> Signed-off-by: Jakub Kicinski <[email-masked]>
> ---
> CC: [email-masked]
> CC: [email-masked]
> CC: [email-masked]
> 
> v2:
>  - add sentence about revisiting later to commit msg
>  - fix spelling
> v1: https://lore.kernel.org/[email-masked]

Thanks. I think this patch has enough tags, so I won't add another.
But, FTR, this version looks good to me.

...