Hi,
as far as I could figure out via bisect commit [1] causes a memory leakage as the way how skbs are consumed / freed has been changed since I first developed this patch. This happens in a setup using bla and when the mesh links between mesh nodes sharing a common backbone become very weak and unstable.
I think the line with the "return NET_RX_DROP;" needs to be replaced by "goto skb_free;". How is the correct procedure to change this? As this patch has already been committed to the git repo I guess I should send a new patch or is a new version of the old patch necessary?
Best regards, Andreas
[1]: https://git.open-mesh.org/batman-adv.git/commitdiff/bfe2a1971f43ef540ef0440d...
.................................................................. PHOENIX CONTACT ELECTRONICS GmbH
Sitz der Gesellschaft / registered office of the company: 31812 Bad Pyrmont USt-Id-Nr.: DE811742156 Amtsgericht Hannover HRB 100528 / district court Hannover HRB 100528 Geschäftsführer / Executive Board: Ulrich Leidecker, Christoph Leifer __________________________________________________________________ Diese E-Mail enthält vertrauliche und/oder rechtlich geschützte Informationen. Wenn Sie nicht der richtige Adressat sind oder diese E-Mail irrtümlich erhalten haben, informieren Sie bitte sofort den Absender und vernichten Sie diese Mail. Das unerlaubte Kopieren, jegliche anderweitige Verwendung sowie die unbefugte Weitergabe dieser Mail ist nicht gestattet. ---------------------------------------------------------------------------------------------------- This e-mail may contain confidential and/or privileged information. If you are not the intended recipient (or have received this e-mail in error) please notify the sender immediately and destroy this e-mail. Any unauthorized copying, disclosure, distribution or other use of the material or parts thereof is strictly forbidden. ___________________________________________________________________
On Freitag, 19. Mai 2017 08:29:29 CEST Andreas Pape wrote: [...]
I think the line with the "return NET_RX_DROP;" needs to be replaced by "goto skb_free;".
Yes
How is the correct procedure to change this? As this patch has already been committed to the git repo I guess I should send a new patch or is a new version of the old patch necessary?
Thanks for debugging this. Please sent a new patch based on master [1] and add following line above your Signed-off-by
Fixes: bfe2a1971f43 ("batman-adv: drop unicast packets from other backbone gw")
We will take care of submitting it to Linux net for Linux 4.12. If you still have problems with your IT department then you can either send the patch from a private mail account (git-send-email will then add an extra "From:" line to keep your phoenixcontact.com mail address as author mail address) or (if not otherwise possible) sent it again as attachment to me or Simon. We can then forward it to the mailing list.
Kind regards, Sven
[1] Your patch is currently only in master. It should actually be in maint but Simon hadn't found the time to release master as 2017.1. Which is not to bad because now we can still incorporate your fix. It would be different when the patch would already be part of maint. Then it would be better when you should create your patch on top of maint
b.a.t.m.a.n@lists.open-mesh.org