From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Serge E. Hallyn" Subject: Re: [PATCH 6/7] have ckpt_err set ctx->errno Date: Mon, 16 Nov 2009 11:20:39 -0600 Message-ID: <20091116172039.GA12466@us.ibm.com> References: <1257465619-1777-1-git-send-email-serue@us.ibm.com> <1257465619-1777-7-git-send-email-serue@us.ibm.com> <4B017CB4.10707@cs.columbia.edu> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <4B017CB4.10707-eQaUEPhvms7ENvBUuze7eA@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org Errors-To: containers-bounces-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org To: Oren Laadan Cc: containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org List-Id: containers.vger.kernel.org Quoting Oren Laadan (orenl-eQaUEPhvms7ENvBUuze7eA@public.gmane.org): > > > serue-r/Jw6+rmf7HQT0dZR+AlfA@public.gmane.org wrote: > > From: Serge E. Hallyn > > > > Move setting of ctx->errno into do_ckpt_msg(). We leave a small separate > > function to notify all restarting tasks in case of failure. We use a > > CKPT_CTX_WOKEN bit for this. If we always notify restarting tasks of > > failure inside do_ckpt_msg() then we will have to be more careful about > > when ckpt_err() is called at restart. Only 4 instances were doing the > > I don't think it is wrong to notify of errors and wakeup tasks > "too early" or multiple times. In fact, the first one to notify > an error (test-and-set of ctx->errno) should do that. What am I > missing ? ckpt_err() is called before the init_completion(&complete). We could just make sure that never happens. -serge