Re: USB4BSD release candidate number 3 - request for review

2008-11-09 Thread Hans Petter Selasky
On Monday 10 November 2008, kevin wrote: > Hans Petter Selasky wrote: > > On Sunday 09 November 2008, kevin wrote: > >> It's a bug? > >> /boot/loader.conf: > >> zfs_load="YES" > >> vm.kmem_size_max="2048M" > >> vm.kmem_size="2048M" > >> vfs.zfs.zil_disable=1 > >> vfs.zfs.prefetch_disable=1 > >> hw.

usb/128745: zyd teoretically supported usb device makes kernel panic

2008-11-09 Thread aGaTHoS
>Number: 128745 >Category: usb >Synopsis: zyd teoretically supported usb device makes kernel panic >Confidential: no >Severity: serious >Priority: medium >Responsible:freebsd-usb >State: open >Quarter: >Keywords: >Date-Required: >Class:

Re: USB4BSD release candidate number 3 - request for review

2008-11-09 Thread kevin
Hans Petter Selasky wrote: On Sunday 09 November 2008, kevin wrote: It's a bug? /boot/loader.conf: zfs_load="YES" vm.kmem_size_max="2048M" vm.kmem_size="2048M" vfs.zfs.zil_disable=1 vfs.zfs.prefetch_disable=1 hw.psm.synaptics_support=1 legal.intel_iwn.license_ack=1 acpi_ibm_load="YES" snd_hda

Re: USB4BSD release candidate number 3 - request for review

2008-11-09 Thread Paul B. Mahol
On 11/9/08, Hans Petter Selasky <[EMAIL PROTECTED]> wrote: > On Saturday 08 November 2008, Paul B. Mahol wrote: >> On 11/7/08, Hans Petter Selasky <[EMAIL PROTECTED]> wrote: >> > On Friday 07 November 2008, Paul B. Mahol wrote: >> >> On 11/7/08, M. Warner Losh <[EMAIL PROTECTED]> wrote: >> >> > : >

Re: Patch to convert usb2 to use cdev

2008-11-09 Thread Hans Petter Selasky
Hi Rick, On Sunday 09 November 2008, Rink Springer wrote: > On Sun, Nov 09, 2008 at 05:28:35PM +0100, Rink Springer wrote: > > I think it makes sense to say that if /dev/ugenX.Y is opened, you > > shouldn't be able to open /dev/ugenX.Y.Z, right? However, what happends > > if /dev/ugenX.Y.Z is open

Re: Patch to convert usb2 to use cdev

2008-11-09 Thread Hans Petter Selasky
On Sunday 09 November 2008, Rink Springer wrote: > Hi Hans, > > On Sun, Nov 09, 2008 at 05:00:05PM +0100, Hans Petter Selasky wrote: > > It is not the same endpoint, but multiple different endpoints on the same > > device through the same file handle. Instead of one endpoint per file, > > you have

Re: Patch to convert usb2 to use cdev

2008-11-09 Thread Rink Springer
On Sun, Nov 09, 2008 at 05:28:35PM +0100, Rink Springer wrote: > I think it makes sense to say that if /dev/ugenX.Y is opened, you > shouldn't be able to open /dev/ugenX.Y.Z, right? However, what happends > if /dev/ugenX.Y.Z is opened? I'd think that opening /dev/ugenX.Y would > be fine, but any io

Re: Patch to convert usb2 to use cdev

2008-11-09 Thread Rink Springer
Hi Hans, On Sun, Nov 09, 2008 at 05:00:05PM +0100, Hans Petter Selasky wrote: > It is not the same endpoint, but multiple different endpoints on the same > device through the same file handle. Instead of one endpoint per file, you > have one file handle for each set of endpoints you need for you

Re: Patch to convert usb2 to use cdev

2008-11-09 Thread Hans Petter Selasky
Hi Rick, > but why would you ever want to be able > to read/write the same endpoint from multiple processes? > The control endpoint is a special endpoint which is shared and might be accessed from multiple process. Else you are right that only one process should access and endpoint at a time.

Re: Patch to convert usb2 to use cdev

2008-11-09 Thread Hans Petter Selasky
Hi Rick, On Sunday 09 November 2008, Rink Springer wrote: > Hi Hans, ... > > > 3) You need to solve the problem about a per-open-call context > > for /dev/ugenX.Y . This device is supposed to be cloneable, that means > > multiple processes are allowed to open it and establish independant > > conne

Re: Patch to convert usb2 to use cdev

2008-11-09 Thread Rink Springer
Hi Hans, On Sun, Nov 09, 2008 at 03:15:25PM +0100, Hans Petter Selasky wrote: > 1) You don't have to create an alias in "usb2_fifo_attach". The > alias /dev/usbX.Y.Z.T is mostly for internal usage. > > + make_dev_alias(f_sc->dev, buf); OK, I'll remove it - I assumed it might be us

Re: Patch to convert usb2 to use cdev

2008-11-09 Thread Hans Petter Selasky
Hi Rick, After going through your patch I have a feeling you quite well understand how USB2 is working with regard to the file system. Some issues: 1) You don't have to create an alias in "usb2_fifo_attach". The alias /dev/usbX.Y.Z.T is mostly for internal usage. + make_dev_alia

Patch to convert usb2 to use cdev

2008-11-09 Thread Rink Springer
Hi everyone, I've made a patch, available at http://rink.nu/tmp/usb-cdev.diff, which removes the custom "/dev/usb " device, associated event handlers and custom ownership/permissions structures and converts the whole deal to use make_dev(9) and friends. The end result is that every USB device will

Re: USB4BSD release candidate number 3 - request for review

2008-11-09 Thread Hans Petter Selasky
On Saturday 08 November 2008, Paul B. Mahol wrote: > On 11/7/08, Hans Petter Selasky <[EMAIL PROTECTED]> wrote: > > On Friday 07 November 2008, Paul B. Mahol wrote: > >> On 11/7/08, M. Warner Losh <[EMAIL PROTECTED]> wrote: > >> > : > After some time it will appear but will start attaching and > >>

Re: USB4BSD release candidate number 3 - request for review

2008-11-09 Thread Hans Petter Selasky
On Sunday 09 November 2008, kevin wrote: > It's a bug? > /boot/loader.conf: > zfs_load="YES" > vm.kmem_size_max="2048M" > vm.kmem_size="2048M" > vfs.zfs.zil_disable=1 > vfs.zfs.prefetch_disable=1 > hw.psm.synaptics_support=1 > legal.intel_iwn.license_ack=1 > acpi_ibm_load="YES" > snd_hda_load="YES"