[tickets] [opensaf:tickets] #3151 mds: memleak for agent enable flow control
- **status**: review --> fixed - **Comment**: commit b8ab2c8a180b5b1ba110a02ecd60a1001ebddbc6 (HEAD -> develop, origin/develop) Author: thuan.tran Date: Mon Feb 10 14:10:23 2020 +0700 mds: fix memleak in agent enable flow control [#3151] Agent enable flow control keep add new portid without remove. Remove portid when svc count become zero then handle portid reset properly, peer A may see portid reset (peer B) then peer B should accept fseq(1) message from peer A. --- ** [tickets:#3151] mds: memleak for agent enable flow control** **Status:** fixed **Milestone:** 5.20.01 **Created:** Wed Feb 12, 2020 02:01 AM UTC by Thuan Tran **Last Updated:** Wed Feb 12, 2020 10:38 AM UTC **Owner:** Thuan Tran Memory usage keep increase for osafimmnd when it enable flow control >From mds log, portid is added seems never remove. ~~~ <141>1 2020-02-01T05:33:42.760481+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033985"] FCTRL: Add portid[node:1001004, ref:995471411 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.76341+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033987"] FCTRL: Remove svc[node:1001004, ref:995471411 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.773691+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033990"] FCTRL: Add portid[node:1001004, ref:1567425026 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.775878+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033992"] FCTRL: Remove svc[node:1001004, ref:1567425026 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.785244+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033995"] FCTRL: Add portid[node:1001004, ref:1741128394 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.786894+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033997"] FCTRL: Remove svc[node:1001004, ref:1741128394 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.796011+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034000"] FCTRL: Add portid[node:1001004, ref:3116260044 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.797643+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034002"] FCTRL: Remove svc[node:1001004, ref:3116260044 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.805084+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034005"] FCTRL: Add portid[node:1001004, ref:2936038224 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.807109+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034007"] FCTRL: Remove svc[node:1001004, ref:2936038224 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.816094+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034010"] FCTRL: Add portid[node:1001004, ref:3630381079 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.817738+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034012"] FCTRL: Remove svc[node:1001004, ref:3630381079 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.826855+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034015"] FCTRL: Add portid[node:1001004, ref:1347521832 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.828468+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034017"] FCTRL: Remove svc[node:1001004, ref:1347521832 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.835071+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034020"] FCTRL: Add portid[node:1001004, ref:3871464460 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:38:18.168011+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1045358"] FCTRL: Add portid[node:1001004, ref:1010076045 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:38:18.17015+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1045360"] FCTRL: Remove svc[node:1001004, ref:1010076045 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:38:18.178384+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1045363"] FCTRL: Add portid[node:1001004, ref:352812852 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:38:18.183857+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1045365"] FCTRL: Remove svc[node:1001004, ref:352812852 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:38:18.192214+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1045368"] FCTRL: Add portid[node:1001004, ref:3778790219 svc_id:26], svc_cnt:1 ~~~ --- Sent from sourceforge.net because opensaf-tickets@lists.sourceforge.net is subscribed to https://sourceforge.net/p/opensaf/tickets/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/opensaf/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.___ Opensaf-tickets mailing list Opensaf-tickets@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-tickets
[tickets] [opensaf:tickets] #3151 mds: memleak for agent enable flow control
- **status**: assigned --> review --- ** [tickets:#3151] mds: memleak for agent enable flow control** **Status:** review **Milestone:** 5.20.01 **Created:** Wed Feb 12, 2020 02:01 AM UTC by Thuan Tran **Last Updated:** Wed Feb 12, 2020 02:01 AM UTC **Owner:** Thuan Tran Memory usage keep increase for osafimmnd when it enable flow control >From mds log, portid is added seems never remove. ~~~ <141>1 2020-02-01T05:33:42.760481+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033985"] FCTRL: Add portid[node:1001004, ref:995471411 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.76341+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033987"] FCTRL: Remove svc[node:1001004, ref:995471411 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.773691+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033990"] FCTRL: Add portid[node:1001004, ref:1567425026 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.775878+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033992"] FCTRL: Remove svc[node:1001004, ref:1567425026 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.785244+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033995"] FCTRL: Add portid[node:1001004, ref:1741128394 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.786894+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033997"] FCTRL: Remove svc[node:1001004, ref:1741128394 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.796011+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034000"] FCTRL: Add portid[node:1001004, ref:3116260044 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.797643+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034002"] FCTRL: Remove svc[node:1001004, ref:3116260044 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.805084+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034005"] FCTRL: Add portid[node:1001004, ref:2936038224 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.807109+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034007"] FCTRL: Remove svc[node:1001004, ref:2936038224 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.816094+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034010"] FCTRL: Add portid[node:1001004, ref:3630381079 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.817738+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034012"] FCTRL: Remove svc[node:1001004, ref:3630381079 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.826855+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034015"] FCTRL: Add portid[node:1001004, ref:1347521832 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.828468+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034017"] FCTRL: Remove svc[node:1001004, ref:1347521832 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.835071+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034020"] FCTRL: Add portid[node:1001004, ref:3871464460 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:38:18.168011+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1045358"] FCTRL: Add portid[node:1001004, ref:1010076045 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:38:18.17015+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1045360"] FCTRL: Remove svc[node:1001004, ref:1010076045 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:38:18.178384+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1045363"] FCTRL: Add portid[node:1001004, ref:352812852 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:38:18.183857+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1045365"] FCTRL: Remove svc[node:1001004, ref:352812852 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:38:18.192214+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1045368"] FCTRL: Add portid[node:1001004, ref:3778790219 svc_id:26], svc_cnt:1 ~~~ --- Sent from sourceforge.net because opensaf-tickets@lists.sourceforge.net is subscribed to https://sourceforge.net/p/opensaf/tickets/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/opensaf/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.___ Opensaf-tickets mailing list Opensaf-tickets@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-tickets
[tickets] [opensaf:tickets] #3151 mds: memleak for agent enable flow control
--- ** [tickets:#3151] mds: memleak for agent enable flow control** **Status:** assigned **Milestone:** 5.20.01 **Created:** Wed Feb 12, 2020 02:01 AM UTC by Thuan Tran **Last Updated:** Wed Feb 12, 2020 02:01 AM UTC **Owner:** Thuan Tran Memory usage keep increase for osafimmnd when it enable flow control >From mds log, portid is added seems never remove. ~~~ <141>1 2020-02-01T05:33:42.760481+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033985"] FCTRL: Add portid[node:1001004, ref:995471411 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.76341+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033987"] FCTRL: Remove svc[node:1001004, ref:995471411 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.773691+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033990"] FCTRL: Add portid[node:1001004, ref:1567425026 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.775878+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033992"] FCTRL: Remove svc[node:1001004, ref:1567425026 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.785244+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033995"] FCTRL: Add portid[node:1001004, ref:1741128394 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.786894+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1033997"] FCTRL: Remove svc[node:1001004, ref:1741128394 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.796011+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034000"] FCTRL: Add portid[node:1001004, ref:3116260044 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.797643+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034002"] FCTRL: Remove svc[node:1001004, ref:3116260044 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.805084+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034005"] FCTRL: Add portid[node:1001004, ref:2936038224 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.807109+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034007"] FCTRL: Remove svc[node:1001004, ref:2936038224 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.816094+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034010"] FCTRL: Add portid[node:1001004, ref:3630381079 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.817738+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034012"] FCTRL: Remove svc[node:1001004, ref:3630381079 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.826855+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034015"] FCTRL: Add portid[node:1001004, ref:1347521832 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:33:42.828468+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034017"] FCTRL: Remove svc[node:1001004, ref:1347521832 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:33:42.835071+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1034020"] FCTRL: Add portid[node:1001004, ref:3871464460 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:38:18.168011+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1045358"] FCTRL: Add portid[node:1001004, ref:1010076045 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:38:18.17015+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1045360"] FCTRL: Remove svc[node:1001004, ref:1010076045 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:38:18.178384+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1045363"] FCTRL: Add portid[node:1001004, ref:352812852 svc_id:26], svc_cnt:1 <141>1 2020-02-01T05:38:18.183857+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1045365"] FCTRL: Remove svc[node:1001004, ref:352812852 svc_id:26], svc_cnt:0 <141>1 2020-02-01T05:38:18.192214+01:00 PL-4 osafimmnd 10676 mds.log [meta sequenceId="1045368"] FCTRL: Add portid[node:1001004, ref:3778790219 svc_id:26], svc_cnt:1 ~~~ --- Sent from sourceforge.net because opensaf-tickets@lists.sourceforge.net is subscribed to https://sourceforge.net/p/opensaf/tickets/ To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/opensaf/admin/tickets/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.___ Opensaf-tickets mailing list Opensaf-tickets@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-tickets
