Re: [PATCH 2.6.24] mm: BadRAM support for broken memory

Rick van Rein
2008-03-04T13:43:39Z

Ingo,

> as i said it in another reply to this thread, it would be perfectly
> acceptable for upstream to merge an easier to use boot option - be that
> badmem=addr$size or excludemem=addr$size.

I'm not sure if that is an optimal replacement of BadRAM at all.

Broken memories usually manifest themselves as a column or row in a DIMM
that stopped working. So there is a pattern in the memory that is to be
excluded, and I'm not wholly sure that combines well with the excludemem
mechanism. (I will look into it to be sure.) I hope to know after the
weekend if the patterns that I am talking about can be turned into to a
cosmetic boot option.

I've also read that people on this list say that it is a waste to throw
out a full page if only 5 bytes (say) are actually faulty. Although it
is certainly interesting from an academic viewpoint to avoid this, it has
never, ever led to complaints. I've been advertising the possibility of
using the pages for allocation slabs, and allow all slabs except for those
with an error contained, but nobody has ever taken that seriously.

> Please send a patch :-)

I've been reluctant to quickly mend things because it could lead to
unstable code. The patch that I am presenting is mature code, with the
possible exception of the somewhat newer 64-bit code. If this list has
a preference for less mature code that is hardly field-tested, I could
of course tailor to that.

Cheers,

Rick van Rein