Re: [PATCH] ure improvement

2017-07-17 Thread Martin Pieuchot
On 15/07/17(Sat) 21:16, sc dying wrote: > Hi, > > This patch does: > > - Enable RX aggregation. Does it work on all chips? > - Fix RX packet buffer alignment, using roundup() macro in sys/params.h. Why is that needed? pktlen should already be aligned, no? > - Call usbd_set_config before conf

Re: rc: reorder_libs: [2/2] Pick archive versions more efficiently

2017-07-17 Thread Vadim Zhukov
2017-07-16 14:55 GMT+03:00 Klemens Nanni : > On Sun, Jul 16, 2017 at 10:26:25AM +, Robert Peichaer wrote: >> But I'd like to stay strict matching the filenames. >> >> + for _liba in /usr/lib/lib{c,crypto}; do >> + _libas="$_libas $(ls $_liba.so.+([0-9.]).a | sort -V | tail >> -

Re: rc: reorder_libs: [2/2] Pick archive versions more efficiently

2017-07-17 Thread Klemens Nanni
On Mon, Jul 17, 2017 at 11:57:02AM +0300, Vadim Zhukov wrote: > > + for _liba in /usr/lib/lib{c,crypto}; do > > + _libas="$_libas $(ls $_liba.so.+([0-9.]).a | sort -V | tail > > -1)" > > done > > + _libas=${_libas# } > > > > # Remount read-write, if /usr/l

Re: rc: reorder_libs: [2/2] Pick archive versions more efficiently

2017-07-17 Thread Vadim Zhukov
2017-07-17 14:03 GMT+03:00 Klemens Nanni : > On Mon, Jul 17, 2017 at 11:57:02AM +0300, Vadim Zhukov wrote: >> > + for _liba in /usr/lib/lib{c,crypto}; do >> > + _libas="$_libas $(ls $_liba.so.+([0-9.]).a | sort -V | >> > tail -1)" >> > done >> > + _libas=${_libas#

Re: rc: Use here document for temporary pf rule set

2017-07-17 Thread Vadim Zhukov
2017-07-17 0:15 GMT+03:00 Klemens Nanni : > On Sun, Jul 16, 2017 at 08:15:38PM +, Robert Peichaer wrote: >> > + ifconfig lo0 inet6 >/dev/null 2>&1 && >> >> Please leave the if-then-fi construct intact. This short form is >> mostly used for on-line commands (with only a few exceptions). > OK.

[libcrypto] Don't build empty ecp_nistp* objects

2017-07-17 Thread Brent Cook
OPENSSL_NO_EC_NISTP_64_GCC_128 has been defined in opensslfeatures.h for a long time, which effectively means that ecp_nistp* are all empty files. So, there is no reason to build them in the first place. OK? Index: Makefile === RCS fi

Re: rc: reorder_libs: [2/2] Pick archive versions more efficiently

2017-07-17 Thread Klemens Nanni
On Mon, Jul 17, 2017 at 02:14:26PM +0300, Vadim Zhukov wrote: > 2017-07-17 14:03 GMT+03:00 Klemens Nanni : > > On Mon, Jul 17, 2017 at 11:57:02AM +0300, Vadim Zhukov wrote: > >> > + for _liba in /usr/lib/lib{c,crypto}; do > >> > + _libas="$_libas $(ls $_liba.so.+([0-9.]).a | sor

Re: rc: reorder_libs: [2/2] Pick archive versions more efficiently

2017-07-17 Thread Theo Buehler
On Mon, Jul 17, 2017 at 02:14:26PM +0300, Vadim Zhukov wrote: > 2017-07-17 14:03 GMT+03:00 Klemens Nanni : > > On Mon, Jul 17, 2017 at 11:57:02AM +0300, Vadim Zhukov wrote: > >> > + for _liba in /usr/lib/lib{c,crypto}; do > >> > + _libas="$_libas $(ls $_liba.so.+([0-9.]).a | sor

Re: inteldrm(4) diff needs review and testing

2017-07-17 Thread Giovanni Bechis
On Sun, Jul 16, 2017 at 03:19:41PM +0200, Mark Kettenis wrote: > Can somebody test the following diff on Ivy Bridge or Haswell (Intel > HD Graphics 2500/4000/4600/4700/5000/5100/5200)? > > When I added support for the command parser, I took a bit of a > shortcut and implemented the hash tables as

Re: [libcrypto] Don't build empty ecp_nistp* objects

2017-07-17 Thread Kinichiro Inoguchi
Absolutely. ok inoguchi@ On Mon, Jul 17, 2017 at 06:26:30AM -0500, Brent Cook wrote: > OPENSSL_NO_EC_NISTP_64_GCC_128 has been defined in opensslfeatures.h for a > long time, which effectively means that ecp_nistp* are all empty files. So, > there is no reason to build them in the first place. OK

Re: rc: reorder_libs: [1/2] Drop unused _l, exit early on failure

2017-07-17 Thread Klemens Nanni
On Sun, Jul 16, 2017 at 09:09:44AM +, Robert Peichaer wrote: > The rationale to picking the library versions before remounting was > to keep the time window having rw /usr as small as possible. > If that's deemed ok, I'm too OK with switching the steps. Considering the fact that the now simplif

Re: rc: reorder_libs: [1/2] Drop unused _l, exit early on failure

2017-07-17 Thread Theo Buehler
On Mon, Jul 17, 2017 at 03:00:34PM +0200, Klemens Nanni wrote: > On Sun, Jul 16, 2017 at 09:09:44AM +, Robert Peichaer wrote: > > The rationale to picking the library versions before remounting was > > to keep the time window having rw /usr as small as possible. > > If that's deemed ok, I'm too

rc: Use IFS when looking for carp interfaces

2017-07-17 Thread Klemens Nanni
The Internal Field Seperator is meant for this so use it instead of reading and stripping ':' again. Feedback? Comments? Index: rc === RCS file: /cvs/src/etc/rc,v retrieving revision 1.508 diff -u -p -r1.508 rc --- rc 17 Jul 2017 12

Re: [PATCH] ure improvement

2017-07-17 Thread sc dying
On 2017/07/17 08:24, Martin Pieuchot wrote: > On 15/07/17(Sat) 21:16, sc dying wrote: >> Hi, >> >> This patch does: >> >> - Enable RX aggregation. > > Does it work on all chips? I don't have all, but it works with mine that have RTL8152 (ver 4c10) and RTL8153 (ver 5c20). > >> - Fix RX packet buff

Re: pledge ifstated

2017-07-17 Thread Rob Pierce
On Sun, Jul 16, 2017 at 04:47:07PM -0400, Rob Pierce wrote: > On Thu, Jul 13, 2017 at 09:16:14PM -0400, Rob Pierce wrote: > > On Mon, Jul 10, 2017 at 01:21:58PM -0400, Rob Pierce wrote: > > > The following diff is loosely based on the approach that was taken for > > > pledging mountd. Other code/ap

Re: rc: reorder_libs: [1/2] Drop unused _l, exit early on failure

2017-07-17 Thread Robert Peichaer
On Mon, Jul 17, 2017 at 03:00:34PM +0200, Klemens Nanni wrote: > On Sun, Jul 16, 2017 at 09:09:44AM +, Robert Peichaer wrote: > > The rationale to picking the library versions before remounting was > > to keep the time window having rw /usr as small as possible. > > If that's deemed ok, I'm too

Re: Add machdep.lidaction=3 - powerdown laptop upon lid closing

2017-07-17 Thread Kevin Chadwick
What about powerup? Currently if u boot say an access point and close the lid to save a watt. I *believe*? you have to wait for sysctl.conf to be read before closing to avoid sleeping?

Re: rc: Use IFS when looking for carp interfaces

2017-07-17 Thread Robert Peichaer
On Mon, Jul 17, 2017 at 03:39:29PM +0200, Klemens Nanni wrote: > The Internal Field Seperator is meant for this so use it instead of > reading and stripping ':' again. > > Feedback? Comments? > > Index: rc > === > RCS file: /cvs/src/

Re: inteldrm(4) diff needs review and testing

2017-07-17 Thread Paul de Weerd
Hi Mark, On Sun, Jul 16, 2017 at 03:19:41PM +0200, Mark Kettenis wrote: | Can somebody test the following diff on Ivy Bridge or Haswell (Intel | HD Graphics 2500/4000/4600/4700/5000/5100/5200)? | | When I added support for the command parser, I took a bit of a | shortcut and implemented the hash

Re: [patch] fake pv drivers installation on xen

2017-07-17 Thread Maxim Khitrov
On Wed, Jan 18, 2017 at 2:16 PM, Dinar Talypov wrote: > I use Xenserver 7.0 with xencenter management console. > without it doesn't allow shutdown or reboot. > Anyway I'll try with hostctl. > > Thanks. Were you able to get this working with hostctl? I'm running OpenBSD 6.1 amd64 on XenServer 7.0.

Re: [patch] fake pv drivers installation on xen

2017-07-17 Thread Mike Belopuhov
On Mon, Jul 17, 2017 at 14:32 -0400, Maxim Khitrov wrote: > On Wed, Jan 18, 2017 at 2:16 PM, Dinar Talypov wrote: > > I use Xenserver 7.0 with xencenter management console. > > without it doesn't allow shutdown or reboot. > > Anyway I'll try with hostctl. > > > > Thanks. > > Were you able to get

newsyslog: skip invalid newsyslog.conf entries

2017-07-17 Thread Jeremie Courreges-Anglas
On Sat, Jul 15 2017, Jeremie Courreges-Anglas wrote: > "Todd C. Miller" writes: > >> On Sat, 15 Jul 2017 00:24:05 +0200, Jeremie Courreges-Anglas wrote: >> >>> Wouldn't it be better if we at least tried to properly free what was >>> allocated in parse_file()? >> >> I'm not sure it is worth it sin

Re: [patch] fake pv drivers installation on xen

2017-07-17 Thread Maxim Khitrov
On Mon, Jul 17, 2017 at 3:40 PM, Mike Belopuhov wrote: > On Mon, Jul 17, 2017 at 14:32 -0400, Maxim Khitrov wrote: >> On Wed, Jan 18, 2017 at 2:16 PM, Dinar Talypov wrote: >> > I use Xenserver 7.0 with xencenter management console. >> > without it doesn't allow shutdown or reboot. >> > Anyway I'l

Re: armv7 sunxi i2c+pmic(=shutdown -p)

2017-07-17 Thread Artturi Alm
On Sun, Jul 16, 2017 at 11:13:35PM +0200, Mark Kettenis wrote: > > Date: Sun, 9 Jul 2017 20:34:29 +0300 > > From: Artturi Alm > > > > Hi, > > > > revived the diff below, i2c tested via pmic's shutdown(), for working > > "shutdown -p now" operation. > > there was only two i2c's w/"status: 'okay'"

clang: emit trap padding between functions

2017-07-17 Thread Todd Mortimer
Hello tech, The patch below teaches clang to treat padding between functions differently than padding inside functions. Padding between functions is completely filled with trap instructions, and padding inside functions is padded as usual (trapsleds on X86, NOPs on everything else). This means tha

SIGBUS short mmap object

2017-07-17 Thread Alexander Bluhm
Hi, The tests regress/sys/kern/siginfo-fault accesses an mmap(2)ed file behind its end. Then it expects an SIGBUS, but gets an SIGSEGV. Related commit message is: Additionally, SIGBUS/BUS_ADRERR should be generated instead of SIGSEGV for access to file mapped pages that exceed the end of

Re: SIGBUS short mmap object

2017-07-17 Thread Theo de Raadt
The tests regress/sys/kern/siginfo-fault accesses an mmap(2)ed file behind its end. Then it expects an SIGBUS, but gets an SIGSEGV. Sorry I disagree. SIGSEGV is the right answer. It was not mapped. That is segmentation. A SIGBUS should be delivered when the mode of access is no

Re: newsyslog: skip invalid newsyslog.conf entries

2017-07-17 Thread Todd C. Miller
On Mon, 17 Jul 2017 22:21:36 +0200, Jeremie Courreges-Anglas wrote: > Here's a diff to skip invalid lines and return a meaningful error > status if meet one. A goto looked like the simplest way here because > of the nested for(;;) loop in parse_file(). > > While here, mention the exit status in

Re: armv7 _dcache_wbinv_all, or _dcache_wb_all ?

2017-07-17 Thread Artturi Alm
On Fri, Jul 14, 2017 at 03:33:32AM +0300, Artturi Alm wrote: > On Fri, Jul 14, 2017 at 03:28:25AM +0300, Artturi Alm wrote: > > Hi, > > > > i'm having hard time choosing, between this diff, and the one that > > renames it to for what it does, which would likely incur cleanup > > elsewhere. current

Re: armv7 sunxi i2c+pmic(=shutdown -p)

2017-07-17 Thread Juan Francisco Cantero Hurtado
On Sun, Jul 09, 2017 at 08:34:29PM +0300, Artturi Alm wrote: > Hi, > > revived the diff below, i2c tested via pmic's shutdown(), for working > "shutdown -p now" operation. > there was only two i2c's w/"status: 'okay'" in the FDT, so not all of > them do attach. > > related part of dmesg: > > com

intel.4: Typofix

2017-07-17 Thread Klemens Nanni
Add missing verb and period. Index: driver/xf86-video-intel/man/intel.man === RCS file: /cvs/xenocara/driver/xf86-video-intel/man/intel.man,v retrieving revision 1.9 diff -u -p -r1.9 intel.man --- driver/xf86-video-intel/man/intel.man

Re: intel.4: Typofix

2017-07-17 Thread Bryan Steele
On Tue, Jul 18, 2017 at 02:48:08AM +0200, Klemens Nanni wrote: > Add missing verb and period. > > Index: driver/xf86-video-intel/man/intel.man > === > RCS file: /cvs/xenocara/driver/xf86-video-intel/man/intel.man,v > retrieving revisi

[patch] enable calling chroot(2) after calling pledge(2)

2017-07-17 Thread Matt Miller
I would like to be able to call chroot(2) in a program that uses pledge(2). The following (untested) patch makes this possible as part of the "rpath" promise. Does this seem like a reasonable idea? --- sys/kern/kern_pledge.cThu Jun 29 00:10:07 2017 +++ sys/kern/kern_pledge.c.with_chroot

filt_soread() & socket lock

2017-07-17 Thread Martin Pieuchot
The socket checks in filt_soread() should be atomic. As found in the past week it is not possible to call solock() there because we're not allowed to sleep. However I'd like to prepare the function for future locking in order to keep the locking diff minimal. ok? Index: kern/uipc_socket.c =

soo_ioctl() & socket lock

2017-07-17 Thread Martin Pieuchot
Where soo_ioctl() modifies `so_state', `so_rcv' or `so_snd' it needs the socket lock. More fields might need the lock in the future but for the moment I'm concentrating on fields accessed in the TCP input path. ok? Index: kern/sys_socket.c

Extend the scope of socket lock in soo_stat()

2017-07-17 Thread Martin Pieuchot
`so_state' and `so_rcv' need to be checked atomically, so extend the scope of the lock. ok? Index: kern/sys_socket.c === RCS file: /cvs/src/sys/kern/sys_socket.c,v retrieving revision 1.30 diff -u -p -r1.30 sys_socket.c --- kern/sys_