On Sat, Jul 19, 2008 at 06:13:28AM -0600, Matthew Wilcox wrote: > On Sat, Jul 19, 2008 at 12:37:11PM +0200, Andi Kleen wrote: > > Russ Anderson <rja@xxxxxxx> writes: > > > > > [PATCH 0/2] Migrate data off physical pages with corrected memory errors (Version 7) > > > > FWIW I discussed this with some hardware people and the general > > opinion was that it was way too aggressive to disable a page on the > > first corrected error like this patchkit currently does. > > I think it's reasonable to take a page out of service on the first error. > Then a user program needs to be notified of which bit is suspected. > It can then subject that page to an intense set of tests (I'd start > by stealing the ones from memtest86+) and if no more errors are found, > it could return the page to service. In general I agree with that approach. One concern is that in the process of testing the memory the diagnostic may hit an uncorrectable error. That is not a problem with Itanium, which is designed to handle uncorrected/poisoned data going into and out of the processor core, but can be a system fatal error (requiring a reboot) on other processor types. Just something to be aware of. -- Russ Anderson, OS RAS/Partitioning Project Lead SGI - Silicon Graphics Inc rja@xxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
|