Re: Fix regression in in_pcbbind(), bug observable with INET6

2016-04-06 Thread Alexander Bluhm
On Wed, Apr 06, 2016 at 05:20:32PM +0200, Vincent Gross wrote: > The regression test in regress/sys/netinet6/autoport is failing because > my merge of in_pcbbind() and in6_pcbbind() introduced a bug. Long story > short, if nam == NULL, then you skip the part where you check if the socket > is

Re: vmm: guest<->host communication via shared memory

2016-04-06 Thread Stefan Kempf
Stefan Kempf wrote: > Hi, > > here comes a diff for vmm, and I'd like to ask people that are > interested in our hypervisor to test this. If you are experimenting > with vmm already, just do what you always do with vmm when running > with this diff :-) > > [...] > > This diff will not go in at

vmm: guest<->host communication via shared memory

2016-04-06 Thread Stefan Kempf
Hi, here comes a diff for vmm, and I'd like to ask people that are interested in our hypervisor to test this. If you are experimenting with vmm already, just do what you always do with vmm when running with this diff :-) How to apply the diff: $ cd /usr/src $ patch < vmm.diff $ doas make

Re: [patch] ftpd: unused include

2016-04-06 Thread Ted Unangst
frit...@alokat.org wrote: > Looks like monitor.h doesn't need to be included > in monitor_fdpass.c. > > Mentioned by att...@stalphonsos.com. it's better if the declaration and definition of functions always match.

[patch] ftpd: unused include

2016-04-06 Thread fritjof
Looks like monitor.h doesn't need to be included in monitor_fdpass.c. Mentioned by att...@stalphonsos.com. --f. Index: monitor_fdpass.c === RCS file: /cvs/src/libexec/ftpd/monitor_fdpass.c,v retrieving revision 1.7 diff -u -r1.7

Fix for a possible ahci issue on intel skylake hardware

2016-04-06 Thread Imre Vadasz
For us, this patch fixed ahci getting stuck during high disk-IO load (mainly writes) from at least 2 threads in parallel. We can reproduce this on a Skylake notebook with a single SSD disk, but running openbsd within a custom virtualization solution. However, I so far failed to reproduce this

remove MPSAFE from makesyscalls.sh

2016-04-06 Thread Michal Mazurek
MPSAFE is never used, and doesn't look like it's even supported (no matching SY_MPSAFE anywhere). Index: sys/kern/makesyscalls.sh === RCS file: /cvs/src/sys/kern/makesyscalls.sh,v retrieving revision 1.11 diff -u -p -r1.11

Re: document NOLOCK in syscalls.master

2016-04-06 Thread Mike Belopuhov
On 6 April 2016 at 14:45, Michal Mazurek wrote: > * document NOLOCK - copy and paste from the commit message. > * keep MPSAFE undocumented, as it is never used. > * mention INDIR and NOLOCK in 'Fields'. > * sort the list of types in 'Fields'. > * sort the list of types in

document NOLOCK in syscalls.master

2016-04-06 Thread Michal Mazurek
* document NOLOCK - copy and paste from the commit message. * keep MPSAFE undocumented, as it is never used. * mention INDIR and NOLOCK in 'Fields'. * sort the list of types in 'Fields'. * sort the list of types in 'types'. * place a dot ('.') at the end of each sentence. Index:

sys_process.c: remove relebad

2016-04-06 Thread Michal Mazurek
relebad used to have more body: relebad: PRELE(t); return (error); But then PRELE(t); was removed. This diff gets rid of what remains of relebad. Index: sys/kern/sys_process.c === RCS file:

fdt: convert function to return EINVAL

2016-04-06 Thread Patrick Wildt
Hi, as the new translation function makes use of errnos as return value, it probably makes sense to change the "get memory" function, too. Also I think we should check that mem is not NULL in this function, as it's exposed in the API. If node or mem is NULL, just return EINVAL. ok? Patrick

pppoe(4): relax ifconfig command sequence requirements

2016-04-06 Thread Mike Belopuhov
Hi, The EBUSY hack that I'm removing imposes an order on the ifconfig commands issued against the pppoe interface used to configure the sppp layer below. Right now ifconfig will fail with "device busy" after the authentication parameters have been set up because it kickstarts the LCP logic down

Re: fdt: translate memory addresses

2016-04-06 Thread Mark Kettenis
> Date: Tue, 5 Apr 2016 18:11:03 +0200 > From: Patrick Wildt > > Updated to include feedback. > > ok? Looks good to me; ok kettenis@ > diff --git sys/dev/ofw/fdt.c sys/dev/ofw/fdt.c > index 4cebbb4..09ddcfc 100644 > --- sys/dev/ofw/fdt.c > +++ sys/dev/ofw/fdt.c > @@ -34,6

Re: Double resched_proc() in setrunnable()

2016-04-06 Thread Mark Kettenis
> Date: Wed, 6 Apr 2016 11:03:29 +0200 > From: Martin Pieuchot > > On 27/03/16(Sun) 12:00, Martin Pieuchot wrote: > > Diff below moves resetpriority() out of updatepri() and resched_proc() > > out of resetpriority() in order to expose a double resched_proc() call > > in

Re: [patch] ftpd: ptr == NULL

2016-04-06 Thread Sebastien Marie
On Wed, Apr 06, 2016 at 06:50:04AM +0200, Sebastien Marie wrote: > On Tue, Apr 05, 2016 at 08:22:22PM +0200, frit...@alokat.org wrote: > > Hi, > > > > cmd is a ptr. > > > > --f. > > I am OK with it (after a small change for fit 80 cols) > Commited. Thanks ! -- Sebastien Marie