svn commit: r359288 - head/sys/netinet

2020-03-24 Thread Michael Tuexen
Author: tuexen
Date: Tue Mar 24 23:04:07 2020
New Revision: 359288
URL: https://svnweb.freebsd.org/changeset/base/359288

Log:
  Only call panic when building with INVARIANTS.
  
  MFC after:1 week

Modified:
  head/sys/netinet/sctputil.c

Modified: head/sys/netinet/sctputil.c
==
--- head/sys/netinet/sctputil.c Tue Mar 24 22:44:36 2020(r359287)
+++ head/sys/netinet/sctputil.c Tue Mar 24 23:04:07 2020(r359288)
@@ -2015,7 +2015,11 @@ sctp_timeout_handler(void *t)
sctp_delete_prim_timer(inp, stcb);
break;
default:
+#ifdef INVARIANTS
panic("Unknown timer type %d", type);
+#else
+   goto get_out;
+#endif
}
 #ifdef SCTP_AUDITING_ENABLED
sctp_audit_log(0xF1, (uint8_t)type);
@@ -2434,7 +2438,11 @@ sctp_timer_start(int t_type, struct sctp_inpcb *inp, s
to_ticks = MSEC_TO_TICKS(stcb->asoc.initial_rto);
break;
default:
+#ifdef INVARIANTS
panic("Unknown timer type %d", t_type);
+#else
+   return;
+#endif
}
KASSERT(tmr != NULL, ("tmr is NULL for timer type %d", t_type));
KASSERT(to_ticks > 0, ("to_ticks == 0 for timer type %d", t_type));
@@ -2715,7 +2723,11 @@ sctp_timer_stop(int t_type, struct sctp_inpcb *inp, st
tmr = >asoc.delete_prim_timer;
break;
default:
+#ifdef INVARIANTS
panic("Unknown timer type %d", t_type);
+#else
+   return;
+#endif
}
KASSERT(tmr != NULL, ("tmr is NULL for timer type %d", t_type));
if ((tmr->type != SCTP_TIMER_TYPE_NONE) &&
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359295 - head/lib/libpmc

2020-03-24 Thread Emmanuel Vadot
Author: manu
Date: Wed Mar 25 02:28:47 2020
New Revision: 359295
URL: https://svnweb.freebsd.org/changeset/base/359295

Log:
  libpmc: Always add the full include path for libpmcstat
  
  Previously it was only added for aarch64, amd64 and i386

Modified:
  head/lib/libpmc/Makefile

Modified: head/lib/libpmc/Makefile
==
--- head/lib/libpmc/MakefileWed Mar 25 01:36:23 2020(r359294)
+++ head/lib/libpmc/MakefileWed Mar 25 02:28:47 2020(r359295)
@@ -5,6 +5,8 @@ LIB=pmc
 SRCS=  libpmc.c pmclog.c libpmc_pmu_util.c libpmc_json.cc
 INCS=  pmc.h pmclog.h pmcformat.h
 
+CFLAGS+= -I${SRCTOP}/${RELDIR:H}/libpmcstat
+
 .if ${MACHINE_ARCH} == "aarch64" || ${MACHINE_ARCH} == "amd64" || \
 ${MACHINE_ARCH} == "i386"
 
@@ -15,8 +17,6 @@ EVENT_ARCH="x86"
 .elif ${MACHINE_ARCH} == "powerpc"
 EVENT_ARCH="powerpc"
 .endif
-
-CFLAGS+= -I${SRCTOP}/${RELDIR:H}/libpmcstat
 
 .if ${MK_DIRDEPS_BUILD} == "yes"
 # avoid circular dependency
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r359168 - head

2020-03-24 Thread Enji Cooper


> On Mar 24, 2020, at 3:57 PM, Scott Long  wrote:
> 
> On Tue, Mar 24, 2020, at 3:45 PM, Gleb Smirnoff wrote:
>> On Fri, Mar 20, 2020 at 04:02:46PM +, Ed Maste wrote:
>> E> Author: emaste
>> E> Date: Fri Mar 20 16:02:45 2020
>> E> New Revision: 359168
>> E> URL: https://svnweb.freebsd.org/changeset/base/359168
>> E> 
>> E> Log:
>> E>   remove ancient pre-2000 ObsoleteFiles.inc entries
>> E>   
>> E>   We support 10.3 as the minimum version to install from, which was
>> E>   released in the mid-2010s.  There's a lot of ancient ObsoleteFiles.inc
>> E>   history that serves no purpose today; start by removing entries from
>> E>   1999 and earlier.
>> 
>> I understand that rationale is to speedup 'make delete-old' times, and
>> trimming the default file definitely makes sense. However, what about
>> keeping the full ObsoleteFiles.inc version, at least for documenting
>> purposes?
>> 
> 
> This is what SCM history is for, IMHO.  The historical entries serve little
> purpose outside of that.

Moreover, splitting them up into separate files will make merges more difficult 
than they need to be/already are.
Cheers,
-Enji
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359294 - in head/libexec/rc: . rc.d

2020-03-24 Thread Emmanuel Vadot
Author: manu
Date: Wed Mar 25 01:36:23 2020
New Revision: 359294
URL: https://svnweb.freebsd.org/changeset/base/359294

Log:
  pkgbase: Stop threating rc files as config files
  
  rc scripts arent configuration files so use FILES instead of CONFS.
  While here put rc scripts into related package (sendmail in the 
FreeBSD-sendmail
  package, wpa_supplicant in the FreeBSD-wpa etc ...)
  
  Reviewed by:  bapt, emaste
  Differential Revision:https://reviews.freebsd.org/D24177

Modified:
  head/libexec/rc/Makefile
  head/libexec/rc/rc.d/Makefile

Modified: head/libexec/rc/Makefile
==
--- head/libexec/rc/MakefileWed Mar 25 01:35:13 2020(r359293)
+++ head/libexec/rc/MakefileWed Mar 25 01:36:23 2020(r359294)
@@ -2,25 +2,28 @@
 
 .include 
 
-CONFGROUPS=CONFETC CONFETCEXEC CONFETCDEFAULTS
-CONFETCDIR=/etc
-CONFETC=   network.subr rc rc.initdiskless rc.subr rc.shutdown 
rc.bsdextended
-CONFETCPACKAGE=rc
+BINDIR=/etc
+FILESGROUPS=   RCETC RCETCEXEC RCETCDEFAULTS
+RCETC= network.subr rc rc.initdiskless rc.subr rc.shutdown rc.bsdextended
+RCETCPACKAGE=  rc
 
 .if ${MK_IPFW} != "no"
-CONFETC+=  rc.firewall
+FILESGROUPS+=  RCIPFW
+RCIPFW+=   rc.firewall
+RCIPFWPACKAGE= ipfw
 .endif
 .if ${MK_SENDMAIL} != "no"
-CONFETC+=  rc.sendmail
+FILESGROUPS+=  RCSENDMAIL
+RCSENDMAIL+=   rc.sendmail
+RCSENDMAILPACKAGE= sendmail
 .endif
-CONFETCMODE=   644
-CONFETCEXEC=   netstart pccard_ether rc.resume rc.suspend
-CONFETCEXECDIR=/etc
-CONFETCEXECMODE=   755
-CONFETCEXECPACKAGE=rc
-CONFETCDEFAULTSDIR=/etc/defaults
-CONFETCDEFAULTS=   rc.conf
-CONFETCDEFAULTSPACKAGE=rc
+RCETCMODE= 644
+RCETCEXEC= netstart pccard_ether rc.resume rc.suspend
+RCETCEXECMODE= 755
+RCETCEXECPACKAGE=  rc
+RCETCDEFAULTSDIR=  /etc/defaults
+RCETCDEFAULTS= rc.conf
+RCETCDEFAULTSPACKAGE=  rc
 
 SUBDIR+=   rc.d
 

Modified: head/libexec/rc/rc.d/Makefile
==
--- head/libexec/rc/rc.d/Makefile   Wed Mar 25 01:35:13 2020
(r359293)
+++ head/libexec/rc/rc.d/Makefile   Wed Mar 25 01:36:23 2020
(r359294)
@@ -2,11 +2,11 @@
 
 .include 
 
-CONFDIR=   /etc/rc.d
-CONFGROUPS=CONFS
-CONFSPACKAGE=  rc
+BINDIR=/etc/rc.d
+FILESGROUPS=   BASERC
+BASERCPACKAGE= rc
 
-CONFS= DAEMON \
+BASERC=DAEMON \
FILESYSTEMS \
LOGIN \
NETWORKING \
@@ -115,7 +115,7 @@ CONFS=  DAEMON \
watchdogd
 
 .if ${MK_NIS} != "no"
-CONFS+=ypbind \
+BASERC+=   ypbind \
ypldap \
yppasswdd \
ypserv \
@@ -125,23 +125,23 @@ CONFS+=   ypbind \
 .endif
 
 .if ${MK_ACCT} != "no"
-CONFGROUPS+=   ACCT
+FILESGROUPS+=  ACCT
 ACCT+= accounting
 ACCTPACKAGE=   acct
 .endif
 
 .if ${MK_ACPI} != "no"
-CONFGROUPS+=   ACPI
+FILESGROUPS+=  ACPI
 ACPI=  power_profile
 ACPIPACKAGE=   acpi
 .endif
 
 .if ${MK_ACPI} != "no" || ${MK_APM} != "no"
-CONFS+=powerd
+BASERC+=   powerd
 .endif
 
 .if ${MK_APM} != "no"
-CONFGROUPS+=   APM
+FILESGROUPS+=  APM
 APM+=  apm
 .if ${MACHINE} == "i386"
 APM+=  apmd
@@ -150,15 +150,17 @@ APMPACKAGE=   apm
 .endif
 
 .if ${MK_AUDIT} != "no"
-CONFGROUPS+=   AUDIT
+FILESGROUPS+=  AUDIT
 AUDIT+=auditd
 AUDIT+=auditdistd
 .endif
 
 .if ${MK_AUTOFS} != "no"
-CONFS+=automount
-CONFS+=automountd
-CONFS+=autounmountd
+FILESGROUPS+=  AUFOFS
+AUTOFS+=   automount
+AUTOFS+=   automountd
+AUTOFS+=   autounmountd
+AUTOFSPACKAGE= autofs
 .endif
 
 .if ${MK_BLACKLIST} != "no"
@@ -166,7 +168,7 @@ _blacklistd+=   blacklistd
 .endif
 
 .if ${MK_BLUETOOTH} != "no"
-CONFGROUPS+=   BLUETOOTH
+FILESGROUPS+=  BLUETOOTH
 BLUETOOTH+=bluetooth \
bthidd \
hcsecd \
@@ -177,71 +179,79 @@ BLUETOOTHPACKAGE= bluetooth
 .endif
 
 .if ${MK_BOOTPARAMD} != "no"
-CONFS+=bootparams
+BASERC+=   bootparams
 .endif
 
 .if ${MK_BSNMP} != "no"
-CONFGROUPS+=   BSNMP
+FILESGROUPS+=  BSNMP
 BSNMP+=bsnmpd
 BSNMPPACKAGE=  bsnmp
 .endif
 
 .if ${MK_CCD} != "no"
-CONFS+=ccd
+FILESGROUPS+=  CCD
+CCD+=  ccd
+CCDPACKAGE=ccdconfig
 .endif
 
 .if ${MK_FTP} != "no"
-CONFS+=ftpd
+BASERC+=   ftpd
 .endif
 
 .if ${MK_HAST} != "no"
-CONFGROUPS+=   HAST
+FILESGROUPS+=  HAST
 HAST=  hastd
 HASTPACKAGE=   hast
 .endif
 
 .if ${MK_INETD} != "no"
-CONFS+=inetd
+BASERC+=   inetd
 .endif
 
 .if ${MK_IPFILTER} != "no"
-CONFS+=ipfilter \
-   ipfs \
-   ipmon \
-   ipnat \
-   ippool
+FILESGROUPS+=  IPFILTER
+IPFILTER+= ipfilter \
+   ipfs \
+   ipmon \
+   ipnat \
+   ippool

svn commit: r359293 - in head: . share/mk

2020-03-24 Thread Emmanuel Vadot
Author: manu
Date: Wed Mar 25 01:35:13 2020
New Revision: 359293
URL: https://svnweb.freebsd.org/changeset/base/359293

Log:
  Re-apply r359267 now that tools are using the proper include path
  
  Original commit message:
  bsd.lib.mk: Do not include bsd.incs.mk for INTERNALLIB
  
  f we're building an internal lib do not bother including bsd.incs.mk so we
  will not install the headers.
  This also "solves" a problem with pkgbase where a libXXX-development package
  is created and due to how packages are created we add a dependency to a
  libXXX package that doesn't exists.

Modified:
  head/ObsoleteFiles.inc
  head/share/mk/bsd.lib.mk

Modified: head/ObsoleteFiles.inc
==
--- head/ObsoleteFiles.inc  Wed Mar 25 01:32:16 2020(r359292)
+++ head/ObsoleteFiles.inc  Wed Mar 25 01:35:13 2020(r359293)
@@ -36,6 +36,11 @@
 #   xargs -n1 | sort | uniq -d;
 # done
 
+# 20200323: INTERNALLIB don't install headers anymore
+OLD_FILES+=usr/include/libelftc.h
+OLD_FILES+=usr/include/libifconfig.h
+OLD_FILES+=usr/include/libpmcstat.h
+
 # 20200320: cx and ctau drivers retired
 OLD_FILES+=usr/share/man/man4/ctau.4.gz
 OLD_FILES+=usr/share/man/man4/cx.4.gz

Modified: head/share/mk/bsd.lib.mk
==
--- head/share/mk/bsd.lib.mkWed Mar 25 01:32:16 2020(r359292)
+++ head/share/mk/bsd.lib.mkWed Mar 25 01:35:13 2020(r359293)
@@ -489,7 +489,10 @@ _libinstall:
 .include 
 .include 
 .include 
+#No need to install header for INTERNALLIB
+.if !defined(INTERNALLIB)
 .include 
+.endif
 .endif
 
 LINKOWN?=  ${LIBOWN}
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359292 - in head: lib/libpmc lib/libpmcstat usr.sbin/pmc usr.sbin/pmcstat

2020-03-24 Thread Emmanuel Vadot
Author: manu
Date: Wed Mar 25 01:32:16 2020
New Revision: 359292
URL: https://svnweb.freebsd.org/changeset/base/359292

Log:
  pmc: Add include path for libpmcstat as it is an internallib
  
  Reviewed by:  bapt
  Differential Revision:https://reviews.freebsd.org/D24173

Modified:
  head/lib/libpmc/Makefile
  head/lib/libpmcstat/Makefile
  head/usr.sbin/pmc/Makefile
  head/usr.sbin/pmcstat/Makefile

Modified: head/lib/libpmc/Makefile
==
--- head/lib/libpmc/MakefileWed Mar 25 01:31:26 2020(r359291)
+++ head/lib/libpmc/MakefileWed Mar 25 01:32:16 2020(r359292)
@@ -16,9 +16,10 @@ EVENT_ARCH="x86"
 EVENT_ARCH="powerpc"
 .endif
 
+CFLAGS+= -I${SRCTOP}/${RELDIR:H}/libpmcstat
+
 .if ${MK_DIRDEPS_BUILD} == "yes"
 # avoid circular dependency
-CFLAGS+= -I${SRCTOP}/${RELDIR:H}/libpmcstat
 GENDIRDEPS_FILTER+= N${RELDIR:H}/libpmcstat
 JEVENTS?= ${HOST_OBJTOP}/${RELDIR}/pmu-events/jevents
 .else

Modified: head/lib/libpmcstat/Makefile
==
--- head/lib/libpmcstat/MakefileWed Mar 25 01:31:26 2020
(r359291)
+++ head/lib/libpmcstat/MakefileWed Mar 25 01:32:16 2020
(r359292)
@@ -10,6 +10,8 @@ SRCS= \
libpmcstat_process.c\
libpmcstat_string.c \
libpmcstat_symbol.c
-INCS=  libpmcstat.h
+
+# If libpmcstat becomes public uncomment this line
+#INCS= libpmcstat.h
 
 .include 

Modified: head/usr.sbin/pmc/Makefile
==
--- head/usr.sbin/pmc/Makefile  Wed Mar 25 01:31:26 2020(r359291)
+++ head/usr.sbin/pmc/Makefile  Wed Mar 25 01:32:16 2020(r359292)
@@ -9,6 +9,7 @@ WARNS?= 3
 CXXFLAGS+= -O0
 CXXSTD= c++14
 CWARNFLAGS.gcc+= -Wno-redundant-decls
+CFLAGS+= -I${SRCTOP}/lib/libpmcstat
 
 LIBADD=kvm pmc m ncursesw pmcstat elf
 

Modified: head/usr.sbin/pmcstat/Makefile
==
--- head/usr.sbin/pmcstat/Makefile  Wed Mar 25 01:31:26 2020
(r359291)
+++ head/usr.sbin/pmcstat/Makefile  Wed Mar 25 01:32:16 2020
(r359292)
@@ -5,6 +5,7 @@
 PROG_CXX=  pmcstat
 MAN=   pmcstat.8
 
+CFLAGS+= -I${SRCTOP}/lib/libpmcstat
 LIBADD=kvm pmc m ncursesw pmcstat elf
 
 SRCS=  pmcstat.c pmcstat.h pmcstat_log.c \
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359291 - in head: lib/libifconfig usr.sbin/wlandebug

2020-03-24 Thread Emmanuel Vadot
Author: manu
Date: Wed Mar 25 01:31:26 2020
New Revision: 359291
URL: https://svnweb.freebsd.org/changeset/base/359291

Log:
  wlandebug: Add include path for libifconfig as it is a internallib
  
  Reviewed by:  bapt
  Differential Revision:https://reviews.freebsd.org/D24172

Modified:
  head/lib/libifconfig/Makefile
  head/usr.sbin/wlandebug/Makefile

Modified: head/lib/libifconfig/Makefile
==
--- head/lib/libifconfig/Makefile   Wed Mar 25 00:31:42 2020
(r359290)
+++ head/lib/libifconfig/Makefile   Wed Mar 25 01:31:26 2020
(r359291)
@@ -10,8 +10,9 @@ SRCS= libifconfig.c libifconfig_carp.c libifconfig_in
 SRCS+= libifconfig_inet6.c libifconfig_internal.c libifconfig_lagg.c
 SRCS+= libifconfig_media.c
 
-INCSDIR=   ${INCLUDEDIR}
-INCS=  libifconfig.h
+# If libifconfig become public uncomment those two lines
+#INCSDIR=  ${INCLUDEDIR}
+#INCS= libifconfig.h
 
 #MAN=  libifconfig.3
 

Modified: head/usr.sbin/wlandebug/Makefile
==
--- head/usr.sbin/wlandebug/MakefileWed Mar 25 00:31:42 2020
(r359290)
+++ head/usr.sbin/wlandebug/MakefileWed Mar 25 01:31:26 2020
(r359291)
@@ -3,6 +3,7 @@
 PROG=  wlandebug
 MAN=   wlandebug.8
 
+CFLAGS+=   -I${SRCTOP}/lib/libifconfig
 LIBADD+=   ifconfig
 
 WARNS?=2
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359290 - in stable: 11/usr.bin/hexdump/tests 12/usr.bin/hexdump/tests

2020-03-24 Thread Kyle Evans
Author: kevans
Date: Wed Mar 25 00:31:42 2020
New Revision: 359290
URL: https://svnweb.freebsd.org/changeset/base/359290

Log:
  MFC r358553: hexdump: tests: take into account byte order
  
  Hexdump test was failling on big endian systems when testing decimal, octal
  and hexa outputs as the tests were designed on a little endian system. This
  revision adds the two distinct flavors of output expected and determines at
  runtime which to compare against.

Added:
  stable/12/usr.bin/hexdump/tests/d_hexdump_dflag_a_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_a_eb.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_dflag_a_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_a_el.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_dflag_b_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_b_eb.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_dflag_b_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_b_el.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_dflag_c_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_c_eb.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_dflag_c_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_c_el.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_oflag_a_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_oflag_a_eb.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_oflag_a_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_oflag_a_el.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_oflag_b_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_oflag_b_eb.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_oflag_b_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_oflag_b_el.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_oflag_c_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_oflag_c_eb.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_oflag_c_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_oflag_c_el.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_xflag_a_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_xflag_a_eb.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_xflag_a_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_xflag_a_el.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_xflag_b_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_xflag_b_eb.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_xflag_b_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_xflag_b_el.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_xflag_c_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_xflag_c_eb.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_xflag_c_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_xflag_c_el.out
Deleted:
  stable/12/usr.bin/hexdump/tests/d_hexdump_dflag_a.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_dflag_b.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_dflag_c.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_oflag_a.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_oflag_b.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_oflag_c.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_xflag_a.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_xflag_b.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_xflag_c.out
Modified:
  stable/12/usr.bin/hexdump/tests/Makefile
  stable/12/usr.bin/hexdump/tests/hexdump_test.sh
Directory Properties:
  stable/12/   (props changed)

Changes in other areas also in this revision:
Added:
  stable/11/usr.bin/hexdump/tests/d_hexdump_dflag_a_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_a_eb.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_dflag_a_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_a_el.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_dflag_b_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_b_eb.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_dflag_b_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_b_el.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_dflag_c_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_c_eb.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_dflag_c_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_c_el.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_oflag_a_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_oflag_a_eb.out
  

svn commit: r359290 - in stable: 11/usr.bin/hexdump/tests 12/usr.bin/hexdump/tests

2020-03-24 Thread Kyle Evans
Author: kevans
Date: Wed Mar 25 00:31:42 2020
New Revision: 359290
URL: https://svnweb.freebsd.org/changeset/base/359290

Log:
  MFC r358553: hexdump: tests: take into account byte order
  
  Hexdump test was failling on big endian systems when testing decimal, octal
  and hexa outputs as the tests were designed on a little endian system. This
  revision adds the two distinct flavors of output expected and determines at
  runtime which to compare against.

Added:
  stable/11/usr.bin/hexdump/tests/d_hexdump_dflag_a_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_a_eb.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_dflag_a_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_a_el.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_dflag_b_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_b_eb.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_dflag_b_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_b_el.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_dflag_c_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_c_eb.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_dflag_c_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_c_el.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_oflag_a_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_oflag_a_eb.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_oflag_a_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_oflag_a_el.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_oflag_b_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_oflag_b_eb.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_oflag_b_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_oflag_b_el.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_oflag_c_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_oflag_c_eb.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_oflag_c_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_oflag_c_el.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_xflag_a_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_xflag_a_eb.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_xflag_a_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_xflag_a_el.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_xflag_b_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_xflag_b_eb.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_xflag_b_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_xflag_b_el.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_xflag_c_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_xflag_c_eb.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_xflag_c_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_xflag_c_el.out
Deleted:
  stable/11/usr.bin/hexdump/tests/d_hexdump_dflag_a.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_dflag_b.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_dflag_c.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_oflag_a.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_oflag_b.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_oflag_c.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_xflag_a.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_xflag_b.out
  stable/11/usr.bin/hexdump/tests/d_hexdump_xflag_c.out
Modified:
  stable/11/usr.bin/hexdump/tests/Makefile
  stable/11/usr.bin/hexdump/tests/hexdump_test.sh
Directory Properties:
  stable/11/   (props changed)

Changes in other areas also in this revision:
Added:
  stable/12/usr.bin/hexdump/tests/d_hexdump_dflag_a_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_a_eb.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_dflag_a_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_a_el.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_dflag_b_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_b_eb.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_dflag_b_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_b_el.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_dflag_c_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_c_eb.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_dflag_c_el.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_dflag_c_el.out
  stable/12/usr.bin/hexdump/tests/d_hexdump_oflag_a_eb.out
 - copied unchanged from r358553, 
head/usr.bin/hexdump/tests/d_hexdump_oflag_a_eb.out
  

Re: svn commit: r359168 - head

2020-03-24 Thread Warner Losh
On Tue, Mar 24, 2020 at 4:58 PM Scott Long  wrote:

>
>
> On Tue, Mar 24, 2020, at 3:45 PM, Gleb Smirnoff wrote:
> > On Fri, Mar 20, 2020 at 04:02:46PM +, Ed Maste wrote:
> > E> Author: emaste
> > E> Date: Fri Mar 20 16:02:45 2020
> > E> New Revision: 359168
> > E> URL: https://svnweb.freebsd.org/changeset/base/359168
> > E>
> > E> Log:
> > E>   remove ancient pre-2000 ObsoleteFiles.inc entries
> > E>
> > E>   We support 10.3 as the minimum version to install from, which was
> > E>   released in the mid-2010s.  There's a lot of ancient
> ObsoleteFiles.inc
> > E>   history that serves no purpose today; start by removing entries from
> > E>   1999 and earlier.
> >
> > I understand that rationale is to speedup 'make delete-old' times, and
> > trimming the default file definitely makes sense. However, what about
> > keeping the full ObsoleteFiles.inc version, at least for documenting
> > purposes?
> >
>
> This is what SCM history is for, IMHO.  The historical entries serve little
> purpose outside of that.
>

We only support upgrading so far into the past. Farther than we can build,
sure, but it's still limited.

If you have systems older than that, or that haven't been preened, you can
get the the last release's tree and do a cleaning.

There's no more reason to retain this stuff than there is to retain ancient
version compat code in drivers, etc.

So while it seems like a good idea to have this back to dirt, in reality it
would get little use and the data can be found easily enough now, and in
the future. That's why I advised Ed to just delete the entries > 20 years
old...

Warner
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r359168 - head

2020-03-24 Thread Gleb Smirnoff
On Fri, Mar 20, 2020 at 04:02:46PM +, Ed Maste wrote:
E> Author: emaste
E> Date: Fri Mar 20 16:02:45 2020
E> New Revision: 359168
E> URL: https://svnweb.freebsd.org/changeset/base/359168
E> 
E> Log:
E>   remove ancient pre-2000 ObsoleteFiles.inc entries
E>   
E>   We support 10.3 as the minimum version to install from, which was
E>   released in the mid-2010s.  There's a lot of ancient ObsoleteFiles.inc
E>   history that serves no purpose today; start by removing entries from
E>   1999 and earlier.

I understand that rationale is to speedup 'make delete-old' times, and
trimming the default file definitely makes sense. However, what about
keeping the full ObsoleteFiles.inc version, at least for documenting
purposes?

-- 
Gleb Smirnoff
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359289 - head

2020-03-24 Thread Mitchell Horne
Author: mhorne
Date: Tue Mar 24 23:25:54 2020
New Revision: 359289
URL: https://svnweb.freebsd.org/changeset/base/359289

Log:
  Makefile.inc1: override MACHINE for native-xtools
  
  For the final step of the native-xtools target, "everything" is built
  with TARGET and TARGET_ARCH set to the architecture we wish to
  cross-build for. However, CROSSENV overwrites the values of MACHINE and
  MACHINE_ARCH, setting them to be identical to TARGET and TARGET_ARCH.
  For native-xtools this is undesirable since we are building binaries to
  run on the host architecture, that can cross compile for the target
  architecture.
  
  When building native-xtools for RISC-V, this issue manifests as an invalid
  argument for "-march". The compiler is invoked with the target triple of the
  host architecture, but the CFLAGS inherited from bsd.cpu.mk are that of the
  target architecture.
  
  Reviewed by:  imp, bdrewery
  MFC after:2 weeks
  Differential Revision:https://reviews.freebsd.org/D23838

Modified:
  head/Makefile.inc1

Modified: head/Makefile.inc1
==
--- head/Makefile.inc1  Tue Mar 24 23:04:07 2020(r359288)
+++ head/Makefile.inc1  Tue Mar 24 23:25:54 2020(r359289)
@@ -2650,6 +2650,7 @@ NXBMAKE+= XCC="${NXBOBJTOP}/tmp/usr/bin/cc" \
XCPP="${NXBOBJTOP}/tmp/usr/bin/cpp"
 .endif
 NXBMAKE+=  ${NXBMAKEENV} ${MAKE} -f Makefile.inc1 ${NXBMAKEARGS} \
+   MACHINE=${MACHINE} MACHINE_ARCH=${MACHINE_ARCH} \
TARGET=${NXB_TARGET} TARGET_ARCH=${NXB_TARGET_ARCH} \
TARGET_TRIPLE=${MACHINE_TRIPLE:Q}
 # NXBDIRS is improperly based on MACHINE rather than NXB_TARGET.  Need to
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r359168 - head

2020-03-24 Thread Scott Long



On Tue, Mar 24, 2020, at 3:45 PM, Gleb Smirnoff wrote:
> On Fri, Mar 20, 2020 at 04:02:46PM +, Ed Maste wrote:
> E> Author: emaste
> E> Date: Fri Mar 20 16:02:45 2020
> E> New Revision: 359168
> E> URL: https://svnweb.freebsd.org/changeset/base/359168
> E> 
> E> Log:
> E>   remove ancient pre-2000 ObsoleteFiles.inc entries
> E>   
> E>   We support 10.3 as the minimum version to install from, which was
> E>   released in the mid-2010s.  There's a lot of ancient ObsoleteFiles.inc
> E>   history that serves no purpose today; start by removing entries from
> E>   1999 and earlier.
> 
> I understand that rationale is to speedup 'make delete-old' times, and
> trimming the default file definitely makes sense. However, what about
> keeping the full ObsoleteFiles.inc version, at least for documenting
> purposes?
> 

This is what SCM history is for, IMHO.  The historical entries serve little
purpose outside of that.

Scott
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359287 - head/sys/netinet

2020-03-24 Thread Michael Tuexen
Author: tuexen
Date: Tue Mar 24 22:44:36 2020
New Revision: 359287
URL: https://svnweb.freebsd.org/changeset/base/359287

Log:
  Another cleanup of the timer code. Also be more pedantic about the
  parameters of the timer start and stop routines. Several inconsistencies
  have been fixed in earlier commits. Now they will be catched when running
  an INVARIANTS system.
  
  MFC after:1 week

Modified:
  head/sys/netinet/sctputil.c

Modified: head/sys/netinet/sctputil.c
==
--- head/sys/netinet/sctputil.c Tue Mar 24 21:37:34 2020(r359286)
+++ head/sys/netinet/sctputil.c Tue Mar 24 22:44:36 2020(r359287)
@@ -1601,10 +1601,36 @@ sctp_handle_addr_wq(void)
}
 }
 
