Messages in this thread | | | Date | Fri, 12 Jun 2009 20:57:41 +0800 | From | Wu Fengguang <> | Subject | Re: [PATCH 1/5] HWPOISON: define VM_FAULT_HWPOISON to 0 when feature is disabled |
| |
Hi Ingo,
On Fri, Jun 12, 2009 at 07:22:58PM +0800, Ingo Molnar wrote: > > * Wu Fengguang <fengguang.wu@intel.com> wrote: > > > So as to eliminate one #ifdef in the c source. > > > > Proposed by Nick Piggin. > > > > CC: Nick Piggin <npiggin@suse.de> > > Signed-off-by: Wu Fengguang <fengguang.wu@intel.com> > > --- > > arch/x86/mm/fault.c | 3 +-- > > include/linux/mm.h | 7 ++++++- > > 2 files changed, 7 insertions(+), 3 deletions(-) > > > > --- sound-2.6.orig/arch/x86/mm/fault.c > > +++ sound-2.6/arch/x86/mm/fault.c > > @@ -819,14 +819,13 @@ do_sigbus(struct pt_regs *regs, unsigned > > tsk->thread.error_code = error_code; > > tsk->thread.trap_no = 14; > > > > -#ifdef CONFIG_MEMORY_FAILURE > > if (fault & VM_FAULT_HWPOISON) { > > printk(KERN_ERR > > "MCE: Killing %s:%d due to hardware memory corruption fault at %lx\n", > > tsk->comm, tsk->pid, address); > > code = BUS_MCEERR_AR; > > } > > -#endif > > Btw., anything like this should happen in close cooperation with the > x86 tree, not as some pure MM feature. I dont see Cc:s and nothing > that indicates that realization. What's going on here?
Ah sorry for the ignorance! Andi has a nice overview of the big picture here: http://lkml.org/lkml/2009/6/3/371
In the above chunk, the process is trying to access the already corrupted page and thus shall be killed, otherwise it will either silently consume corrupted data, or will trigger another (deadly) MCE event and bring down the whole machine.
VM_FAULT_HWPOISON is tagged by the hwpoison code to indicate that the previously mapped page contains corrupted data, and is unrecoverable because there are no valid on-disk copy that can be reloaded.
> It is not at all clear to me whether propagating hardware failures > this widely is desired from a general design POV. Most desktop > hardware wont give a damn about this (and if a hardware fault > happens you want to get as far from the crappy hardware as possible) > so i'm not sure how relevant it is and how well tested it will > become in practice.
Intel Nehalem-EX will have this feature, and is going to ship in volume servers in the coming years. Given that the servers may well be equipped with tons of memory, memory failures (especially soft errors http://en.wikipedia.org/wiki/Soft_error) become un-ignorable.
Sunspot Maximum is underway by 2011 and we must be prepared for it ;)
> I.e. really some wider discussion needs to happen on this.
OK.
Thanks, Fengguang
| |