Re: Improve the accuracy of the TSC frequency calibration - Updated Patch

2017-08-23 Thread Adam Steen
On Thu, Aug 24, 2017 at 2:35 AM, Mike Larkin wrote: > On Wed, Aug 23, 2017 at 09:29:15PM +0800, Adam Steen wrote: >> On Thu, Aug 17, 2017 at 12:19:28PM +0800, Adam Steen wrote: >> > On Tue, Aug 8, 2017 at 10:12 PM, Mike Belopuhov wrote: >> > > On Tue,

memory leak in sdmmc_io_scan()

2017-08-23 Thread Jonathan Gray
Coverity CID 1453042. Index: sdmmc_io.c === RCS file: /cvs/src/sys/dev/sdmmc/sdmmc_io.c,v retrieving revision 1.28 diff -u -p -U4 -r1.28 sdmmc_io.c --- sdmmc_io.c 6 Apr 2017 17:00:53 - 1.28 +++ sdmmc_io.c 24 Aug 2017

freezero(NULL, 0)

2017-08-23 Thread Damien Miller
Hi, memset(NULL, 0, 0) is (strictly speaking) undefined behaviour, but there's no reason that freezero(3) needs to follow suit. This mentions that freezero(NULL, 0) is valid in the manpage, so that anyone who copies this API should get it right too. ok? Index: malloc.3

Re: ospfd: add IMSG_IFADDRADD to deal with "sh /etc/netstart if"

2017-08-23 Thread Remi Locherer
On Wed, Aug 23, 2017 at 12:22:03AM +0200, Florian Riehm wrote: > On 08/21/17 18:57, Remi Locherer wrote: > > On Mon, Jul 24, 2017 at 04:59:46PM +0200, Remi Locherer wrote: > > > On Fri, Jul 21, 2017 at 06:24:06PM +0200, Remi Locherer wrote: > > > > On Fri, Jul 21, 2017 at 02:45:03PM +0200, Florian

iked: Do not accept superfluous arguments

2017-08-23 Thread Klemens Nanni
Calling `iked reload' when I meant `ikectl reload' showed that iked happily returned 0 and and fired up another daemon. Feedback? Index: iked.c === RCS file: /cvs/src/sbin/iked/iked.c,v retrieving revision 1.34 diff -u -p -r1.34

Re: Improve the accuracy of the TSC frequency calibration - Updated Patch

2017-08-23 Thread Mike Larkin
On Wed, Aug 23, 2017 at 09:29:15PM +0800, Adam Steen wrote: > On Thu, Aug 17, 2017 at 12:19:28PM +0800, Adam Steen wrote: > > On Tue, Aug 8, 2017 at 10:12 PM, Mike Belopuhov wrote: > > > On Tue, Aug 08, 2017 at 08:18 +0800, Adam Steen wrote: > > >> On Mon, Jul 31, 2017 at 3:58

Re: Improve the accuracy of the TSC frequency calibration - Updated Patch

2017-08-23 Thread Adam Steen
On Thu, Aug 17, 2017 at 12:19:28PM +0800, Adam Steen wrote: > On Tue, Aug 8, 2017 at 10:12 PM, Mike Belopuhov wrote: > > On Tue, Aug 08, 2017 at 08:18 +0800, Adam Steen wrote: > >> On Mon, Jul 31, 2017 at 3:58 PM, Mike Belopuhov wrote: > >> > On Mon, Jul

Re: allow iwm_stop to sleep

2017-08-23 Thread Stefan Sperling
Has anybody tried this diff? On Mon, Aug 14, 2017 at 09:07:37AM +0200, Stefan Sperling wrote: > This diff makes iwm_stop() always run in a process context. > I want iwm_stop() to be able to sleep so that it can wait for asynchronous > driver tasks, and perhaps even wait for firmware commands, in

Re: [PATCH] Fix likely typo in if_iwm.c

2017-08-23 Thread Stefan Sperling
On Wed, Aug 23, 2017 at 09:57:46PM +0900, Bryan Linton wrote: > I spotted what looks to be a typo in if_iwm.c given the context in > which it occurs. Patch attached. Apologies if I'm mistaken. Committed, thanks. > > -- > Bryan > > Index: if_iwm.c >

[PATCH] Fix likely typo in if_iwm.c

2017-08-23 Thread Bryan Linton
I spotted what looks to be a typo in if_iwm.c given the context in which it occurs. Patch attached. Apologies if I'm mistaken. -- Bryan Index: if_iwm.c === RCS file: /cvs/src/sys/dev/pci/if_iwm.c,v retrieving revision 1.211 diff

Re: relayd interrupted transfers

2017-08-23 Thread Rivo Nurges
Hi! I'll look into it. Rivo On 23/08/2017, 14:42, "Alexander Bluhm" wrote: On Tue, Aug 22, 2017 at 05:31:17PM +, Rivo Nurges wrote: > relay_error() sets se_done even if write buffer is not drained and > relay_write() will close the connection if

Re: relayd interrupted transfers

2017-08-23 Thread Alexander Bluhm
On Tue, Aug 22, 2017 at 05:31:17PM +, Rivo Nurges wrote: > relay_error() sets se_done even if write buffer is not drained and > relay_write() will close the connection if se_done is set I have seen a sporadic fail with chunked encoding in the daily regression test run. So something might be