From nobody Sat Feb 03 19:11:37 2024 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 4TS2Ld4xR6z58DMs; Sat, 3 Feb 2024 19:11:37 +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 4TS2Ld4SC6z40CM; Sat, 3 Feb 2024 19:11:37 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1706987497; 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=r/uOQXxqBeqKEL3z4ux8QXbyF9D8dZrfQ5pMF+vaHLI=; b=RU/7olff4W0O7U0rRv788C5dbgdrFrd6O4v1TgWp37hUZBLRy3ARkmq7i1QC8mxtRlmf3Z UrMyzgv+FgK+UTqn9aTgNmUjUnGy4qdt0Vvw9/UOEGBwjK0oEW9fpk2MZFAgq8LL7vHPyZ cd9NYjl7nPJ3Naox+S6jARkahLVsdWoUKRy5N8nRmtrBraXAtcdq3ud/+CGT7j0RZttnQD Y6OBw34dnA1gYA/kWs8khhW3BI9lDUBYybcZxE+2QQ5IZh/5yzLkEfBza9M2dk/UmnlACp BO6VpmZBiE49aOsN1a9fr2QpBu+ES2GTjuoOa0Lphs6G99xa/OZHyll9TGXWWQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1706987497; 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=r/uOQXxqBeqKEL3z4ux8QXbyF9D8dZrfQ5pMF+vaHLI=; b=yX/YQY8rPmJVJXR8uSGC3odIvQ1yscTqbmiOwnLxlxrzAupXMRbwHVGhsfrRLomiRsJp7c vGYNQnKN4+0tU6obu1d60tzfOOfiYK+/iXqeZOBPFE61WA/+LEQnIPbzBVvbWwBzEcM5Kv QXdJPnuk1l8IX3cQBn4HhpbM0jXcTrSs1ZOG2fHrgrFWP3KzjT9FQLkZ3l4W6maoRdygBc 9kvMpTI6u/a+3KVARGYdLNHHLBVuMkuLRLeBHLDIMXZwYANcQsYtojU6NgGOaOhkc+ZjlZ 6nbI/QWelqcJ4kp6KjhYiz5oDd8aZSiEr9sA5Qj0Ai11wfdied7aQxmoZtVKHA== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1706987497; a=rsa-sha256; cv=none; b=bRh2O9576jGHApu4iuQqYjk5Uvu7IjwOjmdLIokvcFpGgcQX9OBeDMRNQ+mSUR5IX5cyOm fXAMdBXadXr3UEgxiwWhMVruk98werf3vl1KXdetEuG2nCMl3KHnoZPavxrIx+YdTdE6dz qdpNaKoydVr3oww5BQmGqJ8Tv1oFtCQx+GCdT4Ggn1320K+AsVgvX9PWabVUFCfmsNtqKu +jMi9jAiENGD5KFA2yvV2m8+8Jkuv4aCir+fI8SrQpEDIzT2SyD2lLIuVXtT+F4BER5j7T yPkflsUbBRB6RHQKCqebikA/sZJshfv6hyg/zcSnKBJPQIjSaspdh07EygWmBA== 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 4TS2Ld3WvXzQTL; Sat, 3 Feb 2024 19:11:37 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.17.1/8.17.1) with ESMTP id 413JBbVk092984; Sat, 3 Feb 2024 19:11:37 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.17.1/8.17.1/Submit) id 413JBbOY092962; Sat, 3 Feb 2024 19:11:37 GMT (envelope-from git) Date: Sat, 3 Feb 2024 19:11:37 GMT Message-Id: <202402031911.413JBbOY092962@gitrepo.freebsd.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org From: Mark Johnston Subject: git: 93f523ab3668 - stable/14 - netinet: Remove stale references to Giant from comments 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: markj X-Git-Repository: src X-Git-Refname: refs/heads/stable/14 X-Git-Reftype: branch X-Git-Commit: 93f523ab3668516f34d13d844a7772c92411891a Auto-Submitted: auto-generated The branch stable/14 has been updated by markj: URL: https://cgit.FreeBSD.org/src/commit/?id=93f523ab3668516f34d13d844a7772c92411891a commit 93f523ab3668516f34d13d844a7772c92411891a Author: Mark Johnston AuthorDate: 2023-03-12 18:59:42 +0000 Commit: Mark Johnston CommitDate: 2024-02-03 19:10:36 +0000 netinet: Remove stale references to Giant from comments MFC after: 1 week (cherry picked from commit bbf86c65d04d6013fd3f7b6d74a341256c4e7336) --- sys/netinet/igmp.c | 2 -- sys/netinet/in_mcast.c | 15 +++++---------- sys/netinet6/in6_mcast.c | 7 ++----- 3 files changed, 7 insertions(+), 17 deletions(-) diff --git a/sys/netinet/igmp.c b/sys/netinet/igmp.c index 3a60feadcdab..ae58dc0b8b04 100644 --- a/sys/netinet/igmp.c +++ b/sys/netinet/igmp.c @@ -168,7 +168,6 @@ static const struct netisr_handler igmp_nh = { * Any may be taken independently; if any are held at the same * time, the above lock order must be followed. * * All output is delegated to the netisr. - * Now that Giant has been eliminated, the netisr may be inlined. * * IN_MULTI_LIST_LOCK covers in_multi. * * IGMP_LOCK covers igmp_ifsoftc and any global variables in this file, * including the output queue. @@ -1674,7 +1673,6 @@ igmp_fasttimo(void *arg __unused) /* * Fast timeout handler (per-vnet). - * Sends are shuffled off to a netisr to deal with Giant. * * VIMAGE: Assume caller has set up our curvnet. */ diff --git a/sys/netinet/in_mcast.c b/sys/netinet/in_mcast.c index 362e3f25115c..94fc807ccea7 100644 --- a/sys/netinet/in_mcast.c +++ b/sys/netinet/in_mcast.c @@ -93,8 +93,8 @@ static MALLOC_DEFINE(M_IPMSOURCE, "ip_msource", /* * Locking: * - * - Lock order is: Giant, IN_MULTI_LOCK, INP_WLOCK, - * IN_MULTI_LIST_LOCK, IGMP_LOCK, IF_ADDR_LOCK. + * - Lock order is: IN_MULTI_LOCK, INP_WLOCK, IN_MULTI_LIST_LOCK, IGMP_LOCK, + * IF_ADDR_LOCK. * - The IF_ADDR_LOCK is implicitly taken by inm_lookup() earlier, however * it can be taken by code in net/if.c also. * - ip_moptions and in_mfilter are covered by the INP_WLOCK. @@ -1188,9 +1188,9 @@ inm_purge(struct in_multi *inm) /* * Join a multicast group; unlocked entry point. * - * SMPng: XXX: in_joingroup() is called from in_control() when Giant - * is not held. Fortunately, ifp is unlikely to have been detached - * at this point, so we assume it's OK to recurse. + * SMPng: XXX: in_joingroup() is called from in_control(). Fortunately, + * ifp is unlikely to have been detached at this point, so we assume + * it's OK to recurse. */ int in_joingroup(struct ifnet *ifp, const struct in_addr *gina, @@ -1364,8 +1364,6 @@ in_leavegroup_locked(struct in_multi *inm, /*const*/ struct in_mfilter *imf) * The delta-based API applies only to exclusive-mode memberships. * An IGMP downcall will be performed. * - * SMPng: NOTE: Must take Giant as a join may create a new ifma. - * * Return 0 if successful, otherwise return an appropriate error code. */ static int @@ -1552,7 +1550,6 @@ out_inp_locked: * Given an inpcb, return its multicast options structure pointer. Accepts * an unlocked inpcb pointer, but will return it locked. May sleep. * - * SMPng: NOTE: Potentially calls malloc(M_WAITOK) with Giant held. * SMPng: NOTE: Returns with the INP write lock held. */ static struct ip_moptions * @@ -2522,8 +2519,6 @@ inp_set_multicast_if(struct inpcb *inp, struct sockopt *sopt) /* * Atomically set source filters on a socket for an IPv4 multicast group. - * - * SMPng: NOTE: Potentially calls malloc(M_WAITOK) with Giant held. */ static int inp_set_source_filters(struct inpcb *inp, struct sockopt *sopt) diff --git a/sys/netinet6/in6_mcast.c b/sys/netinet6/in6_mcast.c index db2176614f38..a6186568ecb2 100644 --- a/sys/netinet6/in6_mcast.c +++ b/sys/netinet6/in6_mcast.c @@ -101,8 +101,8 @@ RB_GENERATE(ip6_msource_tree, ip6_msource, im6s_link, ip6_msource_cmp); /* * Locking: - * - Lock order is: Giant, IN6_MULTI_LOCK, INP_WLOCK, - * IN6_MULTI_LIST_LOCK, MLD_LOCK, IF_ADDR_LOCK. + * - Lock order is: IN6_MULTI_LOCK, INP_WLOCK, IN6_MULTI_LIST_LOCK, MLD_LOCK, + * IF_ADDR_LOCK. * - The IF_ADDR_LOCK is implicitly taken by in6m_lookup() earlier, however * it can be taken by code in net/if.c also. * - ip6_moptions and in6_mfilter are covered by the INP_WLOCK. @@ -1418,8 +1418,6 @@ in6_leavegroup_locked(struct in6_multi *inm, /*const*/ struct in6_mfilter *imf) * The delta-based API applies only to exclusive-mode memberships. * An MLD downcall will be performed. * - * SMPng: NOTE: Must take Giant as a join may create a new ifma. - * * Return 0 if successful, otherwise return an appropriate error code. */ static int @@ -1581,7 +1579,6 @@ out_in6p_locked: * Given an inpcb, return its multicast options structure pointer. Accepts * an unlocked inpcb pointer, but will return it locked. May sleep. * - * SMPng: NOTE: Potentially calls malloc(M_WAITOK) with Giant held. * SMPng: NOTE: Returns with the INP write lock held. */ static struct ip6_moptions *