From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oren Laadan Subject: Re: Add support for connected INET sockets Date: Mon, 16 Nov 2009 13:38:08 -0500 Message-ID: <4B019C10.8050609@cs.columbia.edu> References: <1257878856-25520-1-git-send-email-danms@us.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1257878856-25520-1-git-send-email-danms-r/Jw6+rmf7HQT0dZR+AlfA@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: Dan Smith Cc: containers-qjLDD68F18O7TbgM5vRIOg@public.gmane.org List-Id: containers.vger.kernel.org The patches look good, see a few comments per patch. Before I pull, please (1) use ckpt_err() instead of ckpt_write_err(), and also use ckpt_err() in the restart path where it makes sense. I should be pushing v19-rc1 by tomorrow, perhaps better wait. Thanks, Oren. Dan Smith wrote: > This set includes the previous INET connected sockets patch with a few > trivial changes suggested by Oren, as well as the readme update. The > first patch fixes socket buffer checkpoint to include fragments, and > includes the changes in the previous first patch that added header mark > information. As a result, this set now also includes a fix to the UNIX > code to cope with the change. > > _______________________________________________ > Containers mailing list > Containers-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org > https://lists.linux-foundation.org/mailman/listinfo/containers >