Please have a look at the output below after I incorporated the
suggestions given.
I will be adding support for -P option for both dcmds.
> ::ill
ADDR NAME CNT VER WQ UP TYPE FLAGS
60012dca688 lo0 0 v4 0 R LOOPB M,V
6001479d528 e1000g0 34 v4 60014f13d58 R ETHER M,R
60013ac69c8 lo0 2 v6 0 R LOOPB M,V
6001479d1a8 e1000g0 20 v6 60016675ad8 R ETHER M
> ::ill -v
ADDR NAME VER WQ UP IPST
TYPE FLAGS
PHYINT CNT GROUP WAITERS
60012dca688 lo0 v4 0 R 60012b0e000
LOOPB M,V
600139d8b88 0 0 0
|
+----> 0 ill_refcnt
0 ill_ire_cnt
0 ill_nce_cnt
0 ill_ilm_cnt
0 ill_walker_cnt
6001479d528 e1000g0 v4 60014f13d58 R 60012b0e000
ETHER M,R
600145988c8 34 0 0
|
+----> 0 ill_refcnt
18 ill_ire_cnt
16 ill_nce_cnt
0 ill_ilm_cnt
0 ill_walker_cnt
.....
....
> ::ipif
ADDR NAME CNT ILL STFLAGS
FLAGS
LCLADDR
600139d8688 lo0:0 2 60012dca688 <>
<UP>
127.0.0.1/8
600145997c8 e1000g0:0 28 6001479d528 <>
<UP,DHCP>
10.8.57.171/24
60014599cc8 lo0:0 1 60013ac69c8 <>
<UP>
::1/128
600139d8408 e1000g0:0 8 6001479d1a8 <>
<UP>
fe80::214:4fff:fe1d:98c6/10
6001669e2c8 e1000g0:1 2 6001479d1a8 <>
<UP,ACONF>
2002:a08:39f0:1:214:4fff:fe1d:98c6/64
> ::ipif -v
ADDR NAME CNT ILL ADR STFLAGS
FLAGS
LCLADDR
BROADCAST
600139d8688 lo0:0 2 60012dca688 Y <>
<UP>
|
+---> 0 ipif_refcnt
1 ipif_ire_cnt
1 ipif_ilm_cnt
127.0.0.1/8
0.0.0.0
600145997c8 e1000g0:0 28 6001479d528 Y <>
<UP,DHCP>
|
+---> 0 ipif_refcnt
26 ipif_ire_cnt
2 ipif_ilm_cnt
10.8.57.171/24
10.8.57.255
...
...
600139d8408 e1000g0:0 8 6001479d1a8 Y <>
<UP>
|
+---> 0 ipif_refcnt
8 ipif_ire_cnt
0 ipif_ilm_cnt
fe80::214:4fff:fe1d:98c6/10
::
6001669e2c8 e1000g0:1 2 6001479d1a8 Y <>
<UP,ACONF>
|
+---> 0 ipif_refcnt
2 ipif_ire_cnt
0 ipif_ilm_cnt
2002:a08:39f0:1:214:4fff:fe1d:98c6/64
::
Thanks,
Vasumathi
Peter Memishian wrote:
> > > ::ill
> > ADDR NAME ILL_WQ TYPE IPST FLAGS REFCNT IRECNT
> > NCECNT
>
> Some comments:
> - the flags should be printed using some char mnemonic, not as a hex
> number (see, for example, the output of ::nce)
>
> - refcnt should not be very large. And usually the irecnt and ncecnt
> should als not be very large, so maybe you can shrink the field width
> a bit and get more space here.
>
> - I personally would prefer to see the ill name to be left justified,
> rather than right justified.
I agree with all of the above. Some other points:
* On 64-bit systems, the address is going to require twice the
field width, so you need to build for that now.
* The name WQ will suffice.
* It's essential to indicate which ill is v4 and which is v6.
* TYPE needs to be printed as an enumeration, not an integer.
* You might consider a single REFCNT field that tallyies up
all of the reference counts, but then will display
broken-up output with -v -- e.g.:
REFCNT
4
|
+-> 2 ILM references
1 IRE reference
1 NCE reference
_______________________________________________
networking-discuss mailing list
[email protected]
