DomainKey-Signature: a=rsa-sha1; c=nofws; d=pobox.com; h=subject:from:to
	:cc:in-reply-to:references:content-type:date:message-id
	:mime-version:content-transfer-encoding; q=dns; s=sasl; b=M8v+pp
	Yi9U77Nv+TQJVyt2i5SJcAyk7+DvMEVoYz1ReQfxnxYq68Ky0u/FyI3IuA1PlIDB
	M3oRkU3aMyHRpw0TEPtGsVOUhTDv3DImjnc1G9MX7ku2tGJs1lnux2wSYTJu100P
	KPX+aVpGxZfmOcdIhreq2nH4DPBCyhK156b74=
Subject: Re: [PATCH 02/10] Introduce mm_has_pending_aio() helper
From: Nathan Lynch <ntl@pobox.com>
To: Jeff Moyer <jmoyer@redhat.com>
Cc: linux-kernel@vger.kernel.org, containers@lists.linux-foundation.org,
        Oren Laadan <orenl@cs.columbia.edu>, linux-aio@kvack.org
In-Reply-To: <x494o7mrgrh.fsf@segfault.boston.devel.redhat.com>
References: <1298936432-29607-1-git-send-email-ntl@pobox.com>
 <1298936432-29607-3-git-send-email-ntl@pobox.com>
 <x494o7mrgrh.fsf@segfault.boston.devel.redhat.com>
Content-Type: text/plain; charset="UTF-8"
Date: Tue, 01 Mar 2011 10:04:24 -0600
Message-ID: <1298995464.461.9.camel@tp-t61>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1056
Lines: 29

On Tue, 2011-03-01 at 10:40 -0500, Jeff Moyer wrote:
> 
> ntl@pobox.com writes:
> 
> > From: Nathan Lynch <ntl@pobox.com>
> >
> > Support for AIO is on the to-do list, but until that is implemented,
> > checkpoint will have to fail if a mm_struct has outstanding AIO
> > contexts.  Add a mm_has_pending_aio() helper function for this
> > purpose.
> 
> Just because a process has an io context, doesn't mean that the process
> has active outstanding requests.  So, is this really what you wanted to
> test?

As a temporary measure, yeah.  We haven't settled on code to
record/restore the io context objects themselves, so we do want to bail
if we encounter any.  I realize now the name of the function doesn't
actually express this well.  Will try to come up with something better
for the next round.

Thanks!


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/