Re: [ovs-dev] [PATCH] netlink-notifier: Support multiple groups.

2016-06-13 Thread Jarno Rajahalme
Thanks for the review Pravin! I fixed this before merging just a minute ago. Jarno > On Jun 7, 2016, at 4:54 PM, pravin shelar wrote: > > On Thu, May 26, 2016 at 4:29 PM, Jarno Rajahalme > wrote: >> A netlink notifier ('nln') already supports multiple notifiers. This >

Re: [ovs-dev] [PATCH] netlink-notifier: Support multiple groups.

2016-06-13 Thread Jarno Rajahalme
Thanks for the review! Applied to master with changes indicated, with one response below, Jarno > On Jun 2, 2016, at 7:19 AM, Thadeu Lima de Souza Cascardo > wrote: > > On Thu, May 26, 2016 at 04:29:28PM -0700, Jarno Rajahalme wrote: >> A netlink notifier ('nln') already supports multiple n

Re: [ovs-dev] [PATCH] netlink-notifier: Support multiple groups.

2016-06-07 Thread pravin shelar
On Thu, May 26, 2016 at 4:29 PM, Jarno Rajahalme wrote: > A netlink notifier ('nln') already supports multiple notifiers. This > patch allows each of these notifiers to subscribe to a different > multicast group. Sharing a single socket for multiple event types > (each on their own multicast gro

Re: [ovs-dev] [PATCH] netlink-notifier: Support multiple groups.

2016-06-02 Thread Thadeu Lima de Souza Cascardo
On Thu, May 26, 2016 at 04:29:28PM -0700, Jarno Rajahalme wrote: > A netlink notifier ('nln') already supports multiple notifiers. This > patch allows each of these notifiers to subscribe to a different > multicast group. Sharing a single socket for multiple event types > (each on their own multi

Re: [ovs-dev] [PATCH] netlink-notifier: Support multiple groups.

2015-05-29 Thread Ben Pfaff
On Thu, Apr 16, 2015 at 12:49:54PM -0700, Jarno Rajahalme wrote: > A netlink notifier ('nln') already supports multiple notifiers. This > patch allows each of these notifiers to subscribe to a different > multicast group. Sharing a single socket for multiple event types > (each on their own multi