This changes the format inet_ntop() uses for IPv4 addresses to be the 

same as userland one (which makes more sense).
The list of current kernel's inet_ntop() consumers with comments:
https://gist.github.com/yuripv/cbf201e7450eab867b510577cdb8188c
The only consumer that requires attention is 

usr/src/uts/common/io/scsi/adapters/iscsi/persistent.c as it uses the IP 

addresses as keys into persistent storage, so I've added a wrapper there 

keeping old inet_ntop() behaviour - adding the logic which would rewrite 

the nvlist names for such entries is just not worth it IMO.
Everything else is using inet_ntop() to print log messages, or was 

already using a wrapper to fix the format (like SMB and iscsit).

Checked that iscsiadm can delete/update the entries in persistent storage that were created in previous BE.