On Wed, 2018-01-17 at 13:01 +0530, tamizhr@xxxxxxxxxxxxxx wrote: > From: Pradeep Kumar Chitrapu <pradeepc@xxxxxxxxxxxxxx> > > Extend cqm rssi notifier apis to include peer mac address. Currently > netlink cqm msg already accepts the mac. With this change just passing > the mac address as an argument to cfg and mac80211 apis. > It seems eminently reasonable to have this functionality, but I'm not sure you can implement it like this? I'd be worried that there's some userspace that just listens for these events and could get confused if another virtual interface shows up with notifications - if it's not doing proper filtering etc.? I'd be more comfortable at least moving the per-station notifications to a new command ID, or making them opt-in by clients by having a registration command and sending them unicast instead. Or actually, perhaps the best would be to just go and create a new multicast group for them, then we don't need a new command ID and existing clients can't get confused, but new clients don't really have to do anything but sign up to the new multicast group. We could check if the group is non-empty before we even build the message too. johannes
