On 10/23/21 08:51, Han Zhou wrote: > > > On Thu, Oct 14, 2021 at 4:46 AM Ilya Maximets <[email protected] > <mailto:[email protected]>> wrote: >> >> Added new function to return memory usage statistics for database >> objects inside IDL. Statistics similar to what ovsdb-server reports. >> Not counting _Server database as it should be small, hence doesn't >> worth adding extra code to the ovsdb-cs module. Can be added later >> if needed. >> >> ovs-vswitchd is a user in OVS, but this API will be mostly useful for >> OVN daemons. >> >> Signed-off-by: Ilya Maximets <[email protected] <mailto:[email protected]>> >> ---
<snip> > > Hi Ilya, I tested in sandbox: > > [hanzhou@fedora tutorial]$ ovs-vsctl add-br br-int > [hanzhou@fedora tutorial]$ ovs-appctl memory/show > handlers:17 ports:1 revalidators:7 rules:5 > > It didn't show idl-xxx memory for vswitchd. Did I miss something? Weird. Are you sure that patch was correctly applied? On my machine: [ovs]# wget -c https://patchwork.ozlabs.org/project/openvswitch/patch/[email protected]/mbox/ --content-disposition [ovs]# git am -3 ovs-dev-ovsdb-idl-Add-memory-report-function..patch [ovs]# make -j8 sandbox [tutorial]# ovs-appctl memory/show idl-cells:17 [tutorial]# ovs-vsctl add-br br-int [tutorial]# ovs-appctl memory/show handlers:11 idl-cells:96 ports:1 revalidators:5 rules:5 [tutorial]# ovs-vsctl del-br br-int [tutorial]# ovs-appctl memory/show idl-cells:17 Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
