On Friday, April 28, 2017 10:26:10 PM CEST Simon Wunderlich wrote: > From: Andreas Pape <ap...@phoenixcontact.com> > > Like in the case of the patch for batadv_bla_tx to handle a race > condition when claiming a mac address for bla, a similar situation > can occur when claiming is triggered via batadv_bla_rx. This patch > solves this with a similar approach as for batadv_bla_tx. > > Signed-off-by: Andreas Pape <ap...@phoenixcontact.com>
Hi Andreas,
thanks again for the patch - in general, I think this looks good, although I
don't follow completely where you saw that. Can you describe the scenario a
little more?
We usually don't process packets from the mesh sent by nodes on the same LAN
segment - we look at the originator and check the BLA group using
batadv_check_claim_group().
There are two things which we could improve documentation-wise:
1.) Have some kernel doc batadv_tt_local_has_timed_out - we want to have
kerneldoc for every new function we add.
2.) Describe the scenario in a comment in batadv_bla_rx(). I find the comment
not too convincing, see above.
Again, if this situation is really happening then I believe this patch
provides a good solution which I'd like to adopt. :)
Thank you,
Simon
signature.asc
Description: This is a digitally signed message part.
