Re: CVS commit: src/sys/dev

2020-03-02 Thread Martin Husemann 
On Tue, Mar 03, 2020 at 03:26:47PM +1100, matthew green wrote:
> without really understanding, it seems that there should be
> a uhid ioctl to enable this mode, and then your userland code
> sets it, instead of this hack.

Or make uhid not attach at all on FIDO devices and instead use ugen from
userland?

Or solve the long standing "every usb devices should be able to eject
its driver and be used as ugen instead" problem, see also the various
JTAG devices showing up as dual uftdi and you need to override in your
kernel config like:

# GuruPlug JTAG debug interface
ugenif* at uhub? vendor 0x9e88 product 0x9e8f interface 0 flags 1


Most devices go via uhid to make userland access easy on windows, but for us
it makes no real difference.

Martin

re: CVS commit: src/sys/dev

2020-03-02 Thread matthew green 
"Christos Zoulas" writes:
> Module Name:  src
> Committed By: christos
> Date: Mon Mar  2 18:15:29 UTC 2020
> 
> Modified Files:
>   src/sys/dev/hid: hid.h
>   src/sys/dev/usb: uhid.c
> 
> Log Message:
> Add fido constants, and turn hid "raw" mode for fido devices.

this is gross.

please don't put device specific stuff into uhid like this.

without really understanding, it seems that there should be
a uhid ioctl to enable this mode, and then your userland code
sets it, instead of this hack.

also, please actually explain what this "raw" mode means.  is
there perhaps a way to avoid this entirely?  what's the real
underlying need here..?


.mrg.

Re: CVS commit: src/sys/kern

2020-03-02 Thread Taylor R Campbell 
> Date: Tue, 3 Mar 2020 12:41:32 +0900
> From: Rin Okuyama 
> 
> On 2020/03/03 1:00, Taylor R Campbell wrote:
> > Include kern_crashme.c in non-DEBUG kernels.
> > 
> > This is useful for simulating crashes in production to test failover.
> 
> I like this.
> 
> Could you please send pullup request to netbsd-9, or can I?

Go for it!

Re: CVS commit: src/sys/kern

2020-03-02 Thread Rin Okuyama 

Hi,

On 2020/03/03 1:00, Taylor R Campbell wrote:

Module Name:src
Committed By:   riastradh
Date:   Mon Mar  2 16:00:54 UTC 2020

Modified Files:
src/sys/kern: files.kern

Log Message:
Include kern_crashme.c in non-DEBUG kernels.

This is useful for simulating crashes in production to test failover.


To generate a diff of this commit:
cvs rdiff -u -r1.43 -r1.44 src/sys/kern/files.kern

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


I like this.

Could you please send pullup request to netbsd-9, or can I?

Thanks,
rin