On Fri, Sep 29, 2023 at 12:08:18PM +0200, Phil Sutter wrote: > On Thu, Sep 28, 2023 at 08:53:11PM +0200, Pablo Neira Ayuso wrote: > > On Thu, Sep 28, 2023 at 06:52:43PM +0200, Phil Sutter wrote: > > > Relieve the dump callback from having to check nlmsg_type upon each > > > call. Prep work for set element reset locking. > > > > Maybe add this as a preparation patch first place in this series, > > rather making this cleanup at this late stage of the batch. > > Sure, no problem. I extracted it from v1 of patch 8 and so they are > closely related. > > Maybe I should split the series up in per-callback ones? I'd start with > the getsetelem_reset one as that is most cumbersome it seems. Thanks. Side note: I also read a comment from Florian regarding the use of ctx.table. You have to be very careful with what you cache in the dump context area, since such pointer might just go away. So far this code caches was "careful" to cache only to check if the table was still there, but iterating over the table list again (another safer approach could be to use the table handle which is unique). All this is also related to the chunked nature of netlink dumps (in other words, userspace retrieves part of it in every netlink_recvmsg() call).
