On 2/16/26 9:26 AM, Eelco Chaudron wrote:
> 
> 
> On 13 Feb 2026, at 21:19, Ilya Maximets wrote:
> 
>> The ovs-parse-backtrace script was supposed to be used for parsing the
>> output of 'ovs-appctl backtrace' command.  But that command was
>> removed back in 2013 while making OVS thread-safe.  The script itself
>> is also not even compatible with python3 (some decoding of byte objects
>> is missing).
>>
>> We could've fixed it and adjusted to parse out backtraces from the
>> logs instead, but a simple grep with addr2line can do a very similar
>> job, e.g.:
>>
>>   grep -o '^0x[0-9a-f]*' ./ovs-vswitchd.log \
>>      | xargs addr2line -a -f -s -p -e $(which ovs-vswitchd)
>>
>> Also, GDB is generally more useful for this task, but that would be a
>> complete re-write of the script that no-one have used for 13 years.
>> We can just add a new one if we ever need it.  For now, just remove
>> the broken script and clean up the mentions of it.
>>
>> Fixes: 94f3e5269e93 ("timeval: Remove backtrace feature.")
>> Signed-off-by: Ilya Maximets <[email protected]>
> 
> Thanks for the cleanup, it looks good to me.
> 
> Acked-by: Eelco Chaudron <[email protected]>
> 

Thanks!  Applied.

We could technically backport this, but it doesn't seem important.
Can backport later if necessary.

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to