Re: disklabel template: percentage of disk optional?

On Thu, Oct 06, 2016 at 08:57:44AM +0200, Otto Moerbeek wrote: > On Tue, Oct 04, 2016 at 01:26:06PM +0200, Otto Moerbeek wrote: > > > On Tue, Oct 04, 2016 at 04:00:50AM -0700, Claus Assmann wrote: > > > > > On Tue, Oct 04, 2016, Dmitrij D. Czarkoff wrote: > > > > > > [please do not Cc me] > > >

Re: regress for etherip and vxlan

On Wed, 5 Oct 2016 15:31:39 +0200 Alexander Bluhm wrote: > On Wed, Oct 05, 2016 at 04:48:55PM +0900, YASUOKA Masahiko wrote: >> The diff add regress scripts for vxlan(4) and etherip(4). >> >> This will be my first commit to regress/. ok? > > I does not run with an obj

Re: Help me testing the netlock

On 10/05/16 20:52, Lampshade wrote: panic: rw_enter: netlock locking against myself Your tree is not up-to-date, you're missing sys/net/route.c r1.332. rt_timer_timer() is now called from the softclock thread.

Re: disklabel template: percentage of disk optional?

On Tue, Oct 04, 2016 at 01:26:06PM +0200, Otto Moerbeek wrote: > On Tue, Oct 04, 2016 at 04:00:50AM -0700, Claus Assmann wrote: > > > On Tue, Oct 04, 2016, Dmitrij D. Czarkoff wrote: > > > > [please do not Cc me] > > > > > I shouldn't have started on sending patches at 3AM. This one should do

gprof(1): remove reference to prof(1)

The prof(1) reference in the See Also section was removed back in 1998 (r1.10). Index: gprof.1 === RCS file: /cvs/src/usr.bin/gprof/gprof.1,v retrieving revision 1.22 diff -u -p -r1.22 gprof.1 --- gprof.1 23 Dec 2011 17:00:47

Re: gprof(1): remove reference to prof(1)

On Thu, Oct 06, 2016 at 08:54:27AM +0200, Anton Lindqvist wrote: > The prof(1) reference in the See Also section was removed back in 1998 > (r1.10). > fixed, thanks. jmc > Index: gprof.1 > === > RCS file:

Re: add in6 multicast support to vxlan(4) ; question on mbufs

On Thu, Oct 06, 2016 at 10:11 +0900, YASUOKA Masahiko wrote: > > On Wed, 5 Oct 2016 14:30:27 +0200 > Mike Belopuhov wrote: > > On Wed, Oct 05, 2016 at 10:58 +0900, YASUOKA Masahiko wrote: > >> On Tue, 4 Oct 2016 17:27:12 +0200 > >> Mike Belopuhov wrote: >

hide iwn firmware error log

Disable the detailed fatal firmware error log in iwn(4) by default. Index: if_iwn.c === RCS file: /cvs/src/sys/dev/pci/if_iwn.c,v retrieving revision 1.172 diff -u -p -r1.172 if_iwn.c --- if_iwn.c5 Sep 2016 08:18:18 -

Re: iwm(4): Improve iwm_rx_addbuf() error handling.

On Thu, Oct 06, 2016 at 10:49:21AM +0200, Imre Vadasz wrote: > Hi, > This patch improves the error handling iwm_rx_addbuf(), specifically in > out-of-memory and mbuf exhaustion cases. > > Keep an additional/spare bus_dmamap_t object around to make error handling > for bus_dmamap_load_mbuf()

iwm(4): Improve iwm_rx_addbuf() error handling.

Hi, This patch improves the error handling iwm_rx_addbuf(), specifically in out-of-memory and mbuf exhaustion cases. Keep an additional/spare bus_dmamap_t object around to make error handling for bus_dmamap_load_mbuf() failures easier in iwm_rx_addbuf(). This seems like an easy way to avoid

Re: armv7 pinmux woes

On Thu, Oct 06, 2016 at 04:43:09AM -0400, Ian Sutton wrote: > On Wed, Oct 05, 2016 at 12:40:42AM -0400, Ian Sutton wrote: > > My driver needs to recognize cases where the root disk is MMC1 and abort > > as to avoid hanging the system when the kernel tries to init userspace > > on a disk it is no

Re: disklabel template: percentage of disk optional?

On Thu, Oct 06, 2016 at 09:15:50AM +0200, Theo Buehler wrote: > On Thu, Oct 06, 2016 at 08:57:44AM +0200, Otto Moerbeek wrote: > > On Tue, Oct 04, 2016 at 01:26:06PM +0200, Otto Moerbeek wrote: > > > > > On Tue, Oct 04, 2016 at 04:00:50AM -0700, Claus Assmann wrote: > > > > > > > On Tue, Oct

Re: Help me testing the netlock

> > panic: rw_enter: netlock locking against myself > > Your tree is not up-to-date, you're missing sys/net/route.c r1.332. My bad. With updated sources I have tested this for few minutes and I don't encountered any kernel panic. CVS sync, wget to download base system from mirror, browsing web

Re: 'make build': simplify 'make includes' step

On Thu, Oct 06, 2016 at 02:02:41PM +0200, Theo Buehler wrote: > Before calling 'make includes' during 'make build', the first two steps > are repetitions of what a direct 'make includes' does anyway, so move > the privdrop up there and simply call 'make includes' from 'make build'. > > While

Re: 'make build': simplify 'make includes' step

On Thu, Oct 06, 2016 at 03:09:49PM +0200, Martin Natano wrote: > On Thu, Oct 06, 2016 at 02:02:41PM +0200, Theo Buehler wrote: > > Before calling 'make includes' during 'make build', the first two steps > > are repetitions of what a direct 'make includes' does anyway, so move > > the privdrop up

