Bug#539406: [Fwd: Bug#539406: linux-image-2.6.30-1-alpha-smp: fails to load fw for 1st scsi adapter]
Hello Ben,
On Wed, Aug 12, 2009 at 7:28 AM, Ben Hutchings<ben@decadent.org.uk> 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@eds.com>
> Reply-to: "R. Scott Bailey" <scott.bailey@eds.com>, 539406@bugs.debian.org
> To: Debian Bug Tracking System <submit@bugs.debian.org>
> 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.
>
Reply to: