From owner-freebsd-scsi@FreeBSD.ORG Mon Oct 18 11:07:06 2010 Return-Path: Delivered-To: freebsd-scsi@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 2300010656E4 for ; Mon, 18 Oct 2010 11:07:06 +0000 (UTC) (envelope-from owner-bugmaster@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 103BE8FC08 for ; Mon, 18 Oct 2010 11:07:06 +0000 (UTC) Received: from freefall.freebsd.org (localhost [127.0.0.1]) by freefall.freebsd.org (8.14.4/8.14.4) with ESMTP id o9IB75Hr029438 for ; Mon, 18 Oct 2010 11:07:05 GMT (envelope-from owner-bugmaster@FreeBSD.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.4/8.14.4/Submit) id o9IB75uj029436 for freebsd-scsi@FreeBSD.org; Mon, 18 Oct 2010 11:07:05 GMT (envelope-from owner-bugmaster@FreeBSD.org) Date: Mon, 18 Oct 2010 11:07:05 GMT Message-Id: <201010181107.o9IB75uj029436@freefall.freebsd.org> X-Authentication-Warning: freefall.freebsd.org: gnats set sender to owner-bugmaster@FreeBSD.org using -f From: FreeBSD bugmaster To: freebsd-scsi@FreeBSD.org Cc: Subject: Current problem reports assigned to freebsd-scsi@FreeBSD.org X-BeenThere: freebsd-scsi@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SCSI subsystem List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 18 Oct 2010 11:07:06 -0000 Note: to view an individual PR, use: http://www.freebsd.org/cgi/query-pr.cgi?pr=(number). The following is a listing of current problems submitted by FreeBSD users. These represent problem reports covering all versions including experimental development code and obsolete releases. S Tracker Resp. Description -------------------------------------------------------------------------------- o docs/151336 scsi Missing documentation of scsi_ and ata_ functions in l o kern/149502 scsi [mpt] Latent buglet in debug print code o kern/148083 scsi [aac] Strange device reporting o kern/147704 scsi [mpt] sys/dev/mpt: new chip revision, partially unsupp o kern/146287 scsi [ciss] ciss(4) cannot see more than one SmartArray con o kern/145768 scsi [mpt] can't perform I/O on SAS based SAN disk in freeb o kern/144648 scsi [aac] Strange values of speed and bus width in dmesg o kern/144301 scsi [ciss] [hang] HP proliant server locks when using ciss o kern/142351 scsi [mpt] LSILogic driver performance problems o kern/141934 scsi [cam] [patch] add support for SEAGATE DAT Scopion 130 o kern/134488 scsi [mpt] MPT SCSI driver probes max. 8 LUNs per device o kern/132250 scsi [ciss] ciss driver does not support more then 15 drive o kern/132206 scsi [mpt] system panics on boot when mirroring and 2nd dri o kern/130621 scsi [mpt] tranfer rate is inscrutable slow when use lsi213 o kern/129602 scsi [ahd] ahd(4) gets confused and wedges SCSI bus o kern/128452 scsi [sa] [panic] Accessing SCSI tape drive randomly crashe o kern/128245 scsi [scsi] "inquiry data fails comparison at DV1 step" [re o kern/127927 scsi [isp] isp(4) target driver crashes kernel when set up o kern/124667 scsi [amd] [panic] FreeBSD-7 kernel page faults at amd-scsi o kern/123674 scsi [ahc] ahc driver dumping o sparc/121676 scsi [iscsi] iscontrol do not connect iscsi-target on sparc o kern/120487 scsi [sg] scsi_sg incompatible with scanners o kern/120247 scsi [mpt] FreeBSD 6.3 and LSI Logic 1030 = only 3.300MB/s o kern/119668 scsi [cam] [patch] certain errors are too verbose comparing o kern/114597 scsi [sym] System hangs at SCSI bus reset with dual HBAs o kern/110847 scsi [ahd] Tyan U320 onboard problem with more than 3 disks o kern/99954 scsi [ahc] reading from DVD failes on 6.x [regression] o kern/94838 scsi Kernel panic while mounting SD card with lock switch o o kern/92798 scsi [ahc] SCSI problem with timeouts o kern/90282 scsi [sym] SCSI bus resets cause loss of ch device o kern/76178 scsi [ahd] Problem with ahd and large SCSI Raid system o kern/74627 scsi [ahc] [hang] Adaptec 2940U2W Can't boot 5.3 s kern/61165 scsi [panic] kernel page fault after calling cam_send_ccb o kern/60641 scsi [sym] Sporadic SCSI bus resets with 53C810 under load o kern/60598 scsi wire down of scsi devices conflicts with config s kern/57398 scsi [mly] Current fails to install on mly(4) based RAID di o kern/52638 scsi [panic] SCSI U320 on SMP server won't run faster than o kern/44587 scsi dev/dpt/dpt.h is missing defines required for DPT_HAND o kern/40895 scsi wierd kernel / device driver bug o kern/39388 scsi ncr/sym drivers fail with 53c810 and more than 256MB m o kern/35234 scsi World access to /dev/pass? (for scanner) requires acce 41 problems total. From owner-freebsd-scsi@FreeBSD.ORG Tue Oct 19 22:11:31 2010 Return-Path: Delivered-To: freebsd-scsi@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 1233) id E99EF1065673; Tue, 19 Oct 2010 22:11:31 +0000 (UTC) Date: Tue, 19 Oct 2010 22:11:31 +0000 From: Alexander Best To: freebsd-scsi@freebsd.org Message-ID: <20101019221131.GA75368@freebsd.org> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="h31gzZEtNLTqOjlF" Content-Disposition: inline Subject: set kern.cam.scsi_delay to 2000ms on all platforms X-BeenThere: freebsd-scsi@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SCSI subsystem List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 Oct 2010 22:11:32 -0000 --h31gzZEtNLTqOjlF Content-Type: text/plain; charset=us-ascii Content-Disposition: inline hi there, i've posted this patch on freebsd-hackers@, but jhb@ recommended i should also send it to freebsd-scsi@. basically what this patch does is to populate kern.cam.scsi_delay=2000 as the default value throughout the entire source tree. in addition to that it gets rid of some legacy Makefile entries, where kern.cam.scsi_delay was set to 150000. cheers. alex -- a13x --h31gzZEtNLTqOjlF Content-Type: text/plain; charset=us-ascii Content-Disposition: attachment; filename="SCSI_DELAY.diff2" diff --git a/share/man/man4/scsi.4 b/share/man/man4/scsi.4 index ad52663..3087aec 100644 --- a/share/man/man4/scsi.4 +++ b/share/man/man4/scsi.4 @@ -24,7 +24,7 @@ .\" SUCH DAMAGE. .\" .\" $FreeBSD$ -.Dd March 4, 2010 +.Dd October 19, 2010 .Dt CAM 4 .Os .Sh NAME @@ -47,7 +47,7 @@ .Cd "options CAM_MAX_HIGHPOWER=4" .Cd "options SCSI_NO_SENSE_STRINGS" .Cd "options SCSI_NO_OP_STRINGS" -.Cd "options SCSI_DELAY=8000" +.Cd "options SCSI_DELAY=2000" .Sh DESCRIPTION The .Nm @@ -116,7 +116,7 @@ Enabling this option for normal use is not recommended, since it slows debugging of .Tn SCSI problems. -.It Dv SCSI_DELAY=8000 +.It Dv SCSI_DELAY=2000 This is the .Tn SCSI "bus settle delay." diff --git a/sys/amd64/conf/GENERIC b/sys/amd64/conf/GENERIC index 3bc6299..df26ea6 100644 --- a/sys/amd64/conf/GENERIC +++ b/sys/amd64/conf/GENERIC @@ -49,7 +49,6 @@ options COMPAT_FREEBSD4 # Compatible with FreeBSD4 options COMPAT_FREEBSD5 # Compatible with FreeBSD5 options COMPAT_FREEBSD6 # Compatible with FreeBSD6 options COMPAT_FREEBSD7 # Compatible with FreeBSD7 -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI options KTRACE # ktrace(1) support options STACK # stack(9) support options SYSVSHM # SYSV-style shared memory diff --git a/sys/amd64/conf/XENHVM b/sys/amd64/conf/XENHVM index 51f1256..47cdc83 100644 --- a/sys/amd64/conf/XENHVM +++ b/sys/amd64/conf/XENHVM @@ -50,7 +50,6 @@ options COMPAT_FREEBSD4 # Compatible with FreeBSD4 options COMPAT_FREEBSD5 # Compatible with FreeBSD5 options COMPAT_FREEBSD6 # Compatible with FreeBSD6 options COMPAT_FREEBSD7 # Compatible with FreeBSD7 -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI options KTRACE # ktrace(1) support options STACK # stack(9) support options SYSVSHM # SYSV-style shared memory diff --git a/sys/arm/conf/BWCT b/sys/arm/conf/BWCT index 0fb3b87..a2ecc07 100644 --- a/sys/arm/conf/BWCT +++ b/sys/arm/conf/BWCT @@ -57,7 +57,6 @@ options BOOTP #options CD9660 #ISO 9660 Filesystem #options PROCFS #Process filesystem (requires PSEUDOFS) options PSEUDOFS #Pseudo-filesystem framework -#options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI #options KTRACE #ktrace(1) support options SYSVSHM #SYSV-style shared memory options SYSVMSG #SYSV-style message queues diff --git a/sys/arm/conf/CNS11XXNAS b/sys/arm/conf/CNS11XXNAS index 76db42b..b14b1b2 100644 --- a/sys/arm/conf/CNS11XXNAS +++ b/sys/arm/conf/CNS11XXNAS @@ -74,7 +74,6 @@ options MSDOSFS #MSDOS Filesystem #options CD9660 #ISO 9660 Filesystem #options PROCFS #Process filesystem (requires PSEUDOFS) options PSEUDOFS #Pseudo-filesystem framework -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI options KTRACE #ktrace(1) support options SYSVSHM #SYSV-style shared memory options SYSVMSG #SYSV-style message queues diff --git a/sys/arm/conf/CRB b/sys/arm/conf/CRB index 2afd080..ff7fc1d 100644 --- a/sys/arm/conf/CRB +++ b/sys/arm/conf/CRB @@ -49,7 +49,6 @@ options NFS_ROOT #NFS usable as /, requires NFSCLIENT options CD9660 #ISO 9660 Filesystem #options PROCFS #Process filesystem (requires PSEUDOFS) options PSEUDOFS #Pseudo-filesystem framework -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI options KTRACE #ktrace(1) support options INTR_FILTER options SYSVSHM #SYSV-style shared memory diff --git a/sys/arm/conf/EP80219 b/sys/arm/conf/EP80219 index 3c2c1aa..24cf837 100644 --- a/sys/arm/conf/EP80219 +++ b/sys/arm/conf/EP80219 @@ -48,7 +48,6 @@ options NFS_ROOT #NFS usable as /, requires NFSCLIENT options CD9660 #ISO 9660 Filesystem #options PROCFS #Process filesystem (requires PSEUDOFS) options PSEUDOFS #Pseudo-filesystem framework -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI options KTRACE #ktrace(1) support options SYSVSHM #SYSV-style shared memory options SYSVMSG #SYSV-style message queues diff --git a/sys/arm/conf/GUMSTIX b/sys/arm/conf/GUMSTIX index e7900f5..973f6e3 100644 --- a/sys/arm/conf/GUMSTIX +++ b/sys/arm/conf/GUMSTIX @@ -53,7 +53,6 @@ options NFS_ROOT #NFS usable as /, requires NFSCLIENT #options PROCFS #Process filesystem (requires PSEUDOFS) options PSEUDOFS #Pseudo-filesystem framework options COMPAT_43 #Compatible with BSD 4.3 [KEEP THIS!] -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI options KTRACE #ktrace(1) support options SYSVSHM #SYSV-style shared memory options SYSVMSG #SYSV-style message queues diff --git a/sys/arm/conf/HL200 b/sys/arm/conf/HL200 index dd46a61..1c06f24 100644 --- a/sys/arm/conf/HL200 +++ b/sys/arm/conf/HL200 @@ -53,7 +53,6 @@ options BOOTP_COMPAT #options CD9660 #ISO 9660 Filesystem #options PROCFS #Process filesystem (requires PSEUDOFS) options PSEUDOFS #Pseudo-filesystem framework -#options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI #options KTRACE #ktrace(1) support options SYSVSHM #SYSV-style shared memory options SYSVMSG #SYSV-style message queues diff --git a/sys/arm/conf/HL201 b/sys/arm/conf/HL201 index 6524cc6..194f454 100644 --- a/sys/arm/conf/HL201 +++ b/sys/arm/conf/HL201 @@ -55,7 +55,6 @@ options ALT_BREAK_TO_DEBUGGER #options CD9660 #ISO 9660 Filesystem #options PROCFS #Process filesystem (requires PSEUDOFS) options PSEUDOFS #Pseudo-filesystem framework -#options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI #options KTRACE #ktrace(1) support options SYSVSHM #SYSV-style shared memory options SYSVMSG #SYSV-style message queues diff --git a/sys/arm/conf/IQ31244 b/sys/arm/conf/IQ31244 index 8b79497..a2e4687 100644 --- a/sys/arm/conf/IQ31244 +++ b/sys/arm/conf/IQ31244 @@ -49,7 +49,6 @@ options NFS_ROOT #NFS usable as /, requires NFSCLIENT options CD9660 #ISO 9660 Filesystem #options PROCFS #Process filesystem (requires PSEUDOFS) options PSEUDOFS #Pseudo-filesystem framework -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI options KTRACE #ktrace(1) support options SYSVSHM #SYSV-style shared memory options SYSVMSG #SYSV-style message queues diff --git a/sys/arm/conf/KB920X b/sys/arm/conf/KB920X index f47e9ac..75980aa 100644 --- a/sys/arm/conf/KB920X +++ b/sys/arm/conf/KB920X @@ -51,7 +51,6 @@ options NFSCLIENT #Network Filesystem Client #options CD9660 #ISO 9660 Filesystem #options PROCFS #Process filesystem (requires PSEUDOFS) options PSEUDOFS #Pseudo-filesystem framework -#options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI #options KTRACE #ktrace(1) support options SYSVSHM #SYSV-style shared memory options SYSVMSG #SYSV-style message queues diff --git a/sys/arm/conf/LN2410SBC b/sys/arm/conf/LN2410SBC index e4f3a54..17a1088 100644 --- a/sys/arm/conf/LN2410SBC +++ b/sys/arm/conf/LN2410SBC @@ -49,7 +49,6 @@ options ROOTDEVNAME=\"ufs:da0s1\" #options NFS_ROOT #NFS usable as root device options PSEUDOFS #Pseudo-filesystem framework -#options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI options KTRACE #ktrace(1) support options SYSVSHM #SYSV-style shared memory options SYSVMSG #SYSV-style message queues diff --git a/sys/arm/conf/NSLU b/sys/arm/conf/NSLU index d921e34..1585689 100644 --- a/sys/arm/conf/NSLU +++ b/sys/arm/conf/NSLU @@ -66,7 +66,6 @@ options NFS_ROOT #NFS usable as /, requires NFSCLIENT options CD9660 #ISO 9660 Filesystem #options PROCFS #Process filesystem (requires PSEUDOFS) options PSEUDOFS #Pseudo-filesystem framework -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI options KTRACE #ktrace(1) support options SYSVSHM #SYSV-style shared memory options SYSVMSG #SYSV-style message queues diff --git a/sys/arm/conf/QILA9G20 b/sys/arm/conf/QILA9G20 index 55839ce..e1858fd 100644 --- a/sys/arm/conf/QILA9G20 +++ b/sys/arm/conf/QILA9G20 @@ -57,7 +57,6 @@ options ALT_BREAK_TO_DEBUGGER #options CD9660 #ISO 9660 Filesystem #options PROCFS #Process filesystem (requires PSEUDOFS) #options PSEUDOFS #Pseudo-filesystem framework -#options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI #options KTRACE #ktrace(1) support options SYSVSHM #SYSV-style shared memory options SYSVMSG #SYSV-style message queues diff --git a/sys/arm/conf/SAM9G20EK b/sys/arm/conf/SAM9G20EK index 6c2e94e..21a3f74 100644 --- a/sys/arm/conf/SAM9G20EK +++ b/sys/arm/conf/SAM9G20EK @@ -56,7 +56,6 @@ options ALT_BREAK_TO_DEBUGGER #options CD9660 #ISO 9660 Filesystem #options PROCFS #Process filesystem (requires PSEUDOFS) #options PSEUDOFS #Pseudo-filesystem framework -#options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI #options KTRACE #ktrace(1) support options SYSVSHM #SYSV-style shared memory options SYSVMSG #SYSV-style message queues diff --git a/sys/arm/conf/SKYEYE b/sys/arm/conf/SKYEYE index b3aad2a..e7c843e 100644 --- a/sys/arm/conf/SKYEYE +++ b/sys/arm/conf/SKYEYE @@ -50,7 +50,6 @@ options NFS_ROOT #NFS usable as /, requires NFSCLIENT options CD9660 #ISO 9660 Filesystem #options PROCFS #Process filesystem (requires PSEUDOFS) options PSEUDOFS #Pseudo-filesystem framework -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI #options KTRACE #ktrace(1) support options SYSVSHM #SYSV-style shared memory options SYSVMSG #SYSV-style message queues diff --git a/sys/i386/conf/GENERIC b/sys/i386/conf/GENERIC index 85f4697..e1c0673 100644 --- a/sys/i386/conf/GENERIC +++ b/sys/i386/conf/GENERIC @@ -50,7 +50,6 @@ options COMPAT_FREEBSD4 # Compatible with FreeBSD4 options COMPAT_FREEBSD5 # Compatible with FreeBSD5 options COMPAT_FREEBSD6 # Compatible with FreeBSD6 options COMPAT_FREEBSD7 # Compatible with FreeBSD7 -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI options KTRACE # ktrace(1) support options STACK # stack(9) support options SYSVSHM # SYSV-style shared memory diff --git a/sys/ia64/conf/GENERIC b/sys/ia64/conf/GENERIC index 71af42c..ff43121 100644 --- a/sys/ia64/conf/GENERIC +++ b/sys/ia64/conf/GENERIC @@ -54,7 +54,6 @@ options PRINTF_BUFR_SIZE=128 # Printf buffering to limit interspersion options PROCFS # Process filesystem (/proc) options PSEUDOFS # Pseudo-filesystem framework options SCHED_ULE # ULE scheduler -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI options SCTP # Stream Control Transmission Protocol options SMP # Symmetric Multi-Processor support options SOFTUPDATES # Enable FFS soft updates support diff --git a/sys/mips/conf/OCTEON1 b/sys/mips/conf/OCTEON1 index a00e95d..db56e7c 100644 --- a/sys/mips/conf/OCTEON1 +++ b/sys/mips/conf/OCTEON1 @@ -65,7 +65,6 @@ options PSEUDOFS # Pseudo-filesystem framework options GEOM_PART_GPT # GUID Partition Tables. options GEOM_LABEL # Provides labelization #options COMPAT_FREEBSD32 # Compatible with o32 binaries (not yet) -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI options KTRACE # ktrace(1) support options STACK # stack(9) support options SYSVSHM # SYSV-style shared memory diff --git a/sys/modules/aha/Makefile b/sys/modules/aha/Makefile index 24d356c..052c020 100644 --- a/sys/modules/aha/Makefile +++ b/sys/modules/aha/Makefile @@ -6,9 +6,4 @@ KMOD= aha SRCS= aha.c aha_isa.c ahareg.h opt_cam.h device_if.h bus_if.h \ opt_scsi.h isa_if.h -.if !defined(KERNBUILDDIR) -opt_scsi.h: - echo "#define SCSI_DELAY 15000" > ${.TARGET} -.endif - .include diff --git a/sys/modules/ahb/Makefile b/sys/modules/ahb/Makefile index 2616226..afb36f3 100644 --- a/sys/modules/ahb/Makefile +++ b/sys/modules/ahb/Makefile @@ -5,9 +5,4 @@ KMOD= ahb SRCS= ahb.c opt_cam.h device_if.h bus_if.h eisa_if.h opt_scsi.h -.if !defined(KERNBUILDDIR) -opt_scsi.h: - echo "#define SCSI_DELAY 15000" > ${.TARGET} -.endif - .include diff --git a/sys/modules/cam/Makefile b/sys/modules/cam/Makefile index df0d77d..4391b15 100644 --- a/sys/modules/cam/Makefile +++ b/sys/modules/cam/Makefile @@ -35,9 +35,4 @@ SRCS+= ata_pmp.c EXPORT_SYMS= YES # XXX evaluate -.if !defined(KERNBUILDDIR) -opt_scsi.h: - echo "#define SCSI_DELAY 15000" > ${.TARGET} -.endif - .include diff --git a/sys/modules/trm/Makefile b/sys/modules/trm/Makefile index 48df9f5..7c42e40 100644 --- a/sys/modules/trm/Makefile +++ b/sys/modules/trm/Makefile @@ -6,9 +6,4 @@ KMOD= trm SRCS= trm.c trm.h opt_cam.h device_if.h bus_if.h \ opt_scsi.h pci_if.h -.if !defined(KERNBUILDDIR) -opt_scsi.h: - echo "#define SCSI_DELAY 15000" > ${.TARGET} -.endif - .include diff --git a/sys/pc98/conf/GENERIC b/sys/pc98/conf/GENERIC index e137297..df937c6 100644 --- a/sys/pc98/conf/GENERIC +++ b/sys/pc98/conf/GENERIC @@ -50,7 +50,6 @@ options COMPAT_FREEBSD4 # Compatible with FreeBSD4 options COMPAT_FREEBSD5 # Compatible with FreeBSD5 options COMPAT_FREEBSD6 # Compatible with FreeBSD6 options COMPAT_FREEBSD7 # Compatible with FreeBSD7 -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI options EPSON_BOUNCEDMA # use bounce buffer for 15-16M #options EPSON_MEMWIN # EPSON memory window support #options LINE30 diff --git a/sys/powerpc/conf/GENERIC b/sys/powerpc/conf/GENERIC index c23e9ac..5f44ee8 100644 --- a/sys/powerpc/conf/GENERIC +++ b/sys/powerpc/conf/GENERIC @@ -54,7 +54,6 @@ options COMPAT_FREEBSD4 #Keep this for a while options COMPAT_FREEBSD5 #Compatible with FreeBSD5 options COMPAT_FREEBSD6 #Compatible with FreeBSD6 options COMPAT_FREEBSD7 #Compatible with FreeBSD7 -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI options KTRACE #ktrace(1) syscall trace support options STACK #stack(9) support options SYSVSHM #SYSV-style shared memory diff --git a/sys/powerpc/conf/GENERIC64 b/sys/powerpc/conf/GENERIC64 index b861e51..9127c01 100644 --- a/sys/powerpc/conf/GENERIC64 +++ b/sys/powerpc/conf/GENERIC64 @@ -53,7 +53,6 @@ options COMPAT_FREEBSD32 #Compatible with FreeBSD/powerpc binaries options COMPAT_FREEBSD5 #Compatible with FreeBSD5 options COMPAT_FREEBSD6 #Compatible with FreeBSD6 options COMPAT_FREEBSD7 #Compatible with FreeBSD7 -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI options KTRACE #ktrace(1) syscall trace support options STACK #stack(9) support options SYSVSHM #SYSV-style shared memory diff --git a/sys/sparc64/conf/GENERIC b/sys/sparc64/conf/GENERIC index 6bbd4e1..60226d0 100644 --- a/sys/sparc64/conf/GENERIC +++ b/sys/sparc64/conf/GENERIC @@ -50,7 +50,6 @@ options GEOM_LABEL # Provides labelization options COMPAT_FREEBSD5 # Compatible with FreeBSD5 options COMPAT_FREEBSD6 # Compatible with FreeBSD6 options COMPAT_FREEBSD7 # Compatible with FreeBSD7 -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI options KTRACE # ktrace(1) support options STACK # stack(9) support options SYSVSHM # SYSV-style shared memory diff --git a/sys/sun4v/conf/GENERIC b/sys/sun4v/conf/GENERIC index 74fc036..0508048 100644 --- a/sys/sun4v/conf/GENERIC +++ b/sys/sun4v/conf/GENERIC @@ -51,7 +51,6 @@ options PROCFS # Process filesystem (requires PSEUDOFS) options PSEUDOFS # Pseudo-filesystem framework options GEOM_PART_GPT # GUID Partition Tables. options GEOM_LABEL # Provides labelization -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI options KTRACE # ktrace(1) support options STACK # stack(9) support options SYSVSHM # SYSV-style shared memory --h31gzZEtNLTqOjlF-- From owner-freebsd-scsi@FreeBSD.ORG Wed Oct 20 01:27:11 2010 Return-Path: Delivered-To: freebsd-scsi@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 1233) id 73823106566C; Wed, 20 Oct 2010 01:27:11 +0000 (UTC) Date: Wed, 20 Oct 2010 01:27:11 +0000 From: Alexander Best To: freebsd-scsi@freebsd.org Message-ID: <20101020012711.GA132@freebsd.org> References: <20101019221131.GA75368@freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20101019221131.GA75368@freebsd.org> Subject: Re: set kern.cam.scsi_delay to 2000ms on all platforms X-BeenThere: freebsd-scsi@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SCSI subsystem List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 20 Oct 2010 01:27:11 -0000 On Tue Oct 19 10, Alexander Best wrote: > hi there, > > i've posted this patch on freebsd-hackers@, but jhb@ recommended i should also > send it to freebsd-scsi@. basically what this patch does is to populate > kern.cam.scsi_delay=2000 as the default value throughout the entire source > tree. in addition to that it gets rid of some legacy Makefile entries, where > kern.cam.scsi_delay was set to 150000. Warner Losh pointed out that opt_scsi.h could be removed from the module makefiles of aha, ahb and trm. cheers. alex > > cheers. > alex > > -- > a13x > diff --git a/share/man/man4/scsi.4 b/share/man/man4/scsi.4 > index ad52663..3087aec 100644 > --- a/share/man/man4/scsi.4 > +++ b/share/man/man4/scsi.4 > @@ -24,7 +24,7 @@ > .\" SUCH DAMAGE. > .\" > .\" $FreeBSD$ > -.Dd March 4, 2010 > +.Dd October 19, 2010 > .Dt CAM 4 > .Os > .Sh NAME > @@ -47,7 +47,7 @@ > .Cd "options CAM_MAX_HIGHPOWER=4" > .Cd "options SCSI_NO_SENSE_STRINGS" > .Cd "options SCSI_NO_OP_STRINGS" > -.Cd "options SCSI_DELAY=8000" > +.Cd "options SCSI_DELAY=2000" > .Sh DESCRIPTION > The > .Nm > @@ -116,7 +116,7 @@ Enabling this option for normal use is not recommended, since it slows > debugging of > .Tn SCSI > problems. > -.It Dv SCSI_DELAY=8000 > +.It Dv SCSI_DELAY=2000 > This is the > .Tn SCSI > "bus settle delay." > diff --git a/sys/amd64/conf/GENERIC b/sys/amd64/conf/GENERIC > index 3bc6299..df26ea6 100644 > --- a/sys/amd64/conf/GENERIC > +++ b/sys/amd64/conf/GENERIC > @@ -49,7 +49,6 @@ options COMPAT_FREEBSD4 # Compatible with FreeBSD4 > options COMPAT_FREEBSD5 # Compatible with FreeBSD5 > options COMPAT_FREEBSD6 # Compatible with FreeBSD6 > options COMPAT_FREEBSD7 # Compatible with FreeBSD7 > -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI > options KTRACE # ktrace(1) support > options STACK # stack(9) support > options SYSVSHM # SYSV-style shared memory > diff --git a/sys/amd64/conf/XENHVM b/sys/amd64/conf/XENHVM > index 51f1256..47cdc83 100644 > --- a/sys/amd64/conf/XENHVM > +++ b/sys/amd64/conf/XENHVM > @@ -50,7 +50,6 @@ options COMPAT_FREEBSD4 # Compatible with FreeBSD4 > options COMPAT_FREEBSD5 # Compatible with FreeBSD5 > options COMPAT_FREEBSD6 # Compatible with FreeBSD6 > options COMPAT_FREEBSD7 # Compatible with FreeBSD7 > -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI > options KTRACE # ktrace(1) support > options STACK # stack(9) support > options SYSVSHM # SYSV-style shared memory > diff --git a/sys/arm/conf/BWCT b/sys/arm/conf/BWCT > index 0fb3b87..a2ecc07 100644 > --- a/sys/arm/conf/BWCT > +++ b/sys/arm/conf/BWCT > @@ -57,7 +57,6 @@ options BOOTP > #options CD9660 #ISO 9660 Filesystem > #options PROCFS #Process filesystem (requires PSEUDOFS) > options PSEUDOFS #Pseudo-filesystem framework > -#options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > #options KTRACE #ktrace(1) support > options SYSVSHM #SYSV-style shared memory > options SYSVMSG #SYSV-style message queues > diff --git a/sys/arm/conf/CNS11XXNAS b/sys/arm/conf/CNS11XXNAS > index 76db42b..b14b1b2 100644 > --- a/sys/arm/conf/CNS11XXNAS > +++ b/sys/arm/conf/CNS11XXNAS > @@ -74,7 +74,6 @@ options MSDOSFS #MSDOS Filesystem > #options CD9660 #ISO 9660 Filesystem > #options PROCFS #Process filesystem (requires PSEUDOFS) > options PSEUDOFS #Pseudo-filesystem framework > -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > options KTRACE #ktrace(1) support > options SYSVSHM #SYSV-style shared memory > options SYSVMSG #SYSV-style message queues > diff --git a/sys/arm/conf/CRB b/sys/arm/conf/CRB > index 2afd080..ff7fc1d 100644 > --- a/sys/arm/conf/CRB > +++ b/sys/arm/conf/CRB > @@ -49,7 +49,6 @@ options NFS_ROOT #NFS usable as /, requires NFSCLIENT > options CD9660 #ISO 9660 Filesystem > #options PROCFS #Process filesystem (requires PSEUDOFS) > options PSEUDOFS #Pseudo-filesystem framework > -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > options KTRACE #ktrace(1) support > options INTR_FILTER > options SYSVSHM #SYSV-style shared memory > diff --git a/sys/arm/conf/EP80219 b/sys/arm/conf/EP80219 > index 3c2c1aa..24cf837 100644 > --- a/sys/arm/conf/EP80219 > +++ b/sys/arm/conf/EP80219 > @@ -48,7 +48,6 @@ options NFS_ROOT #NFS usable as /, requires NFSCLIENT > options CD9660 #ISO 9660 Filesystem > #options PROCFS #Process filesystem (requires PSEUDOFS) > options PSEUDOFS #Pseudo-filesystem framework > -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > options KTRACE #ktrace(1) support > options SYSVSHM #SYSV-style shared memory > options SYSVMSG #SYSV-style message queues > diff --git a/sys/arm/conf/GUMSTIX b/sys/arm/conf/GUMSTIX > index e7900f5..973f6e3 100644 > --- a/sys/arm/conf/GUMSTIX > +++ b/sys/arm/conf/GUMSTIX > @@ -53,7 +53,6 @@ options NFS_ROOT #NFS usable as /, requires NFSCLIENT > #options PROCFS #Process filesystem (requires PSEUDOFS) > options PSEUDOFS #Pseudo-filesystem framework > options COMPAT_43 #Compatible with BSD 4.3 [KEEP THIS!] > -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > options KTRACE #ktrace(1) support > options SYSVSHM #SYSV-style shared memory > options SYSVMSG #SYSV-style message queues > diff --git a/sys/arm/conf/HL200 b/sys/arm/conf/HL200 > index dd46a61..1c06f24 100644 > --- a/sys/arm/conf/HL200 > +++ b/sys/arm/conf/HL200 > @@ -53,7 +53,6 @@ options BOOTP_COMPAT > #options CD9660 #ISO 9660 Filesystem > #options PROCFS #Process filesystem (requires PSEUDOFS) > options PSEUDOFS #Pseudo-filesystem framework > -#options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > #options KTRACE #ktrace(1) support > options SYSVSHM #SYSV-style shared memory > options SYSVMSG #SYSV-style message queues > diff --git a/sys/arm/conf/HL201 b/sys/arm/conf/HL201 > index 6524cc6..194f454 100644 > --- a/sys/arm/conf/HL201 > +++ b/sys/arm/conf/HL201 > @@ -55,7 +55,6 @@ options ALT_BREAK_TO_DEBUGGER > #options CD9660 #ISO 9660 Filesystem > #options PROCFS #Process filesystem (requires PSEUDOFS) > options PSEUDOFS #Pseudo-filesystem framework > -#options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > #options KTRACE #ktrace(1) support > options SYSVSHM #SYSV-style shared memory > options SYSVMSG #SYSV-style message queues > diff --git a/sys/arm/conf/IQ31244 b/sys/arm/conf/IQ31244 > index 8b79497..a2e4687 100644 > --- a/sys/arm/conf/IQ31244 > +++ b/sys/arm/conf/IQ31244 > @@ -49,7 +49,6 @@ options NFS_ROOT #NFS usable as /, requires NFSCLIENT > options CD9660 #ISO 9660 Filesystem > #options PROCFS #Process filesystem (requires PSEUDOFS) > options PSEUDOFS #Pseudo-filesystem framework > -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > options KTRACE #ktrace(1) support > options SYSVSHM #SYSV-style shared memory > options SYSVMSG #SYSV-style message queues > diff --git a/sys/arm/conf/KB920X b/sys/arm/conf/KB920X > index f47e9ac..75980aa 100644 > --- a/sys/arm/conf/KB920X > +++ b/sys/arm/conf/KB920X > @@ -51,7 +51,6 @@ options NFSCLIENT #Network Filesystem Client > #options CD9660 #ISO 9660 Filesystem > #options PROCFS #Process filesystem (requires PSEUDOFS) > options PSEUDOFS #Pseudo-filesystem framework > -#options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > #options KTRACE #ktrace(1) support > options SYSVSHM #SYSV-style shared memory > options SYSVMSG #SYSV-style message queues > diff --git a/sys/arm/conf/LN2410SBC b/sys/arm/conf/LN2410SBC > index e4f3a54..17a1088 100644 > --- a/sys/arm/conf/LN2410SBC > +++ b/sys/arm/conf/LN2410SBC > @@ -49,7 +49,6 @@ options ROOTDEVNAME=\"ufs:da0s1\" > #options NFS_ROOT #NFS usable as root device > > options PSEUDOFS #Pseudo-filesystem framework > -#options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > options KTRACE #ktrace(1) support > options SYSVSHM #SYSV-style shared memory > options SYSVMSG #SYSV-style message queues > diff --git a/sys/arm/conf/NSLU b/sys/arm/conf/NSLU > index d921e34..1585689 100644 > --- a/sys/arm/conf/NSLU > +++ b/sys/arm/conf/NSLU > @@ -66,7 +66,6 @@ options NFS_ROOT #NFS usable as /, requires NFSCLIENT > options CD9660 #ISO 9660 Filesystem > #options PROCFS #Process filesystem (requires PSEUDOFS) > options PSEUDOFS #Pseudo-filesystem framework > -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > options KTRACE #ktrace(1) support > options SYSVSHM #SYSV-style shared memory > options SYSVMSG #SYSV-style message queues > diff --git a/sys/arm/conf/QILA9G20 b/sys/arm/conf/QILA9G20 > index 55839ce..e1858fd 100644 > --- a/sys/arm/conf/QILA9G20 > +++ b/sys/arm/conf/QILA9G20 > @@ -57,7 +57,6 @@ options ALT_BREAK_TO_DEBUGGER > #options CD9660 #ISO 9660 Filesystem > #options PROCFS #Process filesystem (requires PSEUDOFS) > #options PSEUDOFS #Pseudo-filesystem framework > -#options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > #options KTRACE #ktrace(1) support > options SYSVSHM #SYSV-style shared memory > options SYSVMSG #SYSV-style message queues > diff --git a/sys/arm/conf/SAM9G20EK b/sys/arm/conf/SAM9G20EK > index 6c2e94e..21a3f74 100644 > --- a/sys/arm/conf/SAM9G20EK > +++ b/sys/arm/conf/SAM9G20EK > @@ -56,7 +56,6 @@ options ALT_BREAK_TO_DEBUGGER > #options CD9660 #ISO 9660 Filesystem > #options PROCFS #Process filesystem (requires PSEUDOFS) > #options PSEUDOFS #Pseudo-filesystem framework > -#options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > #options KTRACE #ktrace(1) support > options SYSVSHM #SYSV-style shared memory > options SYSVMSG #SYSV-style message queues > diff --git a/sys/arm/conf/SKYEYE b/sys/arm/conf/SKYEYE > index b3aad2a..e7c843e 100644 > --- a/sys/arm/conf/SKYEYE > +++ b/sys/arm/conf/SKYEYE > @@ -50,7 +50,6 @@ options NFS_ROOT #NFS usable as /, requires NFSCLIENT > options CD9660 #ISO 9660 Filesystem > #options PROCFS #Process filesystem (requires PSEUDOFS) > options PSEUDOFS #Pseudo-filesystem framework > -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > #options KTRACE #ktrace(1) support > options SYSVSHM #SYSV-style shared memory > options SYSVMSG #SYSV-style message queues > diff --git a/sys/i386/conf/GENERIC b/sys/i386/conf/GENERIC > index 85f4697..e1c0673 100644 > --- a/sys/i386/conf/GENERIC > +++ b/sys/i386/conf/GENERIC > @@ -50,7 +50,6 @@ options COMPAT_FREEBSD4 # Compatible with FreeBSD4 > options COMPAT_FREEBSD5 # Compatible with FreeBSD5 > options COMPAT_FREEBSD6 # Compatible with FreeBSD6 > options COMPAT_FREEBSD7 # Compatible with FreeBSD7 > -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI > options KTRACE # ktrace(1) support > options STACK # stack(9) support > options SYSVSHM # SYSV-style shared memory > diff --git a/sys/ia64/conf/GENERIC b/sys/ia64/conf/GENERIC > index 71af42c..ff43121 100644 > --- a/sys/ia64/conf/GENERIC > +++ b/sys/ia64/conf/GENERIC > @@ -54,7 +54,6 @@ options PRINTF_BUFR_SIZE=128 # Printf buffering to limit interspersion > options PROCFS # Process filesystem (/proc) > options PSEUDOFS # Pseudo-filesystem framework > options SCHED_ULE # ULE scheduler > -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI > options SCTP # Stream Control Transmission Protocol > options SMP # Symmetric Multi-Processor support > options SOFTUPDATES # Enable FFS soft updates support > diff --git a/sys/mips/conf/OCTEON1 b/sys/mips/conf/OCTEON1 > index a00e95d..db56e7c 100644 > --- a/sys/mips/conf/OCTEON1 > +++ b/sys/mips/conf/OCTEON1 > @@ -65,7 +65,6 @@ options PSEUDOFS # Pseudo-filesystem framework > options GEOM_PART_GPT # GUID Partition Tables. > options GEOM_LABEL # Provides labelization > #options COMPAT_FREEBSD32 # Compatible with o32 binaries (not yet) > -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI > options KTRACE # ktrace(1) support > options STACK # stack(9) support > options SYSVSHM # SYSV-style shared memory > diff --git a/sys/modules/aha/Makefile b/sys/modules/aha/Makefile > index 24d356c..052c020 100644 > --- a/sys/modules/aha/Makefile > +++ b/sys/modules/aha/Makefile > @@ -6,9 +6,4 @@ KMOD= aha > SRCS= aha.c aha_isa.c ahareg.h opt_cam.h device_if.h bus_if.h \ > opt_scsi.h isa_if.h > > -.if !defined(KERNBUILDDIR) > -opt_scsi.h: > - echo "#define SCSI_DELAY 15000" > ${.TARGET} > -.endif > - > .include > diff --git a/sys/modules/ahb/Makefile b/sys/modules/ahb/Makefile > index 2616226..afb36f3 100644 > --- a/sys/modules/ahb/Makefile > +++ b/sys/modules/ahb/Makefile > @@ -5,9 +5,4 @@ > KMOD= ahb > SRCS= ahb.c opt_cam.h device_if.h bus_if.h eisa_if.h opt_scsi.h > > -.if !defined(KERNBUILDDIR) > -opt_scsi.h: > - echo "#define SCSI_DELAY 15000" > ${.TARGET} > -.endif > - > .include > diff --git a/sys/modules/cam/Makefile b/sys/modules/cam/Makefile > index df0d77d..4391b15 100644 > --- a/sys/modules/cam/Makefile > +++ b/sys/modules/cam/Makefile > @@ -35,9 +35,4 @@ SRCS+= ata_pmp.c > > EXPORT_SYMS= YES # XXX evaluate > > -.if !defined(KERNBUILDDIR) > -opt_scsi.h: > - echo "#define SCSI_DELAY 15000" > ${.TARGET} > -.endif > - > .include > diff --git a/sys/modules/trm/Makefile b/sys/modules/trm/Makefile > index 48df9f5..7c42e40 100644 > --- a/sys/modules/trm/Makefile > +++ b/sys/modules/trm/Makefile > @@ -6,9 +6,4 @@ KMOD= trm > SRCS= trm.c trm.h opt_cam.h device_if.h bus_if.h \ > opt_scsi.h pci_if.h > > -.if !defined(KERNBUILDDIR) > -opt_scsi.h: > - echo "#define SCSI_DELAY 15000" > ${.TARGET} > -.endif > - > .include > diff --git a/sys/pc98/conf/GENERIC b/sys/pc98/conf/GENERIC > index e137297..df937c6 100644 > --- a/sys/pc98/conf/GENERIC > +++ b/sys/pc98/conf/GENERIC > @@ -50,7 +50,6 @@ options COMPAT_FREEBSD4 # Compatible with FreeBSD4 > options COMPAT_FREEBSD5 # Compatible with FreeBSD5 > options COMPAT_FREEBSD6 # Compatible with FreeBSD6 > options COMPAT_FREEBSD7 # Compatible with FreeBSD7 > -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI > options EPSON_BOUNCEDMA # use bounce buffer for 15-16M > #options EPSON_MEMWIN # EPSON memory window support > #options LINE30 > diff --git a/sys/powerpc/conf/GENERIC b/sys/powerpc/conf/GENERIC > index c23e9ac..5f44ee8 100644 > --- a/sys/powerpc/conf/GENERIC > +++ b/sys/powerpc/conf/GENERIC > @@ -54,7 +54,6 @@ options COMPAT_FREEBSD4 #Keep this for a while > options COMPAT_FREEBSD5 #Compatible with FreeBSD5 > options COMPAT_FREEBSD6 #Compatible with FreeBSD6 > options COMPAT_FREEBSD7 #Compatible with FreeBSD7 > -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > options KTRACE #ktrace(1) syscall trace support > options STACK #stack(9) support > options SYSVSHM #SYSV-style shared memory > diff --git a/sys/powerpc/conf/GENERIC64 b/sys/powerpc/conf/GENERIC64 > index b861e51..9127c01 100644 > --- a/sys/powerpc/conf/GENERIC64 > +++ b/sys/powerpc/conf/GENERIC64 > @@ -53,7 +53,6 @@ options COMPAT_FREEBSD32 #Compatible with FreeBSD/powerpc binaries > options COMPAT_FREEBSD5 #Compatible with FreeBSD5 > options COMPAT_FREEBSD6 #Compatible with FreeBSD6 > options COMPAT_FREEBSD7 #Compatible with FreeBSD7 > -options SCSI_DELAY=5000 #Delay (in ms) before probing SCSI > options KTRACE #ktrace(1) syscall trace support > options STACK #stack(9) support > options SYSVSHM #SYSV-style shared memory > diff --git a/sys/sparc64/conf/GENERIC b/sys/sparc64/conf/GENERIC > index 6bbd4e1..60226d0 100644 > --- a/sys/sparc64/conf/GENERIC > +++ b/sys/sparc64/conf/GENERIC > @@ -50,7 +50,6 @@ options GEOM_LABEL # Provides labelization > options COMPAT_FREEBSD5 # Compatible with FreeBSD5 > options COMPAT_FREEBSD6 # Compatible with FreeBSD6 > options COMPAT_FREEBSD7 # Compatible with FreeBSD7 > -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI > options KTRACE # ktrace(1) support > options STACK # stack(9) support > options SYSVSHM # SYSV-style shared memory > diff --git a/sys/sun4v/conf/GENERIC b/sys/sun4v/conf/GENERIC > index 74fc036..0508048 100644 > --- a/sys/sun4v/conf/GENERIC > +++ b/sys/sun4v/conf/GENERIC > @@ -51,7 +51,6 @@ options PROCFS # Process filesystem (requires PSEUDOFS) > options PSEUDOFS # Pseudo-filesystem framework > options GEOM_PART_GPT # GUID Partition Tables. > options GEOM_LABEL # Provides labelization > -options SCSI_DELAY=5000 # Delay (in ms) before probing SCSI > options KTRACE # ktrace(1) support > options STACK # stack(9) support > options SYSVSHM # SYSV-style shared memory -- a13x From owner-freebsd-scsi@FreeBSD.ORG Thu Oct 21 05:32:19 2010 Return-Path: Delivered-To: freebsd-scsi@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 1D49A1065670 for ; Thu, 21 Oct 2010 05:32:19 +0000 (UTC) (envelope-from scottl@samsco.org) Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57]) by mx1.freebsd.org (Postfix) with ESMTP id 8DA1B8FC12 for ; Thu, 21 Oct 2010 05:32:18 +0000 (UTC) Received: from [127.0.0.1] (pooker.samsco.org [168.103.85.57]) (authenticated bits=0) by pooker.samsco.org (8.14.4/8.14.4) with ESMTP id o9L5WEuK062259; Wed, 20 Oct 2010 23:32:14 -0600 (MDT) (envelope-from scottl@samsco.org) Mime-Version: 1.0 (Apple Message framework v1081) Content-Type: text/plain; charset=us-ascii From: Scott Long In-Reply-To: <20101019221131.GA75368@freebsd.org> Date: Wed, 20 Oct 2010 23:32:14 -0600 Content-Transfer-Encoding: quoted-printable Message-Id: <63EF6D51-1196-43F1-8521-27756E972263@samsco.org> References: <20101019221131.GA75368@freebsd.org> To: Alexander Best X-Mailer: Apple Mail (2.1081) X-Spam-Status: No, score=-50.0 required=3.8 tests=ALL_TRUSTED, T_RP_MATCHES_RCVD autolearn=unavailable version=3.3.0 X-Spam-Checker-Version: SpamAssassin 3.3.0 (2010-01-18) on pooker.samsco.org Cc: freebsd-scsi@freebsd.org Subject: Re: set kern.cam.scsi_delay to 2000ms on all platforms X-BeenThere: freebsd-scsi@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SCSI subsystem List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 21 Oct 2010 05:32:19 -0000 You've described the "what" but not the "why". Why do you want to = remove this option? Why do you want to change its value? Scott On Oct 19, 2010, at 4:11 PM, Alexander Best wrote: > hi there, >=20 > i've posted this patch on freebsd-hackers@, but jhb@ recommended i = should also > send it to freebsd-scsi@. basically what this patch does is to = populate > kern.cam.scsi_delay=3D2000 as the default value throughout the entire = source > tree. in addition to that it gets rid of some legacy Makefile entries, = where > kern.cam.scsi_delay was set to 150000. >=20 > cheers. > alex >=20 > --=20 > a13x > _______________________________________________ > freebsd-scsi@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-scsi > To unsubscribe, send any mail to = "freebsd-scsi-unsubscribe@freebsd.org" From owner-freebsd-scsi@FreeBSD.ORG Thu Oct 21 06:21:50 2010 Return-Path: Delivered-To: freebsd-scsi@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 1233) id 105F01065673; Thu, 21 Oct 2010 06:21:50 +0000 (UTC) Date: Thu, 21 Oct 2010 06:21:50 +0000 From: Alexander Best To: Scott Long Message-ID: <20101021062150.GA20489@freebsd.org> References: <20101019221131.GA75368@freebsd.org> <63EF6D51-1196-43F1-8521-27756E972263@samsco.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <63EF6D51-1196-43F1-8521-27756E972263@samsco.org> Cc: freebsd-scsi@freebsd.org Subject: Re: set kern.cam.scsi_delay to 2000ms on all platforms X-BeenThere: freebsd-scsi@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SCSI subsystem List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 21 Oct 2010 06:21:50 -0000 a few years ago kern.cam.scsi_delay was set from 150000 to 5000. nowadays a value of 2000 should be sufficient even on old/slow hardware. to have cleaner code it's better to set default value to 2000 instead of doing so in every config file on every platform. cheers. alex On Wed Oct 20 10, Scott Long wrote: > You've described the "what" but not the "why". Why do you want to remove this option? Why do you want to change its value? > > Scott > > On Oct 19, 2010, at 4:11 PM, Alexander Best wrote: > > > hi there, > > > > i've posted this patch on freebsd-hackers@, but jhb@ recommended i should also > > send it to freebsd-scsi@. basically what this patch does is to populate > > kern.cam.scsi_delay=2000 as the default value throughout the entire source > > tree. in addition to that it gets rid of some legacy Makefile entries, where > > kern.cam.scsi_delay was set to 150000. > > > > cheers. > > alex > > > > -- > > a13x > > _______________________________________________ > > freebsd-scsi@freebsd.org mailing list > > http://lists.freebsd.org/mailman/listinfo/freebsd-scsi > > To unsubscribe, send any mail to "freebsd-scsi-unsubscribe@freebsd.org" -- a13x From owner-freebsd-scsi@FreeBSD.ORG Thu Oct 21 13:19:13 2010 Return-Path: Delivered-To: freebsd-scsi@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id A6DCE1065673; Thu, 21 Oct 2010 13:19:13 +0000 (UTC) (envelope-from scottl@samsco.org) Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57]) by mx1.freebsd.org (Postfix) with ESMTP id 52CCF8FC1E; Thu, 21 Oct 2010 13:19:13 +0000 (UTC) Received: from [127.0.0.1] (pooker.samsco.org [168.103.85.57]) (authenticated bits=0) by pooker.samsco.org (8.14.4/8.14.4) with ESMTP id o9LDJ9Gt065065; Thu, 21 Oct 2010 07:19:10 -0600 (MDT) (envelope-from scottl@samsco.org) Mime-Version: 1.0 (Apple Message framework v1081) Content-Type: text/plain; charset=us-ascii From: Scott Long In-Reply-To: <20101021062150.GA20489@freebsd.org> Date: Thu, 21 Oct 2010 07:19:09 -0600 Content-Transfer-Encoding: quoted-printable Message-Id: <8B967C79-CD84-435C-9007-E33467DC92A9@samsco.org> References: <20101019221131.GA75368@freebsd.org> <63EF6D51-1196-43F1-8521-27756E972263@samsco.org> <20101021062150.GA20489@freebsd.org> To: Alexander Best X-Mailer: Apple Mail (2.1081) X-Spam-Status: No, score=-50.0 required=3.8 tests=ALL_TRUSTED, T_RP_MATCHES_RCVD autolearn=unavailable version=3.3.0 X-Spam-Checker-Version: SpamAssassin 3.3.0 (2010-01-18) on pooker.samsco.org Cc: freebsd-scsi@freebsd.org Subject: Re: set kern.cam.scsi_delay to 2000ms on all platforms X-BeenThere: freebsd-scsi@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SCSI subsystem List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 21 Oct 2010 13:19:13 -0000 Ok, I've gotta be a stick in the mud and ask you how you've come to the = conclusion that 2000 is better than 5000. Can you provide your evidence = and/or testing results? Scott On Oct 21, 2010, at 12:21 AM, Alexander Best wrote: > a few years ago kern.cam.scsi_delay was set from 150000 to 5000. = nowadays a > value of 2000 should be sufficient even on old/slow hardware. >=20 > to have cleaner code it's better to set default value to 2000 instead = of doing > so in every config file on every platform. >=20 > cheers. > alex >=20 > On Wed Oct 20 10, Scott Long wrote: >> You've described the "what" but not the "why". Why do you want to = remove this option? Why do you want to change its value? >>=20 >> Scott >>=20 >> On Oct 19, 2010, at 4:11 PM, Alexander Best wrote: >>=20 >>> hi there, >>>=20 >>> i've posted this patch on freebsd-hackers@, but jhb@ recommended i = should also >>> send it to freebsd-scsi@. basically what this patch does is to = populate >>> kern.cam.scsi_delay=3D2000 as the default value throughout the = entire source >>> tree. in addition to that it gets rid of some legacy Makefile = entries, where >>> kern.cam.scsi_delay was set to 150000. >>>=20 >>> cheers. >>> alex >>>=20 >>> --=20 >>> a13x >>> _______________________________________________ >>> freebsd-scsi@freebsd.org mailing list >>> http://lists.freebsd.org/mailman/listinfo/freebsd-scsi >>> To unsubscribe, send any mail to = "freebsd-scsi-unsubscribe@freebsd.org" >=20 > --=20 > a13x From owner-freebsd-scsi@FreeBSD.ORG Thu Oct 21 13:54:47 2010 Return-Path: Delivered-To: freebsd-scsi@freebsd.org Received: by hub.freebsd.org (Postfix, from userid 1233) id B61FB106566B; Thu, 21 Oct 2010 13:54:47 +0000 (UTC) Date: Thu, 21 Oct 2010 13:54:47 +0000 From: Alexander Best To: Scott Long Message-ID: <20101021135447.GC72290@freebsd.org> References: <20101019221131.GA75368@freebsd.org> <63EF6D51-1196-43F1-8521-27756E972263@samsco.org> <20101021062150.GA20489@freebsd.org> <8B967C79-CD84-435C-9007-E33467DC92A9@samsco.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <8B967C79-CD84-435C-9007-E33467DC92A9@samsco.org> Cc: freebsd-scsi@freebsd.org Subject: Re: set kern.cam.scsi_delay to 2000ms on all platforms X-BeenThere: freebsd-scsi@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SCSI subsystem List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 21 Oct 2010 13:54:47 -0000 On Thu Oct 21 10, Scott Long wrote: > Ok, I've gotta be a stick in the mud and ask you how you've come to the conclusion that 2000 is better than 5000. Can you provide your evidence and/or testing results? i'm running with kern.cam.scsi_delay set to 2000 and haven't experienced any timeout issues. my original patch lowered kern.cam.scsi_delay to 5000, but after receiving some feedback on freebsd-current@ [1] i was under the impression that the majority of people responding to my post considered 5000 to be too high. Matthew Jacob e.g. wrote that even 2 seconds is more than enough even for older hardware. John Baldwin also seemed to agree with lowering the timeout value to 2 seconds. this timeout value is not intended to wait for devices to initialize upon boot time. kern.cam.boot_delay must be set in that case. it only applies to the timeout of already succussefully initialzed devices. cheers. alex [1] http://www.mail-archive.com/freebsd-hackers@freebsd.org/msg158316.html > > Scott > > On Oct 21, 2010, at 12:21 AM, Alexander Best wrote: > > > a few years ago kern.cam.scsi_delay was set from 150000 to 5000. nowadays a > > value of 2000 should be sufficient even on old/slow hardware. > > > > to have cleaner code it's better to set default value to 2000 instead of doing > > so in every config file on every platform. > > > > cheers. > > alex > > > > On Wed Oct 20 10, Scott Long wrote: > >> You've described the "what" but not the "why". Why do you want to remove this option? Why do you want to change its value? > >> > >> Scott > >> > >> On Oct 19, 2010, at 4:11 PM, Alexander Best wrote: > >> > >>> hi there, > >>> > >>> i've posted this patch on freebsd-hackers@, but jhb@ recommended i should also > >>> send it to freebsd-scsi@. basically what this patch does is to populate > >>> kern.cam.scsi_delay=2000 as the default value throughout the entire source > >>> tree. in addition to that it gets rid of some legacy Makefile entries, where > >>> kern.cam.scsi_delay was set to 150000. > >>> > >>> cheers. > >>> alex > >>> > >>> -- > >>> a13x > >>> _______________________________________________ > >>> freebsd-scsi@freebsd.org mailing list > >>> http://lists.freebsd.org/mailman/listinfo/freebsd-scsi > >>> To unsubscribe, send any mail to "freebsd-scsi-unsubscribe@freebsd.org" > > > > -- > > a13x -- a13x From owner-freebsd-scsi@FreeBSD.ORG Thu Oct 21 13:57:07 2010 Return-Path: Delivered-To: freebsd-scsi@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 8F5B0106564A for ; Thu, 21 Oct 2010 13:57:07 +0000 (UTC) (envelope-from mj@feral.com) Received: from ns1.feral.com (ns1.feral.com [192.67.166.1]) by mx1.freebsd.org (Postfix) with ESMTP id 4053C8FC25 for ; Thu, 21 Oct 2010 13:57:07 +0000 (UTC) Received: from [192.168.1.2] (m206-63.dsl.tsoft.com [198.144.206.63]) by ns1.feral.com (8.14.3/8.14.3) with ESMTP id o9LDv6VS020568 for ; Thu, 21 Oct 2010 06:57:06 -0700 (PDT) (envelope-from mj@feral.com) Message-ID: <4CC046B8.8000505@feral.com> Date: Thu, 21 Oct 2010 06:57:12 -0700 From: Matthew Jacob Organization: Feral Software User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.11) Gecko/20101013 Thunderbird/3.1.5 MIME-Version: 1.0 To: freebsd-scsi@freebsd.org References: <20101019221131.GA75368@freebsd.org> <63EF6D51-1196-43F1-8521-27756E972263@samsco.org> <20101021062150.GA20489@freebsd.org> <8B967C79-CD84-435C-9007-E33467DC92A9@samsco.org> <20101021135447.GC72290@freebsd.org> In-Reply-To: <20101021135447.GC72290@freebsd.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Greylist: Default is to whitelist mail, not delayed by milter-greylist-4.2.6 (ns1.feral.com [192.67.166.1]); Thu, 21 Oct 2010 06:57:06 -0700 (PDT) Subject: Re: set kern.cam.scsi_delay to 2000ms on all platforms X-BeenThere: freebsd-scsi@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SCSI subsystem List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 21 Oct 2010 13:57:07 -0000 Now would be a good time to find out if there are issues. > On Thu Oct 21 10, Scott Long wrote: >> Ok, I've gotta be a stick in the mud and ask you how you've come to the conclusion that 2000 is better than 5000. Can you provide your evidence and/or testing results? From owner-freebsd-scsi@FreeBSD.ORG Thu Oct 21 14:01:55 2010 Return-Path: Delivered-To: freebsd-scsi@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 103261065698 for ; Thu, 21 Oct 2010 14:01:55 +0000 (UTC) (envelope-from scottl@samsco.org) Received: from pooker.samsco.org (pooker.samsco.org [168.103.85.57]) by mx1.freebsd.org (Postfix) with ESMTP id C47988FC12 for ; Thu, 21 Oct 2010 14:01:54 +0000 (UTC) Received: from [127.0.0.1] (pooker.samsco.org [168.103.85.57]) (authenticated bits=0) by pooker.samsco.org (8.14.4/8.14.4) with ESMTP id o9LE1kMB065282; Thu, 21 Oct 2010 08:01:46 -0600 (MDT) (envelope-from scottl@samsco.org) Mime-Version: 1.0 (Apple Message framework v1081) Content-Type: text/plain; charset=us-ascii From: Scott Long In-Reply-To: <4CC046B8.8000505@feral.com> Date: Thu, 21 Oct 2010 08:01:46 -0600 Content-Transfer-Encoding: quoted-printable Message-Id: <422A732A-355A-4C18-9E8F-8C7D2EE9D122@samsco.org> References: <20101019221131.GA75368@freebsd.org> <63EF6D51-1196-43F1-8521-27756E972263@samsco.org> <20101021062150.GA20489@freebsd.org> <8B967C79-CD84-435C-9007-E33467DC92A9@samsco.org> <20101021135447.GC72290@freebsd.org> <4CC046B8.8000505@feral.com> To: Matthew Jacob X-Mailer: Apple Mail (2.1081) X-Spam-Status: No, score=-50.0 required=3.8 tests=ALL_TRUSTED, T_RP_MATCHES_RCVD autolearn=unavailable version=3.3.0 X-Spam-Checker-Version: SpamAssassin 3.3.0 (2010-01-18) on pooker.samsco.org Cc: freebsd-scsi@freebsd.org Subject: Re: set kern.cam.scsi_delay to 2000ms on all platforms X-BeenThere: freebsd-scsi@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SCSI subsystem List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 21 Oct 2010 14:01:55 -0000 Sure, but what's the plan for validating it? Too often this stuff goes = into HEAD with the best of intentions, and then it basically never = touched again until someone stumbles over it in an release. I'm not = against this change, I just want to know what engineering has gone into = it. Scott On Oct 21, 2010, at 7:57 AM, Matthew Jacob wrote: > Now would be a good time to find out if there are issues. >=20 >> On Thu Oct 21 10, Scott Long wrote: >>> Ok, I've gotta be a stick in the mud and ask you how you've come to = the conclusion that 2000 is better than 5000. Can you provide your = evidence and/or testing results? >=20 > _______________________________________________ > freebsd-scsi@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-scsi > To unsubscribe, send any mail to = "freebsd-scsi-unsubscribe@freebsd.org" From owner-freebsd-scsi@FreeBSD.ORG Thu Oct 21 14:09:36 2010 Return-Path: Delivered-To: freebsd-scsi@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 356FE106564A for ; Thu, 21 Oct 2010 14:09:36 +0000 (UTC) (envelope-from mj@feral.com) Received: from ns1.feral.com (ns1.feral.com [192.67.166.1]) by mx1.freebsd.org (Postfix) with ESMTP id F01618FC12 for ; Thu, 21 Oct 2010 14:09:35 +0000 (UTC) Received: from [192.168.1.2] (m206-63.dsl.tsoft.com [198.144.206.63]) by ns1.feral.com (8.14.3/8.14.3) with ESMTP id o9LE9ZNQ020647 for ; Thu, 21 Oct 2010 07:09:35 -0700 (PDT) (envelope-from mj@feral.com) Message-ID: <4CC049A4.2090007@feral.com> Date: Thu, 21 Oct 2010 07:09:40 -0700 From: Matthew Jacob Organization: Feral Software User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.2.11) Gecko/20101013 Thunderbird/3.1.5 MIME-Version: 1.0 To: freebsd-scsi@freebsd.org References: <20101019221131.GA75368@freebsd.org> <63EF6D51-1196-43F1-8521-27756E972263@samsco.org> <20101021062150.GA20489@freebsd.org> <8B967C79-CD84-435C-9007-E33467DC92A9@samsco.org> <20101021135447.GC72290@freebsd.org> <4CC046B8.8000505@feral.com> <422A732A-355A-4C18-9E8F-8C7D2EE9D122@samsco.org> In-Reply-To: <422A732A-355A-4C18-9E8F-8C7D2EE9D122@samsco.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-Greylist: Default is to whitelist mail, not delayed by milter-greylist-4.2.6 (ns1.feral.com [192.67.166.1]); Thu, 21 Oct 2010 07:09:35 -0700 (PDT) Subject: Re: set kern.cam.scsi_delay to 2000ms on all platforms X-BeenThere: freebsd-scsi@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SCSI subsystem List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 21 Oct 2010 14:09:36 -0000 Absolutely, but let's not lose sight of the fact that timeouts after SCSI bus resets have never been an engineered solution, for any OS. Going back to 1986 the ANSI specifications have been vague or worse about delays after such an action. I can't speak for Alexander about this, but I suspect all of this is going to be "try it, and report failures". A conservative approach would be "let's not change this", but the reason for doing this is to shorten reboot time, which is enough of an important issue that it's worth the risk, IMO. > Sure, but what's the plan for validating it? Too often this stuff goes into HEAD with the best of intentions, and then it basically never touched again until someone stumbles over it in an release. I'm not against this change, I just want to know what engineering has gone into it. > > Scott >