Andreas Dilger wrote:
On Dec 21, 2007  16:39 +0530, Aneesh Kumar K.V wrote:
@@ -3790,7 +3782,9 @@ repeat:
/* if we still need more blocks and some PAs were used, try again */
        if (free < needed && busy) {
+               busy = 0;
                ext4_unlock_group(sb, group);
+               schedule_timeout(HZ);
                goto repeat;
        }

Is there nothing we could actually wait on instead of just sleeping for
1 second?

actually it was done for simplicity - in my tests busy PA happened quite rare.
I have no objection to improve this with special wait queue.

thanks, Alex

-
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to