On Mon, Oct 25, 2021 at 5:39 AM Ilya Maximets <[email protected]> wrote: > > 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?
My bad! I forgot that this patch is not in a series :P Acked-by: Han Zhou <[email protected]> > > 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
