From owner-svn-src-all@freebsd.org Sat Jun 6 20:28:27 2020 Return-Path: Delivered-To: svn-src-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 0442433D4F2; Sat, 6 Jun 2020 20:28:27 +0000 (UTC) (envelope-from tuexen@freebsd.org) Received: from drew.franken.de (drew.ipv6.franken.de [IPv6:2001:638:a02:a001:20e:cff:fe4a:feaa]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "*.franken.de", Issuer "COMODO RSA Domain Validation Secure Server CA" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 49fWLL4QSZz3Y5n; Sat, 6 Jun 2020 20:28:26 +0000 (UTC) (envelope-from tuexen@freebsd.org) Received: from [IPv6:2a02:8109:1140:c3d:89fd:b31a:4c43:a91d] (unknown [IPv6:2a02:8109:1140:c3d:89fd:b31a:4c43:a91d]) (Authenticated sender: macmic) by mail-n.franken.de (Postfix) with ESMTPSA id E3F407220B80B; Sat, 6 Jun 2020 22:28:21 +0200 (CEST) Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 13.4 \(3608.80.23.2.2\)) Subject: Re: svn commit: r361872 - head/sys/netinet From: Michael Tuexen In-Reply-To: Date: Sat, 6 Jun 2020 22:28:21 +0200 Cc: src-committers , svn-src-all@freebsd.org, svn-src-head@freebsd.org Content-Transfer-Encoding: quoted-printable Message-Id: References: <202006061820.056IK9Fa079753@repo.freebsd.org> To: Kevin Bowling X-Mailer: Apple Mail (2.3608.80.23.2.2) X-Spam-Status: No, score=-2.9 required=5.0 tests=ALL_TRUSTED,BAYES_00 autolearn=disabled version=3.4.1 X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on mail-n.franken.de X-Rspamd-Queue-Id: 49fWLL4QSZz3Y5n X-Spamd-Bar: / Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [0.00 / 15.00]; ASN(0.00)[asn:680, ipnet:2001:638::/32, country:DE]; local_wl_from(0.00)[freebsd.org] X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.33 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 06 Jun 2020 20:28:27 -0000 > On 6. Jun 2020, at 20:56, Kevin Bowling = wrote: >=20 > Out of curiosity what is panda? Hi Kevin, it was the name used (#ifdef __Panda__) in the upstream SCTP code for using it in an Cisco proprietary product. Since I don't know if that code is still used, but they haven't contributed anything back, I think it is OK to reduce the #ifdef = complexity upstream and remove it.=20 Best regards Michael >=20 > On Sat, Jun 6, 2020 at 11:20 AM Michael Tuexen = wrote: >>=20 >> Author: tuexen >> Date: Sat Jun 6 18:20:09 2020 >> New Revision: 361872 >> URL: https://svnweb.freebsd.org/changeset/base/361872 >>=20 >> Log: >> Non-functional changes due to cleanup (upstream removing of Panda = support) >> of the code >>=20 >> MFC after: 1 week >>=20 >> Modified: >> head/sys/netinet/sctp_constants.h >> head/sys/netinet/sctp_indata.c >> head/sys/netinet/sctp_os.h >> head/sys/netinet/sctp_output.c >> head/sys/netinet/sctp_pcb.c >> head/sys/netinet/sctp_usrreq.c >>=20 >> Modified: head/sys/netinet/sctp_constants.h >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- head/sys/netinet/sctp_constants.h Sat Jun 6 17:48:55 2020 = (r361871) >> +++ head/sys/netinet/sctp_constants.h Sat Jun 6 18:20:09 2020 = (r361872) >> @@ -576,7 +576,6 @@ __FBSDID("$FreeBSD$"); >> */ >> #define SCTP_ASOC_MAX_CHUNKS_ON_QUEUE 512 >>=20 >> - >> /* >> * Basically the minimum amount of time before I do a early FR. = Making this >> * value to low will cause duplicate retransmissions. >> @@ -756,9 +755,8 @@ __FBSDID("$FreeBSD$"); >> #define SCTP_FROM_SCTP_ASCONF 0x80000000 >> #define SCTP_FROM_SCTP_OUTPUT 0x90000000 >> #define SCTP_FROM_SCTP_PEELOFF 0xa0000000 >> -#define SCTP_FROM_SCTP_PANDA 0xb0000000 >> -#define SCTP_FROM_SCTP_SYSCTL 0xc0000000 >> -#define SCTP_FROM_SCTP_CC_FUNCTIONS 0xd0000000 >> +#define SCTP_FROM_SCTP_SYSCTL 0xb0000000 >> +#define SCTP_FROM_SCTP_CC_FUNCTIONS 0xc0000000 >>=20 >> /* Location ID's */ >> #define SCTP_LOC_1 0x00000001 >>=20 >> Modified: head/sys/netinet/sctp_indata.c >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- head/sys/netinet/sctp_indata.c Sat Jun 6 17:48:55 2020 = (r361871) >> +++ head/sys/netinet/sctp_indata.c Sat Jun 6 18:20:09 2020 = (r361872) >> @@ -2721,8 +2721,7 @@ sctp_process_data(struct mbuf **mm, int iphlen, = int *o >> * cluster... i.e. it is a small packet sent in and yet the = driver >> * underneath allocated a full cluster for it. If so we must = copy it >> * to a smaller mbuf and free up the cluster mbuf. This will = help >> - * with cluster starvation. Note for __Panda__ we don't do = this >> - * since it has clusters all the way down to 64 bytes. >> + * with cluster starvation. >> */ >> if (SCTP_BUF_LEN(m) < (long)MLEN && SCTP_BUF_NEXT(m) =3D=3D = NULL) { >> /* we only handle mbufs that are singletons.. not = chains */ >>=20 >> Modified: head/sys/netinet/sctp_os.h >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- head/sys/netinet/sctp_os.h Sat Jun 6 17:48:55 2020 = (r361871) >> +++ head/sys/netinet/sctp_os.h Sat Jun 6 18:20:09 2020 = (r361872) >> @@ -67,7 +67,6 @@ __FBSDID("$FreeBSD$"); >>=20 >>=20 >>=20 >> - >> /* All os's must implement this address gatherer. If >> * no VRF's exist, then vrf 0 is the only one and all >> * addresses and ifn's live here. >>=20 >> Modified: head/sys/netinet/sctp_output.c >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- head/sys/netinet/sctp_output.c Sat Jun 6 17:48:55 2020 = (r361871) >> +++ head/sys/netinet/sctp_output.c Sat Jun 6 18:20:09 2020 = (r361872) >> @@ -6475,8 +6475,7 @@ error_out: >> appendchain =3D clonechain; >> } else { >> if (!copy_by_ref && >> - (sizeofcpy <=3D = (int)((((SCTP_BASE_SYSCTL(sctp_mbuf_threshold_count) - 1) * MLEN) + = MHLEN))) >> - ) { >> + (sizeofcpy <=3D = (int)((((SCTP_BASE_SYSCTL(sctp_mbuf_threshold_count) - 1) * MLEN) + = MHLEN)))) { >> /* Its not in a cluster */ >> if (*endofchain =3D=3D NULL) { >> /* lets get a mbuf cluster */ >> @@ -13526,12 +13525,6 @@ skip_preblock: >> error =3D sctp_msg_append(stcb, net, top, srcv, 0); >> top =3D NULL; >> if (sinfo_flags & SCTP_EOF) { >> - /* >> - * This should only happen for Panda for the = mbuf >> - * send case, which does NOT yet support EEOR = mode. >> - * Thus, we can just set this flag to do the = proper >> - * EOF handling. >> - */ >> got_all_of_the_send =3D 1; >> } >> } >>=20 >> Modified: head/sys/netinet/sctp_pcb.c >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- head/sys/netinet/sctp_pcb.c Sat Jun 6 17:48:55 2020 = (r361871) >> +++ head/sys/netinet/sctp_pcb.c Sat Jun 6 18:20:09 2020 = (r361872) >> @@ -749,8 +749,7 @@ sctp_del_addr_from_vrf(uint32_t vrf_id, struct = sockadd >>=20 >> /*- >> * The name has priority over the ifn_index >> - * if its given. We do this especially for >> - * panda who might recycle indexes fast. >> + * if its given. >> */ >> if (if_name) { >> if (strncmp(if_name, = sctp_ifap->ifn_p->ifn_name, SCTP_IFNAMSIZ) =3D=3D 0) { >> @@ -3157,8 +3156,7 @@ continue_anyway: >> } else { >> /* >> * Note for BSD we hit here always other O/S's = will >> - * pass things in via the sctp_ifap argument >> - * (Panda). >> + * pass things in via the sctp_ifap argument. >> */ >> ifa =3D sctp_find_ifa_by_addr(&store.sa, >> vrf_id, SCTP_ADDR_NOT_LOCKED); >> @@ -4302,11 +4300,7 @@ sctp_aloc_assoc(struct sctp_inpcb *inp, struct = sockadd >> * If you have not performed a bind, then we need to = do the >> * ephemeral bind for you. >> */ >> - if ((err =3D sctp_inpcb_bind(inp->sctp_socket, >> - (struct sockaddr *)NULL, >> - (struct sctp_ifa *)NULL, >> - p >> - ))) { >> + if ((err =3D sctp_inpcb_bind(inp->sctp_socket, NULL, = NULL, p))) { >> /* bind error, probably perm */ >> *error =3D err; >> return (NULL); >> @@ -4674,7 +4668,6 @@ sctp_clean_up_stream(struct sctp_tcb *stcb, = struct sct >> } >> } >> } >> - >>=20 >> /*- >> * Free the association after un-hashing the remote port. This >>=20 >> Modified: head/sys/netinet/sctp_usrreq.c >> = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D >> --- head/sys/netinet/sctp_usrreq.c Sat Jun 6 17:48:55 2020 = (r361871) >> +++ head/sys/netinet/sctp_usrreq.c Sat Jun 6 18:20:09 2020 = (r361872) >> @@ -71,7 +71,7 @@ sctp_init(void) >> SCTP_BASE_SYSCTL(sctp_max_chunks_on_queue) =3D = (nmbclusters / 8); >> /* >> * Allow a user to take no more than 1/2 the number of = clusters or >> - * the SB_MAX whichever is smaller for the send window. >> + * the SB_MAX, whichever is smaller, for the send window. >> */ >> sb_max_adj =3D (u_long)((u_quad_t)(SB_MAX) * MCLBYTES / (MSIZE = + MCLBYTES)); >> SCTP_BASE_SYSCTL(sctp_sendspace) =3D min(sb_max_adj, >> _______________________________________________ >> svn-src-head@freebsd.org mailing list >> https://lists.freebsd.org/mailman/listinfo/svn-src-head >> To unsubscribe, send any mail to = "svn-src-head-unsubscribe@freebsd.org"