Re: [PATCH] libata-eh: Use switch() instead of sparse array for protocol strings

From: Tejun Heo
Date: Mon Jan 09 2017 - 12:32:16 EST


Hello,

On Mon, Jan 09, 2017 at 09:27:23AM -0800, Christoph Hellwig wrote:
> On Mon, Jan 09, 2017 at 05:30:02PM +0100, Geert Uytterhoeven wrote:
> > > ata_force_param_buf is __initdata and shouldn't really matter.
> >
> > It mainly matters because of e.g. bootloader limitations.
>
> Do we need a full 4k for the force parameters? What would a typical
> command line for it look like?

Maybe a couple hundreds bytes at max, but it's a bit weird to restrict
this given that it is bss, not gigantic and __initdata. What kind of
bootloader limitations are we talking about?

Thanks.

--
tejun