On Monday 25 September 2006 12:11, Muli Ben-Yehuda wrote: > Hi, > > The recently merged aic94xx in mainline requires external firmware > support. This, in turn, necessitates an initrd/initramfs environment > that includes firmware support to load the firmware. Will a patch to > optionally include the firmware inline in the kernel and thus not > having to use an initramfs be acceptable? > > Also, aic94xx does not compile unless FW_LOADER is set in .config due > to missing 'request_firmware'. What's the right thing to do here - > aic94xx selecting it, depending on it, or FW_LOADER providing empty > request_firmware() in case it's compiled out (the last one violates > the principle of least surprise IMHO). While we're at it, already existing aicXXXX driver continues to be insanely large due to excessive inlining of I/O routines: # x86_64-pc-linux-gnu-size aic7xxx.o text data bss dec hex filename 118978 22673 568 142219 22b8b aic7xxx.o This is not even helping performance one iota. Call overhead for these routines pales in comparison to I/O instruction stalls, which is in turn pales in comparison to disk seek delays. Please do something with it. -- vda - 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