Re: [PATCH v2 2/2] tools: ynl: add unicast notification receive support

From: 侯敏熙

Date: Tue May 19 2026 - 06:02:31 EST


Hello Donald

Should I re-spain a new version to fix the yaml check issue?

====== Checking the tree with the patch ======
New warnings added
--- /tmp/tmp.2uYmXlzyoK 2026-05-18 12:59:32.034644770 -0400
+++ /tmp/tmp.UAwK6To8IX 2026-05-18 12:59:32.187643683 -0400
@@ -0,0 +1,3 @@
+ovs_packet.yaml
+ 3:1 warning missing document start "---" (document-start)

+
B.R
Minxi

Donald Hunter <donald.hunter@xxxxxxxxx> 于2026年5月19日周二 17:45写道:
>
> Minxi Hou <houminxi@xxxxxxxxx> writes:
>
> > Add ntf_listen() method to YnlFamily for binding the netlink
> > socket without joining a multicast group. This enables receiving
> > unicast notifications through the existing poll_ntf/check_ntf
> > path.
> >
> > The OVS packet family sends MISS and ACTION upcalls via
> > genlmsg_unicast() to a per-vport PID rather than through a
> > multicast group. The existing ntf_subscribe() couples bind()
> > with setsockopt(ADD_MEMBERSHIP), which does not fit the unicast
> > case. ntf_listen() provides the bind-only alternative.
> >
> > Signed-off-by: Minxi Hou <houminxi@xxxxxxxxx>
>
> Reviewed-by: Donald Hunter <donald.hunter@xxxxxxxxx>