Re: [PATCH] mac80211 : Add support to track mesh peer beacon miss event

[Johannes Berg <johannes@sipsolutions.net>]

On Mon, 2022-05-30 at 12:18 +0530, quic_haric@quicinc.com wrote:
> 
> +++ b/include/net/mac80211.h
> @@ -1732,6 +1732,9 @@ enum ieee80211_offload_flags {
>   *	for read access.
>   * @color_change_color: the bss color that will be used after the change.
>   * @mbssid_tx_vif: Pointer to the transmitting interface if MBSSID is enabled.
> + * @bmiss_threshold: beacon miss threshold that is used to change the beacon miss
> + * timer value
> + *
>   */
>  struct ieee80211_vif {
>  	enum nl80211_iftype type;
> @@ -1764,6 +1767,7 @@ struct ieee80211_vif {
>  	u8 color_change_color;
>  
>  	struct ieee80211_vif *mbssid_tx_vif;
> +	u8 bmiss_threshold;
> 
> 

Why should this be visible to the driver?

> @@ -2495,6 +2499,19 @@ enum ieee80211_hw_flags {
>  };
>  
>  /**
> + * enum ieee80211_dbg_mask - Debug mask to enable and disable logs
> + * in runtime
> + *
> + * @IEEE80211_HW_DBG_BMISS_LOG - To enable/disable mesh beacon miss logs
> + *
> + * @IEEE80211_HW_MAX_DBG_MASK - Max debug mask value
> + */

same here

> @@ -2656,6 +2675,7 @@ struct ieee80211_hw {
>  	u8 tx_sk_pacing_shift;
>  	u8 weight_multiplier;
>  	u32 max_mtu;
> +	u32 dbg_mask;

and here.

>  /*
> + * mesh_bmiss_update - update beacon miss parameters such as

/** I guess, based on the comment formatting

You also have a bunch of robot comments.

johannes