tar/cpio: fix bugs in selecting stdout usage

2016-08-23 Thread Philip Guenther
These are kinda weird corner cases that I found while checking on other things. First, not a bug but a cleanup: when support for TAPE=- was added we ended up with duplicate code recognizing '-' as meaning stdin/stdout. By delaying the handling of that to after the "no -f option? use $TAPE,

Amend history & authors info in bin/expr/expr.1

2016-08-23 Thread Sevan Janiyan
Hello, Attached patch makes some changes to the history and authors section of expr(1) as follows: 1) State the first rewrite was a public domain version 2) The majority of the rewrite work was for NetBSD 1.0 [1] 3) It was largely rewritten by John T. Conklin in 1993 [1] Sevan [1] http://cvsweb.

sync arch with reality in sendbug(1)

2016-08-23 Thread Giovanni Bechis
Remove vax and other discontinued arch, add zaurus and more. Is this correct or we want less ? Cheers Giovanni Index: sendbug.c === RCS file: /var/cvs/src/usr.bin/sendbug/sendbug.c,v retrieving revision 1.76 diff -u -p -r1.76 sendbu

fdt_find_node() bug

2016-08-23 Thread Mark Kettenis
This function is supposed to return NULL if the node isn't found. But the current implementation will always return the node for the last path component that matched. Wchis will be the root node if nothing else matches. So this function will never return NULL if the path starts with '/'. Diff f

Re: ld.so initarray support

2016-08-23 Thread Philip Guenther
On Tue, Aug 23, 2016 at 6:59 AM, Mark Kettenis wrote: >> Date: Mon, 22 Aug 2016 21:27:51 -0700 >> From: Philip Guenther >> >> Right now, DT_NUM is defined in , but that's dumb: it >> should be moved immediately to libexec/ld.so/resolv.h, right above this >> struct, and its value increased to matc

puc(4): serial console, UART's address != PCI-BAR

2016-08-23 Thread SASANO Takayoshi
Hello, I have CH382-based PCI Express UART card and trying to use this port as serial console. But specifing I/O port by "machine comaddr 0xd0c0" at bootloader, kernel crashes with protection fault. Here is the log. pci3 at ppb2 bus 3 puc0 at pci3 dev 0 function 0 "Nanjing QinHeng Elect

Patch - Making iked keepalive timeout configurable

2016-08-23 Thread Michael Lam
This is my first time submitting patch, hopefully it can assist your work. The attached patch adds an configuration option “keepalivetime” after “lifetime” to allow IKED_IKE_SA_ALIVE_TIMEOUT became configurable. If not specified, it will be defaulted to IKED_IKE_SA_ALIVE_TIMEOUT. An option of 0 w

Re: Kill ip6_forward_rt

2016-08-23 Thread Alexander Bluhm
On Tue, Aug 23, 2016 at 01:10:02PM +0200, Martin Pieuchot wrote: > @@ -414,43 +414,23 @@ ip6_input(struct mbuf *m) > goto hbhcheck; > } > > + > /* >* Unicast check >*/ Useless new line. OK bluhm@

Re: ld.so initarray support

2016-08-23 Thread Mark Kettenis
> Date: Mon, 22 Aug 2016 21:27:51 -0700 > From: Philip Guenther > > Right now, DT_NUM is defined in , but that's dumb: it > should be moved immediately to libexec/ld.so/resolv.h, right above this > struct, and its value increased to match the growth here. I killed it's > use in lib/csu for th

Re: No struct route_in6 in icmp6

2016-08-23 Thread Alexander Bluhm
On Tue, Aug 23, 2016 at 01:29:19PM +0200, Martin Pieuchot wrote: > Similar to yesterday's NS/ND in6_selectsrc() conversion here's a diff > for icmp6. > > ok? OK bluhm@ > > Index: netinet6/icmp6.c > === > RCS file: /cvs/src/sys/neti

Re: pax: stop getting mode via strcmp(argv0)

2016-08-23 Thread Todd C. Miller
On Mon, 22 Aug 2016 21:05:11 -0700, Philip Guenther wrote: > pax, tar, and cpio are one binary that changes behavior based on the last > component of argv[0]. Lots of shared code, so sure, but I find the > strcmp() tests in the main code overly complex and perhaps an affront to > clear style.

Re: pkg_add mirror latency testing, setting program

2016-08-23 Thread lists
Tue, 23 Aug 2016 10:32:48 + Luke Small > You can only tell the fastest latency for a download by testing it at your > location. It is very fast. > > On Tue, Aug 23, 2016 at 5:25 AM wrote: > > > Tue, 23 Aug 2016 09:09:38 + Luke Small > > [...] > > > It downloads the ANNOUNCEMENT file

No struct route_in6 in icmp6

2016-08-23 Thread Martin Pieuchot
Similar to yesterday's NS/ND in6_selectsrc() conversion here's a diff for icmp6. ok? Index: netinet6/icmp6.c === RCS file: /cvs/src/sys/netinet6/icmp6.c,v retrieving revision 1.189 diff -u -p -r1.189 icmp6.c --- netinet6/icmp6.c4

Re: pax: stop getting mode via strcmp(argv0)

2016-08-23 Thread Jeremie Courreges-Anglas
Philip Guenther writes: > pax, tar, and cpio are one binary that changes behavior based on the last > component of argv[0]. Lots of shared code, so sure, but I find the > strcmp() tests in the main code overly complex and perhaps an affront to > clear style. (It's _not_ really a perf issue,

Kill ip6_forward_rt

2016-08-23 Thread Martin Pieuchot
This follows the direction of IPv4. The idea is to stop using a single cache entry and continue the cleanup to be able to cache route entries in PF state keys. Since we're no longer using a single cache entry the destination & rtableid checks become obsolete. As for IPv4 the rule I'm trying to f

Re: pkg_add mirror latency testing, setting program

2016-08-23 Thread Stuart Henderson
On 2016/08/23 09:09, Luke Small wrote: > It downloads the ANNOUNCEMENT file from each mirror, which is both small > and has the same name for every release. > > I suspect it still wouldn't be accepted in any form for the base, but is > there any reason I couldn't make a package, even though I have

Re: pkg_add mirror latency testing, setting program

2016-08-23 Thread Luke Small
You can only tell the fastest latency for a download by testing it at your location. It is very fast. On Tue, Aug 23, 2016 at 5:25 AM wrote: > Tue, 23 Aug 2016 09:09:38 + Luke Small > [...] > > It downloads the ANNOUNCEMENT file from each mirror, which is both small > > and has the same nam

Re: pkg_add mirror latency testing, setting program

2016-08-23 Thread lists
Tue, 23 Aug 2016 09:09:38 + Luke Small [...] > It downloads the ANNOUNCEMENT file from each mirror, which is both small > and has the same name for every release. [...] Hi Luke, Two comments, first everyone knows their geographic location, time zone and respectively closest mirror, and then

pkg_add mirror latency testing, setting program

2016-08-23 Thread Luke Small
I had one before that read openbsd.org/ftp.html (which is insecure because it gets sets a mirror from data from an unencrypted connection), but I changed it to read the /etc/examples/pkg.conf file so that there is a more secure method. I pledged it. I tried to do pledge and setuid, but a glitch tha

Re: multi-pool malloc wip diff

2016-08-23 Thread Otto Moerbeek
On Mon, Aug 22, 2016 at 08:59:44PM -0400, Ted Unangst wrote: > Otto Moerbeek wrote: > > > > After a forkl of a threaded program __isthreaded is reset, but > > existing allocations are spread around the pools. But the new single > > threaded child only looks in the first pool. I have to think how