to den 31.03.2005 Klokka 16:54 (+0200) skreiv Ingo Molnar: > i think it's incorrect in its current form, because 'pos' is not > necessarily safe and might be removed from the commit_list?
Right. > the whole loop could be a "while (!list_empty(head))" thing, if it wasnt > for the possibility for an nfs_set_page_writeback_locked() to skip an > entry. Maybe a local list of 'already processed' requests should be > built, and once the main list is emptied, moved back into the main list? > Then the lock-break could be moved to the end of the loop. Should be pretty much unnecessary. See my previous email. Cheers, Trond -- Trond Myklebust <[EMAIL PROTECTED]> - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
