[Swan-dev] logging interface's SO_{SND,RCV}BUF values

[Andrew Cagney]

On Tue, 14 May 2019 at 13:04, Antony Antony <antony at phenome.org> wrote:
>
> On Tue, May 14, 2019 at 10:40:49AM -0400, Andrew Cagney wrote:
> > I'd like to log the SO_{SND,RCV}BUF sizes for each interface (as best
>
> is this going to be dynamic? would there be what is set and what is
> current(can we read the current buffer levels?)

Interesting questions:

- the global pluto_sock_bufsize contains what it should be set and it
can be different to what getsockopt() returns so we might as well dump
both values (a whack command to change the value doesn't take effect
until the interfaces are re-opened)

- I've not seen anything to dump how full the buffer currently is (but
that's probably because I've not looked that hard)


> > I can tell we've no way to validate that interfaces are correctly
> > configured using ike-socket-bufsize et.al.).
> > So my question is where should it be added?
> >
> > - there's show_ifaces_status(void) called by 'show status' which looks like:
> >       "interface %s/%s %s@%d",
> >  so I could append the values to the end of the line, or add a second line
> >  I don't really like this, 'show status' already suffers from TMI.
> >
> > - there's 'show globalstatus' which dumps other stats in the format:
> >      key=value
> >   but it doesn't seem to list interfaces at all, I guess I'd invent my own keys
> >
> > - and there's always the option of adding 'show interfacestatus' ...
> >   hopefully not two painful, and as for output I'd probably print key=value
>
>
> > The middle option seems most straight forward while the last option
> > one seems to be the general trend.  However the last option also seems
> > to crying for a way for whack to tell pluto to dump a heap of stats as
> > a single operation:
> >      show trafficstatus shuntstatus interfacestatus statstatus ....
> > perhaps that works?
> >
> > preference?
>
> third or second seems appeal to me. there would be more changes soon with
> xfrmi or/and removel of KLIPS. keep eth0/eth0 that will be different in
> KLIPS and xfrmi.
>
>