On 10/05/18 04:38, Sven Eckelmann wrote:
Let me rephrase that: Are we sure that this is from intermediate nodes or is there still the possibility that it is from the actual originator? Regarding the latter: Should batadv_tt_prepare_tvlv_local_data be modified or will you wait for Leonardo to debug why it happens in the first place?
Wait. When it is a problem of an intermediate node that doesn't have (for whatever reason) not a valid copy of the entries, shouldn't the intermediate node just ignore the request and not answer it? Otherwise the next OGM with the data from batadv_tt_prepare_tvlv_local_data from the actual node will just trigger the next request by the receiver.
And the next thing is that your commit message is talking about OGMs all the time. But the OGM TVLV are not filled by the batadv_tt_prepare_tvlv_global_data but by the batadv_tt_prepare_tvlv_local_data. I am unable to find out what you actually doing here and why you do it.
I believe batadv_tt_prepare_tvlv_local_data() was the actual function that Marek wanted to modify. But we probably spent too much time digging into translation-table.c these days and I managed to confuse him enough.
Thanks for pointing this out, Sven!
Cheers,