Re: [patch] Add -z and -Z to apmd for automatic suspend/hibernate

2017-08-15 Thread Jesper Wallin
in the getopt-loop, as I would most likely do more harm than good trying to rewrite error(). The syslog(3) line, when suspending, was too long and corrected now. Jesper Wallin Index: usr.sbin/apmd/apmd.8 === RCS file: /cvs/src/usr

Re: [patch] Add -z and -Z to apmd for automatic suspend/hibernate

2017-08-16 Thread Jesper Wallin
On Tue, Aug 15, 2017 at 07:24:47PM -0400, Ted Unangst wrote: > this looks good. will commit soon. > > (one nit, i'll correct, is man page options are upper case first.) Noted for future reference. Thanks!

Re: [patch] Add -z and -Z to apmd for automatic suspend/hibernate

2017-08-13 Thread Jesper Wallin
but two (non functionality) nits > inline for future reference. Thanks for the feedback, greatly appreciated! I've done a few changes and also rewrote some of the manual, as my previous patch was lying. Jesper Wallin Index: usr.sbin/apmd/apmd.8

[patch] Add -z and -Z to apmd for automatic suspend/hibernate

2017-08-12 Thread Jesper Wallin
the specified percentage. Jesper Wallin Index: usr.sbin/apmd/apmd.8 === RCS file: /cvs/src/usr.sbin/apmd/apmd.8,v retrieving revision 1.47 diff -u -p -r1.47 apmd.8 --- usr.sbin/apmd/apmd.812 Feb 2015 14:03:49 - 1.47

[patch] Remove superfluous seek_filesize() from less

2017-09-16 Thread Jesper Wallin
Hi all, I was reading through the code for less/filename.c and noticed that the calling for seek_filesize() in filesize() is superfluous? A wild guess is that it's remains from when BAD_LSEEK was removed? Jesper Wallin Index: usr.bin/less/filename.c

[patch] ftp(1): change mtime for http/https links

2017-09-22 Thread Jesper Wallin
hing files. Jesper Wallin Index: fetch.c === RCS file: /cvs/src/usr.bin/ftp/fetch.c,v retrieving revision 1.163 diff -u -p -r1.163 fetch.c --- fetch.c 7 Mar 2017 08:00:23 - 1.163 +++ fetch.c 22 Sep 2017 19:5

Re: [patch] ftp(1): change mtime for http/https links

2017-09-23 Thread Jesper Wallin
Yeah, the SHA256.sig should be enough and just veify that everything is correct. If not, fetch the incorrect sets again. I just wanted to give a brief explaination on how I noticed this inconsistency. But yeah, thanks for pointing me to a better solution. On Sat, Sep 23, 2017 at 05:12:32AM

Re: [patch] ftp(1): change mtime for http/https links

2017-09-23 Thread Jesper Wallin
Yeah, fully understandable. I'll bump this once 6.2 is out. I'll apply the changes Philip suggested (thanks!) and then build a release to verify that everything works. On Fri, Sep 22, 2017 at 11:35:03PM +0100, Stuart Henderson wrote: > I like this, but it's rather late in the release cycle to

Re: [patch] ftp(1): change mtime for http/https links

2017-09-23 Thread Jesper Wallin
> On Sat, Sep 23, 2017 at 09:43:12AM +0200, Jesper Wallin wrote: > Yeah, fully understandable. I'll bump this once 6.2 is out. > > I'll apply the changes Philip suggested (thanks!) and then build a > release to verify that everything works. > > Woups, seems like I m

[patch] In malloc.conf(5), explain what options S includes/enables.

2017-09-17 Thread Jesper Wallin
Hi all, The mentioning of the S option in malloc.conf(5) won't actually tell the user what flags it enables. Rephrased this and list the options. Jesper Wallin Index: ./share/man/man5/malloc.conf.5 === RCS file: /cvs/src/share

Re: apmd: manual tweak

2017-10-16 Thread Jesper Wallin
On Mon, Oct 16, 2017 at 07:51:47PM +0200, Anton Lindqvist wrote: > On Mon, Oct 16, 2017 at 06:10:02PM +0100, Jason McIntyre wrote: > > On Mon, Oct 16, 2017 at 06:38:31PM +0200, Anton Lindqvist wrote: > > > > > > Good call. Without knowing too much about the apmd internals, here's a > > > first

[patch] Mention pf(4) and pf.conf(5) in afterboot(8)

2017-10-14 Thread Jesper Wallin
), when things like lpd(8), smtpd(8) and packages are. The patch below adds a small section, pointing the admin to the pf(4) and pf.conf(5) man pages. Jesper Wallin Index: afterboot.8 === RCS file: /cvs/src/share/man/man8/afterboot

