On Wed, Jun 30, 2010 at 12:21:08PM +0200, Sven Eckelmann wrote:
Linus Lüssing wrote:
From: mesh-node <mesh-node@mesh-node1.(none)>
I think the person who applies this should remove the From and the next empty line... or do you have a secret dwarf named mesh-node who works for you? :D
Ah no, I wanted to keep this a secret... I also just noticed this when I reread my post here, didn't notice the wrong From in the patch file. Hope the commiter doesn't mind changing that to not make other kernel devs jealous about my dwarf? :)
Right now, there is no easy/intuitive way to find out whether a node might have vanished. This commit adds the time when a node was last seen to the originator table output, so that a common user is able to tell whether a node might have gone without having to wait PURGE_TIMEOUT seconds until that node gets "garbe-collected".
The rest looks good, but was a little bit irritated about the change of the table header (which makes sense to split it in two lines, but didn't expected that this patch does that).
Oki doki, just did that in the same patch because the extra info made the line a little bit long. Is adding a line like this fine?
"It also puts the the versioning information in an extra line, as the first one of this debug output would otherwise get too long."
Best regards, Sven
Thanks for reviewing, Sven
Cheers, Linus