From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Marek Lindner Date: Wed, 23 Jun 2010 17:58:39 +0200 References: <201006231620.11058.lindner_marek@yahoo.de> <1277302839-20197-1-git-send-email-lindner_marek@yahoo.de> <20100623150913.GJ15420@lunn.ch> In-Reply-To: <20100623150913.GJ15420@lunn.ch> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <201006231758.40146.lindner_marek@yahoo.de> Subject: Re: [B.A.T.M.A.N.] [PATCH 1/2] batman-adv: add routing debug log accessible via debugfs Reply-To: The list for a Better Approach To Mobile Ad-hoc Networking List-Id: The list for a Better Approach To Mobile Ad-hoc Networking List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: b.a.t.m.a.n@lists.open-mesh.org On Wednesday 23 June 2010 17:09:13 Andrew Lunn wrote: > There will be a ring buffer per mesh. When debugging problems between > meshes it is likely both logs will be combined performing a merge > based on the timestamp. However at this point you have lost > information about which line belongs to which mesh. Could we maybe > include the soft interface name in the output? Actually, this information is (indirectly) present, since each log file resides inside debugfs/batman_adv/$mesh_if/log. It would be a little bit redundant to add this on every line, wouldn't you agree ? If someone merges 2 mesh files it still can be added ? > Also, do you really want to do jiffies / HZ? You are throwing away > useful information. Either keep it as jiffies, or scale the remainder > to ms and print it as well? You are right - we could leave it as jiffies. Might be the best solution. Regards, Marek