Re: [PATCH 0 of 4] Generic AIO by scheduling stacks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Tue, 30 Jan 2007, Zach Brown wrote:
>
> This very rough patch series introduces a different way to provide AIO support
> for system calls.

Yee-haa!

I looked at this approach a long time ago, and basically gave up because 
it looked like too much work.

I heartily approve, although I only gave the actual patches a very cursory 
glance. I think the approach is the proper one, but the devil is in the 
details. It might be that the stack allocation overhead or some other 
subtle fundamental problem ends up making this impractical in the end, but 
I would _really_ like for this to basically go in.

One of the biggest issues I see is signalling. Your mention it, and I 
think that's going to be one of the big issues.

It won't matter at all for a certain class of calls (a lot of the people 
who want to do AIO really end up doing non-interruptible things, and 
signalling is a non-issue), but not only is it going to matter for some 
others, we will almost certainly want to have a way to not just signal a 
task, but a single "fibril" (and let me say that I'm not convinced about 
your naming, but I don't hate it either ;)

But from a quick overview of the patches, I really don't see anything 
fundamentally wrong. It needs some error checking and some limiting (I 
_really_ don't think we want a regular user starting a thousand fibrils 
concurrently), but it actually looks much less invasive than I thought it 
would be.

So yay! Consider me at least preliminarily very happy.

		Linus
-
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/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux