Re: pf state key link inp

2017-12-23 Thread Alexandr Nedvedicky
Hello, > ok? sure, no objections/comments OK sashan@

sdmmc(4) DMA map tweak

2017-12-23 Thread Mark Kettenis
I have a diff to add DMA support to dwmmc(4). The hardware is a bit dumb and can only transfer up to 4K in a single DMA "segment". Currently sdmmc(4) allocates a bus DMA mapping structure that allows for segments up to MAXPHYS. Subdividing such segments in my driver seems a bit dumb. So here is

Re: uniq: add -i option

2017-12-23 Thread Jeremie Courreges-Anglas
On Sun, Dec 24 2017, Theo Buehler wrote: >> Obviously, the relevant condition should have been >> >> if ((iflag ? strcasecmp : strcmp)(t1, t2) != 0) >> >> instead of awkwardly messing with logical AND and OR. > > Indeed, this is much better. I like your version, but perhaps others > might l

Re: uniq: add -i option

2017-12-23 Thread Theo Buehler
On Sun, Dec 24, 2017 at 12:01:17AM +, kshe wrote: > On Sat, 23 Dec 2017 23:13:22 +, Theo Buehler wrote: > > > Obviously, the relevant condition should have been > > > > > > if ((iflag ? strcasecmp : strcmp)(t1, t2) != 0) > > > > > > instead of awkwardly messing with logical AND and OR.

Re: uniq: add -i option

2017-12-23 Thread kshe
On Sat, 23 Dec 2017 23:13:22 +, Theo Buehler wrote: > > Obviously, the relevant condition should have been > > > > if ((iflag ? strcasecmp : strcmp)(t1, t2) != 0) > > > > instead of awkwardly messing with logical AND and OR. > > Indeed, this is much better. I like your version, but perhaps

Re: Propagate http_proxy from installer to rc.firsttime

2017-12-23 Thread trondd
trondd wrote: > If you need to use a proxy to fetch the base sets, you'll likely need > it when you reboot in order for fw_update and syspatch to work. It's > helpful for fw_update especially as it has a painfully long timeout. > > If the http_proxy was set, simply pass it on to rc.firsttime so

Re: less(1): `!' command

2017-12-23 Thread Jeremie Courreges-Anglas
On Fri, Dec 22 2017, Stuart Henderson wrote: > On 2017/12/22 19:47, Nicholas Marriott wrote: >> I don't think we should bring ! back. >> >> I wanted to remove v and | (and some other stuff) shortly afterwards, but >> several people objected. >> >> I did suggest having a lightweight less in base

Re: uniq: add -i option

2017-12-23 Thread Theo Buehler
> Obviously, the relevant condition should have been > > if ((iflag ? strcasecmp : strcmp)(t1, t2) != 0) > > instead of awkwardly messing with logical AND and OR. Indeed, this is much better. I like your version, but perhaps others might like this one more: if ((iflag && strcasec

Re: less(1): `!' command

2017-12-23 Thread kshe
On Fri, 22 Dec 2017 22:21:12 +, Stuart Henderson wrote: > On 2017/12/22 19:47, Nicholas Marriott wrote: > > I don't think we should bring ! back. > > > > I wanted to remove v and | (and some other stuff) shortly afterwards, but > > several people objected. > > > > I did suggest having a lightwe

dc(1): minor simplification in bexp()

2017-12-23 Thread kshe
Hi, When p->scale is not zero, bexp() calculates the same value twice, first in split_number() and then in normalize(). This can be avoided by simply reusing the result produced by the former. (While here, also fold a long line.) Index: bcode.c ==

Re: uniq: add -i option

2017-12-23 Thread kshe
Hi, This change causes uniq(1) to compare equal lines twice when run without `-i', once in strcmp(3) and once again in strcasecmp(3). In the worst case, which is also one of the most common, the main loop spends about half of its time copying buffers and looking for newlines in fgets(3), and the

Re: arm64: init a var in _bootstrap()

2017-12-23 Thread Artturi Alm
On Sat, Dec 23, 2017 at 03:41:02PM +0200, Artturi Alm wrote: > Hi, > > spotted this while looking at how deep the hole suggested by mpi@ is. > memhook is used by mmrw minor case 0, and the diff below fixes that. > rest of it should be logically solid improvement, but given the state > of arm64 int

pom: use round(3), fmod(3)

2017-12-23 Thread Scott Cheloha
Hey, I have a few patches here that spruce up pom(6). First up: use more libm. adj360() is a modulo operation, so we ought to leverage fmod(3). Adding 0.5 to the phase here to "steer" integer truncation toward the desired principal phase classification is hokey. Using round(3) with its documen

Generic hw.cpuspeed implementation for arm64

2017-12-23 Thread Mark Kettenis
The device tree node for the cpu is supposed to have a "clocks" property that references its clock. Using the generic clock framework code we can get the frequency of that clock and report it. Note that some device trees are incomplete and won't have a "clocks" property. In that case hw.cpuspeed

Re: realpath(3): some buffer overflows and conformance issues

2017-12-23 Thread Todd C. Miller
Here's a diff for libc and ld.so without the asserts. - todd Index: lib/libc/stdlib/realpath.c === RCS file: /cvs/src/lib/libc/stdlib/realpath.c,v retrieving revision 1.21 diff -u -p -u -r1.21 realpath.c --- lib/libc/stdlib/realpath

malloc cleanup and small optimizations

2017-12-23 Thread Otto Moerbeek
Hi, Largely cleanup and small optimizations: - remove over-initializaition of chunks: canary is set only once, other fields only as-needed. - remember position of last free bit in chunk_info, so we can start at the proper spot the next time. To be able to do this a field was added to chunk_info.

arm64: init a var in _bootstrap()

2017-12-23 Thread Artturi Alm
Hi, spotted this while looking at how deep the hole suggested by mpi@ is. memhook is used by mmrw minor case 0, and the diff below fixes that. rest of it should be logically solid improvement, but given the state of arm64 internals w/all the "// XXXshit", and how sacred these arch-md bootstraps ar

Re: release(8): join exports

2017-12-23 Thread Anton Lindqvist
On Fri, Dec 22, 2017 at 10:39:20PM +0100, Mark Kettenis wrote: > > Date: Fri, 22 Dec 2017 22:12:19 +0100 > > From: Anton Lindqvist > > > > Hi, > > Since export accepts several variables, put them on a single line. > > > > Comments? OK? > > Feels like unnecessary chrun to me. The exports are al

Re: less(1): `!' command

2017-12-23 Thread Alexander Hall
On December 22, 2017 11:21:12 PM GMT+01:00, Stuart Henderson wrote: >On 2017/12/22 19:47, Nicholas Marriott wrote: >> I don't think we should bring ! back. >> >> I wanted to remove v and | (and some other stuff) shortly afterwards, >but >> several people objected. >> >> I did suggest having a

Re: fc-list.1 bug

2017-12-23 Thread Matthieu Herrb
On Fri, Dec 22, 2017 at 06:56:14PM -0600, Edgar Pettijohn wrote: > $ diff -u fc-list.1.orig fc-list.1 > > --- fc-list.1.orig Fri Dec 22 18:47:19 2017 > +++ fc-list.1 Fri Dec 22 18:54:24 2017 > @@ -69,7 +69,7 @@ > \fBfc-scan\fR(1) >