On Fri, Dec 18, 2015 at 12:25:35PM +0100, Christoph Hellwig wrote: > [can you trim your replies please? I had to trim almost 150 lines of > junk before getting to your reply!] > > > > spin_unlock_irqrestore(&log->io_list_lock, flags); > > > > > > or is that too simplistic? > > > > maybe add a new list and run the list at the begining of r5l_do_reclaim(). > > What's the advantage of another list? I mean simply waking up reclaim thread doesn't work as r5l_do_reclaim() will return if reclaimable == 0. There is no guarantee reclaimable space is available in the allocation failure. So we'd better move the retry at the begining of r5l_do_reclaim(). If yes, we can't reuse the no_space_stripes list. Thanks, Shaohua -- To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
