> > That's an idea. Or make separate called functions for each ioctl and
> > declare the structures inside them?
>
> and this method looks like a possibility.
>
> Gerd, Andrew says that the kmalloc() time will be small compared to
> the memset()'s that the function is already doing.
That is wrong, at least the 2k memset/call mentioned by Andrew. There
are lots of memset() calls, but they all are within the case switches
for the ioctls and zero out only the structs which are used in that code
path, so it is actually much smaller (~50 -> ~300 bytes maybe, depending
on the ioctl).
> Do you want to do anything about this, or want me to, or want me
> to drop it?
It is on my todo list, just havn't found yet the time to do that.
Probably I'll split it into smaller functions, this looks like the
best way to me.
Gerd
-
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 Mar 15 2003 - 22:00:24 EST