On Tue, May 17, 2016 at 11:24 AM, David Ahern <d...@cumulusnetworks.com> wrote:
> As I mentioned we can print the unsupported once or per socket matched and
> with the socket params. e.g.,
>
> + } else if (errno == EOPNOTSUPP) {
> + printf("Operation not supported for:\n");
> + inet_show_sock(h, diag_arg->f, diag_arg->protocol);
>
> Actively suppressing all error messages is just wrong. I get the flooding
> issue so I'm fine with just printing it once.
I disagree, but then I'm the one who wrote it in the first place, so
you wouldn't expect me to agree. :-) Let's see what Stephen says.