Re: [patch] Remove local nameservers from dhclient.conf during an upgrade.

2017-09-08 Thread Jesper Wallin
ent away. But yeah, it's also stupid to bloat /bsd.rd with a fix for an uncommon cases like this. :-) Jesper Wallin

Re: [patch] Remove local nameservers from dhclient.conf during an upgrade.

2017-09-08 Thread Jesper Wallin
On Fri, Sep 08, 2017 at 12:29:57PM +0100, Raf Czlonka wrote: > > One small comment regarding your patch - since a good while, sed(1) > can do in place file edits so the above is a useless use of cat :^) > Hehe, that too. Thanks for pointing it out! It's been a long week with way too little

[patch] Remove local nameservers from dhclient.conf during an upgrade.

2017-09-08 Thread Jesper Wallin
name instead, before givig the https error, but seeing that host, dig or nslookup is unavailable, I decided to go with this instead. Jesper Wallin Index: distrib/miniroot/install.sub === RCS file: /cvs/src/distrib/miniroot/install.sub,v

[patch] kbd should give an error unless we're root

2017-08-29 Thread Jesper Wallin
or otherwise error out. Jesper Wallin Index: sbin/kbd/main.c === RCS file: /cvs/src/sbin/kbd/main.c,v retrieving revision 1.8 diff -u -p -r1.8 main.c --- sbin/kbd/main.c 30 Dec 2007 13:50:43 - 1.8 +++ sbin/kbd/main.c 29 Aug

Re: [patch] kbd should give an error unless we're root

2017-08-30 Thread Jesper Wallin
On Wed, Aug 30, 2017 at 08:14:16AM +0200, Jeremie Courreges-Anglas wrote: > > Well, the behavior of kbd depends on the permissions of the device > files, not on the fact that you're root or not. Maybe something like > this would be more appropriate? > Ah, you're absolutely right! No idea why,

Re: [patch] ftp(1): change mtime for http/https links

2017-09-24 Thread Jesper Wallin
On Sat, Sep 23, 2017 at 11:50:46PM +0200, Jesper Wallin wrote: > Woups, seems like I managed to break ftp(1) in the installer due to > pledge being a bit too tight. Here is an updated version of the patch > and with Philips changes as well. ...and hopefully a final version, sorry for

[patch] vmd.c: Keep the ownership when rebooting a VM

2017-09-24 Thread Jesper Wallin
the console, but it's not possible to stop the machine since the vmd owner is root. The patch below simply leave vm->vm_uid untouched if the keeptty is set. Jesper Wallin Index: vmd.c === RCS file: /cvs/src/usr.sbin/vmd/vmd.

Re: [patch] Remove superfluous seek_filesize() from less

2017-09-25 Thread Jesper Wallin
Shameless bump, now when we're out of beta. :-) On Sat, Sep 16, 2017 at 10:53:47PM +0200, Jesper Wallin wrote: > Hi all, > > I was reading through the code for less/filename.c and noticed that the > calling for seek_filesize() in filesize() is superfluous? A wild guess > is th

Re: [patch] Remove superfluous seek_filesize() from less

2017-09-25 Thread Jesper Wallin
On Mon, Sep 25, 2017 at 10:36:18AM +0200, Landry Breuil wrote: > On Mon, Sep 25, 2017 at 10:19:16AM +0200, Jesper Wallin wrote: > > Shameless bump, now when we're out of beta. :-) > > being out of beta means we're in release mode.. Oh! Of course, silly me. Apologize for the noise.

Re: [patch] Add -z and -Z to apmd for automatic suspend/hibernate

2017-08-28 Thread Jesper Wallin
On Tue, Aug 15, 2017 at 07:24:47PM -0400, Ted Unangst wrote: > this looks good. will commit soon. > > (one nit, i'll correct, is man page options are upper case first.) Sorry if this comes on as a bit pushy, but any news here? I don't know how 6.2-beta affects this, but maybe it would be nice to

[patch] Mention syspatch(8) in afterboot(8)

2017-09-05 Thread Jesper Wallin
Hi, Now when syspatch(8) is available and has been tested for a few months, perhaps mention it in afterboot(8) under the Errata section? Jesper Wallin Index: share/man/man8/afterboot.8 === RCS file: /cvs/src/share/man/man8

Re: [patch] vmd.c: Keep the ownership when rebooting a VM

2017-10-05 Thread Jesper Wallin
Hi again, Bumping this, as I assume it wasn't prioritized during the beta. On Sun, Sep 24, 2017 at 06:01:00PM +0200, Jesper Wallin wrote: > Hi all, > > If a machine is configured in vm.conf to have the owner of a regular > user, the ownership of the machine is lost upon reboot and

Re: [patch] ftp(1): change mtime for http/https links

2017-10-05 Thread Jesper Wallin
...and another bump. :-) *ping sthen@* On Sun, Sep 24, 2017 at 07:25:33PM +0200, Jesper Wallin wrote: > On Sat, Sep 23, 2017 at 11:50:46PM +0200, Jesper Wallin wrote: > > Woups, seems like I managed to break ftp(1) in the installer due to > > pledge being a bit too tight. Her

Re: [patch] Remove superfluous seek_filesize() from less

2017-10-05 Thread Jesper Wallin
...and a third bump, this time we're actually out of beta for real. ;-) On Mon, Sep 25, 2017 at 10:19:16AM +0200, Jesper Wallin wrote: > Shameless bump, now when we're out of beta. :-) > > On Sat, Sep 16, 2017 at 10:53:47PM +0200, Jesper Wallin wrote: > > Hi all, > > >

Re: [patch] add missing pledge to aucat(1).

2018-05-07 Thread Jesper Wallin
lso moved the pledge calls back to the main to make it easier to see what promises are set and where. I've still not been able to test this using MIDI devices, but everything else seems to work as far as I can tell. Regards, Jesper Wallin Index: aucat.c =

Re: [patch] add missing pledge to aucat(1).

2018-05-07 Thread Jesper Wallin
On Mon, May 07, 2018 at 03:30:19PM +0200, Jesper Wallin wrote: > I've still not been able to test this using MIDI devices, but everything > else seems to work as far as I can tell. As Theo (tb@) kindly pointed out to me off-list, aucat(1) supports the use of multiple -i and/or -o flags, wh

Re: [patch] add missing pledge to aucat(1).

2018-05-04 Thread Jesper Wallin
On Fri, May 04, 2018 at 12:20:02PM +0200, Alexandre Ratchov wrote: > AFAIU, only "sndio" is needed in offline(), because files are already > opened/created. Ah! Yeha, of course.. :-) Regards, Jesper Wallin Index: aucat.c ===

Re: [patch] add missing pledge to aucat(1).

2018-05-04 Thread Jesper Wallin
is meant to be used so far down in a program, but I drop all pledges except stdio and audio once sio_open(2) is finished. Regards, Jesper Wallin Index: aucat.c === RCS file: /cvs/src/usr.bin/aucat/aucat.c,v retrieving revision 1

Re: [patch] add missing pledge to aucat(1).

2018-05-06 Thread Jesper Wallin
se file headers, then configure the device. This is > not trivial. > Yeah, I noticed this as well when I tried to do the refactoring. Unfortunately, I've been a bit busy lately, but I might give this another try whenever I have the time. Regards, Jesper Wallin

Re: [patch] Skip background scan if bssid is set

2018-05-25 Thread Jesper Wallin
Sperling wrote: > On Sun, Apr 29, 2018 at 03:39:07AM +0200, Jesper Wallin wrote: > > Hi all, > > > > I recently learned that my AP behaves badly and I have packet loss when > > the background scan is running. > > Hi Jesper, > > Can you please check

[patch] add missing pledge to aucat(1).

2018-05-03 Thread Jesper Wallin
obvious. Regards, Jesper Wallin Index: aucat.c === RCS file: /cvs/src/usr.bin/aucat/aucat.c,v retrieving revision 1.172 diff -u -p -r1.172 aucat.c --- aucat.c 7 Nov 2017 11:39:24 - 1.172 +++ aucat.c 3 May 2018 19:13:14

Re: [PATCH] mv -P

2018-04-28 Thread Jesper Wallin
Hi, Adding a -P to your command might of course be easier, but since you have -P in rm(1) and -v was added recently, won't this do: mv newfile "$(rm -Pv oldfile)" Regards Jesper Wallin

[patch] Skip background scan if bssid is set

2018-04-28 Thread Jesper Wallin
the background scan has other use-cases that I'm not aware of, if so, I apologize in advance. The patch below simply check if a bssid is specified and if so, skip the background scan. Regards, Jesper Wallin Index: ieee80211.c === RCS file

Re: [patch] Skip background scan if bssid is set

2018-04-29 Thread Jesper Wallin
. Thanks for the pointers! Brainfart at my end due to lack of coffee. Regards, Jesper Wallin Index: ieee80211_input.c === RCS file: /cvs/src/sys/net80211/ieee80211_input.c,v retrieving revision 1.199 diff -u -p -r1.199 ieee8021

Re: [patch] Skip background scan if bssid is set

2018-04-29 Thread Jesper Wallin
However, when I set the bssid and it aborts the scan, the debug message of "end background scan" appears before the actual list of APs. Regards, Jesper Wallin Index: ieee80211_input.c === RCS file: /cvs/src/sys/net80211/ieee8

Re: [patch] Skip background scan if bssid is set

2018-04-29 Thread Jesper Wallin
er testing and get some sort of tcpdump or so from the Mac perhaps and see how OpenBSD behaves compared to another client. Regards, Jesper Wallin

[patch] make ifconfig report 'SIOCSIFMEDIA' when ioctl fails

2017-10-26 Thread Jesper Wallin
ithout any notices or errors, even if the command fails. Jesper Wallin Index: ifconfig.c === RCS file: /cvs/src/sbin/ifconfig/ifconfig.c,v retrieving revision 1.348 diff -u -p -r1.348 ifconfig.c --- ifconfig.c 29 Aug 2017 21:10:20 -

[patch] skip wait4() for power{up,down} in do_etc_file()

2018-04-05 Thread Jesper Wallin
t it also feels like a bug if apmd becomes unresponsive if one doesn't make apm execute in the background. The patch below adds the exception. Regards, Jesper Wallin Index: apmd.c === RCS file: /cvs/src/usr.sbin/apmd/apmd.c,v retrievi

[patch] urtwn(4): accept control frames when in monitor mode

2019-01-23 Thread Jesper Wallin
monitor up Check with tcpdump if you receive any control frames (ack, rts, cts): $ doas tcpdump -i urtwn0 -n -v -y IEEE802_11_RADIO Also, please make sure that the NIC works BSS ("normal") mode as well. Jesper Wallin Ind

Re: [patch] urtwn(4): accept control frames when in monitor mode

2019-01-24 Thread Jesper Wallin
On Thu, Jan 24, 2019 at 03:56:17PM +0800, Kevin Lo wrote: > > First off, thanks for the diff. Right, urtwn(4) doesn't see any control > frames in monitor mode, I think we also need to set R92C_RCR_ADF, > R92C_RCR_ACF, > and R92C_RCR_AMF bits in R92C_RCR to accept data/ctrl/mgmt frames

[patch] replace 'php5' with 'php' in ports(7)

2019-05-30 Thread Jesper Wallin
Hi all, Replace the reference of 'php5' with 'php' in ports(7) since 'php5' is deprecated and no longer exists in the ports tree. Jesper Wallin Index: share/man/man7/ports.7 === RCS file: /cvs/src/share/man/man7/ports.7,v

[patch] make bioctl(8) print the correct error

2019-05-24 Thread Jesper Wallin
_F_INVALID error. I guess the error could be moved to the function above, but then we don't know the device name. Jesper Wallin Index: sys/dev/softraid.c === RCS file: /cvs/src/sys/dev/softraid.c,v retrieving revision 1.394 diff -u

Re: [patch] ex/vi(1): wait with pledge until *after* ~/.nexrc is read

2019-05-21 Thread Jesper Wallin
ere fix a > lie that secure has an off parameter and inform the user that it can't > be turned off again. Nice! Your patch is more elegant and good point about not failing hard. Jesper Wallin

[patch] ex/vi(1): wait with pledge until *after* ~/.nexrc is read

