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

Rick van Rein
2008-03-02T17:42:54Z

Hello Kosaki,

> in general,
> Agreed with we need bad memory treatness.

Glad to hear that.

> > +#define PG_badram 20 /* BadRam page */
>
> some architecture use PG_reserved for treat bad memory.
> Why do you want introduce new page flag?

It is clearer to properly name a flag, I suppose.
Is the use that you are mentioning the intended, and only use of the flag?
If not, I think it is clearer to use a separate flag instead of overloading
one.

> for show_mem() improvement?

For code clarity.

Thanks,
-Rick