Re: lots and lots of disks again

Christoph Hellwig
2004-02-10T15:52:10Z

On Tue, Feb 10, 2004 at 04:47:51PM +0100, Kurt Garloff wrote:
>
> Right, Matthew provided the reverse mapping, despite the fact that
> we don't need them, as it's just done by pointer deref ...
>
> We may need them later though, if we ever go for 64 partitions per disk.
> So I left them in. Probably we should comment them out, so the compiler
> does not see them for now. Or stick them in the docu.

Please don't add #if 0'ed code. Just leave it out and whoever needs it
can readd it.

> +#define SD_DISKS 32768 // anything between 256 and 262144

Even if C99 allows C++-style comments now please don't use the in the kernel

Else I'm okay with the patch although I'm not too happy with having the
huge array, but I think we can declare that a "who complains may fix it"
issue.