Re: [PATCH] kill i_dev

From: Linus Torvalds (torvalds@transmeta.com)
Date: Thu Nov 21 2002 - 18:56:23 EST


On Fri, 22 Nov 2002 Andries.Brouwer@cwi.nl wrote:
>
> The i_dev field is deleted and the few uses are replaced
> by i_sb->s_dev.

Applied.

> There is a single side effect: a stat on a socket now sees
> a nonzero st_dev. There is nothing against that - FreeBSD
> has a nonzero value as well - but there is at least one
> utility (fuser) that will need an update.

Looking at the patch (not testing it), as far as I can tell we'll return a
basically random number that is just whatever the anonymous super-block
was allocated, right?

I'm not convinced that returning random numbers to user space is
necessarily a great idea.. That said, I think we already do it for unnamed
pipes anyway, so I'm more wondering if we should have some way to map
these numbews (in user space) to a valid thing, so that they wouldn't just
be random numbers.

(In other words: I like the patch, and I'm not really complaining about
this new behavour at all. It's just the "randomness" as far as user space
goes that bothers me a bit, since it seems to imply bad interface design).

                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/



This archive was generated by hypermail 2b29 : Sat Nov 23 2002 - 22:00:38 EST