Thread (17 messages) 17 messages, 4 authors, 2022-09-29

Re: [PATCH net-next 1/6] docs: add more netlink docs (incl. spec docs)

From: Guillaume Nault <hidden>
Date: 2022-09-29 15:52:39
Also in: linux-doc

On Thu, Sep 29, 2022 at 08:47:15AM -0700, Jakub Kicinski wrote:
quoted hunk ↗ jump to hunk
Folded it a little bit:
diff --git a/Documentation/core-api/netlink.rst b/Documentation/core-api/netlink.rst
index 2a97f765d0d2..7b98dd48a6af 100644
--- a/Documentation/core-api/netlink.rst
+++ b/Documentation/core-api/netlink.rst
@@ -37,15 +37,15 @@ added whether it replies with a full message or only an ACK is uAPI and
 cannot be changed. It's better to err on the side of replying.
 
 Specifically NEW and ADD commands should reply with information identifying
-the created object such as the allocated object's ID.
-
-Having to rely on ``NLM_F_ECHO`` is a hack, not a valid design.
+the created object such as the allocated object's ID (without having to
+resort to using ``NLM_F_ECHO``).
 
 NLM_F_ECHO
 ----------
 
 Make sure to pass the request info to genl_notify() to allow ``NLM_F_ECHO``
-to take effect.
+to take effect.  This is useful for programs that need precise feedback
+from the kernel (for example for logging purposes).
That's very clear, thanks!
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help