public inbox for b.a.t.m.a.n@lists.open-mesh.org
 help / color / mirror / Atom feed
From: Marek Lindner <lindner_marek@yahoo.de>
To: The list for a Better Approach To Mobile Ad-hoc Networking
	<b.a.t.m.a.n@open-mesh.net>
Subject: Re: [B.A.T.M.A.N.] Kernel crashes with batgat installed
Date: Thu, 21 May 2009 03:02:22 +0800	[thread overview]
Message-ID: <200905210302.22639.lindner_marek@yahoo.de> (raw)
In-Reply-To: <4313f3060905201001u5658c660p5a9be3d63dbff150@mail.gmail.com>

On Thursday 21 May 2009 01:01:43 Nathan Wharton wrote:
> In this case, it does the same thing whether or not batgat is installed.

Ok.

I miss a couple of things in your output - do you use the plain sources from 
open-mesh.net or do you apply custom patches ?


> Debug level 4 gives:
> ========================================
> WARNING: You are using the unstable batman branch. If you are
> interested in *using* batman get the lat
> est stable release !
> Deleting throw route to 10.1.3.0/24 via 0.0.0.0 (table 66 - unknown)
> Deleting throw route to 10.1.3.0/24 via 0.0.0.0 (table 66 - unknown)
> Deleting throw route to 10.1.3.0/24 via 0.0.0.0 (table 65 - unknown)
> Deleting throw route to 10.1.3.0/24 via 0.0.0.0 (table 65 - unknown)
> Deleting throw route to 10.1.3.0/24 via 0.0.0.0 (table 67 - unknown)
> Deleting throw route to 10.1.3.0/24 via 0.0.0.0 (table 67 - unknown)

Your log indicates that all routes are still present and batman tries to clean 
them up while starting. As you can see here table 68 is not mentioned. On my 
machine I get:

Deleting throw route to 105.0.0.0/8 via 0.0.0.0 (table 68 - unknown)


> [        30] Adding throw route to 10.1.3.0/24 via 0.0.0.0 (table 65)
> [        30] Adding throw route to 10.1.3.0/24 via 0.0.0.0 (table 66)

Here we lack the message that says we found a new HNA:
Adding HNA to announce network list: 105.0.0.0/8


> It does this while not connected.

I could make a patch that produces more debug output to get to the root of it 
but first we have to make sure we run the same code ...


> If you consider 1 bug being the debug_malloc stuff not working, and
> the other being batgat possibly crashing the kernel, then yes.
> If I turn off debug malloc, then everything works fine, except using
> batgat and going from gateway to routing class.

Ok, lets do the malloc stuff first and then we move to the batgat issue.

Just to be clear here: DEBUG_MALLOC is not the problem - it just makes the 
problem visible. Everytime batman allocates memory the debugger will allocate 
more than needed to add its debugging information. Now the debugging 
information gets overwritten and the debugger tells you that (including a 
direction towards the source of the problem). If you deactivate the debugger 
the memory will still be overwritten but you don't notice it! 
It can destroy arbitrary structures in the memory that need hours to lead to a 
crash (if it all). May be it leads to broken routing entries ..

Regards,
Marek



  reply	other threads:[~2009-05-20 19:02 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-19 14:27 [B.A.T.M.A.N.] Kernel crashes with batgat installed Nathan Wharton
2009-05-19 19:21 ` Sven Eckelmann
2009-05-19 20:38   ` Nathan Wharton
2009-05-20  1:30     ` Marek Lindner
2009-05-20 14:34       ` Nathan Wharton
2009-05-20 16:10         ` Marek Lindner
2009-05-20 17:01           ` Nathan Wharton
2009-05-20 19:02             ` Marek Lindner [this message]
2009-05-20 19:39               ` Nathan Wharton
2009-05-28 10:40 ` [B.A.T.M.A.N.] [PATCH] [batman] Add padding around allocation debugger structures Sven Eckelmann
2009-05-29  7:02   ` Marek Lindner
2009-05-29 14:00     ` Nathan Wharton
2009-06-01 16:44       ` Sven Eckelmann
2009-06-01 18:03         ` Nathan Wharton
2009-06-01 19:35           ` Sven Eckelmann
2009-06-01 21:50             ` Nathan Wharton
2009-06-02  4:36             ` Marek Lindner
2009-06-02 17:50               ` [B.A.T.M.A.N.] " Sven Eckelmann
2009-06-02 17:56                 ` [B.A.T.M.A.N.] [PATCH] [batman] Align pointers in hna list elements Sven Eckelmann
2009-06-02 18:56                   ` Nathan Wharton
2009-06-03 10:39                   ` [B.A.T.M.A.N.] [PATCHv2] " Sven Eckelmann
2009-06-03 11:16                     ` Marek Lindner
2009-05-28 11:36 ` [B.A.T.M.A.N.] [PATCH 2/3] [batman] Make TYPE_OF_WORD the largest integral type Sven Eckelmann
2009-05-28 11:36 ` [B.A.T.M.A.N.] [PATCH 3/3] [batman] Word-Align char buffer which are later casted to larger data types Sven Eckelmann

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=200905210302.22639.lindner_marek@yahoo.de \
    --to=lindner_marek@yahoo.de \
    --cc=b.a.t.m.a.n@open-mesh.net \
    /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