Re: [RFC][PATCH] pstore: EFI Support

[Greg KH]

On Tue, May 10, 2011 at 11:00:44AM -0400, Seiji Aguchi wrote:
> Description of boot paremeters is following.
> 
>  - efi_pstore_enable
>    enable EFI support of pstore.
> 
>  - efi_pstore_len
>    Sets the buffer size of EFI variable space used by pstore.

Please don't add new boot parameters if at all possible.  Distros will
not know to enable them, and users don't know how to either.

Use sane defaults, and provide ways to override them if needed, but
don't rely on them for new functionality if at all possible.

Why would this option ever _not_ be something that should be enabled?

thanks,

greg k-h
--
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/