netdev - Re: [PATCH v3 iproute] ip: Add support for netdev events to monitor

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <0a7efd86-3345-223a-7a39-8c8cd1a42f14@cumulusnetworks.com>
Date:   Sat, 8 Apr 2017 18:24:06 -0400
From:   David Ahern <dsa@...ulusnetworks.com>
To:     Vladislav Yasevich <vyasevic@...hat.com>, netdev@...r.kernel.org
Cc:     roopa@...ulusnetworks.com, dsa@...ulusnetworks.com,
        davem@...emloft.net
Subject: Re: [PATCH v3 iproute] ip: Add support for netdev events to monitor

On 4/4/17 9:25 AM, Vladislav Yasevich wrote:
> Add IFLA_EVENT handling so that event types can be viewed with
> 'monitor' command.  This gives a little more information for why
> a given message was receivied.
> 
> Signed-off-by: Vladislav Yasevich <vyasevic@...hat.com>
> ---
>  include/linux/if_link.h | 21 +++++++++++++++++++++
>  ip/ipaddress.c          | 31 +++++++++++++++++++++++++++++++
>  2 files changed, 52 insertions(+)

per comments on the email thread about reducing notifications, the
kernel patch for this should be reverted (and hence this iproute2 patch
is not needed) in favor of using a bitmask. Right now there are too many
redundant notifications to userspace.

Vlad: can you look at a bitmask version of IFLA_EVENT that shows what
changed in the newlink message from do_setlink()?