Add netdev name to NL80211_CMD_STOP_AP event?

While trying to make 'iw' able to print out something useful for
the 'stop_ap' event, I notice there is not really any human friendly
info in the message (just phy id (which may not match phy name), ifindex, and some other wiphy related numeric ID.

Any reason not to add the netdev name (NL80211_ATTR_IFNAME) to this and related messages?

Quoted 7 lines · Ben Greear

While trying to make 'iw' able to print out something useful for
the 'stop_ap' event, I notice there is not really any human friendly
info in the message (just phy id (which may not match phy name), ifindex, and some other wiphy related numeric ID.

Any reason not to add the netdev name (NL80211_ATTR_IFNAME) to this and related messages?

We're not really adding that *anywhere*, and you can trivially look it up based on the ifindex, so why bother?

johannes

I was thinking netdevice might go away before user-space can decode the message ( I was assuming that on delete you would stop and very shortly after that delete the netdev).

It's a small patch to iw and the kernel as far as I can tell...

Quoted 3 lines · Ben Greear

I was thinking netdevice might go away before user-space can decode the message ( I was assuming that on delete you would stop and very shortly after that delete the netdev).

Yeah, but in that case ... does it matter? :)

Quoted 1 line

It's a small patch to iw and the kernel as far as I can tell...

It's small *now*, but if we add it to all events it'll end up pretty big...