FreeBSD source tree https://cgit-dev.freebsd.org/src-test2/atom?h=release%2F5.2.1_cvs 2004-02-23T15:32:56Z 2004-02-23T15:32:56Z cvs2svn cvs2svn@FreeBSD.org 2004-02-23T15:32:56Z urn:sha1:84a7b99b4e02a260bf7ff9b540032f4cb7bc612a 'RELENG_5_2_1_RELEASE'. This commit was manufactured to restore the state of the 5.2.1-RELEASE image. Releases prior to 5.3-RELEASE are omitting the secure/ and crypto/ subdirs. 2003-09-03T05:35:37Z Kenneth D. Merry ken@FreeBSD.org 2003-09-03T05:35:37Z urn:sha1:e0254f1068a478799796d627e0dd744f5d45133c Pointed out by: njl MFC after: 3 days 2003-09-03T04:46:28Z Kenneth D. Merry ken@FreeBSD.org 2003-09-03T04:46:28Z urn:sha1:cb32189e239c6c7f43369390b08dd504816ce4db out of cdregister() and daregister(), which are run from interrupt context. The sysctl code does blocking mallocs (M_WAITOK), which causes problems if malloc(9) actually needs to sleep. The eventual fix for this issue will involve moving the CAM probe process inside a kernel thread. For now, though, I have fixed the issue by moving dynamic sysctl variable creation for these two drivers to a task queue running in a kernel thread. The existing task queues (taskqueue_swi and taskqueue_swi_giant) run in software interrupt handlers, which wouldn't fix the problem at hand. So I have created a new task queue, taskqueue_thread, that runs inside a kernel thread. (It also runs outside of Giant -- clients must explicitly acquire and release Giant in their taskqueue functions.) scsi_cd.c: Remove sysctl variable creation code from cdregister(), and move it to a new function, cdsysctlinit(). Queue cdsysctlinit() to the taskqueue_thread taskqueue once we have fully registered the cd(4) driver instance. scsi_da.c: Remove sysctl variable creation code from daregister(), and move it to move it to a new function, dasysctlinit(). Queue dasysctlinit() to the taskqueue_thread taskqueue once we have fully registered the da(4) instance. taskqueue.h: Declare the new taskqueue_thread taskqueue, update some comments. subr_taskqueue.c: Create the new kernel thread taskqueue. This taskqueue runs outside of Giant, so any functions queued to it would need to explicitly acquire/release Giant if they need it. cd.4: Update the cd(4) man page to talk about the minimum command size sysctl/loader tunable. Also note that the changer variables are available as loader tunables as well. da.4: Update the da(4) man page to cover the retry_count, default_timeout and minimum_cmd_size sysctl variables/loader tunables. Remove references to /dev/r???, they aren't used any longer. cd.9: Update the cd(9) man page to describe the CD_Q_10_BYTE_ONLY quirk. taskqueue.9: Update the taskqueue(9) man page to describe the new thread task queue, and the taskqueue_swi_giant queue. MFC after: 3 days 2002-03-15T12:04:49Z Ruslan Ermilov ru@FreeBSD.org 2002-03-15T12:04:49Z urn:sha1:2eff01c7e0d668ac329abfbd0ee6634751d0ddde .Bd erroneously defaulted to -compact mode in the SYNOPSIS section. 2001-12-26T23:14:04Z Chad David davidc@FreeBSD.org 2001-12-26T23:14:04Z urn:sha1:f16b3c0de48d0b845357e7cca843f645bc3117e3 the current state of the system. Approved by: alfred 2001-10-01T16:09:29Z Ruslan Ermilov ru@FreeBSD.org 2001-10-01T16:09:29Z urn:sha1:32eef9aeb1f39a1623cea55da147c89abbd5b9a5 2001-07-09T09:54:33Z Dima Dorfman dd@FreeBSD.org 2001-07-09T09:54:33Z urn:sha1:70d51341bf6ad3be0ea5b02ef489691d80719249 2000-11-22T16:11:48Z Ruslan Ermilov ru@FreeBSD.org 2000-11-22T16:11:48Z urn:sha1:b92a189eb9b32f7e9db74343d46434b554a67d04 2000-05-28T16:53:50Z Doug Rabson dfr@FreeBSD.org 2000-05-28T16:53:50Z urn:sha1:823234556fb17f72e92d5616c111dfeadc15b92a