Re: hashfree: sizes for free(9) when using hashinit.

On Wed, Sep 21, 2016 at 1:30 AM, Mathieu - wrote: > Ted Unangst wrote: >> Mathieu - wrote: >> > Hello list, >> > >> > I'm introducing hashfree, a counterpart to hashinit in order to pass the >> > size to free(9) while hiding the implementation details. >> > Most of the api users

Re: timeout_set_proc(9)

On Fri, 23 Sep 2016, Christiano F. Haesbaert wrote: ... > The diff as it is will deadlock against SCHED_LOCK. > tsleep() calls sleep_setup(), which grabs SCHED_LOCK, > Then sleep_setup_timeout() will grab timeout_mutex in timeout_add() > > On softclock() you have the opposite: > Grabs

Re: Some arm cleanups suggested by clang

On Fri, Sep 23, 2016 at 01:43:04PM +0200, Mark Kettenis wrote: > Most of these are warnings about static symbols that aren't used. The > pmap_get_pde_pte() bit fixes: > > ../../../../arch/arm/arm/pmap7.c:2220:10: warning: comparison of array > 'pm->pm_l2' equal to a null pointer is always

Re: dwc2 cleanup diff

On Fri, Sep 23, 2016 at 01:48:39PM +0200, Mark Kettenis wrote: > Essentially fixes for clang warnings. Clang doesn't like the > > sc = sc; > > statements in dwc2_shutdown(), dwc2_childdet() and dwc2_activate(). > Those functions aren't actually used. This diff moves them inside the >

correct option listing in switchd(8)

Sync the option list with reality. Don't document the internal -I and -P options that set the instance number and title on purpose. Index: switchd.8 === RCS file: /cvs/src/usr.sbin/switchd/switchd.8,v retrieving revision 1.1 diff -u

etherip alignment issues

Here's another alignment issue. I have configured two machines as follows. ifconfig vether0 inet ... ifconfig etherip0 tunnel ... ... ifconfig bridge0 add vether0 add etherip0 An amd64 machine works fine in this configuration, however armv7 and sparc64 both have trouble. This happens very

attach SR drive by force even if not all chunks provide native metadata

Hello, following patch fixes issue while attempting to attach SR RAID1 drive where not all chunks provide native metadata. I.e. one chunk is dd zeroed. The complain of SR is good one, but I'd think that force parameter should overcome it and really enforce SR to attach such drive. Thanks, Karel

libcrypto/Makefile CLEANFILES

There are two lines in the Makefile setting CLEANFILES with the latter one overriding the former. The result: libcrypto.pc is not removed on make clean. Ok? natano Index: Makefile === RCS file: /cvs/src/lib/libcrypto/Makefile,v

Re: Remove more duplicated includes

Frederic Cambus writes: > Hi tech@, > > A few remaining duplicated includes to remove. > > Comments? OK? ok except for ixgbe.h > Index: sys/arch/sparc64/include/asm.h > === > RCS file:

ftp5.usa.openbsd.org going down Saturday September 24th at 10pm - Back Sunday morning

Due to a power outage happening this Sunday morning, ftp5.usa.openbsd.org will be going down around 10pm EDT (UTC-4) on Saturday September 24th. I will bring it back up when the power comes back at 9:30am EDT on Sunday September 25th, so it should be back up by 11am EDT. FYI --Kurt

Re: switchd(8): set the pktbuf for packet_in messages

On Thu, Sep 22, 2016 at 09:48:48PM +0200, Rafael Zalamena wrote: > The pkt_buf variable is never set in incoming packet_in messages and this > diff fixes it. > > ok? > I wonder how this happened, thanks. Skip the space before the cast, > + pkt->pkt_buf = (uint8_t *) eh;

Re: switchd(8): fix memory leak and loop

On Thu, Sep 22, 2016 at 09:46:50PM +0200, Rafael Zalamena wrote: > This diff fixes a memory leak in ofp_read() that happens in every message > and a infinite loop that happens when the remote switch closes the > connection. > > ok? > I'm planning to replace this part of the code (network I/O).

Re: switchd(8): more debug messages

On Thu, Sep 22, 2016 at 09:45:05PM +0200, Rafael Zalamena wrote: > Enable more debug messages to help developing the flow modification messages. > > ok? > OK, sure. Reyk > Index: ofp13.c > === > RCS file:

switchd(8): set the pktbuf for packet_in messages

The pkt_buf variable is never set in incoming packet_in messages and this diff fixes it. ok? Index: packet.c === RCS file: /home/obsdcvs/src/usr.sbin/switchd/packet.c,v retrieving revision 1.3 diff -u -p -r1.3 packet.c --- packet.c

switchd(8): fix memory leak and loop

This diff fixes a memory leak in ofp_read() that happens in every message and a infinite loop that happens when the remote switch closes the connection. ok? Index: ofp.c === RCS file: /home/obsdcvs/src/usr.sbin/switchd/ofp.c,v

switchd(8): more debug messages

Enable more debug messages to help developing the flow modification messages. ok? Index: ofp13.c === RCS file: /home/obsdcvs/src/usr.sbin/switchd/ofp13.c,v retrieving revision 1.5 diff -u -p -r1.5 ofp13.c --- ofp13.c 21 Jul 2016

dwc2 cleanup diff

Essentially fixes for clang warnings. Clang doesn't like the sc = sc; statements in dwc2_shutdown(), dwc2_childdet() and dwc2_activate(). Those functions aren't actually used. This diff moves them inside the #if 0 that's already there for dwc2_suspend() and dwc2_resume(). But

Some arm cleanups suggested by clang

Most of these are warnings about static symbols that aren't used. The pmap_get_pde_pte() bit fixes: ../../../../arch/arm/arm/pmap7.c:2220:10: warning: comparison of array 'pm->pm_l2' equal to a null pointer is always false [-Wtautological-pointer-compare] if (pm->pm_l2 ==

Re: timeout_set_proc(9)

Am Mittwoch, 21. September 2016 schrieb Martin Pieuchot : > On 21/09/16(Wed) 16:29, David Gwynne wrote: > > [...] > > the point i was trying to make was that the existing stuff (tasks, > timeouts) can be used together to get the effect we want. my point was very > poorly made though. > > > > i