public inbox for b.a.t.m.a.n@lists.open-mesh.org
 help / color / mirror / Atom feed
From: cmsv <cmsv@wirelesspt.net>
To: b.a.t.m.a.n@lists.open-mesh.org, daniel@makrotopia.org
Subject: Re: [B.A.T.M.A.N.] batman-adv: memory leak?
Date: Mon, 27 Jan 2014 12:55:09 -0500	[thread overview]
Message-ID: <52E69D7D.80905@wirelesspt.net> (raw)
In-Reply-To: <52E5341F.30800@meshcoding.com>

[-- Attachment #1: Type: text/plain, Size: 2761 bytes --]

Here is an update of some tests i ran in the past 24h with the following
build:

routers used:
dlink dir 601a and tplink wr703n in "ng" mode. (atheros)

My current AA DISTRIB_REVISION="r39154"
mac80211 r39150 from openwrt trunk
hostapd r39155 from trunk

From batman-adv i am using the following patches:
ls  feeds/routing/batman-adv/patches/
0001-batman-adv-fix-batman-adv-header-overhead-calculatio.patch

From d72756b97529b3c6afa08933216aaa912bb16ce6 Mon Sep 17 00:00:00 2001
From: Marek Lindner <mareklindner@neomailbox.ch>
Date: Wed, 15 Jan 2014 20:31:18 +0800
Subject: [PATCH] batman-adv: fix batman-adv header overhead calculation


batman-adv/Makefile
# $Id: Makefile 5624 2006-11-23 00:29:07Z nbd $

include $(TOPDIR)/rules.mk

PKG_NAME:=batman-adv

PKG_VERSION:=2014.0.0
BATCTL_VERSION:=2014.0.0
PKG_RELEASE:=1
PKG_MD5SUM:=8d58ecaede17dc05aab1b549dc09fa7d
BATCTL_MD5SUM:=b0bcf29fef80ddcc33769e13f5937d0a


I tried to find any memory leaks that could be causing reboots and i was
unable to find any after having compiled the build with
batman-adv-header-overhead-calculatio.patch. Before this patch i did get
reboots caused by the leak.


I keep monitoring memory usage with top, htop, ps and /proc/meminfo
since i was not able to install valgrind due to lack of available flash
memory given the size of the valgrind package.

Got some tips from here:
http://blog.thewebsitepeople.org/2011/03/linux-memory-leak-detection

Additionally i ran iperf tests on both routers against each other to force
them under heavy load during 24h: iperf -c <ip> -t 99999 -i 5

The mtu is 1560 for the adhoc.

After 24h i still had 6 mb of ram and above on both routers. Once i
stopped the tests; the ram increased.

Dmesh and logread output nothing wrong and or errors.

No reboots happened during this time which leads me to conclude that the
problem might not be all from batman-adv side or maybe not even at all
or maybe only happens when in use with something very specific.

I would like to run a few more tests to be more sure about possible
leaks but are there any other tools that someone might recommend ?

@ daniel
What did you use to find the leak and or how did you troubleshoot it ?



On 01/26/2014 11:13 AM, Antonio Quartulli wrote:
> On 26/01/14 17:07, Antonio Quartulli wrote:Can you explain in what
>>
>> This is a patch to fix the memleak we were discussing about.
>> This bug appeared with and it is meant to be applied on
>> batman-adv-2014.0.0 (regardless of the openwrt revision).
> 
> sorry, bad copy/paste.
> 
> The patch is for batman-adv-2014.0.0 (I don't know what version you have
> in AA).
> It fixes the memleak bug that we were discussing about.
> 
> 


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 901 bytes --]

  reply	other threads:[~2014-01-27 17:55 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-21 10:22 [B.A.T.M.A.N.] [PATCH maint] batman-adv: fix soft-interface MTU computation Antonio Quartulli
2014-01-21 10:31 ` Antonio Quartulli
2014-01-27  7:57   ` Marek Lindner
2014-01-21 18:43 ` Russell Senior
2014-01-21 19:00   ` Antonio Quartulli
2014-01-22  6:04   ` Russell Senior
2014-01-22  7:03     ` Antonio Quartulli
2014-01-22  7:04     ` [B.A.T.M.A.N.] memleak (Was: [PATCH maint] batman-adv: fix soft-interface MTU computation) Antonio Quartulli
2014-01-22  7:37     ` [B.A.T.M.A.N.] [PATCH maint] batman-adv: fix soft-interface MTU computation Daniel
2014-01-22 17:45       ` [B.A.T.M.A.N.] batman-adv: memory leak? Russell Senior
2014-01-22 17:46         ` Antonio Quartulli
2014-01-22 19:18           ` Russell Senior
2014-01-22 20:49           ` cmsv
2014-01-22 23:57             ` Russell Senior
2014-01-23  0:10               ` cmsv
2014-01-23  3:35                 ` Daniel
2014-01-26 12:57                   ` Daniel
2014-01-26 14:21                     ` Antonio Quartulli
2014-01-26 14:24                       ` [B.A.T.M.A.N.] [PATCH maint] batman-adv: release vlan object after checking the CRC Antonio Quartulli
2014-01-27 18:43                         ` Russell Senior
2014-01-26 16:05                       ` [B.A.T.M.A.N.] batman-adv: memory leak? cmsv
2014-01-26 16:07                         ` Antonio Quartulli
2014-01-26 16:13                           ` Antonio Quartulli
2014-01-27 17:55                             ` cmsv [this message]
2014-01-28  1:21                               ` Russell Senior
2014-01-28  1:30                                 ` cmsv
2014-01-29  8:10                                   ` Russell Senior
2014-01-29 21:48                                     ` cmsv
2014-02-08  3:08                                       ` cmsv
2014-02-08 10:53                                         ` Felix Fietkau
2014-02-12  7:23                                         ` Antonio Quartulli
2014-02-12 10:40                                           ` cmsv
2014-02-12 11:41                                             ` Antonio Quartulli
2014-02-13  0:55                                               ` cmsv
2014-02-13  7:23                                                 ` Antonio Quartulli

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=52E69D7D.80905@wirelesspt.net \
    --to=cmsv@wirelesspt.net \
    --cc=b.a.t.m.a.n@lists.open-mesh.org \
    --cc=daniel@makrotopia.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox