Re: [ovs-dev] [PATCH 0/3] netdev-offload-tc: Improved debug logs for bad/unused bit masks.

2022-08-04 Thread Ilya Maximets
On 7/29/22 16:30, Ilya Maximets wrote:
> Emerged from Roi's suggestion while discussing tunnel offload fixes.
> It will probably make sense to backport these down to 2.17 as it is
> debug-only and 2.17 will be our new LTS.
> 
> Ilya Maximets (3):
>   dynamic-string: Add function for a sparse hex dump.
>   netdev-offload-tc: Print unused mask bits on failure.
>   tc: Use sparse hex dump while printing inconsistencies.
> 
>  include/openvswitch/dynamic-string.h |  2 ++
>  lib/dynamic-string.c | 36 +---
>  lib/netdev-offload-tc.c  | 12 +-
>  lib/tc.c | 12 +-
>  4 files changed, 47 insertions(+), 15 deletions(-)
> 

Thanks, Roi, for review!  Applied and backported down to 2.17
since it's a debug-only change, so it will be easier to debug
issues on a future LTS branch.

best regards, Ilya Maximets.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


[ovs-dev] [PATCH 0/3] netdev-offload-tc: Improved debug logs for bad/unused bit masks.

2022-07-29 Thread Ilya Maximets
Emerged from Roi's suggestion while discussing tunnel offload fixes.
It will probably make sense to backport these down to 2.17 as it is
debug-only and 2.17 will be our new LTS.

Ilya Maximets (3):
  dynamic-string: Add function for a sparse hex dump.
  netdev-offload-tc: Print unused mask bits on failure.
  tc: Use sparse hex dump while printing inconsistencies.

 include/openvswitch/dynamic-string.h |  2 ++
 lib/dynamic-string.c | 36 +---
 lib/netdev-offload-tc.c  | 12 +-
 lib/tc.c | 12 +-
 4 files changed, 47 insertions(+), 15 deletions(-)

-- 
2.34.3

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev