Re: Use km_alloc(9) in drm

2022-02-06 Thread Sebastien Marie
On Sat, Feb 05, 2022 at 11:44:03AM +0100, Mark Kettenis wrote: > We want to get rid of the uvm_km_valloc() interfaces in favour of > km_alloc(). This changes the calls in drm(4) over. The kv_physwait > struct is made static to prevent collission with a symbol in > vm_machdep.c on some

Re: Use km_alloc(9) in drm

2022-02-06 Thread Jonathan Gray
On Sat, Feb 05, 2022 at 02:26:48PM +0100, Mark Kettenis wrote: > > Date: Sat, 5 Feb 2022 22:34:10 +1100 > > From: Jonathan Gray > > > > On Sat, Feb 05, 2022 at 11:44:03AM +0100, Mark Kettenis wrote: > > > We want to get rid of the uvm_km_valloc() interfaces in favour of > > > km_alloc(). This

Re: head(1): check for stdio errors

2022-02-06 Thread Scott Cheloha
> On Feb 6, 2022, at 20:07, Todd C. Miller wrote: > > Since the input is opened read-only I don't see the point in checking > the fclose() return value. However, if you are going to do so, you > might as well combine it with the ferror() check. E.g. > >if (ferror(fp) || fclose(fp) ==

Re: hardware checksum ix and ixl

2022-02-06 Thread David Gwynne
On Fri, Feb 04, 2022 at 09:29:56PM +1000, David Gwynne wrote: > On Fri, Jan 28, 2022 at 05:26:01PM +0100, Alexander Bluhm wrote: > > On Wed, Jan 26, 2022 at 11:05:51AM +0100, Claudio Jeker wrote: > > > On Wed, Jan 26, 2022 at 01:29:42AM +0100, Alexander Bluhm wrote: > > > > Hi, > > > > > > > >

Re: head(1): check for stdio errors

2022-02-06 Thread Todd C . Miller
Since the input is opened read-only I don't see the point in checking the fclose() return value. However, if you are going to do so, you might as well combine it with the ferror() check. E.g. if (ferror(fp) || fclose(fp) == EOF) { warn("%s", name); status

head(1): check for stdio errors

2022-02-06 Thread Scott Cheloha
Add missing stdio error checks to head(1): - Output errors are terminal. The output is always stdout. - Input errors yield a warning and cause the program to fail gracefully. - Restructure the getc(3)/putchar(3) loop in head_file() to accomodate checking for errors. ok? P.S.

Re: Ship ubsan_minimal library in base?

2022-02-06 Thread Greg Steuck
Greg Steuck writes: > Sweet, let's look at the two patches below. No sets yet. The trivial change below produced a comp70.tgz with the new library after `make release` on amd64. >From 7bbf84dfa72b4417fec5bf337fd2fdd310f70205 Mon Sep 17 00:00:00 2001 From: Greg Steuck Date: Sun, 6 Feb 2022

Re: Add rtable capability to login.conf

2022-02-06 Thread Ted Unangst
On 2022-02-05, Matthew Martin wrote: > On Sat, Jan 29, 2022 at 06:25:32PM -0600, Matthew Martin wrote: > > On Sat, Jan 29, 2022 at 07:10:00PM -0500, Ted Unangst wrote: > > > I believe it would be better to add setrtable to id pledge. > > ping > > Also are there any opinions on adding

Re: wskbd_set_mixervolume

2022-02-06 Thread Alexandre Ratchov
On Sat, Feb 05, 2022 at 06:30:43PM +0100, Claudio Jeker wrote: > On Sat, Feb 05, 2022 at 12:28:08PM +0100, Mark Kettenis wrote: > > > Date: Sat, 5 Feb 2022 09:29:42 +0100 > > > From: Anton Lindqvist > > > > > > Hi, > > > I recently got a USB headset with physical volume buttons, handled by > > >

Re: wskbd_set_mixervolume

2022-02-06 Thread Alexandre Ratchov
On Sat, Feb 05, 2022 at 09:29:42AM +0100, Anton Lindqvist wrote: > Hi, > I recently got a USB headset with physical volume buttons, handled by > ucc(4). However, after enabling the device in sndiod the volume buttons > does not cause the volume to change. Turns out wskbd_set_mixervolume() > is