libc: remove pointless off_t, size_t, void* and char* casts

I'm pretty sure I got all of the "no effect" casts to off_t and size_t. I picked up the void* and char* casts with a less comprehensize search, so there may be some that slipped through. This diff was checked by comparing the .o files before and after, building on both an LP64 arch (amd64)

Re: [PATCH] www: mention /usr/src in upgrade notes

On Mon, Sep 19, 2016 at 07:25:08PM -0700, Philip Guenther wrote: > Listing what something *doesn't* do is generally a bad plan, because > the list has no end, so where to draw the line becomes a matter of > opinion. Better to list what the installer does update and say that's > the complete list,

Re: [PATCH] www: mention /usr/src in upgrade notes

On Mon, Sep 19, 2016 at 7:12 PM, Simon Ruderich wrote: ... > However I still would like to see a quick note telling the user > that he should remember to update all source-based data because > the installer won't do it for them . Maybe something like "The > installer doesn't

Re: [PATCH] www: mention /usr/src in upgrade notes

> However I still would like to see a quick note telling the user > that he should remember to update all source-based data because > the installer won't do it for them . Maybe something like "The > installer doesn't automatically update source files, for example > in /usr/src". Yeah, we could

Re: [PATCH] www: mention /usr/src in upgrade notes

On Mon, Sep 19, 2016 at 03:00:40AM +0200, Theo Buehler wrote: > On Mon, Sep 19, 2016 at 01:59:20AM +0200, Simon Ruderich wrote: >> I'm a relatively new OpenBSD user and just updated my system from >> 5.9 to 6.0 but forgot to update the files in /usr/src. Thus when >> I applied the latest patches

netinet/ ansify

Hello - Ansify netinet/ Index: ip_ecn.c === RCS file: /cvs/src/sys/netinet/ip_ecn.c,v retrieving revision 1.7 diff -u -p -r1.7 ip_ecn.c --- ip_ecn.c19 Dec 2014 17:14:40 - 1.7 +++ ip_ecn.c20 Sep 2016 01:54:47 -

Re: [PATCH] switch.4 awkward sentence reorder

On Mon, Sep 19, 2016 at 06:24:54PM -0700, Bryan Vyhmeister wrote: > On Tue, Sep 20, 2016 at 02:11:41AM +0100, Stuart Henderson wrote: > > Still seems a bit awkward, how about this? > > > > The > > .Nm > > device and the controller communicate using the > > OpenFlow 1.3 protocol. > > Even better.

Re: [PATCH] switch.4 awkward sentence reorder

On Tue, Sep 20, 2016 at 02:11:41AM +0100, Stuart Henderson wrote: > Still seems a bit awkward, how about this? > > The > .Nm > device and the controller communicate using the > OpenFlow 1.3 protocol. Even better. I am very happy with that. Bryan

my final netinet6 bcopy->memcpy

Hello - Here are the final bcopy->memcpy conversions in netinet6 that I am comfortable with. There is also one (bcmp()) to (memcmp() != 0) conversion since the memory is not overlapping. This, with the last netinet6 diff, has dropped the number of bcopy() calls from 62 to 29 in netinet6/

Re: [PATCH] switch.4 awkward sentence reorder

On 2016/09/19 18:05, Bryan Vyhmeister wrote: > This is a patch to reorder an awkward sentence in the switch(4) man > page. This may not be desired but it looks less awkward to me. > > Bryan > > > Index: switch.4 > === > RCS file:

[PATCH] switch.4 awkward sentence reorder

This is a patch to reorder an awkward sentence in the switch(4) man page. This may not be desired but it looks less awkward to me. Bryan Index: switch.4 === RCS file: /cvs/src/share/man/man4/switch.4,v retrieving revision 1.3 diff

Re: Enabling rasops24

On Mon, Sep 19, 2016 at 06:52:00PM -0400, sven falempin wrote: > Tested here and it works ! @OK > > The terminal is white on red for first part of boot then goes back to white > on > blue. > Also some kind of flicker/bufering during the start, > aftfer the kernel loading no update for like half a

work-in-progress iwm performance fix

I'd like to know if this diff helps with iwm(4) performance issues some people have been reporting. This is not done yet and some details don't really make sense, especially the #if 0 hiding of what should be correct code -- yet, enabling that code makes the problem come back. But hopefully this

Re: Enabling rasops24

Tested here and it works ! @OK The terminal is white on red for first part of boot then goes back to white on blue. Also some kind of flicker/bufering during the start, aftfer the kernel loading no update for like half a second. But it s booting completly : i got some dmesg: sysctl:

Re: db_{interface,trace}.c: ansify function definitions

On Mon, Sep 19, 2016 at 11:02 AM, Jasper Lievisse Adriaanse wrote: > OK? > > Index: alpha/alpha/db_trace.c ... ok guenther@

Re: Remove duplicated includes in stdlib.h and termios.h

On Mon, Sep 19, 2016 at 11:50 AM, Frederic Cambus wrote: > Hi tech@, > > This diff removes duplicated includes in stdlib.h and termios.h > > Comments? OK? ok guenther@

Remove duplicated includes in stdlib.h and termios.h

Hi tech@, This diff removes duplicated includes in stdlib.h and termios.h Comments? OK? Index: include/stdlib.h === RCS file: /cvs/src/include/stdlib.h,v retrieving revision 1.66 diff -u -p -r1.66 stdlib.h --- include/stdlib.h9