+/*-
+ * The following table shows which pointers for the inp, stcb, or net are
+ * stored for each timer after it was started.
+ *
+ *|Name |Timer|inp |stcb|net |
+ *|-|-||||
+ *|SCTP_TIMER_TYPE_SEND |net->rxt_timer   |Yes |Yes |Yes |
+ *|SCTP_TIMER_TYPE_INIT |net->rxt_timer   |Yes |Yes |Yes |
+ *|SCTP_TIMER_TYPE_RECV |stcb->asoc.dack_timer|Yes |Yes |No  |
+ *|SCTP_TIMER_TYPE_SHUTDOWN |net->rxt_timer   |Yes |Yes |Yes |
+ *|SCTP_TIMER_TYPE_HEARTBEAT|net->hb_timer|Yes |Yes |Yes |
+ *|SCTP_TIMER_TYPE_COOKIE   |net->rxt_timer   |Yes |Yes |Yes |
+ *|SCTP_TIMER_TYPE_NEWCOOKIE|inp->sctp_ep.signature_change|Yes |No  |No  |
+ *|SCTP_TIMER_TYPE_PATHMTURAISE |net->pmtu_timer  |Yes |Yes |Yes |
+ *|SCTP_TIMER_TYPE_SHUTDOWNACK  |net->rxt_timer   |Yes |Yes |Yes |
+ *|SCTP_TIMER_TYPE_ASCONF   |stcb->asoc.asconf_timer  |Yes |Yes |Yes |
+ *|SCTP_TIMER_TYPE_SHUTDOWNGUARD|stcb->asoc.shut_guard_timer  |Yes |Yes |No  |
+ *|SCTP_TIMER_TYPE_AUTOCLOSE|stcb->asoc.autoclose_timer   |Yes |Yes |No  |
+ *|SCTP_TIMER_TYPE_STRRESET |stcb->asoc.strreset_timer|Yes |Yes |No  |
+ *|SCTP_TIMER_TYPE_INPKILL  |inp->sctp_ep.signature_change|Yes |No  |No  |
+ *|SCTP_TIMER_TYPE_ASOCKILL |stcb->asoc.strreset_timer|Yes |Yes |No  |
+ *|SCTP_TIMER_TYPE_ADDR_WQ  |SCTP_BASE_INFO(addr_wq_timer)|No  |No  |No  |
+ *|SCTP_TIMER_TYPE_PRIM_DELETED |stcb->asoc.delete_prim_timer |Yes |Yes |No  |
+ */
+
 void
 sctp_timeout_handler(void *t)
 {
struct epoch_tracker et;
+   struct timeval tv;
struct sctp_inpcb *inp;
struct sctp_tcb *stcb;
struct sctp_nets *net;
@@ -1615,6 +1641,7 @@ sctp_timeout_handler(void *t)
 #endif
int did_output;
int type;
+   int i, secret;
 
tmr = (struct sctp_timer *)t;
inp = (struct sctp_inpcb *)tmr->ep;
@@ -1629,48 +1656,31 @@ sctp_timeout_handler(void *t)
 #endif
 
/* sanity checks... */
-   if (tmr->self != (void *)tmr) {
-   /*
-* SCTP_PRINTF("Stale SCTP timer fired (%p), ignoring...\n",
-* (void *)tmr);
-*/
-   CURVNET_RESTORE();
-   return;
-   }
+   KASSERT(tmr->self == tmr, ("tmr->self corrupted"));
+   KASSERT(SCTP_IS_TIMER_TYPE_VALID(tmr->type), ("Invalid timer type %d", 
tmr->type));
+   type = tmr->type;
tmr->stopped_from = 0xa001;
-   if (!SCTP_IS_TIMER_TYPE_VALID(tmr->type)) {
-   /*
-* SCTP_PRINTF("SCTP timer fired with invalid type: 0x%x\n",
-* tmr->type);
-*/
-   CURVNET_RESTORE();
-   return;
-   }
-   tmr->stopped_from = 0xa002;
-   if ((tmr->type != SCTP_TIMER_TYPE_ADDR_WQ) && (inp == NULL)) {
-   CURVNET_RESTORE();
-   return;
-   }
-   /* if this is an iterator timeout, get the struct and clear inp */
-   tmr->stopped_from = 0xa003;
if (inp) {
SCTP_INP_INCR_REF(inp);
if ((inp->sctp_socket == NULL) &&
-   ((tmr->type != SCTP_TIMER_TYPE_INPKILL) &&
-   (tmr->type != SCTP_TIMER_TYPE_INIT) &&
-   (tmr->type != SCTP_TIMER_TYPE_SEND) &&
-   (tmr->type != SCTP_TIMER_TYPE_RECV) &&
-   (tmr->type != SCTP_TIMER_TYPE_HEARTBEAT) &&
-   (tmr->type != SCTP_TIMER_TYPE_SHUTDOWN) &&
-   (tmr->type != SCTP_TIMER_TYPE_SHUTDOWNACK) &&
-   (tmr->type != SCTP_TIMER_TYPE_SHUTDOWNGUARD) &&
-   (tmr->type != SCTP_TIMER_TYPE_ASOCKILL))) {
+   ((type != SCTP_TIMER_TYPE_INPKILL) &&
+   (type != SCTP_TIMER_TYPE_INIT) &&
+   (type != SCTP_TIMER_TYPE_SEND) &&
+   (type != SCTP_TIMER_TYPE_RECV) &&
+   (type != SCTP_TIMER_TYPE_HEARTBEAT) &&
+   (type != SCTP_TIMER_TYPE_SHUTDOWN) &&
+   (type != 

Re: svn commit: r359168 - head

2020-03-24 Thread Oliver Pinter
Wouldn't there been a better way to split up the global obsoletefiles.inc
file and put them in to specific components directory or at least split
them up by release basis to easier the future deletions?

On Tuesday, March 24, 2020, Gleb Smirnoff  wrote:

> On Fri, Mar 20, 2020 at 04:02:46PM +, Ed Maste wrote:
> E> Author: emaste
> E> Date: Fri Mar 20 16:02:45 2020
> E> New Revision: 359168
> E> URL: https://svnweb.freebsd.org/changeset/base/359168
> E>
> E> Log:
> E>   remove ancient pre-2000 ObsoleteFiles.inc entries
> E>
> E>   We support 10.3 as the minimum version to install from, which was
> E>   released in the mid-2010s.  There's a lot of ancient ObsoleteFiles.inc
> E>   history that serves no purpose today; start by removing entries from
> E>   1999 and earlier.
>
> I understand that rationale is to speedup 'make delete-old' times, and
> trimming the default file definitely makes sense. However, what about
> keeping the full ObsoleteFiles.inc version, at least for documenting
> purposes?
>
> --
> Gleb Smirnoff
> ___
> svn-src-h...@freebsd.org mailing list
> https://lists.freebsd.org/mailman/listinfo/svn-src-head
> To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"
>
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359286 - head

2020-03-24 Thread Ed Maste
Author: emaste
Date: Tue Mar 24 21:37:34 2020
New Revision: 359286
URL: https://svnweb.freebsd.org/changeset/base/359286

Log:
  Makefile.inc1: remove old (2018) stale dependency hacks
  
  We handle a number of stale dependency issues on an ad-hoc basis, in
  order to support ongoing NO_CLEAN builds.  These hacks do not need to
  be maintained indefinitely; now remove those for issues that are more
  than a year old.
  
  Sponsored by: The FreeBSD Foundation

Modified:
  head/Makefile.inc1

Modified: head/Makefile.inc1
==
--- head/Makefile.inc1  Tue Mar 24 21:28:48 2020(r359285)
+++ head/Makefile.inc1  Tue Mar 24 21:37:34 2020(r359286)
@@ -962,9 +962,8 @@ _cleanobj_fast_depend_hack: .PHONY
fi
 .endif
 .endfor
-# 20180604  r334626  brk sbrk
 # 20190916  r352703  shm_open
-.for f in brk sbrk shm_open
+.for f in shm_open
@if [ -e "${OBJTOP}/lib/libc/.depend.${f}.o" ] && \
egrep -qw '${f}\.[sS]' ${OBJTOP}/lib/libc/.depend.${f}.o; then \
echo "Removing stale dependencies for ${f} syscall wrappers"; \
@@ -972,37 +971,6 @@ _cleanobj_fast_depend_hack: .PHONY
   ${_LIBCOMPAT:D${LIBCOMPAT_OBJTOP}/lib/libc/.depend.${f}.*}; \
fi
 .endfor
-# 20181013  r339348  bcopy reimplemented as .c
-.for f in bcopy memcpy memmove
-   @if [ -e "${OBJTOP}/lib/libc/.depend.${f}.o" ] && \
-   egrep -qw 'bcopy\.[sS]' ${OBJTOP}/lib/libc/.depend.${f}.o; then \
-   echo "Removing stale dependencies for bcopy"; \
-   rm -f ${OBJTOP}/lib/libc/.depend.${f}.* \
-  ${_LIBCOMPAT:D${LIBCOMPAT_OBJTOP}/lib/libc/.depend.${f}.*}; \
-   fi
-.endfor
-# 20181115  r340463  bzero reimplemented as .c
-   @if [ -e "${OBJTOP}/lib/libc/.depend.bzero.o" ] && \
-   egrep -qw 'bzero\.[sS]' ${OBJTOP}/lib/libc/.depend.bzero.o; then \
-   echo "Removing stale dependencies for bzero"; \
-   rm -f ${OBJTOP}/lib/libc/.depend.bzero.* \
-  ${_LIBCOMPAT:D${LIBCOMPAT_OBJTOP}/lib/libc/.depend.bzero.*}; 
\
-   fi
-# 20181009 track migration from ntp's embedded libevent to updated one
-   @if [ -e 
"${OBJTOP}/usr.sbin/ntp/libntpevent/.depend.bufferevent_openssl.o" ] && \
-   egrep -q 'contrib/ntp/sntp/libevent/bufferevent_openssl.c' \
-   ${OBJTOP}/usr.sbin/ntp/libntpevent/.depend.bufferevent_openssl.o ; 
then \
-   echo "Removing stale libevent dependencies"; \
-   rm -f ${OBJTOP}/usr.sbin/ntp/libntpevent/.depend.*; \
-   fi
-
-# 20181209  r341759 track migration across wpa update
-   @if [ -e "${OBJTOP}/usr.sbin/wpa/wpa_supplicant/.depend.rrm.o" ] && \
-   egrep -q 'src/ap/rrm.c' \
-   ${OBJTOP}/usr.sbin/wpa/wpa_supplicant/.depend.rrm.o; then \
-   echo "Removing stale wpa dependencies"; \
-   rm -f ${OBJTOP}/usr.sbin/wpa/*/.depend*; \
-   fi
 
 _worldtmp: .PHONY
@echo
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359285 - head/sys/dev/ichwd

2020-03-24 Thread Justin Hibbits
Author: jhibbits
Date: Tue Mar 24 21:28:48 2020
New Revision: 359285
URL: https://svnweb.freebsd.org/changeset/base/359285

Log:
  ichwd: Add Atom C3000 watchdog ID.
  
  MFC after:3 days
  Sponsored by: Juniper Networks, Inc

Modified:
  head/sys/dev/ichwd/ichwd.c
  head/sys/dev/ichwd/ichwd.h

Modified: head/sys/dev/ichwd/ichwd.c
==
--- head/sys/dev/ichwd/ichwd.c  Tue Mar 24 19:33:21 2020(r359284)
+++ head/sys/dev/ichwd/ichwd.c  Tue Mar 24 21:28:48 2020(r359285)
@@ -297,6 +297,7 @@ static struct ichwd_device ichwd_devices[] = {
 static struct ichwd_device ichwd_smb_devices[] = {
{ DEVICEID_LEWISBURG_SMB, "Lewisburg watchdog timer",   10, 4 },
{ DEVICEID_SRPTLP_SMB,"Sunrise Point-LP watchdog timer",10, 4 },
+   { DEVICEID_C3000, "Intel Atom C3000 watchdog timer",10, 4 },
{ 0, NULL, 0, 0 },
 };
 

Modified: head/sys/dev/ichwd/ichwd.h
==
--- head/sys/dev/ichwd/ichwd.h  Tue Mar 24 19:33:21 2020(r359284)
+++ head/sys/dev/ichwd/ichwd.h  Tue Mar 24 21:28:48 2020(r359285)
@@ -67,6 +67,7 @@ struct ichwd_softc {
 
 #defineVENDORID_INTEL  0x8086
 #defineDEVICEID_BAYTRAIL   0x0f1c
+#defineDEVICEID_C3000  0x19df
 #defineDEVICEID_CPT0   0x1c40
 #defineDEVICEID_CPT1   0x1c41
 #defineDEVICEID_CPT2   0x1c42
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359284 - in head/lib/libc: gen sys

2020-03-24 Thread Warner Losh
Author: imp
Date: Tue Mar 24 19:33:21 2020
New Revision: 359284
URL: https://svnweb.freebsd.org/changeset/base/359284

Log:
  exec{l,v}{e,p} arrived in 7th Edition research Unix to support the Bourne 
Shell
  which introduced environment variables. Document that here. Verified by
  consulting the TUHS archive.

Modified:
  head/lib/libc/gen/exec.3
  head/lib/libc/sys/execve.2

Modified: head/lib/libc/gen/exec.3
==
--- head/lib/libc/gen/exec.3Tue Mar 24 19:20:10 2020(r359283)
+++ head/lib/libc/gen/exec.3Tue Mar 24 19:33:21 2020(r359284)
@@ -322,6 +322,14 @@ and
 functions appeared in
 .At v2 .
 The
+.Fn execlp ,
+.Fn execle ,
+.Fn execve ,
+and
+.Fn execvp
+functions appeared in
+.At v7 .
+The
 .Fn execvP
 function first appeared in
 .Fx 5.2 .

Modified: head/lib/libc/sys/execve.2
==
--- head/lib/libc/sys/execve.2  Tue Mar 24 19:20:10 2020(r359283)
+++ head/lib/libc/sys/execve.2  Tue Mar 24 19:33:21 2020(r359284)
@@ -346,7 +346,7 @@ system call conforms to The Open Group Extended API Se
 The
 .Fn execve
 system call appeared in
-.Bx 4.2 .
+.At V7 .
 The
 .Fn fexecve
 system call appeared in
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359276 - head/usr.sbin/newsyslog

2020-03-24 Thread Mark Johnston
Author: markj
Date: Tue Mar 24 18:16:36 2020
New Revision: 359276
URL: https://svnweb.freebsd.org/changeset/base/359276

Log:
  newsyslog: Fix stack corruption when initializing a zipwork structure.
  
  This happens when compressing a previously uncompressed already-rotated
  file, as happens when handling the 'p' flag in newsyslog.conf.  The file
  name is stored in a flexible array member, so these structures cannot be
  stack allocated.
  
  Also make sure that we call change_attrs() and do_zipwork() in dry-run
  mode; they handle this properly, contrary to the commit log message for
  r327451.
  
  CID:  1008168
  Github PR:https://github.com/freebsd/freebsd/pull/427
  MFC after:2 weeks
  Submitted by: Radek Brich (original version)

Modified:
  head/usr.sbin/newsyslog/newsyslog.c

Modified: head/usr.sbin/newsyslog/newsyslog.c
==
--- head/usr.sbin/newsyslog/newsyslog.c Tue Mar 24 18:16:02 2020
(r359275)
+++ head/usr.sbin/newsyslog/newsyslog.c Tue Mar 24 18:16:36 2020
(r359276)
@@ -1829,17 +1829,23 @@ do_rotate(const struct conf_entry *ent)
else {
/* XXX - Ought to be checking for failure! */
(void)rename(zfile1, zfile2);
-   change_attrs(zfile2, ent);
-   if (ent->compress && !strlen(logfile_suffix)) {
-   /* compress old rotation */
-   struct zipwork_entry zwork;
+   }
+   change_attrs(zfile2, ent);
+   if (ent->compress && strlen(logfile_suffix) == 0) {
+   /* compress old rotation */
+   struct zipwork_entry *zwork;
+   size_t sz;
 
-   memset(, 0, sizeof(zwork));
-   zwork.zw_conf = ent;
-   zwork.zw_fsize = sizefile(zfile2);
-   strcpy(zwork.zw_fname, zfile2);
-   do_zipwork();
-   }
+   sz = sizeof(*zwork) + strlen(zfile2) + 1;
+   zwork = calloc(1, sz);
+   if (zwork == NULL)
+   err(1, "calloc");
+
+   zwork->zw_conf = ent;
+   zwork->zw_fsize = sizefile(zfile2);
+   strcpy(zwork->zw_fname, zfile2);
+   do_zipwork(zwork);
+   free(zwork);
}
}
 
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359283 - head/sys/net

2020-03-24 Thread Conrad Meyer
Author: cem
Date: Tue Mar 24 19:20:10 2020
New Revision: 359283
URL: https://svnweb.freebsd.org/changeset/base/359283

Log:
  Fix PNP matching for iflib NIC drivers
  
  The previous descriptor string specified that all fields were significant for
  match.  However, the only significant fields for in-tree drivers are
  vendor:devid, and the fictitious zero values constructed by PVID() did not
  match real subvendor, subdevice, revision, and/or class values, resulting in 
no
  automatic probe.
  
  If a future iflib driver needs to match on other criteria, the descriptor
  string can be updated accordingly.  (E.g., "V32" and ~0 for unspecified values
  in PVID().)
  
  Reported by:  mav
  Sponsored by: Dell EMC Isilon

Modified:
  head/sys/net/iflib.h

Modified: head/sys/net/iflib.h
==
--- head/sys/net/iflib.hTue Mar 24 19:12:28 2020(r359282)
+++ head/sys/net/iflib.hTue Mar 24 19:20:10 2020(r359283)
@@ -173,8 +173,9 @@ typedef struct pci_vendor_info {
 #define PVID_OEM(vendor, devid, svid, sdevid, revid, name) {vendor, devid, 
svid, sdevid, revid, 0, name}
 #define PVID_END {0, 0, 0, 0, 0, 0, NULL}
 
-#define IFLIB_PNP_DESCR "U32:vendor;U32:device;U32:subvendor;U32:subdevice;" \
-"U32:revision;U32:class;D:#"
+/* No drivers in tree currently match on anything except vendor:device. */
+#define IFLIB_PNP_DESCR "U32:vendor;U32:device;U32:#;U32:#;" \
+"U32:#;U32:#;D:#"
 #define IFLIB_PNP_INFO(b, u, t) \
 MODULE_PNP_INFO(IFLIB_PNP_DESCR, b, u, t, nitems(t) - 1)
 
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359282 - head/usr.sbin/ypldap

2020-03-24 Thread Sergio Carlavilla Delgado
Author: carlavilla (doc committer)
Date: Tue Mar 24 19:12:28 2020
New Revision: 359282
URL: https://svnweb.freebsd.org/changeset/base/359282

Log:
  Correct path in EXAMPLES ypldap.conf.5
  
  PR:   244743
  Submitted by: a...@i.org.ua
  Patch by: a...@i.org.ua
  Approved by:  bcr@(mentor), 0mp
  MFC after:1 day
  Differential Revision:https://reviews.freebsd.org/D24144

Modified:
  head/usr.sbin/ypldap/ypldap.conf.5

Modified: head/usr.sbin/ypldap/ypldap.conf.5
==
--- head/usr.sbin/ypldap/ypldap.conf.5  Tue Mar 24 19:09:37 2020
(r359281)
+++ head/usr.sbin/ypldap/ypldap.conf.5  Tue Mar 24 19:12:28 2020
(r359282)
@@ -15,7 +15,7 @@
 .\" ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
 .\" OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 .\"
-.Dd $Mdocdate: January 13 2016 $
+.Dd $Mdocdate: March 22 2020 $
 .Dt YPLDAP.CONF 5
 .Os
 .Sh NAME
@@ -155,7 +155,7 @@ Use the supplied LDAP filter to retrieve password entr
 .It Pa /etc/ypldap.conf
 .Xr ypldap 8
 configuration file.
-.It Pa /usr/share/example/ypldap/ypldap.conf
+.It Pa /usr/share/examples/ypldap/ypldap.conf
 .Xr ypldap 8
 configuration file example.
 .El
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359281 - stable/12/lib/libc/gen

2020-03-24 Thread Sergio Carlavilla Delgado
Author: carlavilla (doc committer)
Date: Tue Mar 24 19:09:37 2020
New Revision: 359281
URL: https://svnweb.freebsd.org/changeset/base/359281

Log:
  MFC r359210:
  Add HISTORY section to exec(3)
  
  Submitted by: gbergl...@gmail.com
  Approved by:  bcr@(mentor)

Modified:
  stable/12/lib/libc/gen/exec.3
Directory Properties:
  stable/12/   (props changed)

Modified: stable/12/lib/libc/gen/exec.3
==
--- stable/12/lib/libc/gen/exec.3   Tue Mar 24 18:43:23 2020
(r359280)
+++ stable/12/lib/libc/gen/exec.3   Tue Mar 24 19:09:37 2020
(r359281)
@@ -28,7 +28,7 @@
 .\" @(#)exec.3 8.3 (Berkeley) 1/24/94
 .\" $FreeBSD$
 .\"
-.Dd July 28, 2018
+.Dd March 22, 2020
 .Dt EXEC 3
 .Os
 .Sh NAME
@@ -310,6 +310,11 @@ and
 functions
 conform to
 .St -p1003.1-88 .
+.Sh HISTORY
+The
+.Fn exec
+function appeared in
+.At v1 .
 The
 .Fn execvP
 function first appeared in
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359280 - in head/sys: arm64/arm64 riscv/riscv

2020-03-24 Thread Mark Johnston
Author: markj
Date: Tue Mar 24 18:43:23 2020
New Revision: 359280
URL: https://svnweb.freebsd.org/changeset/base/359280

Log:
  Remove the secondary_stacks array in arm64 and riscv kernels.
  
  Instead, dynamically allocate a page for the boot stack of each AP when
  starting them up, like we do on x86.  This shrinks the bss by
  MAXCPU*KSTACK_PAGES pages, which corresponds to 4MB on arm64 and 256KB
  on riscv.
  
  Duplicate the logic used on x86 to free the bootstacks, by using a
  sysinit to wait for each AP to switch to a thread before freeing its
  stack.
  
  While here, mark some static MD variables as such.
  
  Reviewed by:  kib
  MFC after:1 month
  Sponsored by: Juniper Networks, Klara Inc.
  Differential Revision:https://reviews.freebsd.org/D24158

Modified:
  head/sys/arm64/arm64/locore.S
  head/sys/arm64/arm64/mp_machdep.c
  head/sys/riscv/riscv/locore.S
  head/sys/riscv/riscv/mp_machdep.c

Modified: head/sys/arm64/arm64/locore.S
==
--- head/sys/arm64/arm64/locore.S   Tue Mar 24 18:35:33 2020
(r359279)
+++ head/sys/arm64/arm64/locore.S   Tue Mar 24 18:43:23 2020
(r359280)
@@ -214,11 +214,10 @@ ENTRY(mpentry)
br  x15
 
 mp_virtdone:
-   ldr x4, =secondary_stacks
-   mov x5, #(PAGE_SIZE * KSTACK_PAGES)
-   mul x5, x0, x5
-   add sp, x4, x5
-
+   /* Start using the AP boot stack */
+   ldr x4, =bootstack
+   ldr x4, [x4]
+   mov sp, x4
b   init_secondary
 END(mpentry)
 #endif

Modified: head/sys/arm64/arm64/mp_machdep.c
==
--- head/sys/arm64/arm64/mp_machdep.c   Tue Mar 24 18:35:33 2020
(r359279)
+++ head/sys/arm64/arm64/mp_machdep.c   Tue Mar 24 18:43:23 2020
(r359280)
@@ -123,7 +123,6 @@ static void ipi_preempt(void *);
 static void ipi_rendezvous(void *);
 static void ipi_stop(void *);
 
-struct mtx ap_boot_mtx;
 struct pcb stoppcbs[MAXCPU];
 
 /*
@@ -136,10 +135,18 @@ static int cpu0 = -1;
 void mpentry(unsigned long cpuid);
 void init_secondary(uint64_t);
 
-uint8_t secondary_stacks[MAXCPU - 1][PAGE_SIZE * KSTACK_PAGES] __aligned(16);
+/* Synchronize AP startup. */
+static struct mtx ap_boot_mtx;
 
+/* Stacks for AP initialization, discarded once idle threads are started. */
+void *bootstack;
+static void *bootstacks[MAXCPU];
+
+/* Count of started APs, used to synchronize access to bootstack. */
+static volatile int aps_started;
+
 /* Set to 1 once we're ready to let the APs out of the pen. */
-volatile int aps_ready = 0;
+static volatile int aps_ready;
 
 /* Temporary variables for init_secondary()  */
 void *dpcpu[MAXCPU - 1];
@@ -205,14 +212,14 @@ init_secondary(uint64_t cpu)
"mov x18, %0 \n"
"msr tpidr_el1, %0" :: "r"(pcpup));
 
-   /* Spin until the BSP releases the APs */
-   while (!aps_ready)
+   /* Signal the BSP and spin until it has released all APs. */
+   atomic_add_int(_started, 1);
+   while (!atomic_load_int(_ready))
__asm __volatile("wfe");
 
/* Initialize curthread */
KASSERT(PCPU_GET(idlethread) != NULL, ("no idle thread"));
pcpup->pc_curthread = pcpup->pc_idlethread;
-   pcpup->pc_curpcb = pcpup->pc_idlethread->td_pcb;
 
/* Initialize curpmap to match TTBR0's current setting. */
pmap0 = vmspace_pmap();
@@ -250,6 +257,11 @@ init_secondary(uint64_t cpu)
 
kcsan_cpu_init(cpu);
 
+   /*
+* Assert that smp_after_idle_runnable condition is reasonable.
+*/
+   MPASS(PCPU_GET(curpcb) == NULL);
+
/* Enter the scheduler */
sched_throw(NULL);
 
@@ -257,6 +269,24 @@ init_secondary(uint64_t cpu)
/* NOTREACHED */
 }
 
+static void
+smp_after_idle_runnable(void *arg __unused)
+{
+   struct pcpu *pc;
+   int cpu;
+
+   for (cpu = 1; cpu < mp_ncpus; cpu++) {
+   if (bootstacks[cpu] != NULL) {
+   pc = pcpu_find(cpu);
+   while (atomic_load_ptr(>pc_curpcb) == NULL)
+   cpu_spinwait();
+   kmem_free((vm_offset_t)bootstacks[cpu], PAGE_SIZE);
+   }
+   }
+}
+SYSINIT(smp_after_idle_runnable, SI_SUB_SMP, SI_ORDER_ANY,
+smp_after_idle_runnable, NULL);
+
 /*
  *  Send IPI thru interrupt controller.
  */
@@ -391,7 +421,7 @@ start_cpu(u_int id, uint64_t target_cpu)
struct pcpu *pcpup;
vm_paddr_t pa;
u_int cpuid;
-   int err;
+   int err, naps;
 
/* Check we are able to start this cpu */
if (id > mp_maxid)
@@ -405,7 +435,7 @@ start_cpu(u_int id, uint64_t target_cpu)
 
/*
 * Rotate the CPU IDs to put the boot CPU as CPU 0. We keep the other
-* CPUs ordered as the are likely grouped into clusters so it can be
+* CPUs ordered as they are likely 

svn commit: r359279 - head/sys/dev/ismt

2020-03-24 Thread Justin Hibbits
Author: jhibbits
Date: Tue Mar 24 18:35:33 2020
New Revision: 359279
URL: https://svnweb.freebsd.org/changeset/base/359279

Log:
  ismt: Fix ISMT_DESC_ADDR_RW macro, slave addresses are already left-shifted
  
  Reverts r293369.  The macro was orginally correct, since our SMBus
  framework, unlike i2c, already requires addresses to be 8-bit, LSB-cleared.
  
  MFC after:3 days
  Sponsored by: Juniper Networks, Inc

Modified:
  head/sys/dev/ismt/ismt.c

Modified: head/sys/dev/ismt/ismt.c
==
--- head/sys/dev/ismt/ismt.cTue Mar 24 18:17:10 2020(r359278)
+++ head/sys/dev/ismt/ismt.cTue Mar 24 18:35:33 2020(r359279)
@@ -72,7 +72,7 @@ __FBSDID("$FreeBSD$");
 #define ISMT_DESC_LPR  0x80/* Large Packet Received */
 
 /* Macros */
-#define ISMT_DESC_ADDR_RW(addr, is_read) ((addr << 1) | (is_read))
+#define ISMT_DESC_ADDR_RW(addr, is_read) ((addr) | (is_read))
 
 /* iSMT General Register address offsets (SMBBAR + ) */
 #define ISMT_GR_GCTRL  0x000   /* General Control */
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359278 - head/usr.sbin/newsyslog

2020-03-24 Thread Mark Johnston
Author: markj
Date: Tue Mar 24 18:17:10 2020
New Revision: 359278
URL: https://svnweb.freebsd.org/changeset/base/359278

Log:
  newsyslog: Add fallthrough comments to appease Coverity.
  
  CID:  1008165, 1008166, 1008167
  MFC after:1 week
  Sponsored by: The FreeBSD Foundation

Modified:
  head/usr.sbin/newsyslog/ptimes.c

Modified: head/usr.sbin/newsyslog/ptimes.c
==
--- head/usr.sbin/newsyslog/ptimes.cTue Mar 24 18:16:56 2020
(r359277)
+++ head/usr.sbin/newsyslog/ptimes.cTue Mar 24 18:17:10 2020
(r359278)
@@ -147,18 +147,22 @@ parse8601(struct ptime_data *ptime, const char *s)
case 8:
tm.tm_year = ((l / 100) - 19) * 100;
l = l % 100;
+   /* FALLTHROUGH */
case 6:
ptime->tmspec |= TSPEC_YEAR;
tm.tm_year -= tm.tm_year % 100;
tm.tm_year += l / 1;
l = l % 1;
+   /* FALLTHROUGH */
case 4:
ptime->tmspec |= TSPEC_MONTHOFYEAR;
tm.tm_mon = (l / 100) - 1;
l = l % 100;
+   /* FALLTHROUGH */
case 2:
ptime->tmspec |= TSPEC_DAYOFMONTH;
tm.tm_mday = l;
+   /* FALLTHROUGH */
case 0:
break;
default:
@@ -180,12 +184,15 @@ parse8601(struct ptime_data *ptime, const char *s)
case 6:
tm.tm_sec = l % 100;
l /= 100;
+   /* FALLTHROUGH */
case 4:
tm.tm_min = l % 100;
l /= 100;
+   /* FALLTHROUGH */
case 2:
ptime->tmspec |= TSPEC_HOUROFDAY;
tm.tm_hour = l;
+   /* FALLTHROUGH */
case 0:
break;
default:
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359277 - head/usr.sbin/newsyslog/tests

2020-03-24 Thread Mark Johnston
Author: markj
Date: Tue Mar 24 18:16:56 2020
New Revision: 359277
URL: https://svnweb.freebsd.org/changeset/base/359277

Log:
  Add regression tests for newsyslog.conf's p flag.
  
  While here do a bit of cleanup:
  - declare local variables as such,
  - make tmpdir_create() clean up logfile directories, to handle a
previously interrupt test run more gracefully.
  
  MFC after:2 weeks
  Sponsored by: The FreeBSD Foundation

Modified:
  head/usr.sbin/newsyslog/tests/legacy_test.sh

Modified: head/usr.sbin/newsyslog/tests/legacy_test.sh
==
--- head/usr.sbin/newsyslog/tests/legacy_test.shTue Mar 24 18:16:36 
2020(r359276)
+++ head/usr.sbin/newsyslog/tests/legacy_test.shTue Mar 24 18:16:56 
2020(r359277)
@@ -194,7 +194,8 @@ ckstr()
 
 tmpdir_create()
 {
-   mkdir -p ${TMPDIR}/log ${TMPDIR}/alog
+   rm -rf ${TMPDIR}/log ${TMPDIR}/alog
+   mkdir ${TMPDIR}/log ${TMPDIR}/alog
cd ${TMPDIR}/log
 }
 
@@ -211,6 +212,8 @@ run_newsyslog()
 }
 
 tests_normal_rotate() {
+   local dir ext name_postfix newsyslog_args
+
ext="$1"
dir="$2"
 
@@ -286,6 +289,8 @@ tests_normal_rotate() {
 }
 
 tests_normal_rotate_keepn() {
+   local cnt dir ext name_postfix newsyslog_args
+
cnt="$1"
ext="$2"
dir="$3"
@@ -354,6 +359,8 @@ tests_normal_rotate_keepn() {
 }
 
 tests_time_rotate() {
+   local dir ext name_postfix newsyslog_args
+
ext="$1"
dir="$2"
 
@@ -413,6 +420,8 @@ tests_time_rotate() {
 }
 
 tests_rfc5424() {
+   local dir ext name_postfix newsyslog_args
+
ext="$1"
dir="$2"
 
@@ -451,7 +460,73 @@ tests_rfc5424() {
tmpdir_clean
 }
 
-echo 1..180
+tests_p_flag_rotate() {
+   local ext
+
+   ext="$1"
+
+   tmpdir_create
+
+   begin "create file"
+   run_newsyslog -C
+   ckfe $LOGFNAME
+   cknt ${LOGFNAME}.0
+   cknt ${LOGFNAME}.0${ext}
+   end
+
+   begin "rotate p flag 1 ${ext}"
+   run_newsyslog
+   ckfe $LOGFNAME
+   ckfe ${LOGFNAME}.0
+   cknt ${LOGFNAME}.0${ext}
+   run_newsyslog
+   ckfe $LOGFNAME
+   ckfe ${LOGFNAME}.0
+   cknt ${LOGFNAME}.0${ext}
+   ckfe ${LOGFNAME}.1${ext}
+   run_newsyslog
+   ckfe $LOGFNAME
+   ckfe ${LOGFNAME}.0
+   cknt ${LOGFNAME}.0${ext}
+   ckfe ${LOGFNAME}.1${ext}
+   ckfe ${LOGFNAME}.2${ext}
+   end
+
+   tmpdir_clean
+}
+
+tests_normal_rotate_recompress() {
+   local ext
+
+   ext=".gz"
+
+   tmpdir_create
+
+   begin "create file recompress"
+   run_newsyslog -C
+   ckfe $LOGFNAME
+   cknt ${LOGFNAME}.0${ext}
+   end
+
+   begin "rotate normal 1"
+   run_newsyslog
+   ckfe $LOGFNAME
+   ckfe ${LOGFNAME}.0${ext}
+   cknt ${LOGFNAME}.1${ext}
+   end
+
+   begin "rotate recompress 1"
+   gunzip ${LOGFNAME}.0${ext}
+   ckfe ${LOGFNAME}.0
+   cknt ${LOGFNAME}.0${ext}
+   run_newsyslog
+   ckfe $LOGFNAME
+   ckfe ${LOGFNAME}.0${ext}
+   ckfe ${LOGFNAME}.1${ext}
+   end
+}
+
+echo 1..185
 mkdir -p ${TMPDIR}
 cd ${TMPDIR}
 
@@ -556,5 +631,11 @@ tests_time_rotate "zst" "${TMPDIR}/alog/"
 echo "$LOGFPATH5424640  3 *@T00  NCT" > newsyslog.conf
 echo "$LOGFPATH640  3 *@T00  NC" >> newsyslog.conf
 tests_rfc5424
+
+echo "$LOGFPATH 640  3 *@T00  NCpZ" > newsyslog.conf
+tests_p_flag_rotate ".gz"
+
+echo "$LOGFPATH 640  3 *@T00  NCZ" > newsyslog.conf
+tests_normal_rotate_recompress
 
 rm -rf "${TMPDIR}"
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359275 - head

2020-03-24 Thread John Baldwin
Author: jhb
Date: Tue Mar 24 18:16:02 2020
New Revision: 359275
URL: https://svnweb.freebsd.org/changeset/base/359275

Log:
  Disable rarely used architecture variants in make universe by default.
  
  If EXTRA_TARGETS is defined, build all supported architecture
  variants.  By default, build architecture variants needed to provide
  code coverage or that are commonly used.
  
  Use this to disable building of all the hard-float and little-endian
  MIPS architecture variants along with n32 by default.
  
  Reviewed by:  rpokala
  Discussed with:   imp, emaste
  Differential Revision:https://reviews.freebsd.org/D24178

Modified:
  head/Makefile

Modified: head/Makefile
==
--- head/Makefile   Tue Mar 24 17:54:34 2020(r359274)
+++ head/Makefile   Tue Mar 24 18:16:02 2020(r359275)
@@ -483,16 +483,17 @@ worlds: .PHONY
 .if make(universe) || make(universe_kernels) || make(tinderbox) || \
 make(targets) || make(universe-toolchain)
 #
-# Always build architectures supported by clang.  Only build architectures
-# only supported by GCC if a suitable toolchain is present or enabled.
-# In all cases, if the user specifies TARGETS on the command line,
-# honor that most of all.
+# Don't build rarely used architectures unless requested.
 #
+.if defined(EXTRA_TARGETS)
+EXTRA_ARCHES_mips= mipsel mipshf mipselhf mips64el mips64hf mips64elhf
+EXTRA_ARCHES_mips+=mipsn32
+.endif
 TARGETS?=amd64 arm arm64 i386 mips powerpc riscv
 _UNIVERSE_TARGETS= ${TARGETS}
 TARGET_ARCHES_arm?=armv6 armv7
 TARGET_ARCHES_arm64?=  aarch64
-TARGET_ARCHES_mips?=   mipsel mips mips64el mips64 mipsn32 mipselhf mipshf 
mips64elhf mips64hf
+TARGET_ARCHES_mips?=   mips mips64 ${EXTRA_ARCHES_mips}
 # powerpcspe excluded until clang fixed
 TARGET_ARCHES_powerpc?=powerpc powerpc64
 TARGET_ARCHES_riscv?=  riscv64 riscv64sf
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359274 - head/sys/net

2020-03-24 Thread Ed Maste
Author: emaste
Date: Tue Mar 24 17:54:34 2020
New Revision: 359274
URL: https://svnweb.freebsd.org/changeset/base/359274

Log:
  iflib: simplify MPASS assertion
  
  Submitted by: andrew

Modified:
  head/sys/net/iflib.c

Modified: head/sys/net/iflib.c
==
--- head/sys/net/iflib.cTue Mar 24 17:25:56 2020(r359273)
+++ head/sys/net/iflib.cTue Mar 24 17:54:34 2020(r359274)
@@ -6193,14 +6193,8 @@ iflib_rx_intr_deferred(if_ctx_t ctx, int rxqid)
 void
 iflib_admin_intr_deferred(if_ctx_t ctx)
 {
-#ifdef INVARIANTS
-   struct grouptask *gtask;
 
-   gtask = >ifc_admin_task;
-   MPASS(gtask != NULL);
-   MPASS(gtask->gt_taskqueue != NULL);
-#endif
-
+   MPASS(ctx->ifc_admin_task.gt_taskqueue != NULL);
GROUPTASK_ENQUEUE(>ifc_admin_task);
 }
 
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359272 - head/sys/kern

2020-03-24 Thread Konstantin Belousov
Author: kib
Date: Tue Mar 24 17:16:52 2020
New Revision: 359272
URL: https://svnweb.freebsd.org/changeset/base/359272

Log:
  kern_copy_file_range(): check the file type.
  
  The syscall can only operate on valid vnode types.
  
  Reported and tested by:   pho
  Sponsored by: The FreeBSD Foundation

Modified:
  head/sys/kern/vfs_syscalls.c

Modified: head/sys/kern/vfs_syscalls.c
==
--- head/sys/kern/vfs_syscalls.cTue Mar 24 12:27:02 2020
(r359271)
+++ head/sys/kern/vfs_syscalls.cTue Mar 24 17:16:52 2020
(r359272)
@@ -4735,9 +4735,25 @@ kern_copy_file_range(struct thread *td, int infd, off_
error = fget_read(td, infd, _read_rights, );
if (error != 0)
goto out;
+   if (infp->f_ops == ) {
+   error = EBADF;
+   goto out;
+   }
+   if (infp->f_vnode == NULL) {
+   error = EINVAL;
+   goto out;
+   }
error = fget_write(td, outfd, _write_rights, );
if (error != 0)
goto out;
+   if (outfp->f_ops == ) {
+   error = EBADF;
+   goto out;
+   }
+   if (outfp->f_vnode == NULL) {
+   error = EINVAL;
+   goto out;
+   }
 
/* Set the offset pointers to the correct place. */
if (inoffp == NULL)
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359273 - head/sys/net

2020-03-24 Thread Ed Maste
Author: emaste
Date: Tue Mar 24 17:25:56 2020
New Revision: 359273
URL: https://svnweb.freebsd.org/changeset/base/359273

Log:
  iflib: split compound assertion
  
  ThunderX cluster systems are panicking on boot with a failed assertion
  MPASS(gtask != NULL && gtask->gt_taskqueue != NULL).  Split the
  assertion so that it's clear which part is failing.

Modified:
  head/sys/net/iflib.c

Modified: head/sys/net/iflib.c
==
--- head/sys/net/iflib.cTue Mar 24 17:16:52 2020(r359272)
+++ head/sys/net/iflib.cTue Mar 24 17:25:56 2020(r359273)
@@ -6197,7 +6197,8 @@ iflib_admin_intr_deferred(if_ctx_t ctx)
struct grouptask *gtask;
 
gtask = >ifc_admin_task;
-   MPASS(gtask != NULL && gtask->gt_taskqueue != NULL);
+   MPASS(gtask != NULL);
+   MPASS(gtask->gt_taskqueue != NULL);
 #endif
 
GROUPTASK_ENQUEUE(>ifc_admin_task);
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359271 - head/sbin/ipfw

2020-03-24 Thread Andrey V. Elsukov
Author: ae
Date: Tue Mar 24 12:27:02 2020
New Revision: 359271
URL: https://svnweb.freebsd.org/changeset/base/359271

Log:
  Use IP_FW_NAT44_DESTROY opcode for IP_FW3 socket option to destroy
  NAT instance.
  
  The NAT44 group of opcodes for IP_FW3 socket option is modern way
  to control NAT instances and this method can be used in future to
  switch from numeric to named NAT instances, like was done for ipfw
  tables.
  The IP_FW_NAT_DEL opcode is the last remnant of old ipfw_ctl control
  plane that doesn't support versioned operations. This interface will
  be retired soon.
  
  Reviewed by:  melifaro
  MFC after:10 days
  Sponsored by: Yandex LLC

Modified:
  head/sbin/ipfw/ipfw2.c
  head/sbin/ipfw/ipfw2.h
  head/sbin/ipfw/nat.c

Modified: head/sbin/ipfw/ipfw2.c
==
--- head/sbin/ipfw/ipfw2.c  Tue Mar 24 07:08:39 2020(r359270)
+++ head/sbin/ipfw/ipfw2.c  Tue Mar 24 12:27:02 2020(r359271)
@@ -3328,13 +3328,7 @@ ipfw_delete(char *av[])
j = strtol(sep + 1, NULL, 10);
av++;
if (co.do_nat) {
-   exitval = do_cmd(IP_FW_NAT_DEL, , sizeof i);
-   if (exitval) {
-   exitval = EX_UNAVAILABLE;
-   if (co.do_quiet)
-   continue;
-   warn("nat %u not available", i);
-   }
+   exitval = ipfw_delete_nat(i);
} else if (co.do_pipe) {
exitval = ipfw_delete_pipe(co.do_pipe, i);
} else {

Modified: head/sbin/ipfw/ipfw2.h
==
--- head/sbin/ipfw/ipfw2.h  Tue Mar 24 07:08:39 2020(r359270)
+++ head/sbin/ipfw/ipfw2.h  Tue Mar 24 12:27:02 2020(r359271)
@@ -387,6 +387,7 @@ extern int resvd_set_number;
 /* first-level command handlers */
 void ipfw_add(char *av[]);
 void ipfw_show_nat(int ac, char **av);
+int ipfw_delete_nat(int i);
 void ipfw_config_pipe(int ac, char **av);
 void ipfw_config_nat(int ac, char **av);
 void ipfw_sets_handler(char *av[]);

Modified: head/sbin/ipfw/nat.c
==
--- head/sbin/ipfw/nat.cTue Mar 24 07:08:39 2020(r359270)
+++ head/sbin/ipfw/nat.cTue Mar 24 12:27:02 2020(r359271)
@@ -939,6 +939,34 @@ ipfw_config_nat(int ac, char **av)
}
 }
 
+static void
+nat_fill_ntlv(ipfw_obj_ntlv *ntlv, int i)
+{
+
+   ntlv->head.type = IPFW_TLV_EACTION_NAME(1); /* it doesn't matter */
+   ntlv->head.length = sizeof(ipfw_obj_ntlv);
+   ntlv->idx = 1;
+   ntlv->set = 0; /* not yet */
+   snprintf(ntlv->name, sizeof(ntlv->name), "%d", i);
+}
+
+int
+ipfw_delete_nat(int i)
+{
+   ipfw_obj_header oh;
+   int ret;
+
+   memset(, 0, sizeof(oh));
+   nat_fill_ntlv(, i);
+   ret = do_set3(IP_FW_NAT44_DESTROY, , sizeof(oh));
+   if (ret == -1) {
+   if (!co.do_quiet)
+   warn("nat %u not available", i);
+   return (EX_UNAVAILABLE);
+   }
+   return (EX_OK);
+}
+
 struct nat_list_arg {
uint16_tcmd;
int is_all;
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359270 - head/usr.sbin/auditdistd

2020-03-24 Thread Emmanuel Vadot
Author: manu
Date: Tue Mar 24 07:08:39 2020
New Revision: 359270
URL: https://svnweb.freebsd.org/changeset/base/359270

Log:
  auditdistd: Remove useless linking with libl

Modified:
  head/usr.sbin/auditdistd/Makefile

Modified: head/usr.sbin/auditdistd/Makefile
==
--- head/usr.sbin/auditdistd/Makefile   Tue Mar 24 07:08:02 2020
(r359269)
+++ head/usr.sbin/auditdistd/Makefile   Tue Mar 24 07:08:39 2020
(r359270)
@@ -21,7 +21,7 @@ SRCS+=sandbox.c sender.c subr.c
 SRCS+= token.l trail.c
 MAN=   auditdistd.8 auditdistd.conf.5
 
-LIBADD+=   l pthread util crypto ssl
+LIBADD+=   pthread util crypto ssl
 
 YFLAGS+=-v
 
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r359269 - head/rescue/rescue

2020-03-24 Thread Emmanuel Vadot
Author: manu
Date: Tue Mar 24 07:08:02 2020
New Revision: 359269
URL: https://svnweb.freebsd.org/changeset/base/359269

Log:
  rescue: Remove useless linking with libl

Modified:
  head/rescue/rescue/Makefile

Modified: head/rescue/rescue/Makefile
==
--- head/rescue/rescue/Makefile Tue Mar 24 01:29:18 2020(r359268)
+++ head/rescue/rescue/Makefile Tue Mar 24 07:08:02 2020(r359269)
@@ -55,7 +55,7 @@ CRUNCH_PROGS_bin= cat chflags chio chmod cp date dd df
 ed expr getfacl hostname kenv kill ln ls mkdir mv  \
 pkill ps pwd realpath rm rmdir setfacl sh sleep stty   \
 sync test
-CRUNCH_LIBS+= -lcrypt -ledit -ljail -lkvm -lelf -ll -ltermcapw -lutil -lxo
+CRUNCH_LIBS+= -lcrypt -ledit -ljail -lkvm -lelf -ltermcapw -lutil -lxo
 CRUNCH_BUILDTOOLS+= bin/sh
 
 # Additional options for specific programs
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r359267 - in head: . share/mk

2020-03-24 Thread Baptiste Daroussin
24 mars 2020 02:24:21 Bryan Drewery :

> On 3/23/2020 6:09 PM, Emmanuel Vadot wrote:
>
> > Author: manu
> > Date: Tue Mar 24 01:09:04 2020
> > New Revision: 359267
> > URL: https://svnweb.freebsd.org/changeset/base/359267
> >
> > Log:
> > bsd.lib.mk: Do not include bsd.incs.mk for INTERNALLIB
> >
> > If we're building an internal lib do not bother including bsd.incs.mk so we
> > will not install the headers.
> > This also "solves" a problem with pkgbase where a libXXX-development package
> > is created and due to how packages are created we add a dependency to a
> > libXXX package that doesn't exists.
> >
> > Reported by: pizzamig
> > Reviewed by: pizzamig bapt emaste
> > Differential Revision: https://reviews.freebsd.org/D24166
> >
> > Modified:
> > head/ObsoleteFiles.inc
> > head/share/mk/bsd.lib.mk
> >
> > Modified: head/ObsoleteFiles.inc
> > ==
> > --- head/ObsoleteFiles.inc Tue Mar 24 01:08:06 2020 (r359266)
> > +++ head/ObsoleteFiles.inc Tue Mar 24 01:09:04 2020 (r359267)
> > @@ -36,6 +36,11 @@
> > # xargs -n1 | sort | uniq -d;
> > # done
> >
> > +# 20200323: INTERNALLIB don't install headers anymore
> > +OLD_FILES+=/usr/include/libelftc.h
> > +OLD_FILES+=/usr/include/libifconfig.h
> > +OLD_FILES+=/usr/include/libpmcstat.h
> > +
> >
>
> lib/libelftc/Makefile:INCS= libelftc.h
> lib/libifconfig/Makefile:INCSDIR= ${INCLUDEDIR}
> lib/libifconfig/Makefile:INCS= libifconfig.h
> lib/libpmcstat/Makefile:INCS= libpmcstat.h
>
> This commit seems incomplete or wrong due to the leftover logic.
>
>

This is on purpose so if anyone wants to promote those from internallib to 
something else, they can

They just have to remove INTERNALLIB

Best regards,
Bapt



___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"