2019-05-20 Thread Jesper Wallin
feels ugly and "too deep". Jesper Wallin Index: usr.bin/vi/common/main.c === RCS file: /cvs/src/usr.bin/vi/common/main.c,v retrieving revision 1.41 diff -u -p -r1.41 main.c --- usr.bin/vi/common/main.c10 Nov

The default value of kern.bufcachepercent is 20, not 80.

2019-05-05 Thread Jesper Wallin
Hi all, The kern.bufcachepercent entry in sysctl(2) still claims the default value is 80, even though it was changed to 20 recently. Index: ./lib/libc/sys/sysctl.2 === RCS file: /cvs/src/lib/libc/sys/sysctl.2,v retrieving revision

Re: iwm(4): fix ccmp decrypt edge cases

2019-08-13 Thread Jesper Wallin
) at pool_do_put+0x40 pool_put(81de50d0,fd823c04d7e0) at pool_put+0x5a futex_requeue(ed8068f5fd0,7fff,0,0,2) at futex_requeue+0x150 sys_futex(800043f0eaa8,800044682370,8000446823d0) at sys_futex+0xec syscall(800044682440) at syscall+0x389 Xsyscall(0,53,7f7d6b01,53,ed80c78e4d0,ed8068f5fd0) at Xsyscall+0x128 end of kernel end trace frame: 0x7f7d6a10, count: 251 --8<-- Jesper Wallin

[patch] Re-add 'proc' to vi(1) when running in secure mode.

2019-07-19 Thread Jesper Wallin
Hi all, When using vi(1) with secure mode (-S), both 'proc' and 'exec' are stripped from the pledge promise. This breaks the :pre[serve] command as it uses fork(2). This is broken on 6.4, 6.5 and -current. Re-add the 'proc' promise, even when running in secure mode. Jesper Wallin Index

Re: [patch] Re-add 'proc' to vi(1) when running in secure mode.

2019-07-19 Thread Jesper Wallin
On Fri, Jul 19, 2019 at 05:14:03PM -0400, Bryan Steele wrote: > I suspect that in secure/-S mode, the :pre[serve] should either be > disabled, or modified to stop calling sendmail. The mail it is sending > is purely advisory, and should be easy to disable. See common/recover.c. Oh, you're right.

Re: [patch] Re-add 'proc' to vi(1) when running in secure mode.

2019-07-22 Thread Jesper Wallin
hat the code actually does, before trying to fix it. 2. Test, test with ktrace, test again and then run more tests. ;-) Thanks a *lot* for your wise pointers and your thorough analysis! Jesper Wallin

Re: [patch] Re-add 'proc' to vi(1) when running in secure mode.

2019-07-22 Thread Jesper Wallin
On Mon, Jul 22, 2019 at 06:24:28PM +0200, Ingo Schwarze wrote: > But make sure that doesn't cause bugs to not get reported at all > because the process causes too much work or takes too long. :) > Oh yeah, no worries! :-)

Re: [patch] make bioctl(8) print the correct error

2019-06-16 Thread Jesper Wallin
*bump* On Fri, May 24, 2019 at 07:17:29PM +0200, Jesper Wallin wrote: > Hi tech@ > > A few days ago, I decided to set kern.securelevel=2 just for the sake > of locking down my machine as much as possible. Today I learned that > this affects bioctl(8) when trying to attach a crypt

[patch] relayd.conf.5, DHE params seems incorrect/outdated.

2020-05-01 Thread Jesper Wallin
ects "none", "legacy" or "auto". My guess is that tls_config_set_dheparams() has been updated and the manual for relayd.conf(5) has not. Here's a diff that hopefully solves that. Jesper Wallin Index: relayd.conf.5 ==

Re: [patch] dhclient(8) crashes with vm.malloc_conf=J

2020-07-07 Thread Jesper Wallin
On Tue, Jul 07, 2020 at 12:18:20PM +1000, David Gwynne wrote: > you might want to put the memset after the check to see if the malloc > failed... > Oh, wow... *goes to stand in the corner of shame*

Re: 11n Tx aggregation for iwm(4)

2020-06-27 Thread Jesper Wallin
Tested on a "Intel Dual Band Wireless-AC 9260" rev 0x29, msix (hw rev 0x320, fw ver 34.3125811985.0) I seem to be getting "iwm0: fatal firmware error" a few seconds after the 4-way handshake. I can send a few packets, so it sure connects and all, but then it fails shortly after. iwm0: begin