Re: video(1): Don't limit maximum frame size to display size

Hi, > [...] > This seems to work fine with inteldrm, a 1280x800 display and > [...] Good to hear I didn't break anything :) > [...] > No idea either... if you wanted to reduce the risk of breakage, I > guess you could limit the resolution only if -O is not passed. > [...] I've read up on Xv a

db_{interface,trace}.c: ansify function definitions

OK? Index: alpha/alpha/db_trace.c === RCS file: /cvs/src/sys/arch/alpha/alpha/db_trace.c,v retrieving revision 1.18 diff -u -p -r1.18 db_trace.c --- alpha/alpha/db_trace.c 19 Sep 2016 17:59:18 - 1.18 +++

Re: Build libc++ and libc++abi with -nostdinc++

On Mon, Sep 19, 2016 at 2:40 AM, Mark Kettenis wrote: >> Date: Sun, 18 Sep 2016 00:12:19 +0200 (CEST) >> From: Mark Kettenis >> >> Without this, they don't build with clang++ because it tries to use >> the installed headers. Doesn't hurt

few quick arcofi(4) tests needed

I'm trying to switch the arcofi(4) driver to the "new" (as from 1998) audio interface, but I have no access to the hardware. Anyone with a hp9000 with a working arcofi(4) device to run few tests for me? (to check if it works, just play a .wav file) Thanks

Re: TCP timers & tasks

On Mon, Sep 12, 2016 at 01:54:34PM +0200, Martin Pieuchot wrote: > I extended the G/C timeout to 1sec, because there's no reference count > for 'struct tcpcb'. And the only race possible is described in the > comment below: > > /* >* If one of the timer tasks is already running, it

Re: syslogd validate client certificates

On Mon, Sep 05, 2016 at 03:25:22AM +0200, Alexander Bluhm wrote: > Hi, > > Add an option to give syslogd a server CA that is used to validate > client certificates. This prevent that malicious clients can send > fake messages. > > ok? Could anyone have a look at the validation code. I will

Re: gmtime return value

On Mon, Sep 19, 2016 at 06:48:32AM -0600, Todd C. Miller wrote: > Committed, thanks. > > - todd > Thanks. Is there any chance of this going into -stable? This was found because it was causing test failures in Tor and tor devs are worried that there might be some edge-cases where it can cause

Re: timeout_set_proc(9)

> Date: Mon, 19 Sep 2016 13:47:25 +1000 > From: David Gwynne > > On Fri, Sep 16, 2016 at 04:58:39PM +0200, Mark Kettenis wrote: > > > Date: Thu, 15 Sep 2016 16:29:45 +0200 > > > From: Martin Pieuchot > > > > > > After discussing with a few people about a

Re: timeout_set_proc(9)

On Thu, Sep 15, 2016 at 04:29:45PM +0200, Martin Pieuchot wrote: > After discussing with a few people about a new "timed task" API I came > to the conclusion that mixing timeouts and tasks will result in: > > - always including a 'struct timeout' in a 'struct task', or the other > the way

Re: We don't need cross-unwinding support

> Date: Mon, 19 Sep 2016 16:08:33 +0200 (CEST) > From: Mark Kettenis > > > Date: Mon, 19 Sep 2016 14:08:49 +0200 (CEST) > > From: Mark Kettenis > > > > Saves a bit code and a bit of stack space, which might be important > > important in some

Re: We don't need cross-unwinding support

> Date: Mon, 19 Sep 2016 14:08:49 +0200 (CEST) > From: Mark Kettenis > > Saves a bit code and a bit of stack space, which might be important > important in some unwinding scenarios. > > ok? Transplanted the diff into the wrong file. Here is the correct one. Index:

Re: gmtime return value

Committed, thanks. - todd

We don't need cross-unwinding support

Saves a bit code and a bit of stack space, which might be important important in some unwinding scenarios. ok? Index: Makefile === RCS file: /cvs/src/lib/libcxx/Makefile,v retrieving revision 1.2 diff -u -p -r1.2 Makefile ---

Re: Division by zero in ehci(4)

On Mon, Sep 19, 2016 at 11:47:40AM +0200, Martin Pieuchot wrote: > On 19/09/16(Mon) 14:20, Jonathan Gray wrote: > > On Sat, Sep 17, 2016 at 04:36:12PM +0200, Martin Pieuchot wrote: > > > One of the non-checked value read from an untrusted descriptor is the > > > "maximum packet size" of an

Add support for a few more arm relocations

>From FreeBSD, still under GPLv2. I've seen R_ARM_MOVW_ABS_NC in code generated by clang. ok? Index: gnu/usr.bin/binutils-2.17/bfd/bfd-in2.h === RCS file: /cvs/src/gnu/usr.bin/binutils-2.17/bfd/bfd-in2.h,v retrieving revision 1.6

arm libcompiler_rt

Diff below adds the relevant arm-specific code to the build. It leaves out the VFP bits. Those don't compile with our current toolchain (gcc/binutils) and I believe they're only necessary for the hard-float ABI anyway. ok? Index: Makefile

Re: timeout_set_proc(9)

On 19/09/16(Mon) 13:47, David Gwynne wrote: > [...] > id rather not grow the timeout (or task for that matter) apis just > yet. theyre nice and straightforward to read and understand so far. So better introduce a new API, right? > for this specific problem can we do a specific fix for it? the