From nobody Mon Apr 10 04:16:46 2023 X-Original-To: dev-commits-src-all@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4Pvwf74VjYz44kg2; Mon, 10 Apr 2023 04:16:47 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4Pvwf72Fbnz4MsG; Mon, 10 Apr 2023 04:16:47 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1681100207; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=rgvl8Mqc2SeufOrK7CpEX65SXsm19tVhzZDgAN/r9ho=; b=rrMEGd4u0dd3ymJGpRYF/IvqzFm+ILPgWXhWDZacYe5Uy2sCy74H1dutwkzRbEyhqOawbx gz22yFeJtI4rcdAlkndY8uRfhCKx2rIx8hdQIHu9hAizWUe1W8o2axuArELUgCUC/J1iyS UvCQVaDOLoKkKaqo5K249xzvapBOeDTHOmd1d2TjtS8sctRGh3P14qFeawU7He4LNfiPCG XYVblfPr8/THE/JJiXh5MrV4kNBM+zMjbm+G/H6jKz7WxAum7qIDo8IALY1YjqKdqPsYiV ul3TGYhgqBk3gyce5sC3bfPpZnZauNwXA5fFZRY75xXF3tiACj+2JsNLQAry3g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1681100207; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=rgvl8Mqc2SeufOrK7CpEX65SXsm19tVhzZDgAN/r9ho=; b=hjoB2lXrd8V3k1M9MbYts/tkjoIMIMnuLPu+pmdT1NWvhNxX3XbRC4UtHvdx1HAHADW+FF knMBndokwhYVMK3MJTCDCjxTp0ZmrO4W9qO4TloRTadfn6VY7JkfugANjjGN7XzXev/7xa Kr+SdcT/jpr++xtwGxph77OPBiUH9uAIgjCjrGxCZUGCtUx4G6EhZNwXpHlTpHmbzhT57B L67e80DEVUo7qYyKsrOFgkj0W5fbNRixAblIVz6DXL9+KQmDLjyPBobZdTJvn1BHnGs4RG f0bZMSbsWNj1h8jEQS6U0Z2x0Fb6IPqztBs/qADCaBatiBMNBuW+m0O9Xo7EMw== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1681100207; a=rsa-sha256; cv=none; b=qpeje0LUAuZAsVbHTU/0yzZ5jUfGO1LH67vRhSwrqEaMecmCUNfX5oSD6zMHUewDJrgVLl tprBdoR53VPlw1Wm4M7iuOl//koadTJ4yU0yGzx7vIf5rNVDM0fwUQfjGwDVCA1+M+AdD5 WjMC1FbL+IeVxu5HcnAsy+KmNvgFG4Rz6OLrcB4lXkR8/qhqrLB/N6gsXyT4eFMigqd4ir Vat/zzxEdB5EX3VPUbDER+1/AWxfzem1/ttnRSvSmJY60/mrn+RSrpfqizZwNhDME3LoHW NRZ4iZ27wVaqFVXvBHgMur3rlSs95yXO+fviE1DAICjhx0fPBgVMNeFtP9BF6g== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 4Pvwf706YCzbMq; Mon, 10 Apr 2023 04:16:47 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.16.1/8.16.1) with ESMTP id 33A4Gkrd039601; Mon, 10 Apr 2023 04:16:46 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.16.1/8.16.1/Submit) id 33A4GkMC039600; Mon, 10 Apr 2023 04:16:46 GMT (envelope-from git) Date: Mon, 10 Apr 2023 04:16:46 GMT Message-Id: <202304100416.33A4GkMC039600@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org From: Zhenlei Huang Subject: git: d1d5d8f69659 - stable/13 - lacp: Use C99 bool for boolean return value List-Id: Commit messages for all branches of the src repository List-Archive: https://lists.freebsd.org/archives/dev-commits-src-all List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-src-all@freebsd.org X-BeenThere: dev-commits-src-all@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: zlei X-Git-Repository: src X-Git-Refname: refs/heads/stable/13 X-Git-Reftype: branch X-Git-Commit: d1d5d8f69659c4d3573e64a62a9306ee944d9e4d Auto-Submitted: auto-generated X-ThisMailContainsUnwantedMimeParts: N The branch stable/13 has been updated by zlei: URL: https://cgit.FreeBSD.org/src/commit/?id=d1d5d8f69659c4d3573e64a62a9306ee944d9e4d commit d1d5d8f69659c4d3573e64a62a9306ee944d9e4d Author: Zhenlei Huang AuthorDate: 2023-03-31 17:48:36 +0000 Commit: Zhenlei Huang CommitDate: 2023-04-10 04:15:04 +0000 lacp: Use C99 bool for boolean return value This improves readability. No functional change intended. MFC after: 1 week (cherry picked from commit 5a8abd0a298e6e7a8bf938a7eb171b647b1860cd) --- sys/net/ieee8023ad_lacp.c | 38 +++++++++++++++++--------------------- sys/net/ieee8023ad_lacp.h | 15 ++++++--------- sys/net/if_lagg.c | 2 +- 3 files changed, 24 insertions(+), 31 deletions(-) diff --git a/sys/net/ieee8023ad_lacp.c b/sys/net/ieee8023ad_lacp.c index 6656ebb2b400..8770e2a3e398 100644 --- a/sys/net/ieee8023ad_lacp.c +++ b/sys/net/ieee8023ad_lacp.c @@ -116,9 +116,9 @@ static void lacp_fill_aggregator_id(struct lacp_aggregator *, const struct lacp_port *); static void lacp_fill_aggregator_id_peer(struct lacp_peerinfo *, const struct lacp_peerinfo *); -static int lacp_aggregator_is_compatible(const struct lacp_aggregator *, +static bool lacp_aggregator_is_compatible(const struct lacp_aggregator *, const struct lacp_port *); -static int lacp_peerinfo_is_compatible(const struct lacp_peerinfo *, +static bool lacp_peerinfo_is_compatible(const struct lacp_peerinfo *, const struct lacp_peerinfo *); static struct lacp_aggregator *lacp_aggregator_get(struct lacp_softc *, @@ -1367,44 +1367,40 @@ lacp_fill_aggregator_id_peer(struct lacp_peerinfo *lpi_aggr, * lacp_aggregator_is_compatible: check if a port can join to an aggregator. */ -static int +static bool lacp_aggregator_is_compatible(const struct lacp_aggregator *la, const struct lacp_port *lp) { if (!(lp->lp_state & LACP_STATE_AGGREGATION) || !(lp->lp_partner.lip_state & LACP_STATE_AGGREGATION)) { - return (0); + return (false); } - if (!(la->la_actor.lip_state & LACP_STATE_AGGREGATION)) { - return (0); - } + if (!(la->la_actor.lip_state & LACP_STATE_AGGREGATION)) + return (false); - if (!lacp_peerinfo_is_compatible(&la->la_partner, &lp->lp_partner)) { - return (0); - } + if (!lacp_peerinfo_is_compatible(&la->la_partner, &lp->lp_partner)) + return (false); - if (!lacp_peerinfo_is_compatible(&la->la_actor, &lp->lp_actor)) { - return (0); - } + if (!lacp_peerinfo_is_compatible(&la->la_actor, &lp->lp_actor)) + return (false); - return (1); + return (true); } -static int +static bool lacp_peerinfo_is_compatible(const struct lacp_peerinfo *a, const struct lacp_peerinfo *b) { if (memcmp(&a->lip_systemid, &b->lip_systemid, - sizeof(a->lip_systemid))) { - return (0); + sizeof(a->lip_systemid)) != 0) { + return (false); } - if (memcmp(&a->lip_key, &b->lip_key, sizeof(a->lip_key))) { - return (0); - } + if (memcmp(&a->lip_key, &b->lip_key, sizeof(a->lip_key)) != 0) + return (false); - return (1); + return (true); } static void diff --git a/sys/net/ieee8023ad_lacp.h b/sys/net/ieee8023ad_lacp.h index 0610ed855d50..629bdc25cfc5 100644 --- a/sys/net/ieee8023ad_lacp.h +++ b/sys/net/ieee8023ad_lacp.h @@ -307,7 +307,7 @@ void lacp_linkstate(struct lagg_port *); void lacp_req(struct lagg_softc *, void *); void lacp_portreq(struct lagg_port *, void *); -static __inline int +static __inline bool lacp_isactive(struct lagg_port *lgp) { struct lacp_port *lp = LACP_PORT(lgp); @@ -315,26 +315,23 @@ lacp_isactive(struct lagg_port *lgp) struct lacp_aggregator *la = lp->lp_aggregator; /* This port is joined to the active aggregator */ - if (la != NULL && la == lsc->lsc_active_aggregator) - return (1); - - return (0); + return (la != NULL && la == lsc->lsc_active_aggregator); } -static __inline int +static __inline bool lacp_iscollecting(struct lagg_port *lgp) { struct lacp_port *lp = LACP_PORT(lgp); - return ((lp->lp_state & LACP_STATE_COLLECTING) != 0); + return (lp->lp_state & LACP_STATE_COLLECTING); } -static __inline int +static __inline bool lacp_isdistributing(struct lagg_port *lgp) { struct lacp_port *lp = LACP_PORT(lgp); - return ((lp->lp_state & LACP_STATE_DISTRIBUTING) != 0); + return (lp->lp_state & LACP_STATE_DISTRIBUTING); } /* following constants don't include terminating NUL */ diff --git a/sys/net/if_lagg.c b/sys/net/if_lagg.c index 8a62286db81b..74f8c41a125e 100644 --- a/sys/net/if_lagg.c +++ b/sys/net/if_lagg.c @@ -2577,7 +2577,7 @@ lagg_lacp_input(struct lagg_softc *sc, struct lagg_port *lp, struct mbuf *m) * If the port is not collecting or not in the active aggregator then * free and return. */ - if (lacp_iscollecting(lp) == 0 || lacp_isactive(lp) == 0) { + if (!lacp_iscollecting(lp) || !lacp_isactive(lp)) { m_freem(m); return (NULL); }