Re: linux-next: block tree build failure

From: Christoph Hellwig
Date: Tue Jul 07 2009 - 09:57:19 EST


On Tue, Jul 07, 2009 at 08:38:46AM +0200, Jens Axboe wrote:
> > SYSCALL_DEFINE0(sync)
> > {
> > - wakeup_pdflush(0);
> > + wakeup_flusher_threads(0);
> > sync_filesystems(0);
> > sync_filesystems(1);
> > if (unlikely(laptop_mode))
>
> That is correct! I have just now updated for-next as well, so your next
> pull should lose this fixup.

It's not correct at all. We'll how have various flusher threads doing
async syncs, just to wait for them again synchronously. The right thing
to do here is to queue up the data integrity sync to per-bdi threads and
execute those in parallel.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/