Re: [PATCH v6 00/15] Add allocation annotation to more functions

[David Marchand <david.marchand@redhat.com>]

On Sat, Jan 25, 2025 at 7:40 PM Stephen Hemminger
<stephen@networkplumber.org> wrote:
>
> This patch series builds on the allocation function attributes
> added in 24.11 release. These annotations will allow for compiler
> to flag cases where a pointer is allocated with one function
> but incorrectly passed to a different free function.
>
> The current code base does this correctly now, but adding
> attributes will catch future bugs, or errors in user programs.
>
> For each of these patches, the free function prototype
> needs to be reordered to be before the function attribute
> of the allocator.
>
> Checkpatch perl script falsely complains in a couple patches
> because it doesn't really understand C syntax for attributes.
>
> Note: this won't work for functions that are marked experimental.
> The problem is that function attribute for deallocator can't handle the
> case of experimental API. Even if the function in question is experimental
> the use of experimental free function in the attribute attribute causes problems.
>
> v6 - drop the pipeline patch (see note)

You forgot to drop the associated RN.

Also fixed the list of functions in RN as an empty line was missing.

Series applied, thanks.


-- 
David Marchand