On Thu, May 19, 2016 at 02:03:43AM +0200, Alexander Bluhm wrote: > Hi, > > All msg buf counters are long, so lmin(9) should be used here. > > ok?
Looks good to me. > > bluhm > > Index: kern/subr_log.c > =================================================================== > RCS file: /cvs/src/sys/kern/subr_log.c,v > retrieving revision 1.43 > diff -u -p -r1.43 subr_log.c > --- kern/subr_log.c 18 May 2016 23:42:12 -0000 1.43 > +++ kern/subr_log.c 19 May 2016 00:00:23 -0000 > @@ -145,7 +145,7 @@ msgbuf_putchar(struct msgbuf *mbp, const > return; > > mbp->msg_bufc[mbp->msg_bufx++] = c; > - mbp->msg_bufl = min(mbp->msg_bufl+1, mbp->msg_bufs); > + mbp->msg_bufl = lmin(mbp->msg_bufl+1, mbp->msg_bufs); > if (mbp->msg_bufx < 0 || mbp->msg_bufx >= mbp->msg_bufs) > mbp->msg_bufx = 0; > /* If the buffer is full, keep the most recent data. */ >
