# Re: \[PATCH v3 2/3\] net: add kcov handle to skb extensions

- From: Aleksandr Nogikh \<hidden\>
- Date: 2020-10-27 12:31:38 
- List: netdev
- Page: https://ratatoskr.run/netdev/2020/10/6151385
- Thread: https://ratatoskr.run/netdev/2020/10/6149492/t.md

```text
On Mon, Oct 26, 2020 at 7:57 PM Willem de Bruijn
[ref] wrote:
[...]
> If the handle does not need to be set if zero, why then set it if the
> skb has extensions?

The point of that condition is to avoid unnecessary allocations of skb extension
objects. At the same time, one would expect skb_get_kcov_handle to return the
latest value that was set via skb_set_kcov_handle. So if a buffer already has a
non-zero kcov_handle and skb_set_kcov_handle is called to set it to zero, it
should be set to zero.

> skb_ext_add and skb_ext_find are not defined unless CONFIG_SKB_EXTENSIONS.
>
> Perhaps CONFIG_KCOV should be made to select that?

Yes, thank you for pointing it out. I’ll fix it in the next version.
```