Re: statfs() / statvfs() syscall ballsup...

[Joel Becker]

On Fri, Oct 10, 2003 at 07:59:34AM -0700, Linus Torvalds wrote:
> The interface is fundamentally flawed, it has nasty security issues, it 
> lacks any kind of sane synchronization, and it exposes stuff that 
> shouldn't be exposed to user space.

	Um, sure, the interface as implemented has a few "don't do
that"s.  Yes, we've found security issues.  Those can be fixed.  That
doesn't make the concept bad.

> I hope disk-based databases die off quickly.

	As opposed to what?  Not a challenge, just interested in what
you think they should be.

> Yeah, I see where you are
> working, but where I'm coming from, I see all the _crap_ that Oracle tries
> to push down to the kernel, and most of the time I go "huh - that's a
> f**king bad design".

	I'm hoping that you've seen a marked improvement in the stuff
Oracle requests over the past couple years.  We've worked hard to filter
out the junk that really, really is bad.
	Where I work doesn't change the need for O_DIRECT.  If your Big
App has it's own cache, why copy the cache in the kernel?  That just
wastes RAM.  If your app is sharing data, whether physical disk, logical
disk, or via some network filesystem or storage device, you must
absolutely guarantee that reads and writes hit the storage, not the
kernel cache which has no idea whether another node wrote an update or
needs a cache flush.
	Putting my employer's hat back on, Oracle uses O_DIRECT because
it was the existing API for this.  If Linux came up with a better,
cleaner method, Oracle might change.  I can't guarantee that, but I know
I push like hell for obvious improvements.

Joel

-- 

"I don't want to achieve immortality through my work; I want to
 achieve immortality through not dying."
        - Woody Allen

Joel Becker
Senior Member of Technical Staff
Oracle Corporation
E-mail: joel.becker@xxxxxxxxxx
Phone: (650) 506-8127
-
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/