>>> How come my computer is the only one with a reply button? Hey, I've got one. ;-) 2007/8/16, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>: > On Thu, 16 Aug 2007 01:59:56 +0200 Heiko Carstens <heiko.carstens@xxxxxxxxxx> wrote: > > > > So yes, I'd say this is a bug in DM. > > > > > > Also, __rh_alloc() is called under read_lock(), via __rh_find(). If > > > __rh_alloc()'s mempool_alloc() fails, it will perform a sleeping allocation > > > under read_lock(), which is deadlockable and will generate might_sleep() > > > warnings > > > > The read_lock() is unlocked at the beginning of the function. > > Oh, OK. Looks odd, but whatever. > The major trick, if I am not wrong, is to use GFP_ATOMIC on that mempool_alloc(). This prevents the sleeping allocation but fails, if memory as well as the pool is exhausted. > > It'd be better to fix the kmirrord design so that it can use mempools > properly. One possible way of doing that might be to notice when mempool > exhaustion happens, submit whatever IO is thus-far buffered up and then do > a sleeping mempool allocation, to wait for that memory to come free (via IO > completion). > > That would be a bit abusive of the mempool intent though. A more idiomatic > fix would be to change kmirrord so that it no longer can consume all of the > mempool's reserves without having submitted any I/O (which is what I assume > it is doing). > The problem is, that only the same thread, that allocates from the pool would return memory back. This would be done before the new allocations. But, if there is very high memory pressure, the pool might get drained in the allocation cycle. Then mempool_alloc() waits to be woken from mempool_free(). And this never happens, since the thread will be stuck. So I guess the fix would be to somehow separate the allocation and freeing functionality. If I remember correctly back, the patch was always seen as "not quite correctly, but seems to work". However, due to lack of time, nobody ever came up with a better solution. Stefan -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
