Thread (20 messages) 20 messages, 6 authors, 2022-10-02

Re: [PATCH net-next] docs: netlink: clarify the historical baggage of Netlink flags

From: Florent Fourcot <hidden>
Date: 2022-09-28 08:04:57

Hello,

About NLM_F_EXCL, I'm not sure that my comment is relevant for your 
intro.rst document, but it has another usage in ipset submodule. For 
IPSET_CMD_DEL, setting NLM_F_EXCL means "raise an error if entry does 
not exist before the delete".

For IPSET_CMD_ADD, setting NLM_F_EXCL will raises an error if it already 
exists (that is fine with your documentation). But without the flag, 
IPSET_CMD_ADD is more or less like a "replace" operation, since one can 
update fields like counters values (IPSET_ATTR_PACKETS / 
IPSET_ATTR_BYTES), without using NLM_F_REPLACE flag.

Best regards

-- 
*Ce message et toutes les pièces jointes (ci-après le "message") sont 
établis à l’intention exclusive des destinataires désignés. Il contient des 
informations confidentielles et pouvant être protégé par le secret 
professionnel. Si vous recevez ce message par erreur, merci d'en avertir 
immédiatement l'expéditeur et de détruire le message. Toute utilisation de 
ce message non conforme à sa destination, toute diffusion ou toute 
publication, totale ou partielle, est interdite, sauf autorisation expresse 
de l'émetteur*
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help