lkml.org 
[lkml]   [2011]   [Jun]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
DateTue, 14 Jun 2011 16:43:34 +0300
FromAvi Kivity <>
SubjectRe: [PATCH 08/10] NOTIFIER: Take over TIF_MCE_NOTIFY and implement task return notifier
On 06/14/2011 04:33 PM, Borislav Petkov wrote:
> >
> > Even with mce -> irq_work -> rt thread, we're unlikely to return to
> > the task as the rt thread will displace the task. It may be migrated
> > to an idle cpu, but even then we may be able to drop the page before
> > it gets back to userspace.
>
> This doesn't give you the guarantee that the realtime task manages to
> unmap the page from all pagetables before another process running on
> another core accesses it.

Right, it's not about a guarantee, it's about maintaining decent
performance.

> I think your previous suggestion of making the memory failure handling
> code reentrant would cover all holes.

I think it's required, yes.

Since we can't have nested #MC (due to the IST mechanism resetting %rsp
and cloberring the previous invocation's stack), we have to clear MCIP
outside the #MC handler. And that means irq_work_queue()

(note that this changes the behaviour from memory corruption to shutdown
state; both suck, but one more than the other).

> Even marking all processes mapping a faulty page STOPPED or
> UNINTERRUPTIBLE won't work in all cases since you have to go out and
> find which those processes are. And this is what the rt thread will do.

Yes.

--
error compiling committee.c: too many arguments to function



\
 
 \ /
  Last update: 2011-06-14 15:47    [from the cache]
©2003-2010