On Fri, Jun 13, 2008 at 05:26:17PM -0400, Trond Myklebust wrote: > On Fri, 2008-06-13 at 17:17 -0400, J. Bruce Fields wrote: > > On Thu, Jun 12, 2008 at 03:22:00PM -0400, Trond Myklebust wrote: > > > Since the credentials may be allocated during the call to rpc_new_task(), > > > which again may be called by a memory allocator... > > > > Most of these are only called in gssd process's context when it > > performs the downcall. Which doesn't change the fact that they're > > needed for the rpc_new_task() to succeed, OK, so this isn't an objection > > to this patch--but is there any plan for how we'll deal with e.g. memory > > allocations that gssd does on its own? > > You mean alloc_enc_pages()? We should really try to avoid sleeping > there... No, that may be a problem too, but I mean: if we're worried about a deadlock due to some allocations that gssd performs on the downcall at the end of the context initiation, then shouldn't we also be worried about all the other allocations that gssd performs itself? --b. -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html