Re: [PATCH for-next] scsi: qla2xxx: Hide unavailable firmware

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2015-10-08 at 15:46 +0000, Himanshu Madhani wrote:
> 
> On 10/7/15, 4:41 PM, "Julian Calaby" <julian.calaby@xxxxxxxxx> wrote:
> 
> >Hi Xose,
> >
> >On Thu, Oct 8, 2015 at 2:13 AM, Xose Vazquez Perez
> ><xose.vazquez@xxxxxxxxx> wrote:
> >> On Fri, May 22, 2015 at 10:00 AM, Julian Calaby
> >><julian.calaby@xxxxxxxxx> wrote:
> >>
> >>> Some qla2xxx devices have firmware stored in flash on the device,
> >>> however for debugging and triage purposes, Qlogic staff like to
> >>> be able to load known-good versions of these firmwares through
> >>> request_firmware().
> >>>
> >>> These firmware files were never distributed and are unlikely to ever
> >>> be released publically, so to hide these missing firmware files from
> >>> scripts which check such things, (e.g. Debian's initramfs-tools) put
> >>> them behind a new EXPERT Kconfig option.
> >>
> >>
> >> What is state of this patch ?
> >
> >Apparently nobody cared, either from qLogic or linux-scsi.
> >
> >I'm not overly fussed whether it goes in or not, it was more a point
> >in the discussion that proceeded it, however it does solve the
> >problems in the discussion that preceded it.
> 
> This patch Looks good.
> 
> Acked-By: Himanshu Madhani <himanshu.madhani@xxxxxxxxxx>

Actually, this isn't helpful.  You now add another option over which the
distributions have to make a choice.  Is this interface necessary and
useful?  If yes, then it should be compiled in and if not, just remove
it ... don't do death by 1000 Kconfig options.

James


--
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



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]
  Powered by Linux