Re: Kill sysctl net.inet6.ip6.v6only

2016-05-18 Thread Claudio Jeker
On Thu, May 19, 2016 at 01:15:57AM +0200, Jeremie Courreges-Anglas wrote: > > As noticed by djm it doesn't make much sense to expose this in sysctl(8) > output. > > ok? OK, especially since it makes the setsockopt code path actually understandable. > Index: sbin/sysctl/sysctl.8 >

msg buf lmin

2016-05-18 Thread Alexander Bluhm
Hi, All msg buf counters are long, so lmin(9) should be used here. ok? bluhm Index: kern/subr_log.c === RCS file: /cvs/src/sys/kern/subr_log.c,v retrieving revision 1.43 diff -u -p -r1.43 subr_log.c --- kern/subr_log.c 18 May

Kill sysctl net.inet6.ip6.v6only

2016-05-18 Thread Jeremie Courreges-Anglas
As noticed by djm it doesn't make much sense to expose this in sysctl(8) output. ok? Index: sbin/sysctl/sysctl.8 === RCS file: /cvs/src/sbin/sysctl/sysctl.8,v retrieving revision 1.193 diff -u -p -r1.193 sysctl.8 ---

Enforce ether_output() requirements

2016-05-18 Thread Martin Pieuchot
Back in the old cool days you could simply call ether_output() with a stuffed sockaddr and it will do the L2 resolution for you... Well as long as you do not care about IPv6 it's true. During the last years I tried to reduce the number of places where ether_output() was called without passing it

Re: usb: cache vendor, product and serial

2016-05-18 Thread Ted Unangst
Patrick Wildt wrote: > The same issue can be reproduced by calling usbdevs(8) in a loop. It > uses the same ioctl and "breaks" this usb mass storage. As a rule, we should limit the contact users have with hardware. "Safe" readonly operations are no exception.

Re: usb: cache vendor, product and serial

2016-05-18 Thread Martin Pieuchot
On 18/05/16(Wed) 18:28, Patrick Wildt wrote: > Hi, > > I had the pleasure of debugging a USB mass storage device that showed > interesting behaviour when used with our stack and in combination with > the blink(1) usb device. > > The blink(1)-tool depends on libusb. Libusb calls

Re: New 12" Retina Macbook (Macbook9,1)

2016-05-18 Thread Ted Unangst
Bryan C. Everly wrote: > I just received my 12" Retina Macbook (the Macbook9,1) which is the > latest Skylake version. I would really like to get OpenBSD running on > this and am happy to trace any code, build patches, stand on my head, > etc. if I could get someone interested in working with me

Re: libfuse has moved to github

2016-05-18 Thread Ray Lai
Looks good to me > On May 18, 2016, at 11:14 PM, Jason McIntyre wrote: > >> On Wed, May 18, 2016 at 12:13:14AM +0800, Ray Lai wrote: >> Also separate out the specification (does that count as a "standard"?) >> and the implementation. > > i've tweaked the diff a little to

Re: New 12" Retina Macbook (Macbook9,1)

2016-05-18 Thread Mike Larkin
On Wed, May 18, 2016 at 09:32:55AM -0500, joshua stein wrote: > On Wed, 18 May 2016 at 09:33:36 -0400, Bryan C. Everly wrote: > > 2. The keyboard works at the boot prompt, but does not work once the > > install/upgrade/shell prompt comes up > > The keyboard and trackpad appear to be connected

usb: cache vendor, product and serial

2016-05-18 Thread Patrick Wildt
Hi, I had the pleasure of debugging a USB mass storage device that showed interesting behaviour when used with our stack and in combination with the blink(1) usb device. The blink(1)-tool depends on libusb. Libusb calls ioctl(USB_DEVICEINFO) plenty of times to get a map of the USB tree. Per

Re: libfuse has moved to github

2016-05-18 Thread Jason McIntyre
On Wed, May 18, 2016 at 12:13:14AM +0800, Ray Lai wrote: > Also separate out the specification (does that count as a "standard"?) > and the implementation. > i've tweaked the diff a little to shorten it and keep the links in one section. STANDARDS seems the better choice. i also don;t think the

Re: New 12" Retina Macbook (Macbook9,1)

2016-05-18 Thread joshua stein
On Wed, 18 May 2016 at 09:33:36 -0400, Bryan C. Everly wrote: > 2. The keyboard works at the boot prompt, but does not work once the > install/upgrade/shell prompt comes up The keyboard and trackpad appear to be connected over SPI: https://bugzilla.kernel.org/show_bug.cgi?id=99891 While there

Re: [ntpd] Simultaneously listen on IPv4 and IPv6

