Re: [PATCH] [2.5] Non-blocking write can block

Linus Torvalds (torvalds@transmeta.com)
Wed, 4 Jun 2003 07:35:47 -0700 (PDT)


On Wed, 4 Jun 2003, Christoph Hellwig wrote:
>
> The else should be on the same line as the closing brace, else
> the patch looks fine.

No no no, it's wrong.

If you do something like this, then you also have to teach "select()"
about this, otherwise you just get busy looping in applications.

In general, we shouldn't do this, unless somebody can show an application
where it really matters. Taking internal kernel locking into account for
"blockingness" easily gets quite complicated, and there is seldom any real
point to it.

Remember: perfect is the enemy of good. I'll happily apply the patch (if
it also updates the tty poll() functionality), _if_ there is some
real-world situation where it matters.

Linus

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