On Tue, 30 Jun 2026 at 23:33, Timothy Redaelli via dev <[email protected]> wrote: > > Split libopenvswitch into two libraries: > > - libopenvswitchutils: containers, utilities, I/O, threading, > logging, and OVSDB client modules. > - libopenvswitch: datapath, OpenFlow, netdev, dpif, flow, > conntrack, and related modules. Links libopenvswitchutils.
The vswitch idl code is also part of the libopenvswitch library. > > Programs that do not need datapath functionality now link only > libopenvswitchutils, reducing their dependency footprint: > ovsdb-server, ovsdb-client, ovsdb-tool, ovs-appctl, vtep-ctl. > > To break a circular dependency between the two libraries, > ip_parse(), ipv6_parse(), and ipv6_string_mapped() are made > static inline in packets.h. These are trivial wrappers around > inet_pton()/inet_ntop() with no side effects or state. > > Signed-off-by: Timothy Redaelli <[email protected]> > --- > Changes since v4: > - Squashed into a single patch. > - Dropped net-proto rename per Ilya's review: keep packets.c/h > as-is, no new net-proto.c/h files. > - Dropped pop_mpls and other function moves to dp-packet per > Ilya's review. > - Inlined ip_parse(), ipv6_parse(), ipv6_string_mapped() in > packets.h to resolve cross-library dependency (trivial > wrappers around inet_pton/inet_ntop). > - Fixed shared library builds: link libopenvswitchutils.la > explicitly in every executable that also links libopenvswitch.la > (utilities, vswitchd, tests, oss-fuzz), so the directly > referenced utils symbols resolve under --enable-shared. > - Fixed debian Python extension: link libopenvswitchutils > instead of libopenvswitch. > - Fixed sparse checker: include ordering in packets.h. > - Kept the generated vswitch-idl in libopenvswitch (datapath and > schema consumers); only dirs and ovsdb-server-idl moved to the > libopenvswitchutils sources. Well, on this last point, that forces tools like ovs-vsctl to link against libopenvswitch which in turn pulls in dpdk. Are you still gaining space from this rework (asking, as I see no number in the commitlog)? Maybe this is enough as a first step. -- David Marchand _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