2016-05-18 Thread Jeremie Courreges-Anglas
Stuart Henderson writes: > On 2016/05/18 14:01, Jeremie Courreges-Anglas wrote: >> Stuart Henderson writes: >> >> > Removing the sysctl should be very pretty safe as far as ports goes (I >> > did wonder if thing s might read the sysctl and change

New 12" Retina Macbook (Macbook9,1)

2016-05-18 Thread Bryan C. Everly
Hello tech@ I just received my 12" Retina Macbook (the Macbook9,1) which is the latest Skylake version. I would really like to get OpenBSD running on this and am happy to trace any code, build patches, stand on my head, etc. if I could get someone interested in working with me to figure this

Re: [ntpd] Simultaneously listen on IPv4 and IPv6

2016-05-18 Thread lists
Tue, 17 May 2016 08:27:42 -0500 Brent Cook > This patch came by way of the openntpd github. Linux (and possibly others) > will attempt to bind to 0.0.0.0 when binding to '::' and return an error if > it can't, unless IPV6_V6ONLY is set. See >

Re: [ntpd] Simultaneously listen on IPv4 and IPv6

2016-05-18 Thread Theo de Raadt
> Yes, the ioctl has to stay for exactly the reasons you give. > These are all over the tree. It's not 1 ports, but pretty > much *everything* that talks v6 and listens on dual sockets > apart from maybe some niche BSD-centric things. These > programs won't run on Linux without it. (They will

Re: [ntpd] Simultaneously listen on IPv4 and IPv6

2016-05-18 Thread Stuart Henderson
On 2016/05/18 14:01, Jeremie Courreges-Anglas wrote: > Stuart Henderson writes: > > > Removing the sysctl should be very pretty safe as far as ports goes (I > > did wonder if thing s might read the sysctl and change behaviour but it > > seems that's not the case). Looks

Re: [ntpd] Simultaneously listen on IPv4 and IPv6

2016-05-18 Thread Jeremie Courreges-Anglas
Stuart Henderson writes: > Removing the sysctl should be very pretty safe as far as ports goes (I > did wonder if thing s might read the sysctl and change behaviour but it > seems that's not the case). Looks like only nsh will break from doing > that and it's easily fixed.

Re: [patch] daemon.3

2016-05-18 Thread Remco
Op 05/18/16 om 13:55 schreef Ingo Schwarze: Hi, Remco wrote on Wed, May 18, 2016 at 10:29:40AM +0200: Op 05/18/16 om 01:49 schreef Edgar Pettijohn: The wording made it hard for me to understand at first. The "unless" "non-zero" seem like double negatives. Actually, quadruple negation:

Re: [patch] daemon.3

2016-05-18 Thread Ingo Schwarze
Hi, Remco wrote on Wed, May 18, 2016 at 10:29:40AM +0200: > Op 05/18/16 om 01:49 schreef Edgar Pettijohn: >> The wording made it hard for me to understand at first. >> The "unless" "non-zero" seem like double negatives. Actually, quadruple negation: 1. Unless 2. nochdir is 3. non- 4.

Re: [PATCH] Allow softraid crypto to work with write-protected keys

2016-05-18 Thread bytevolcano
My apologies for the noise; the previous one was the wrong revision (r1.126 instead of 1.127) because both patches look similar; here is the most recent catch: Index: sys/dev/softraid_crypto.c === RCS file:

Re: [PATCH] Allow softraid crypto to work with write-protected keys

2016-05-18 Thread bytevolcano
Ted Unangst wrote: > i removed these last two lines, since they were incorrect. thanks for spotting > that. however the vop_close at the end still needs updating. Thanks Ted, Also I found another stray VOP_CLOSE() setup. I have also put in the VOP_OPEN(vn, FREAD, ...) as I am not sure about

Update iwm(4) to newer firmware

2016-05-18 Thread Stefan Sperling
Update iwm(4) to firmware version 16.242414.0 (API 16). New firmware has been available in fw_update(1) for several weeks already. Among many API changes this firmware version introduces a wide command header for some commands because the old API format has run out command number space. This

ehci panic fix

2016-05-18 Thread Martin Pieuchot
New version of my fix for the "ehci_device_clear_toggle: queue active" panic. This diff gets bigger and bigger after the years but the logic remains the same. It changes when QH are added and removed to/from the async list. Instead of keeping a QH on the list as long as a pipe is open we now

Fewer if_get(9)

2016-05-18 Thread Martin Pieuchot
Now what we are calling ARP input routines directly from ether_input() there's no need to use another if_get()/if_put() dance. ok? Index: net/if_ethersubr.c === RCS file: /cvs/src/sys/net/if_ethersubr.c,v retrieving revision 1.235

Re: Accept cnmac as a valid rootdev from uboot on octeon

2016-05-18 Thread Kim Lidström
Visa Hankala wrote: > On Tue, May 17, 2016 at 02:02:37AM +0200, Kim Lidström wrote: > > Is this patch better? I have tested it by trying both cnmac0, cnmac1 and > > cnmac2 as rootdev and it seems to work properly. > > I also removed the outdated comment, changed the pointless

Re: [patch] daemon.3

2016-05-18 Thread Remco
Op 05/18/16 om 01:49 schreef Edgar Pettijohn: The wording made it hard for me to understand at first. The "unless" "non-zero" seem like double negatives. Index: daemon.3 === RCS file: /cvs/src/lib/libc/gen/daemon.3,v retrieving

Re: AMRR improvements for rt2860

2016-05-18 Thread Stefan Sperling
On Wed, May 18, 2016 at 06:24:15AM +0800, Nathanael Rensen wrote: > On 27 April 2016 at 17:33, Stefan Sperling wrote: > > > This version includes minor tweak: When the AP goes down, we don't > > need to send disassoc frames to nodes in COLLECT state. > > While testing this diff

Re: [ntpd] Simultaneously listen on IPv4 and IPv6

2016-05-18 Thread Stuart Henderson
Removing the sysctl should be very pretty safe as far as ports goes (I did wonder if thing s might read the sysctl and change behaviour but it seems that's not the case). Looks like only nsh will break from doing that and it's easily fixed. I did a search for things using the ioctl too. It

Re: [patch] daemon.3

2016-05-18 Thread Theo de Raadt
> i agree your text is easier to read. actually reading things like this > seems clear until you try and think about what it means. > > still, you are changing the emphasis of the text. it seems the > author chose that wording deliberately. it reads like the author > wants to say that it would

Re: [patch] daemon.3

2016-05-18 Thread Jason McIntyre
On Tue, May 17, 2016 at 06:49:27PM -0500, Edgar Pettijohn wrote: > The wording made it hard for me to understand at first. The "unless" > "non-zero" seem like double negatives. > i agree your text is easier to read. actually reading things like this seems clear until you try and think about