| Message ID | 20201125042026.25374-4-rdunlap@infradead.org (mailing list archive) |
|---|---|
| State | Changes Requested |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | net/tipc: fix all kernel-doc and add TIPC networking chapter | expand |
| Context | Check | Description |
|---|---|---|
| netdev/apply | fail | Patch does not apply to net-next |
| netdev/cover_letter | success | Link |
| netdev/fixes_present | success | Link |
| netdev/patch_count | success | Link |
| netdev/tree_selection | success | Clearly marked for net-next |
| netdev/subject_prefix | success | Link |
| netdev/source_inline | success | Was 0 now: 0 |
| netdev/verify_signedoff | success | Link |
| netdev/module_param | success | Was 0 now: 0 |
| netdev/build_32bit | success | Errors and warnings before: 263 this patch: 263 |
| netdev/kdoc | success | Errors and warnings before: 0 this patch: 0 |
| netdev/verify_fixes | success | Link |
| netdev/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 109 lines checked |
| netdev/build_allmodconfig_warn | success | Errors and warnings before: 263 this patch: 263 |
| netdev/header_inline | success | Link |
| netdev/stable | success | Stable not CCed |
Please see my comments about parameters marked with "FIXME": On 11/25/20 12:20 PM, Randy Dunlap wrote: > Fix tipc header files for adding to the networking docbook. > > Remove some uses of "/**" that were not kernel-doc notation. > > Fix some source formatting to eliminate Sphinx warnings. > > Add missing struct member and function argument kernel-doc descriptions. > > Documentation/networking/tipc:18: ../net/tipc/name_table.h:65: WARNING: Unexpected indentation. > Documentation/networking/tipc:18: ../net/tipc/name_table.h:66: WARNING: Block quote ends without a blank line; unexpected unindent. > > ../net/tipc/bearer.h:128: warning: Function parameter or member 'min_win' not described in 'tipc_media' > ../net/tipc/bearer.h:128: warning: Function parameter or member 'max_win' not described in 'tipc_media' > > ../net/tipc/bearer.h:171: warning: Function parameter or member 'min_win' not described in 'tipc_bearer' > ../net/tipc/bearer.h:171: warning: Function parameter or member 'max_win' not described in 'tipc_bearer' > ../net/tipc/bearer.h:171: warning: Function parameter or member 'disc' not described in 'tipc_bearer' > ../net/tipc/bearer.h:171: warning: Function parameter or member 'up' not described in 'tipc_bearer' > ../net/tipc/bearer.h:171: warning: Function parameter or member 'refcnt' not described in 'tipc_bearer' > > ../net/tipc/name_distr.h:68: warning: Function parameter or member 'port' not described in 'distr_item' > > ../net/tipc/name_table.h:111: warning: Function parameter or member 'services' not described in 'name_table' > ../net/tipc/name_table.h:111: warning: Function parameter or member 'cluster_scope_lock' not described in 'name_table' > ../net/tipc/name_table.h:111: warning: Function parameter or member 'rc_dests' not described in 'name_table' > ../net/tipc/name_table.h:111: warning: Function parameter or member 'snd_nxt' not described in 'name_table' > > ../net/tipc/subscr.h:67: warning: Function parameter or member 'kref' not described in 'tipc_subscription' > ../net/tipc/subscr.h:67: warning: Function parameter or member 'net' not described in 'tipc_subscription' > ../net/tipc/subscr.h:67: warning: Function parameter or member 'service_list' not described in 'tipc_subscription' > ../net/tipc/subscr.h:67: warning: Function parameter or member 'conid' not described in 'tipc_subscription' > ../net/tipc/subscr.h:67: warning: Function parameter or member 'inactive' not described in 'tipc_subscription' > ../net/tipc/subscr.h:67: warning: Function parameter or member 'lock' not described in 'tipc_subscription' > > Signed-off-by: Randy Dunlap <rdunlap@infradead.org> > Cc: Jon Maloy <jmaloy@redhat.com> > Cc: Ying Xue <ying.xue@windriver.com> > Cc: netdev@vger.kernel.org > Cc: tipc-discussion@lists.sourceforge.net > Cc: "David S. Miller" <davem@davemloft.net> > Cc: Jakub Kicinski <kuba@kernel.org> > --- > net/tipc/bearer.h | 10 +++++++--- > net/tipc/crypto.h | 6 +++--- > net/tipc/name_distr.h | 2 +- > net/tipc/name_table.h | 9 ++++++--- > net/tipc/subscr.h | 11 +++++++---- > 5 files changed, 24 insertions(+), 14 deletions(-) > > --- linux-next-20201102.orig/net/tipc/bearer.h > +++ linux-next-20201102/net/tipc/bearer.h > @@ -93,7 +93,8 @@ struct tipc_bearer; > * @raw2addr: convert from raw addr format to media addr format > * @priority: default link (and bearer) priority > * @tolerance: default time (in ms) before declaring link failure > - * @window: default window (in packets) before declaring link congestion > + * @min_win: minimum window (in packets) before declaring link congestion > + * @max_win: maximum window (in packets) before declaring link congestion > * @mtu: max packet size bearer can support for media type not dependent on > * underlying device MTU > * @type_id: TIPC media identifier > @@ -138,12 +139,15 @@ struct tipc_media { > * @pt: packet type for bearer > * @rcu: rcu struct for tipc_bearer > * @priority: default link priority for bearer > - * @window: default window size for bearer > + * @min_win: minimum window (in packets) before declaring link congestion > + * @max_win: maximum window (in packets) before declaring link congestion > * @tolerance: default link tolerance for bearer > * @domain: network domain to which links can be established > * @identity: array index of this bearer within TIPC bearer array > - * @link_req: ptr to (optional) structure making periodic link setup requests > + * @disc: ptr to link setup request > * @net_plane: network plane ('A' through 'H') currently associated with bearer > + * @up: bearer up flag (bit 0) > + * @refcnt: tipc_bearer reference counter > * > * Note: media-specific code is responsible for initialization of the fields > * indicated below when a bearer is enabled; TIPC's generic bearer code takes > --- linux-next-20201102.orig/net/tipc/crypto.h > +++ linux-next-20201102/net/tipc/crypto.h > @@ -1,5 +1,5 @@ > /* SPDX-License-Identifier: GPL-2.0 */ > -/** > +/* > * net/tipc/crypto.h: Include file for TIPC crypto > * > * Copyright (c) 2019, Ericsson AB > @@ -53,7 +53,7 @@ > #define TIPC_AES_GCM_IV_SIZE 12 > #define TIPC_AES_GCM_TAG_SIZE 16 > > -/** > +/* > * TIPC crypto modes: > * - CLUSTER_KEY: > * One single key is used for both TX & RX in all nodes in the cluster. > @@ -69,7 +69,7 @@ enum { > extern int sysctl_tipc_max_tfms __read_mostly; > extern int sysctl_tipc_key_exchange_enabled __read_mostly; > > -/** > +/* > * TIPC encryption message format: > * > * 3 3 2 2 2 2 2 2 2 2 2 2 1 1 1 1 1 1 1 1 1 1 0 0 0 0 0 0 0 0 0 0 > --- linux-next-20201102.orig/net/tipc/name_distr.h > +++ linux-next-20201102/net/tipc/name_distr.h > @@ -46,7 +46,7 @@ > * @type: name sequence type > * @lower: name sequence lower bound > * @upper: name sequence upper bound > - * @ref: publishing port reference > + * @port: publishing port reference > * @key: publication key > * > * ===> All fields are stored in network byte order. <=== > --- linux-next-20201102.orig/net/tipc/name_table.h > +++ linux-next-20201102/net/tipc/name_table.h > @@ -60,8 +60,8 @@ struct tipc_group; > * @key: publication key, unique across the cluster > * @id: publication id > * @binding_node: all publications from the same node which bound this one > - * - Remote publications: in node->publ_list > - * Used by node/name distr to withdraw publications when node is lost > + * - Remote publications: in node->publ_list; > + * Used by node/name distr to withdraw publications when node is lost > * - Local/node scope publications: in name_table->node_scope list > * - Local/cluster scope publications: in name_table->cluster_scope list > * @binding_sock: all publications from the same socket which bound this one > @@ -92,13 +92,16 @@ struct publication { > > /** > * struct name_table - table containing all existing port name publications > - * @seq_hlist: name sequence hash lists > + * @services: name sequence hash lists > * @node_scope: all local publications with node scope > * - used by name_distr during re-init of name table > * @cluster_scope: all local publications with cluster scope > * - used by name_distr to send bulk updates to new nodes > * - used by name_distr during re-init of name table > + * @cluster_scope_lock: lock for accessing @cluster_scope > * @local_publ_count: number of publications issued by this node > + * @rc_dests: broadcast destinations counter (FIXME) @rc_dests: destination node counter > + * @snd_nxt: next sequence number to be used > */ > struct name_table { > struct hlist_head services[TIPC_NAMETBL_SIZE]; > --- linux-next-20201102.orig/net/tipc/subscr.h > +++ linux-next-20201102/net/tipc/subscr.h > @@ -47,12 +47,15 @@ struct tipc_conn; > > /** > * struct tipc_subscription - TIPC network topology subscription object > - * @subscriber: pointer to its subscriber > - * @seq: name sequence associated with subscription > + * @kref: reference count for this subscription > + * @net: network namespace associated with subscription > * @timer: timer governing subscription duration (optional) > - * @nameseq_list: adjacent subscriptions in name sequence's subscription list > + * @service_list: adjacent subscriptions in name sequence's subscription list > * @sub_list: adjacent subscriptions in subscriber's subscription list > * @evt: template for events generated by subscription > + * @conid: connection ID for this subscription (FIXME) @conid: connection identifier of topology server > + * @inactive: true if this subscription is inactive > + * @lock: serialize up/down and timer events > */ > struct tipc_subscription { > struct kref kref; > @@ -63,7 +66,7 @@ struct tipc_subscription { > struct tipc_event evt; > int conid; > bool inactive; > - spinlock_t lock; /* serialize up/down and timer events */ > + spinlock_t lock; > }; > > struct tipc_subscription *tipc_sub_subscribe(struct net *net, >
--- linux-next-20201102.orig/net/tipc/bearer.h +++ linux-next-20201102/net/tipc/bearer.h @@ -93,7 +93,8 @@ struct tipc_bearer; * @raw2addr: convert from raw addr format to media addr format * @priority: default link (and bearer) priority * @tolerance: default time (in ms) before declaring link failure - * @window: default window (in packets) before declaring link congestion + * @min_win: minimum window (in packets) before declaring link congestion + * @max_win: maximum window (in packets) before declaring link congestion * @mtu: max packet size bearer can support for media type not dependent on * underlying device MTU * @type_id: TIPC media identifier @@ -138,12 +139,15 @@ struct tipc_media { * @pt: packet type for bearer * @rcu: rcu struct for tipc_bearer * @priority: default link priority for bearer - * @window: default window size for bearer + * @min_win: minimum window (in packets) before declaring link congestion + * @max_win: maximum window (in packets) before declaring link congestion * @tolerance: default link tolerance for bearer * @domain: network domain to which links can be established * @identity: array index of this bearer within TIPC bearer array - * @link_req: ptr to (optional) structure making periodic link setup requests + * @disc: ptr to link setup request * @net_plane: network plane ('A' through 'H') currently associated with bearer + * @up: bearer up flag (bit 0) + * @refcnt: tipc_bearer reference counter * * Note: media-specific code is responsible for initialization of the fields * indicated below when a bearer is enabled; TIPC's generic bearer code takes --- linux-next-20201102.orig/net/tipc/crypto.h +++ linux-next-20201102/net/tipc/crypto.h @@ -1,5 +1,5 @@ /* SPDX-License-Identifier: GPL-2.0 */ -/** +/* * net/tipc/crypto.h: Include file for TIPC crypto * * Copyright (c) 2019, Ericsson AB @@ -53,7 +53,7 @@ #define TIPC_AES_GCM_IV_SIZE 12 #define TIPC_AES_GCM_TAG_SIZE 16 -/** +/* * TIPC crypto modes: * - CLUSTER_KEY: * One single key is used for both TX & RX in all nodes in the cluster. @@ -69,7 +69,7 @@ enum { extern int sysctl_tipc_max_tfms __read_mostly; extern int sysctl_tipc_key_exchange_enabled __read_mostly; -/** +/* * TIPC encryption message format: * * 3 3 2 2 2 2 2 2 2 2 2 2 1 1 1 1 1 1 1 1 1 1 0 0 0 0 0 0 0 0 0 0 --- linux-next-20201102.orig/net/tipc/name_distr.h +++ linux-next-20201102/net/tipc/name_distr.h @@ -46,7 +46,7 @@ * @type: name sequence type * @lower: name sequence lower bound * @upper: name sequence upper bound - * @ref: publishing port reference + * @port: publishing port reference * @key: publication key * * ===> All fields are stored in network byte order. <=== --- linux-next-20201102.orig/net/tipc/name_table.h +++ linux-next-20201102/net/tipc/name_table.h @@ -60,8 +60,8 @@ struct tipc_group; * @key: publication key, unique across the cluster * @id: publication id * @binding_node: all publications from the same node which bound this one - * - Remote publications: in node->publ_list - * Used by node/name distr to withdraw publications when node is lost + * - Remote publications: in node->publ_list; + * Used by node/name distr to withdraw publications when node is lost * - Local/node scope publications: in name_table->node_scope list * - Local/cluster scope publications: in name_table->cluster_scope list * @binding_sock: all publications from the same socket which bound this one @@ -92,13 +92,16 @@ struct publication { /** * struct name_table - table containing all existing port name publications - * @seq_hlist: name sequence hash lists + * @services: name sequence hash lists * @node_scope: all local publications with node scope * - used by name_distr during re-init of name table * @cluster_scope: all local publications with cluster scope * - used by name_distr to send bulk updates to new nodes * - used by name_distr during re-init of name table + * @cluster_scope_lock: lock for accessing @cluster_scope * @local_publ_count: number of publications issued by this node + * @rc_dests: broadcast destinations counter (FIXME) + * @snd_nxt: next sequence number to be used */ struct name_table { struct hlist_head services[TIPC_NAMETBL_SIZE]; --- linux-next-20201102.orig/net/tipc/subscr.h +++ linux-next-20201102/net/tipc/subscr.h @@ -47,12 +47,15 @@ struct tipc_conn; /** * struct tipc_subscription - TIPC network topology subscription object - * @subscriber: pointer to its subscriber - * @seq: name sequence associated with subscription + * @kref: reference count for this subscription + * @net: network namespace associated with subscription * @timer: timer governing subscription duration (optional) - * @nameseq_list: adjacent subscriptions in name sequence's subscription list + * @service_list: adjacent subscriptions in name sequence's subscription list * @sub_list: adjacent subscriptions in subscriber's subscription list * @evt: template for events generated by subscription + * @conid: connection ID for this subscription (FIXME) + * @inactive: true if this subscription is inactive + * @lock: serialize up/down and timer events */ struct tipc_subscription { struct kref kref; @@ -63,7 +66,7 @@ struct tipc_subscription { struct tipc_event evt; int conid; bool inactive; - spinlock_t lock; /* serialize up/down and timer events */ + spinlock_t lock; }; struct tipc_subscription *tipc_sub_subscribe(struct net *net,
Fix tipc header files for adding to the networking docbook. Remove some uses of "/**" that were not kernel-doc notation. Fix some source formatting to eliminate Sphinx warnings. Add missing struct member and function argument kernel-doc descriptions. Documentation/networking/tipc:18: ../net/tipc/name_table.h:65: WARNING: Unexpected indentation. Documentation/networking/tipc:18: ../net/tipc/name_table.h:66: WARNING: Block quote ends without a blank line; unexpected unindent. ../net/tipc/bearer.h:128: warning: Function parameter or member 'min_win' not described in 'tipc_media' ../net/tipc/bearer.h:128: warning: Function parameter or member 'max_win' not described in 'tipc_media' ../net/tipc/bearer.h:171: warning: Function parameter or member 'min_win' not described in 'tipc_bearer' ../net/tipc/bearer.h:171: warning: Function parameter or member 'max_win' not described in 'tipc_bearer' ../net/tipc/bearer.h:171: warning: Function parameter or member 'disc' not described in 'tipc_bearer' ../net/tipc/bearer.h:171: warning: Function parameter or member 'up' not described in 'tipc_bearer' ../net/tipc/bearer.h:171: warning: Function parameter or member 'refcnt' not described in 'tipc_bearer' ../net/tipc/name_distr.h:68: warning: Function parameter or member 'port' not described in 'distr_item' ../net/tipc/name_table.h:111: warning: Function parameter or member 'services' not described in 'name_table' ../net/tipc/name_table.h:111: warning: Function parameter or member 'cluster_scope_lock' not described in 'name_table' ../net/tipc/name_table.h:111: warning: Function parameter or member 'rc_dests' not described in 'name_table' ../net/tipc/name_table.h:111: warning: Function parameter or member 'snd_nxt' not described in 'name_table' ../net/tipc/subscr.h:67: warning: Function parameter or member 'kref' not described in 'tipc_subscription' ../net/tipc/subscr.h:67: warning: Function parameter or member 'net' not described in 'tipc_subscription' ../net/tipc/subscr.h:67: warning: Function parameter or member 'service_list' not described in 'tipc_subscription' ../net/tipc/subscr.h:67: warning: Function parameter or member 'conid' not described in 'tipc_subscription' ../net/tipc/subscr.h:67: warning: Function parameter or member 'inactive' not described in 'tipc_subscription' ../net/tipc/subscr.h:67: warning: Function parameter or member 'lock' not described in 'tipc_subscription' Signed-off-by: Randy Dunlap <rdunlap@infradead.org> Cc: Jon Maloy <jmaloy@redhat.com> Cc: Ying Xue <ying.xue@windriver.com> Cc: netdev@vger.kernel.org Cc: tipc-discussion@lists.sourceforge.net Cc: "David S. Miller" <davem@davemloft.net> Cc: Jakub Kicinski <kuba@kernel.org> --- net/tipc/bearer.h | 10 +++++++--- net/tipc/crypto.h | 6 +++--- net/tipc/name_distr.h | 2 +- net/tipc/name_table.h | 9 ++++++--- net/tipc/subscr.h | 11 +++++++---- 5 files changed, 24 insertions(+), 14 deletions(-)