syslogd closefrom

Hi, With closefrom(4) we can avoid some global file descriptor variables in syslogd(8). ok? bluhm Index: usr.sbin/syslogd/privsep.c === RCS file: /data/mirror/openbsd/cvs/src/usr.sbin/syslogd/privsep.c,v retrieving revision 1.62

Re: ps.1 tweak

I realised the text actually means to say "terminal width - 1", not "terminal width: -1" How about this instead? Index: bin/ps/ps.1 === RCS file: /cvs/src/bin/ps/ps.1,v retrieving revision 1.109 diff -u -p -r1.109 ps.1 ---

Re: ps.1 tweak

On Thu, Oct 06, 2016 at 08:34:31PM +0100, Jason McIntyre wrote: > On Thu, Oct 06, 2016 at 09:27:21PM +0200, Michal Mazurek wrote: > > > > Don't place a space after the minus sign. Change from this: > > width in columns. Otherwise, ps defaults to the terminal width > > - 1, or 79

Re: ps.1 tweak

On Thu, Oct 06, 2016 at 09:27:21PM +0200, Michal Mazurek wrote: > > Don't place a space after the minus sign. Change from this: > width in columns. Otherwise, ps defaults to the terminal width > - 1, or 79 columns if none of stdout, stderr and stdin are a > To this: > width in

Re: ps.1 tweak

On Thu, Oct 06, 2016 at 09:27:21PM +0200, Michal Mazurek wrote: > > Don't place a space after the minus sign. Change from this: > width in columns. Otherwise, ps defaults to the terminal width > - 1, or 79 columns if none of stdout, stderr and stdin are a > To this: > width in

Re: ps.1 tweak

Theo Buehler wrote: > On Thu, Oct 06, 2016 at 08:34:31PM +0100, Jason McIntyre wrote: > > On Thu, Oct 06, 2016 at 09:27:21PM +0200, Michal Mazurek wrote: > > > > > > Don't place a space after the minus sign. Change from this: > > > width in columns. Otherwise, ps defaults to the terminal

ps.1 tweak

Don't place a space after the minus sign. Change from this: width in columns. Otherwise, ps defaults to the terminal width - 1, or 79 columns if none of stdout, stderr and stdin are a To this: width in columns. Otherwise, ps defaults to the terminal width -1, or 79

athn: allow lower tx retry rates with RTS

athn(4) has a hack which disables lower Tx retry rates if RTS is used. I don't understand why this was added. Perhaps the assumption was that RTS will prevent transmission failures outright. However, not letting the hardware retry at lower rates seems to hurt at least my athn-based AP a little.

user(8): Don't create default config file

There doesn't seem to be any reason for user to write out a config file with the defaults if none exists. I've never seen anything else in base do this, so kill it. - Matthew Martin Index: user.c === RCS file:

Re: Help me testing the netlock

Hi Martin - As soon as machine boots and first packet hits the VPN, it hits the assertion: rw_status() == RW_WRITE (hand-typed) TID PID UID PRFLAGS PFLAGS CPU COMMAND *78967 78967 0 0x14000 0x200 1 crypto __assert at0x25 ip_output() at 0x843

Re: 'make build': simplify 'make includes' step

On Thu, Oct 06, 2016 at 03:55:14PM +0200, Theo Buehler wrote: > On Thu, Oct 06, 2016 at 03:09:49PM +0200, Martin Natano wrote: > > > > Seems right to me. I wonder whether we should enforce root for includes > > like we do for build. Either in the command list of the target, or even > > something

Re: regress for etherip and vxlan

On Thu, Oct 06, 2016 at 05:31:22PM +0900, YASUOKA Masahiko wrote: > Fixed the scripts according to your comments. > -SUBDIR +=pf_divert pf_forward pf_fragment pf_print rdomains rtable > +SUBDIR +=etherip pf_divert pf_forward pf_fragment pf_print rdomains > +SUBDIR +=rtable Do you

'make build': simplify 'make includes' step

Before calling 'make includes' during 'make build', the first two steps are repetitions of what a direct 'make includes' does anyway, so move the privdrop up there and simply call 'make includes' from 'make build'. While there, use the exec idiom also for 'make cleandirs'. ok? Index: Makefile

httpd: fcgi memleak

Hi, It looks like httpd(8) is leaking memory in fcgi. The last line, which is empty, does not get free'd. Thus save the return value, then free the line and return. While there I realized that the extra check for ':' is unnecessary. strchr() will either return NULL or a ptr to ':'. As the

Re: better implementation of malloc option C: canaries

On Mon, Oct 03, 2016 at 07:57:13AM +0200, Otto Moerbeek wrote: > Hi, > > I have been working on a diff to do canaries in a better way. > > Canaries (enabled by the C malloc option) are values stored after the > requested size that are checked for being overwritten on calling > free(3). At the

Re: better implementation of malloc option C: canaries

Otto Moerbeek wrote: > > This reduces the size of the canary to 32 bytes max and also prints > the size of the chunk and the offset corruption was spotted. > > a.out(22658) in free(): error: chunk canary corrupted: 0x00c800e7 > > I allocated 200 (0xc8) bytes and overwrote a byte at offset 231

iwm: rts for long frames only

Stop using RTS for every data frame sent by iwm(4). RTS adds unneccessary overhead if small data frames are sent. The USE_RTS flag in iwm's LQ command enables RTS unconditionally, so only set it while the AP is enforcing protection. The flag will be kept up-to-date as a side effect of