Re: [PATCH net-next] rtnl: Add support for netdev event to link messages

2017-03-30 Thread David Ahern
On 3/30/17 9:21 AM, Vladislav Yasevich wrote: > > So, something like the patch below would be better in your opinion as a > starting point. It'll can at least get the discussion strarted on whether > an event would usefull to user space or not. IMO that is a more direct, explicit statement of wh

Re: [PATCH net-next] rtnl: Add support for netdev event to link messages

2017-03-30 Thread Vladislav Yasevich
On 03/30/2017 10:11 AM, David Ahern wrote: > On 3/30/17 7:47 AM, Vlad Yasevich wrote: >> But, NETDEV_PRECHANGEMTU will be a unnecessary notification to userspace >> without >> changes. There are already enough notifications generated for links (I >> know you are not >> suggest

Re: [PATCH net-next] rtnl: Add support for netdev event to link messages

2017-03-30 Thread David Ahern
On 3/30/17 7:47 AM, Vlad Yasevich wrote: > But, NETDEV_PRECHANGEMTU will be a unnecessary notification to userspace > without > changes. There are already enough notifications generated for links (I > know you are not > suggesting adding it here) Actually, this one a

Re: [PATCH net-next] rtnl: Add support for netdev event to link messages

2017-03-30 Thread Vlad Yasevich
On 03/30/2017 09:39 AM, Vlad Yasevich wrote: > On 03/29/2017 03:11 PM, David Ahern wrote: >> On 3/29/17 11:05 AM, Vlad Yasevich wrote: >>> On 03/29/2017 12:37 PM, Roopa Prabhu wrote: On 3/29/17, 5:23 AM, Vlad Yasevich wrote: > [ resending to list. hit the wrong reply button last time ] >>

Re: [PATCH net-next] rtnl: Add support for netdev event to link messages

2017-03-30 Thread Vlad Yasevich
On 03/29/2017 03:11 PM, David Ahern wrote: > On 3/29/17 11:05 AM, Vlad Yasevich wrote: >> On 03/29/2017 12:37 PM, Roopa Prabhu wrote: >>> On 3/29/17, 5:23 AM, Vlad Yasevich wrote: [ resending to list. hit the wrong reply button last time ] On 03/27/2017 06:58 PM, David Miller wrote:

Re: [PATCH net-next] rtnl: Add support for netdev event to link messages

2017-03-29 Thread David Ahern
On 3/29/17 11:05 AM, Vlad Yasevich wrote: > On 03/29/2017 12:37 PM, Roopa Prabhu wrote: >> On 3/29/17, 5:23 AM, Vlad Yasevich wrote: >>> [ resending to list. hit the wrong reply button last time ] >>> >>> On 03/27/2017 06:58 PM, David Miller wrote: From: Vladislav Yasevich Date: Sat, 25

Re: [PATCH net-next] rtnl: Add support for netdev event to link messages

2017-03-29 Thread Vlad Yasevich
On 03/29/2017 12:37 PM, Roopa Prabhu wrote: > On 3/29/17, 5:23 AM, Vlad Yasevich wrote: >> [ resending to list. hit the wrong reply button last time ] >> >> On 03/27/2017 06:58 PM, David Miller wrote: >>> From: Vladislav Yasevich >>> Date: Sat, 25 Mar 2017 21:59:47 -0400 >>> RTNL currently g

Re: [PATCH net-next] rtnl: Add support for netdev event to link messages

2017-03-29 Thread Roopa Prabhu
On 3/29/17, 5:23 AM, Vlad Yasevich wrote: > [ resending to list. hit the wrong reply button last time ] > > On 03/27/2017 06:58 PM, David Miller wrote: >> From: Vladislav Yasevich >> Date: Sat, 25 Mar 2017 21:59:47 -0400 >> >>> RTNL currently generates notifications on some netdev notifier events

Re: [PATCH net-next] rtnl: Add support for netdev event to link messages

2017-03-29 Thread Vlad Yasevich
[ resending to list. hit the wrong reply button last time ] On 03/27/2017 06:58 PM, David Miller wrote: > From: Vladislav Yasevich > Date: Sat, 25 Mar 2017 21:59:47 -0400 > >> RTNL currently generates notifications on some netdev notifier events. >> However, user space has no idea what changed.

Re: [PATCH net-next] rtnl: Add support for netdev event to link messages

2017-03-27 Thread David Miller
From: Vladislav Yasevich Date: Sat, 25 Mar 2017 21:59:47 -0400 > RTNL currently generates notifications on some netdev notifier events. > However, user space has no idea what changed. All it sees is the > data and has to infer what has changed. For some events that is not > possible. > > This

[PATCH net-next] rtnl: Add support for netdev event to link messages

2017-03-25 Thread Vladislav Yasevich
RTNL currently generates notifications on some netdev notifier events. However, user space has no idea what changed. All it sees is the data and has to infer what has changed. For some events that is not possible. This patch adds a new field to RTM_NEWLINK message called IFLA_EVENT that would ha