Eric, I have to have a similar compat file for the IPMI drivers backported onto RHEL3, RHEL4, and SLES9. They aren't in mainline of course, but each OS has a slightly different copy for its needs, so my DKMS packages carry it. In general, this construct: > > -#if (LINUX_VERSION_CODE < KERNEL_VERSION(2,6,6)) > > -static int inline scsi_device_online(struct scsi_device *sdev) > > -{ > > - return sdev->online; > > -} > > -#endif is better tested as: #ifndef scsi_device_inline static int inline scsi_device_online(struct scsi_device *sdev) { return sdev->online; } #endif when you can. It cleanly eliminates the version test, and tests for exactly what you're looking for - is this function defined. Thanks, Matt -- Matt Domsch Software Architect Dell Linux Solutions linux.dell.com & www.dell.com/linux Linux on Dell mailing lists @ http://lists.us.dell.com - : 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