Hello Ben, On Wed, Aug 12, 2009 at 7:28 AM, Ben Hutchings<ben@xxxxxxxxxxxxxxx> wrote: > There is a bug in the firmware loading code in qla1280, as reported > below: it calls request_firmware() while holding a spinlock. I think > that it needs to drop ha->host->host_lock around the call to > request_firmware() but I'm not sure that's correct. > Yes, you are right, we need to unlock while requesting firmware. But qla1280_mailbox_command() again needs the lock. I am sorry, currently I do not have access to this hardware, can you please test it. Thanks for your help, -- JSR > Ben. > > -------- Forwarded Message -------- > From: R. Scott Bailey <scott.bailey@xxxxxxx> > Reply-to: "R. Scott Bailey" <scott.bailey@xxxxxxx>, 539406@xxxxxxxxxxxxxxx > To: Debian Bug Tracking System <submit@xxxxxxxxxxxxxxx> > Subject: Bug#539406: linux-image-2.6.30-1-alpha-smp: fails to load fw for 1st scsi adapter > Date: Fri, 31 Jul 2009 11:36:38 -0400 > > Package: linux-image-2.6.30-1-alpha-smp > Version: 2.6.30-2 > Severity: important > > Well, the firmware loading logic has been flaky on my system for a few > releases now. I previously reported bug 527265 against 2.6.29, where the > qla1040 firmware would not load at all, and that was resolved in a later > 2.6.29 image. > > In this release (2.6.30-2) I get new/different bad behavior. My system > has three QLA1040 cards in it. In the attached console output from the boot, > notice that at time 23.256824, the first adapter (scsi0) is located, > generates a stack trace trying to load firmware, and fails to initialize > the card. (There goes my tape drive!) A little later, at 83.499957, the > second adapter is found and this time the firmware loads fine. Ditto for > the third adapter after that. > > -- Package-specific info: > > -- System Information: > Debian Release: 5.0.2 > Architecture: alpha > > Kernel: Linux 2.6.29-2-alpha-smp (SMP w/1 CPU core) > Locale: LANG=C, LC_CTYPE=C (charmap=ANSI_X3.4-1968) > Shell: /bin/sh linked to /bin/bash > > Versions of packages linux-image-2.6.30-1-alpha-smp depends on: > ii debconf [debconf-2.0] 1.5.24 Debian configuration management sy > ii initramfs-tools [linux-initra 0.92o tools for generating an initramfs > ii module-init-tools 3.4-1 tools for managing Linux kernel mo > > linux-image-2.6.30-1-alpha-smp recommends no packages. > > Versions of packages linux-image-2.6.30-1-alpha-smp suggests: > ii aboot 1.0~pre20040408-3 Linux bootloader for the SRM conso > ii fdutils 5.5-20060227-3 Linux floppy utilities > pn linux-doc-2.6.30 <none> (no description available) > > -- debconf information: > linux-image-2.6.30-1-alpha-smp/postinst/depmod-error-initrd-2.6.30-1-alpha-smp: false > linux-image-2.6.30-1-alpha-smp/postinst/create-kimage-link-2.6.30-1-alpha-smp: true > linux-image-2.6.30-1-alpha-smp/preinst/lilo-initrd-2.6.30-1-alpha-smp: true > linux-image-2.6.30-1-alpha-smp/preinst/abort-install-2.6.30-1-alpha-smp: > linux-image-2.6.30-1-alpha-smp/postinst/depmod-error-2.6.30-1-alpha-smp: false > linux-image-2.6.30-1-alpha-smp/prerm/removing-running-kernel-2.6.30-1-alpha-smp: true > linux-image-2.6.30-1-alpha-smp/prerm/would-invalidate-boot-loader-2.6.30-1-alpha-smp: true > linux-image-2.6.30-1-alpha-smp/postinst/bootloader-test-error-2.6.30-1-alpha-smp: > linux-image-2.6.30-1-alpha-smp/preinst/initrd-2.6.30-1-alpha-smp: > linux-image-2.6.30-1-alpha-smp/postinst/kimage-is-a-directory: > shared/kernel-image/really-run-bootloader: true > linux-image-2.6.30-1-alpha-smp/preinst/lilo-has-ramdisk: > linux-image-2.6.30-1-alpha-smp/preinst/elilo-initrd-2.6.30-1-alpha-smp: true > linux-image-2.6.30-1-alpha-smp/preinst/overwriting-modules-2.6.30-1-alpha-smp: true > linux-image-2.6.30-1-alpha-smp/postinst/bootloader-error-2.6.30-1-alpha-smp: > linux-image-2.6.30-1-alpha-smp/preinst/abort-overwrite-2.6.30-1-alpha-smp: > linux-image-2.6.30-1-alpha-smp/preinst/bootloader-initrd-2.6.30-1-alpha-smp: true > linux-image-2.6.30-1-alpha-smp/postinst/old-initrd-link-2.6.30-1-alpha-smp: true > linux-image-2.6.30-1-alpha-smp/postinst/old-dir-initrd-link-2.6.30-1-alpha-smp: true > linux-image-2.6.30-1-alpha-smp/postinst/old-system-map-link-2.6.30-1-alpha-smp: true > linux-image-2.6.30-1-alpha-smp/preinst/failed-to-move-modules-2.6.30-1-alpha-smp: > > -- > Ben Hutchings > Unix is many things to many people, > but it's never been everything to anybody. > -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
