From nobody Sun Jun 11 12:40:53 2023 X-Original-To: dev-commits-ports-main@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 4QfDx40lTnz4ctSW; Sun, 11 Jun 2023 12:42:32 +0000 (UTC) (envelope-from grembo@freebsd.org) Received: from mail.evolve.de (mail.evolve.de [213.239.217.29]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA512 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mail.evolve.de", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4QfDx21dXGz4bMP; Sun, 11 Jun 2023 12:42:30 +0000 (UTC) (envelope-from grembo@freebsd.org) Authentication-Results: mx1.freebsd.org; dkim=none; spf=softfail (mx1.freebsd.org: 213.239.217.29 is neither permitted nor denied by domain of grembo@freebsd.org) smtp.mailfrom=grembo@freebsd.org; dmarc=none Received: by mail.evolve.de (OpenSMTPD) with ESMTP id c53d2c5f; Sun, 11 Jun 2023 12:42:27 +0000 (UTC) Received: by mail.evolve.de (OpenSMTPD) with ESMTPSA id 7a8bb9bd (TLSv1.3:TLS_AES_256_GCM_SHA384:256:NO); Sun, 11 Jun 2023 12:42:27 +0000 (UTC) Date: Sun, 11 Jun 2023 14:40:53 +0200 From: Michael Gmelin To: "Herbert J. Skuhra" Cc: ports-committers@freebsd.org, dev-commits-ports-all@freebsd.org, dev-commits-ports-main@freebsd.org Subject: Re: git: 637c0bace261 - main - sysutils/pftop: Fix build on 14.0-CURRENT Message-ID: <20230611144053.7d2ae2b8.grembo@freebsd.org> In-Reply-To: <25969E38-1EA2-4F9A-8644-AF378AC2E857@freebsd.org> References: <87wn0afhdx.wl-herbert@gojira.at> <25969E38-1EA2-4F9A-8644-AF378AC2E857@freebsd.org> X-Face: $wrgCtfdVw_H9WAY?S&9+/F"!41z'L$uo*WzT8miX?kZ~W~Lr5W7v?j0Sde\mwB&/ypo^}> +a'4xMc^^KroE~+v^&^#[B">soBo1y6(TW6#UZiC]o>C6`ej+i Face: iVBORw0KGgoAAAANSUhEUgAAADAAAAAwBAMAAAClLOS0AAAAJFBMVEWJBwe5BQDl LASZU0/LTEWEfHbyj0Txi32+sKrp1Mv944X8/fm1rS+cAAAACXBIWXMAAAsTAAAL EwEAmpwYAAAAB3RJTUUH3wESCxwC7OBhbgAAACFpVFh0Q29tbWVudAAAAAAAQ3Jl YXRlZCB3aXRoIFRoZSBHSU1QbbCXAAAAAghJREFUOMu11DFvEzEUAGCfEhBVFzuq AKkLd0O6VrIQsLXVSZXoWE5N1K3DobBBA9fQpRWc8OkWouaIjedWKiyREOKs+3PY fvalCNjgLVHeF7/3bMtBzV8C/VsQ8tecEgCcDgrzjekwKZ7TwsJZd/ywEKwwP+ZM 8P3drTsAwWn2mpWuDDuYiK1bFs6De0KUUFw0tWxm+D4AIhuuvZqtyWYeO7jQ4Aea 7jUqI+ixhQoHex4WshEvSXdood7stlv4oSuFOC4tqGcr0NjEqXgV4mMJO38nld4+ xKNxRDon7khyKVqY7YR4d+Cg0OMrkWXZOM7YDkEfKiilCn1qYv4mighZiynuHHOA Wq9QJq+BIES7lMFUtcikMnkDGHUoncA+uHgrP0ctIEqfwLHzeSo+eUA66AqzwN6n 2ZHJhw6Qh/PoyC/QENyEyC/AyNjq74Bs+3UH0xYwzDUC4B97HgLocg1QLYgDDO1v f3UX9Y307Ew4AHh67YAFFsxEpkXwpXY3eIgMhAAE3R19L919nNnuD2wlPcDE3UeT L2ytEICQib9BXgS2fU8PrD82ToYO1OEmMSnYTjSqSv9wdC0tPYC+rQRQD9ESnldF CyqfmiYW+tlALt8gH2xrMdC/youbjzPXEun+/ReXsMCDyve3dZc09fn2Oas8oXGc Jj6/fOeK5UmSMPmf/jL+GD8BEj0k/Fn6IO4AAAAASUVORK5CYII= List-Id: Commits to the main branch of the FreeBSD ports repository List-Archive: https://lists.freebsd.org/archives/dev-commits-ports-main List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-ports-main@freebsd.org X-BeenThere: dev-commits-ports-main@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-Spamd-Result: default: False [-2.09 / 15.00]; MID_CONTAINS_FROM(1.00)[]; NEURAL_HAM_LONG(-1.00)[-1.000]; NEURAL_HAM_MEDIUM(-1.00)[-1.000]; NEURAL_HAM_SHORT(-0.99)[-0.993]; MIME_GOOD(-0.10)[text/plain]; TO_MATCH_ENVRCPT_SOME(0.00)[]; ARC_NA(0.00)[]; FROM_EQ_ENVFROM(0.00)[]; MLMMJ_DEST(0.00)[dev-commits-ports-all@freebsd.org,dev-commits-ports-main@freebsd.org]; MIME_TRACE(0.00)[0:+]; R_DKIM_NA(0.00)[]; RCVD_COUNT_THREE(0.00)[3]; R_SPF_SOFTFAIL(0.00)[~all:c]; DMARC_NA(0.00)[freebsd.org]; FREEFALL_USER(0.00)[grembo]; RCVD_VIA_SMTP_AUTH(0.00)[]; ASN(0.00)[asn:24940, ipnet:213.239.192.0/18, country:DE]; FROM_HAS_DN(0.00)[]; TO_DN_SOME(0.00)[]; RCPT_COUNT_THREE(0.00)[4]; RCVD_TLS_LAST(0.00)[] X-Rspamd-Queue-Id: 4QfDx21dXGz4bMP X-Spamd-Bar: -- X-ThisMailContainsUnwantedMimeParts: N On Sun, 11 Jun 2023 12:12:44 +0200 Michael Gmelin wrote: > > On 11. Jun 2023, at 12:00, Herbert J. Skuhra > > wrote: =EF=BB=BFOn Sun, 11 Jun 2023 11:25:45 +0200, Michael Gmelin wrot= e: =20 > >>=20 > >>=20 > >> =20 > >>> On 11. Jun 2023, at 10:27, Herbert J. Skuhra > >>> wrote: On Sat, 10 Jun 2023 12:06:09 +0200, > >>> Michael Gmelin wrote: =20 > >>>> The branch main has been updated by grembo: > >>>> URL: > >>>> https://cgit.FreeBSD.org/ports/commit/?id=3D637c0bace26138529a36232e= 948549ad59342ba9 > >>>> commit 637c0bace26138529a36232e948549ad59342ba9 Author: > >>>> Michael Gmelin AuthorDate: 2023-06-10 > >>>> 10:03:39 +0000 Commit: Michael Gmelin > >>>> CommitDate: 2023-06-10 10:03:39 +0000 > >>>> sysutils/pftop: Fix build on 14.0-CURRENT > >>>> --- > >>>> sysutils/pftop/Makefile | 10 ++++++++-- > >>>> sysutils/pftop/files/extra-patch-config.h | 6 +++++- > >>>> 2 files changed, 13 insertions(+), 3 deletions(-) > >>>> diff --git a/sysutils/pftop/Makefile b/sysutils/pftop/Makefile > >>>> index f3c6d879f637..cba2ecd65aeb 100644 > >>>> --- a/sysutils/pftop/Makefile > >>>> +++ b/sysutils/pftop/Makefile > >>>> @@ -1,6 +1,6 @@ > >>>> PORTNAME=3D pftop > >>>> PORTVERSION=3D 0.8 > >>>> -PORTREVISION=3D 2 > >>>> +PORTREVISION=3D 3 > >>>> CATEGORIES=3D sysutils net > >>>> MAINTAINER=3D grembo@FreeBSD.org > >>>> @@ -22,7 +22,13 @@ EXTRA_PATCHES+=3D > >>>> ${FILESDIR}/extra-patch-bpf_dump.c \ > >>>> ${FILESDIR}/extra-patch-sf-gencode.h MAKE_ARGS=3D > >>>> LOCALBASE=3D"${PREFIX}" \ > >>>> - OSLEVEL=3D45 > >>>> + > >>>> +.if ${OPSYS} =3D=3D FreeBSD && ${OSVERSION} >=3D 1400090 > >>>> +MAKE_ARGS+=3D OSLEVEL=3D46 > >>>> +.else > >>>> +MAKE_ARGS+=3D OSLEVEL=3D45 > >>>> +.endif > >>>> + > >>>> CFLAGS+=3D -DHAVE_SNPRINTF=3D1 -DHAVE_VSNPRINTF=3D1 \ > >>>> -DHAVE_FINE_GRAINED_LOCKING=3D1 > >>>> diff --git a/sysutils/pftop/files/extra-patch-config.h > >>>> b/sysutils/pftop/files/extra-patch-config.h index > >>>> 6d2873c42ab1..d24f88179718 100644 --- > >>>> a/sysutils/pftop/files/extra-patch-config.h +++ > >>>> b/sysutils/pftop/files/extra-patch-config.h @@ -1,7 +1,7 @@ > >>>> $OpenBSD: patch-config_h,v 1.4 2008/12/20 04:36:11 canacar Exp $ > >>>> --- config.h.orig Tue Nov 6 22:34:18 2007 > >>>> +++ config.h Fri Dec 19 20:28:01 2008 > >>>> -@@ -74,11 +74,20 @@ > >>>> +@@ -74,11 +74,24 @@ > >>>> #define HAVE_PFSYNC_STATE > >>>> #endif > >>>> @@ -11,7 +11,11 @@ $OpenBSD: patch-config_h,v 1.4 2008/12/20 > >>>> 04:36:11 canacar Exp $ +#endif > >>>> + > >>>> #ifdef HAVE_PFSYNC_STATE > >>>> ++#if OS_LEVEL > 45 > >>>> ++typedef struct pfsync_state_1400 pf_state_t; =20 > >>> Are you sure that this is correct? > >>> If I replace pfsync_state_1400 with pfsync_state_1301 the port > >>> builds and the output looks sane. =20 > >>=20 > >> Hi, thanks for reporting, could you please add some details (like, > >> how the output differs)? =20 > >=20 > > With your change: > >=20 > > sctp Out (null)[13715] (null)[0] > > 0:255 0 * * * 237 In (null)[0] > > (null)[0] NO_TRAFFIC:NO_TRAFFIC 0 0 > > * * ip In (null)[0] (null)[0] > > NO_TRAFFIC:NO_TRAFFIC 0 9324h * * ip In > > (null)[7185] (null)[512] NO_TRAFFIC:NO_TRAFFIC > > 0 53 * 2048G ip In (null)[4097] > > (null)[49408] NO_TRAFFIC:NO_TRAFFIC 0 0 0 > > 1 ip In (null)[9732] (null)[4992] > > 9:0 0 0 42 73728M cpnx In (null)[0] > > (null)[0] 255:0 0 * > > * * ipenca In (null)[0] (null)[0] > > NO_TRAFFIC:NO_TRAFFIC 0 0 * * ip In > > (null)[0] (null)[0] 0:9 > > * * * * > >=20 > > Garbage? > >=20 > > With attached patch I see ipv[46] addresses and port numbers again. > > :-) > >=20 > > I am running main-n263493-4e8d558c9d1c. > > =20 >=20 > The question is if pftop shouldn=E2=80=99t use pfsync_state_1400, or if t= here > is a kernel problem with that struct. I=E2=80=99ll look into it. >=20 > Thanks Hi Herbert, Apparently what happened is that I tested using 1301, which worked, then changed to 1400, which also seemed to work and committed the change. Apparently I tested the wrong binary, as rebuilding and testing I see the same results as you do. Therefore I'll switch to using 1301, thanks again for reporting. Best Michael --=20 Michael Gmelin