Re: [PATCH] scsi: make scsi_devinfo infrastructure optional

From: Bartlomiej Zolnierkiewicz
Date: Thu Feb 10 2011 - 04:18:56 EST


On Wed, Feb 9, 2011 at 7:00 PM, James Bottomley <James.Bottomley@xxxxxxx> wrote:
> On Wed, 2011-02-09 at 15:15 +0100, Bartlomiej Zolnierkiewicz wrote:
>> Add SCSI_QUIRKS config option (default y and dependent on EMBEDDED
>> config option) to allow disabling of scsi_devinfo infrastructure.
>>
>> The output code size savings are ~14k for CONFIG_SCSI_QUIRKS=n
>> (as measured on x86-32):
>
> I don't understand the point of this patch ... without the quirks SCSI
> will do the wrong thing on a whole bunch of stuff.  The savings look to
> be tiny ... since the SCSI module is habitually a lot larger than your
> figures suggest.

The patch was originally done for embedded ATA-only setups.

Thanks,
Bartlomiej
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/