ssh(1), getrrsetbyname(3), SSHFP and DNSSEC

2020-07-17 Thread Jesper Wallin
) though. As a side note, I noticed that the default value of _res.options was the same value as RES_DEFAULT, so I changed it to RES_DEFAULT instead, for the sake of consistency. Thoughts? Yours, Jesper Wallin [0] https://marc.info/?t=14998384322=1=2 Index: include/resolv.h

Re: ssh(1), getrrsetbyname(3), SSHFP and DNSSEC

2020-07-29 Thread Jesper Wallin
On Wed, Jul 29, 2020 at 02:46:06PM +0200, Jeremie Courreges-Anglas wrote: > On Mon, Jul 27 2020, Jesper Wallin wrote: > > [...] > > > I still think the RES_USE_AD option might be a useful though, for when > > you want to decide on an application-l

Re: ssh(1), getrrsetbyname(3), SSHFP and DNSSEC

2020-07-27 Thread Jesper Wallin
On Mon, Jul 27, 2020 at 03:30:52AM +0200, Jeremie Courreges-Anglas wrote: > On Sun, Jul 26 2020, Jesper Wallin wrote: > > On Sat, Jul 25, 2020 at 02:01:06PM +0200, Jeremie Courreges-Anglas wrote: > >> > >> For those two reasons I think the feature should be opt-in. >

[patch] dhclient(8) crashes with vm.malloc_conf=J

2020-07-06 Thread Jesper Wallin
y that my fix is the prettiest, but I get an extra byte and zero out the buffer. Maybe someone has a more elegant fix for this. Yours, Jesper Wallin Index: clparse.c === RCS file: /cvs/src/sbin/dhclient/clparse.c,v retrieving revision 1.1

Re: ssh(1), getrrsetbyname(3), SSHFP and DNSSEC

2020-07-25 Thread Jesper Wallin
fected by it. Yours, Jesper Wallin

httpd: escape double-quotes in server logs

2020-12-15 Thread Jesper Wallin
ned that it would break the 'combined' log parsing for Webalizer and GoAccess. I have no idea how my patch would affect those, but we already encode backslash, newline and tabs the same way so I guess they handle it well. A more interesting question is, how do they handle double-quotes in the refe

Re: [patch] dhcpleased(8): No new lease when trunk(4) failover

2021-08-02 Thread Jesper Wallin
On Mon, Aug 02, 2021 at 08:28:00PM +0200, Florian Obser wrote: > On 2021-07-28 23:02 +02, Jesper Wallin wrote: > > Hi tech@ > > > > I've setup my machine to use trunk(4) with re(4) and iwm(4) as failover, > > to make life easier when switching between wired and wireless

Re: [patch] dhcpleased(8): No new lease when trunk(4) failover

2021-08-02 Thread Jesper Wallin
On Mon, Aug 02, 2021 at 11:36:47PM +0200, Jesper Wallin wrote: > On Mon, Aug 02, 2021 at 08:28:00PM +0200, Florian Obser wrote: > > On 2021-07-28 23:02 +02, Jesper Wallin wrote: > > > Hi tech@ > > > > > > I've setup my machine to use trunk(4) with re(4) and iw

[patch] dhcpleased(8): No new lease when trunk(4) failover

2021-07-28 Thread Jesper Wallin
Hi tech@ I've setup my machine to use trunk(4) with re(4) and iwm(4) as failover, to make life easier when switching between wired and wireless networking. The wired network at home is on a different subnet from the wireless network, so when I unplug the cable, a DHCP request is required to get

Re: [patch] pool_sethardlimit(9): Actually print warnmess to console

2021-09-01 Thread Jesper Wallin
Ping? On Sat, Aug 21, 2021 at 08:21:55PM +0200, Jesper Wallin wrote: > Hi tech@ > > I'm trying to add log message when the pf(4) state table becomes > exhausted/full. After reading the code, I learned that it's using > pool_sethardlimit(9) to manage the resources. &g

[patch] pool_sethardlimit(9): Actually print warnmess to console

2021-08-21 Thread Jesper Wallin
Hi tech@ I'm trying to add log message when the pf(4) state table becomes exhausted/full. After reading the code, I learned that it's using pool_sethardlimit(9) to manage the resources. Conveniently, pool_sethardlimit(9) takes the argument warnmess, which is printed to the console when the