unlock bge(4)

This brings bge(4) up to about where em(4) is. It involves a few different changes, notably: - per-ring refill timeouts, to ensure we don't try to refill a ring from the timeout and an interrupt at the same time - removing the list of tx dma maps and just assigning a map to use based on the

Re: more on tame

perror(3) is being used instead of err(3) in a few places; is that on purpose? If it's an oversight, I also noticed the same in patch(1). On 10/04/15 01:34, Theo de Raadt wrote: > 42 tame calls have been commited to 28 userland programs so far. > For instance gzip, md5, ping, traceroute,

Re: tame(2) nologin(8)

On 2015-10-04 01.33.12 +0200, Mike Burns wrote: > I suspect that the paths argument is unused or not yet ready, but I > include in here regardless merely so that I can ask about it. Consolidated feedback from off-list: The paths argument is not yet to be used. Also, it must be NULL-terminated.

Re: tame userland diff

On 2015-10-04 07.15.47 +0200, Sebastien Marie wrote: > On Sat, Oct 03, 2015 at 09:52:13PM +0200, Mike Burns wrote: > > On 2015-10-03 09.53.54 -0600, Theo de Raadt wrote: > > > > > > I don't know why you added "proc". I don't see a need for it. Do > > > you have a seperate test cases that

Re: ahci.c memory leak in error path

On Fri, Oct 02, 2015 at 02:37:18PM +0200, Benjamin Baier wrote: > Fix memory leak in error path. > Found by llvm/scan-build. Thanks, I've just committed this. > > Also port (sc->sc_ports[port]) was not assigned, so "goto freeport;" > seems wrong. > > Index: ahci.c >

Re: acpidump(8): fix usage message

On Sat, Oct 03, 2015 at 02:30:38AM -0400, Michael Reed wrote: > This brings the usage info in line with most other programs. > > fixed, thanks. jmc > > Index: acpidump.c > === > RCS file: /cvs/src/usr.sbin/acpidump/acpidump.c,v >

Re: tame userland diff

Sebastien Marie wrote: > - if an exec'ed program starts with herited TAME flags: the > initialisation of the program would be difficult as it would be > already tamed. i've been thinking about this some more. true in some cases, but i think in many cases, what we are banning should be

Re: iwm(4) newstate task (again)

2015-10-03 22:33 GMT+02:00 Stefan Sperling : > On Sun, Sep 27, 2015 at 08:00:19PM +0200, Stefan Sperling wrote: >> This is yet another attempt at improving the iwm(4) newstate task. > > This diff has been working nicely for me, with many suspend/resume cycles. > Never had a problem

ftpd popen

as seen in cron, make the popen replacement nicer. this also repairs two abuses of comma operators and an unnecessary function pointer. Index: extern.h === RCS file: /cvs/src/libexec/ftpd/extern.h,v retrieving revision 1.18 diff -u

Re: CVS: cvs.openbsd.org: src

On Wed, Sep 30, 2015 at 06:07:54PM +0200, Joerg Jung wrote: > On Wed, Sep 30, 2015 at 05:13:31PM +0200, Martijn van Duren wrote: > > > > What I find somewhat strange is that although the dmesg says it has "2 > > lights", it only shows one illuminance sensor in my sysctl. > > This is expected.

Re: more on tame

> perror(3) is being used instead of err(3) in a few places; is > that on purpose? If it's an oversight, I also noticed the same > in patch(1). I am using perror in older programs.

Re: CVS: cvs.openbsd.org: src

On Sun, Oct 04, 2015 at 09:20:06PM +0200, Mark Kettenis wrote: > > Here is what I get on the Macmini 1,1 now: > > asmc0 at isa0 port 0x300/32 > asmc0: rev 1.3f503, 137 keys, 5 temperatures, 1 fan, 0 lights, kbdled Thanks for testing! I think printing "0 lights" is not very useful, so I will

pkg_(add|delete|info): fix synopses

The use of an ellipsis in manuals usually seems to already imply optional arguments, e.g., `.Op Ar file ...', so constructs like `Ar pkg-name Op Ar ...' seem redundant to me. While here, also denote the `pkg-name' argument to pkg_delete as optional, since you can do things like `pkg_delete -nX'.

Re: CVS: cvs.openbsd.org: src

> Date: Sun, 4 Oct 2015 14:10:18 +0200 > From: Joerg Jung > > On Wed, Sep 30, 2015 at 06:07:54PM +0200, Joerg Jung wrote: > > On Wed, Sep 30, 2015 at 05:13:31PM +0200, Martijn van Duren wrote: > > > > > > What I find somewhat strange is that although the dmesg says it has "2 > >

Re: Happy Birthday OpenBSD!

On Sun, Oct 04, 2015 at 05:27:51PM -0600, Bob Beck wrote: > > ** OpenBSD is turning 20, on January 18th 2015 ** Ok, and I'm an idiot.. OCTOBER 18th, 2015 I.E. coming up in two weeks from today :) > > There will be an informal Birthday Party upstairs at the Hose and

Happy Birthday OpenBSD!

** OpenBSD is turning 20, on January 18th 2015 ** There will be an informal Birthday Party upstairs at the Hose and Hound pub in Calgary (http://www.thehose.ca/) At the very least, Theo de Raadt and I will be there starting from about 5 PM. Any and all are welcome to stop by

style.9 references "user land"

When I read "user land" in style.9 it seemed weird. I checked and all other man pages refer to "userland". I also checked the source tree and found well over 400 instances of userland, and only 3 other instances of "user land": share/man/man9/style.9 lib/libc/rpc/svc.c

update lex to use __progname

Lex is currently using progam_name = argv[0]. Change to __progname? Regards, Index: flexdef.h === RCS file: /cvs/src/usr.bin/lex/flexdef.h,v retrieving revision 1.7 diff -u -p -r1.7 flexdef.h --- flexdef.h 3 Feb 2004 21:20:17

Re: ftpd popen

On Sun, 04 Oct 2015 05:40:39 -0400, "Ted Unangst" wrote: > as seen in cron, make the popen replacement nicer. this also repairs two > abuses of comma operators and an unnecessary function pointer. OK millert@ - todd