Re: [Cbe-oss-dev] [patch 01/20] cell: add RAS support

From: Geoff Levand
Date: Tue Jun 20 2006 - 15:01:47 EST


Olof Johansson wrote:
> On Tue, Jun 20, 2006 at 06:26:53PM +0200, Arnd Bergmann wrote:
>> On Tuesday 20 June 2006 17:43, Olof Johansson wrote:
>> >
>> > > This is a first version of support for the Cell BE "Reliability,
>> > > Availability and Serviceability" features.
>> >
>> > Does it really make sense to do this under a config option? I don't
> see
>> > why anyone would not want to know that their machine is about to
> melt.
>> >
>> You can only have that when running on bare metal. Machines that run
>> on a hypervisor can't run that code.
>
> Well, it's harmless to build it in even on hypervisor systems, right?


Harmless if you have a lot of RAM...


>> It probably makes sense to auto-select that option for
> CONFIG_CELL_BLADE
>> though.
>
> Sounds like a reasonable trade-off.


If its always built in, then I guess we don't need the conditionals
in cell/setup.c:

+#ifdef CONFIG_CBE_RAS
+ cbe_ras_init();
+#endif


-Geoff

-
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/