On Thu, Sep 29, 2022 at 08:47:15AM -0700, Jakub Kicinski wrote: > 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!