Re: [PATCH] cfg80211/nl80211: add DFS offload flag
Sergey Matyukevich writes: > Hello Kalle, > >> > Friendly advice: you may want to avoid using this footer in emails on >> > the kernel.org mailing lists. >> >> Indeed. Most of the time I don't even reply to mails with such >> disclaimers and patches containing that will be automatically dropped. > > Based on our previous experience we already know that sending patches from > our 'normal' emails with disclaimer is being frowned upon. So all the > patches are being sent from 'oss' emails. But so far we had impression > that participating in further discussion was allowed even from email > addresses with disclaimer. I think it's just rude to send emails to public with such disclaimers, this is a public list where people voluntarily use their time to reply to emails. I guess some people don't mind but there are people who do not like it, myself included. -- Kalle Valo
Re: [PATCH] cfg80211/nl80211: add DFS offload flag
Hello Kalle, > > Friendly advice: you may want to avoid using this footer in emails on > > the kernel.org mailing lists. > > Indeed. Most of the time I don't even reply to mails with such > disclaimers and patches containing that will be automatically dropped. Based on our previous experience we already know that sending patches from our 'normal' emails with disclaimer is being frowned upon. So all the patches are being sent from 'oss' emails. But so far we had impression that participating in further discussion was allowed even from email addresses with disclaimer. If this is a problem, we will do the following. Patches will be posted from 'oss' emails, and authors will participate in discussions from their personal emails for the time being. Meanwhile we will work with our management to get 'oss' emails for all the engineers in the team. Regards, Sergey
Re: [PATCH] cfg80211/nl80211: add DFS offload flag
Arend van Spriel writes: > Friendly advice: you may want to avoid using this footer in emails on > the kernel.org mailing lists. Indeed. Most of the time I don't even reply to mails with such disclaimers and patches containing that will be automatically dropped. -- Kalle Valo
Re: [PATCH] cfg80211/nl80211: add DFS offload flag
On 2/21/2018 12:25 PM, Dmitrii Lebed wrote: On 21.02.2018 14:03, Arend van Spriel wrote: On 2/21/2018 11:56 AM, Sergey Matyukevich wrote: From: Dmitry Lebed Add wiphy flag to indicate that HW does all DFS actions by itself. User-space functionality already implemented in hostapd using vendor-specific (QCA) OUI to advertise HW support. Need to introduce generic flag to inform about DFS offload support. For deivces with DFS_OFFLOAD flag set user-space will no longer need to issue CAC or do any actions in response to "radar detected" events. HW will do everything by itself and send events to user-space to idicate that CAC was started/finished, etc. Signed-off-by: Dmitrii Lebed --- include/net/cfg80211.h | 7 ++- include/uapi/linux/nl80211.h | 9 + net/wireless/nl80211.c | 7 +++ 3 files changed, 22 insertions(+), 1 deletion(-) diff --git a/include/net/cfg80211.h b/include/net/cfg80211.h index ed3a28105d6c..b2d17993ae4d 100644 --- a/include/net/cfg80211.h +++ b/include/net/cfg80211.h @@ -3241,6 +3241,11 @@ struct cfg80211_ops { * @WIPHY_FLAG_IBSS_RSN: The device supports IBSS RSN. * @WIPHY_FLAG_MESH_AUTH: The device supports mesh authentication by routing * auth frames to userspace. See @NL80211_MESH_SETUP_USERSPACE_AUTH. + * @WIPHY_FLAG_DFS_OFFLOAD: Device will do all DFS-related actions by itself, + * informing user-space about CAC progress, radar detection event, + * channel change triggered by radar detection event. + * No need to start CAC from user-space, no need to react to + * "radar detected" event. wiphy flags are kinda deprecated. At least no new flags should be added. Instead use EXT_FEATURE flag. Regards, Arend Thanks for clarification, Arend. I'll change it to use new EXT_FEATURE flag. BTW, are there any concerns regarding adding generic DFS_OFFLOAD driver capability? I want to add DFS_OFFLOAD support to qtnfmac driver and fix hostapd to support generic EXT_FEATURE flag as well as vendor-specific capabilities. I suppose you are referring to the qcom vendor-specific stuff for DFS offload. I can not speak for Kalle nor Johannes, but you have my thumbs up. I am not familiar enough with the qcom implementation to say whether there are concerns. Thanks, Dmitrii * @WIPHY_FLAG_SUPPORTS_FW_ROAM: The device supports roaming feature in the * firmware. * @WIPHY_FLAG_AP_UAPSD: The device supports uapsd on AP. This email, including its contents and any attachment(s), may contain confidential information of Quantenna Communications, Inc. and is solely for the intended recipient(s). If you may have received this in error, please contact the sender and permanently delete this email, its contents and any attachment(s). Friendly advice: you may want to avoid using this footer in emails on the kernel.org mailing lists. Regards, Arend
Re: [PATCH] cfg80211/nl80211: add DFS offload flag
On 21.02.2018 14:03, Arend van Spriel wrote: On 2/21/2018 11:56 AM, Sergey Matyukevich wrote: From: Dmitry Lebed Add wiphy flag to indicate that HW does all DFS actions by itself. User-space functionality already implemented in hostapd using vendor-specific (QCA) OUI to advertise HW support. Need to introduce generic flag to inform about DFS offload support. For deivces with DFS_OFFLOAD flag set user-space will no longer need to issue CAC or do any actions in response to "radar detected" events. HW will do everything by itself and send events to user-space to idicate that CAC was started/finished, etc. Signed-off-by: Dmitrii Lebed --- include/net/cfg80211.h | 7 ++- include/uapi/linux/nl80211.h | 9 + net/wireless/nl80211.c | 7 +++ 3 files changed, 22 insertions(+), 1 deletion(-) diff --git a/include/net/cfg80211.h b/include/net/cfg80211.h index ed3a28105d6c..b2d17993ae4d 100644 --- a/include/net/cfg80211.h +++ b/include/net/cfg80211.h @@ -3241,6 +3241,11 @@ struct cfg80211_ops { * @WIPHY_FLAG_IBSS_RSN: The device supports IBSS RSN. * @WIPHY_FLAG_MESH_AUTH: The device supports mesh authentication by routing * auth frames to userspace. See @NL80211_MESH_SETUP_USERSPACE_AUTH. + * @WIPHY_FLAG_DFS_OFFLOAD: Device will do all DFS-related actions by itself, + * informing user-space about CAC progress, radar detection event, + * channel change triggered by radar detection event. + * No need to start CAC from user-space, no need to react to + * "radar detected" event. wiphy flags are kinda deprecated. At least no new flags should be added. Instead use EXT_FEATURE flag. Regards, Arend Thanks for clarification, Arend. I'll change it to use new EXT_FEATURE flag. BTW, are there any concerns regarding adding generic DFS_OFFLOAD driver capability? I want to add DFS_OFFLOAD support to qtnfmac driver and fix hostapd to support generic EXT_FEATURE flag as well as vendor-specific capabilities. Thanks, Dmitrii * @WIPHY_FLAG_SUPPORTS_FW_ROAM: The device supports roaming feature in the * firmware. * @WIPHY_FLAG_AP_UAPSD: The device supports uapsd on AP. This email, including its contents and any attachment(s), may contain confidential information of Quantenna Communications, Inc. and is solely for the intended recipient(s). If you may have received this in error, please contact the sender and permanently delete this email, its contents and any attachment(s).
Re: [PATCH] cfg80211/nl80211: add DFS offload flag
On 2/21/2018 11:56 AM, Sergey Matyukevich wrote: From: Dmitry Lebed Add wiphy flag to indicate that HW does all DFS actions by itself. User-space functionality already implemented in hostapd using vendor-specific (QCA) OUI to advertise HW support. Need to introduce generic flag to inform about DFS offload support. For deivces with DFS_OFFLOAD flag set user-space will no longer need to issue CAC or do any actions in response to "radar detected" events. HW will do everything by itself and send events to user-space to idicate that CAC was started/finished, etc. Signed-off-by: Dmitrii Lebed --- include/net/cfg80211.h | 7 ++- include/uapi/linux/nl80211.h | 9 + net/wireless/nl80211.c | 7 +++ 3 files changed, 22 insertions(+), 1 deletion(-) diff --git a/include/net/cfg80211.h b/include/net/cfg80211.h index ed3a28105d6c..b2d17993ae4d 100644 --- a/include/net/cfg80211.h +++ b/include/net/cfg80211.h @@ -3241,6 +3241,11 @@ struct cfg80211_ops { * @WIPHY_FLAG_IBSS_RSN: The device supports IBSS RSN. * @WIPHY_FLAG_MESH_AUTH: The device supports mesh authentication by routing *auth frames to userspace. See @NL80211_MESH_SETUP_USERSPACE_AUTH. + * @WIPHY_FLAG_DFS_OFFLOAD: Device will do all DFS-related actions by itself, + * informing user-space about CAC progress, radar detection event, + * channel change triggered by radar detection event. + * No need to start CAC from user-space, no need to react to + * "radar detected" event. wiphy flags are kinda deprecated. At least no new flags should be added. Instead use EXT_FEATURE flag. Regards, Arend * @WIPHY_FLAG_SUPPORTS_FW_ROAM: The device supports roaming feature in the *firmware. * @WIPHY_FLAG_AP_UAPSD: The device supports uapsd on AP.