netdev - Re: [PATCH] wifi: mac80211: use bullet list for amsdu_mesh

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Thu, 23 Mar 2023 19:25:56 +0100
From:   Simon Horman <simon.horman@...igine.com>
To:     Bagas Sanjaya <bagasdotme@...il.com>
Cc:     Linux Wireless Development <linux-wireless@...r.kernel.org>,
        Linux Networking <netdev@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Linux Documentation <linux-doc@...r.kernel.org>,
        Johannes Berg <johannes@...solutions.net>,
        "David S. Miller" <davem@...emloft.net>,
        Eric Dumazet <edumazet@...gle.com>,
        Jakub Kicinski <kuba@...nel.org>,
        Paolo Abeni <pabeni@...hat.com>, Felix Fietkau <nbd@....name>,
        Ryder Lee <ryder.lee@...iatek.com>,
        Stephen Rothwell <sfr@...b.auug.org.au>
Subject: Re: [PATCH] wifi: mac80211: use bullet list for amsdu_mesh_control
 formats list

On Thu, Mar 23, 2023 at 04:24:54PM +0700, Bagas Sanjaya wrote:
> Commit fe4a6d2db3bad4 ("wifi: mac80211: implement support for yet
> another mesh A-MSDU format") expands amsdu_mesh_control list to
> multi-line list. However, the expansion triggers Sphinx warning:

FWIIW, checkpatch complains about the way that the commit is referenced
above. It's probably not worth respinning for this. But a way
to address this would be.

The referenced commit expands amsdu_mesh_control list to multi-line list.
However, the expansion triggers Sphinx warning:

> 
> Documentation/driver-api/80211/mac80211-advanced:214: ./net/mac80211/sta_info.h:628: WARNING: Unexpected indentation.
> 
> Use bullet list instead to fix the warning.
> 
> Link: https://lore.kernel.org/linux-next/20230323141548.659479ef@canb.auug.org.au/
> Fixes: fe4a6d2db3bad4 ("wifi: mac80211: implement support for yet another mesh A-MSDU format")
> Reported-by: Stephen Rothwell <sfr@...b.auug.org.au>

Checkpatch also says that the Link: tag should go here.
TBH, I'm not sure if that is important.

> Signed-off-by: Bagas Sanjaya <bagasdotme@...il.com>

Patch itself looks good :)

Reviewed-by: Simon Horman <simon.horman@...igine.com>

> ---
>  net/mac80211/sta_info.h | 12 +++++++-----
>  1 file changed, 7 insertions(+), 5 deletions(-)
> 
> diff --git a/net/mac80211/sta_info.h b/net/mac80211/sta_info.h
> index f354d470e1740c..195b563132d6c5 100644
> --- a/net/mac80211/sta_info.h
> +++ b/net/mac80211/sta_info.h
> @@ -622,11 +622,13 @@ struct link_sta_info {
>   *	taken from HT/VHT capabilities or VHT operating mode notification
>   * @cparams: CoDel parameters for this station.
>   * @reserved_tid: reserved TID (if any, otherwise IEEE80211_TID_UNRESERVED)
> - * @amsdu_mesh_control: track the mesh A-MSDU format used by the peer
> - *	(-1: not yet known,
> - *	  0: non-mesh A-MSDU length field
> - *	  1: big-endian mesh A-MSDU length field
> - *	  2: little-endian mesh A-MSDU length field)
> + * @amsdu_mesh_control: track the mesh A-MSDU format used by the peer:
> + *
> + *	  * -1: not yet known
> + *	  * 0: non-mesh A-MSDU length field
> + *	  * 1: big-endian mesh A-MSDU length field
> + *	  * 2: little-endian mesh A-MSDU length field
> + *
>   * @fast_tx: TX fastpath information
>   * @fast_rx: RX fastpath information
>   * @tdls_chandef: a TDLS peer can have a wider chandef that is compatible to
> 
> base-commit: 0dd45ebc08de2449efe1a0908147796856a5f824
> -- 
> An old man doll... just what I always wanted! - Clara
>