Re: [PATCH net-next 1/2] netdevsim: print human readable IP address

From: Stanislav Fomichev
Date: Wed Oct 09 2024 - 11:31:48 EST


On 10/09, Simon Horman wrote:
> On Tue, Oct 08, 2024 at 12:21:33PM +0000, Hangbin Liu wrote:
> > Currently, IPSec addresses are printed in hexadecimal format, which is
> > not user-friendly. e.g.
> >
> > # cat /sys/kernel/debug/netdevsim/netdevsim0/ports/0/ipsec
> > SA count=2 tx=20
> > sa[0] rx ipaddr=0x00000000 00000000 00000000 0100a8c0
> > sa[0] spi=0x00000101 proto=0x32 salt=0x0adecc3a crypt=1
> > sa[0] key=0x3167608a ca4f1397 43565909 941fa627
> > sa[1] tx ipaddr=0x00000000 00000000 00000000 00000000
> > sa[1] spi=0x00000100 proto=0x32 salt=0x0adecc3a crypt=1
> > sa[1] key=0x3167608a ca4f1397 43565909 941fa627
> >
> > This patch updates the code to print the IPSec address in a human-readable
> > format for easier debug. e.g.
> >
> > # cat /sys/kernel/debug/netdevsim/netdevsim0/ports/0/ipsec
> > SA count=4 tx=40
> > sa[0] tx ipaddr=0.0.0.0
> > sa[0] spi=0x00000100 proto=0x32 salt=0x0adecc3a crypt=1
> > sa[0] key=0x3167608a ca4f1397 43565909 941fa627
> > sa[1] rx ipaddr=192.168.0.1
> > sa[1] spi=0x00000101 proto=0x32 salt=0x0adecc3a crypt=1
> > sa[1] key=0x3167608a ca4f1397 43565909 941fa627
> > sa[2] tx ipaddr=::
> > sa[2] spi=0x00000100 proto=0x32 salt=0x0adecc3a crypt=1
> > sa[2] key=0x3167608a ca4f1397 43565909 941fa627
> > sa[3] rx ipaddr=2000::1
> > sa[3] spi=0x00000101 proto=0x32 salt=0x0adecc3a crypt=1
> > sa[3] key=0x3167608a ca4f1397 43565909 941fa627
> >
> > Signed-off-by: Hangbin Liu <liuhangbin@xxxxxxxxx>
>
> Reviewed-by: Simon Horman <horms@xxxxxxxxxx>
>

Can you also update tools/testing/selftests/net/rtnetlink.sh
accordingly? There is a part that diffs this file and it now fails due
to new format.

---
pw-bot: cr