[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <200810240804.42500.major@openvz.org>
Date: Fri, 24 Oct 2008 08:04:41 +0400
From: Andrey Mirkin <major@...nvz.org>
To: Dave Hansen <dave@...ux.vnet.ibm.com>
Cc: Cedric Le Goater <clg@...ibm.com>,
containers@...ts.linux-foundation.org,
linux-kernel@...r.kernel.org, Pavel Emelyanov <xemul@...nvz.org>
Subject: Re: [Devel] Re: [PATCH 08/10] Introduce functions to restart a process
On Thursday 23 October 2008 17:49 Dave Hansen wrote:
> On Thu, 2008-10-23 at 13:54 +0400, Andrey Mirkin wrote:
> > We are putting special structure on stack, which is used at the very end
> > of the whole restart procedure to restore complex states (ptrace is one
> > of such cases). Right now I don't need to use this structure as we have a
> > deal with simple cases, but reservation of 256 bytes on stack is needed
> > for future.
>
> Wow. So you're saying that, if this patch is accepted, we simply need
> to accept that anything being checkpointed will use an extra 256 bytes
> of stack? Seems like something to perhaps put in the changelog rather
> than some completely undocumented assembly nugget.
This 256 bytes will be used only during restart procedure and only by our
module. As you can see in i386_ret_from_resume we are restoring it back. So,
when process will return to user space it will not have extra 256 bytes
reserved on stack already. I will add information about it to documentation
and changelog.
Andrey
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists