[ovs-dev] [PATCH 1/1] ovsdb-tool: fix memory leak while converting cluster into standalone database

aginwala aginwala at asu.edu
Tue Mar 24 19:35:21 UTC 2020


Sounds good Ben.

I sent https://patchwork.ozlabs.org/patch/1260937/ and
https://patchwork.ozlabs.org/patch/1260935/ for 2.11.

On Tue, Mar 24, 2020 at 9:53 AM Ben Pfaff <blp at ovn.org> wrote:

> Probably best to post a rebased series.
>
> On Fri, Mar 20, 2020 at 11:43:41AM -0700, aginwala wrote:
> > Hi Ben:
> >
> > I checked the history for 2.11 and this patch is failing to backport due
> to
> > this commit
> >
> https://github.com/openvswitch/ovs/commit/5832e6a4f103a1eddd62c9ba459ce669e3a5d132
> > as it updates OVSB fast re-sync in news and its ported 2.12 . Hence,
> > backport to 2.12 worked. If we get
> >
> https://github.com/openvswitch/ovs/commit/5832e6a4f103a1eddd62c9ba459ce669e3a5d132
> > in
> > too to 2.11 it will apply cleanly. Let me know else I will have to amend
> > the news and it will fail backport for fast re-syc if we move that to
> 2.11
> > in future.
> >
> > On Thu, Mar 19, 2020 at 4:22 PM aginwala <aginwala at asu.edu> wrote:
> >
> > > Ok sounds good. Will do that. Thanks a ton!
> > >
> > > On Thu, Mar 19, 2020 at 3:53 PM Ben Pfaff <blp at ovn.org> wrote:
> > >
> > >> I backported both to 2.12.
> > >>
> > >> They need a manual backport to 2.11, will you take care of it?
> > >>
> > >> On Thu, Mar 19, 2020 at 03:39:58PM -0700, aginwala wrote:
> > >> > Oh I see it seems the previous patch needs to be backported too
> > >> >
> > >>
> https://lists.linuxfoundation.org/pipermail/ovs-dev/2019-September/362925.html
> > >> > .
> > >> > Please see if you can get that too on 2.11 and 2.12
> > >> >
> > >> >
> > >> >
> > >> >
> > >> > On Thu, Mar 19, 2020 at 3:35 PM Ben Pfaff <blp at ovn.org> wrote:
> > >> >
> > >> > > It doesn't apply cleanly.
> > >> > >
> > >> > > On Thu, Mar 19, 2020 at 03:07:39PM -0700, aginwala wrote:
> > >> > > > Hi Ben:
> > >> > > >
> > >> > > > Thanks for backporting previous patches. Please see if you can
> back
> > >> port
> > >> > > > this one too to 2.11 and 2.12.
> > >> > > >
> > >> > > > On Thu, Mar 19, 2020 at 1:53 PM aginwala <aginwala at asu.edu>
> wrote:
> > >> > > >
> > >> > > > > Hi Ben:
> > >> > > > > Can you also backport this patch to 2.12 and 2.11 too?
> > >> > > > >
> > >> > > > > On Mon, Oct 7, 2019 at 11:22 AM Ben Pfaff <blp at ovn.org>
> wrote:
> > >> > > > >
> > >> > > > >> On Mon, Oct 07, 2019 at 10:10:34AM +0200, Damijan Skvarc
> wrote:
> > >> > > > >> > memory leak is reported by valgrind while executing
> functional
> > >> test
> > >> > > > >> > "ovsdb-tool convert-to-standalone"
> > >> > > > >> >
> > >> > > > >> > ==13842== 2,850 (280 direct, 2,570 indirect) bytes in 7
> blocks
> > >> are
> > >> > > > >> definitely lost in loss record 20 of 20
> > >> > > > >> > ==13842==    at 0x4C2DB8F: malloc (in
> > >> > > > >> /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
> > >> > > > >> > ==13842==    by 0x45EE2E: xmalloc (util.c:138)
> > >> > > > >> > ==13842==    by 0x43E386: json_create (json.c:1451)
> > >> > > > >> > ==13842==    by 0x43BDD2: json_object_create (json.c:254)
> > >> > > > >> > ==13842==    by 0x43DEE3: json_parser_push_object
> (json.c:1273)
> > >> > > > >> > ==13842==    by 0x43E167: json_parser_input (json.c:1371)
> > >> > > > >> > ==13842==    by 0x43D6EA: json_lex_input (json.c:991)
> > >> > > > >> > ==13842==    by 0x43DAC1: json_parser_feed (json.c:1149)
> > >> > > > >> > ==13842==    by 0x40D108: parse_body (log.c:411)
> > >> > > > >> > ==13842==    by 0x40D386: ovsdb_log_read (log.c:476)
> > >> > > > >> > ==13842==    by 0x406A0B: do_convert_to_standalone
> > >> > > (ovsdb-tool.c:1571)
> > >> > > > >> > ==13842==    by 0x406A0B: do_cluster_standalone
> > >> (ovsdb-tool.c:1606)
> > >> > > > >> > ==13842==    by 0x438670: ovs_cmdl_run_command__
> > >> > > (command-line.c:223)
> > >> > > > >> > ==13842==    by 0x438720: ovs_cmdl_run_command
> > >> (command-line.c:254)
> > >> > > > >> > ==13842==    by 0x405A4C: main (ovsdb-tool.c:79)
> > >> > > > >> >
> > >> > > > >> > The problem was in do_convert_to_standalone() function
> which
> > >> while
> > >> > > > >> reading log file
> > >> > > > >> > allocate json object which was not deallocated at the end.
> > >> > > > >> >
> > >> > > > >> > Signed-off-by: Damijan Skvarc <damjan.skvarc at gmail.com>
> > >> > > > >>
> > >> > > > >> Applied to master, thanks.
> > >> > > > >> _______________________________________________
> > >> > > > >> dev mailing list
> > >> > > > >> dev at openvswitch.org
> > >> > > > >> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> > >> > > > >>
> > >> > > > >
> > >> > >
> > >>
> > >
>


More information about the dev mailing list