CVS commit: [netbsd-9] src/sys/kern

["Martin Husemann" <martin%netbsd.org@localhost>]

Module Name:    src
Committed By:   martin
Date:           Sat Apr 12 12:18:43 UTC 2025

Modified Files:
        src/sys/kern [netbsd-9]: subr_log.c

Log Message:
Pull up following revision(s) (requested by rin in ticket #1939):

        sys/kern/subr_log.c: revision 1.64
        sys/kern/subr_log.c: revision 1.65
        sys/kern/subr_log.c: revision 1.66

logread: Drop unnecessary (int) cast for uiomove(9)'s count
which was changed to size_t back to 2002!
https://cvsweb.netbsd.org/bsdweb.cgi/src/sys/sys/systm.h#rev1.149

No practical problem is expected although.

logread: Use ulmin(9) for long v.s. size_t comparison
instead of uimin(9).
No real harm is expected IIUC.

logread: Stop reading msgbuf without log_lock being held
Idea taken from FreeBSD.

XXX
The size of temporal buffer on stack (== 128) is also taken from FreeBSD.
We are not very sure whether this value is optimal or not. But we don't
expect performance regression from this choice.

No new regression for full ATF run on amd64 with DIAGNOSTIC+LOCKDEBUG
kernel and rump kernel.

Authored by knakahara@.


To generate a diff of this commit:
cvs rdiff -u -r1.59 -r1.59.4.1 src/sys/kern/subr_log.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.