On Sat, Jan 3, 2015 at 7:11 PM, Jens Axboe <axboe@xxxxxx> wrote: > On 01/03/2015 07:21 AM, Ming Lei wrote: >> axmap_first_free() is always called with axmap->lock held, >> so needn't to acquire the lock inside the function. >> >> The deadlock is introduced in commit 12bde3697fc230d7a(axmap: >> ensure we lock down the maps for shared access). >> >> Given axmap_first_free() is only called inside lib/axmap.c, >> this patch declares the function as static. In the future, >> if external users need the function, axmap lock can be >> considered at that time. > > Thanks Ming, applied. > Do you use the "Fixes:" tag in fio development [1]? Fixes: 12bde3697fc2 ("axmap: ensure we lock down the maps for shared access") - Sedat - [1] http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/Documentation/SubmittingPatches#n159 -- To unsubscribe from this list: send the line "unsubscribe fio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html