I had to fix several formatting and true bugs while integrating your
changes, for example:
+static inline int solaris_m(unsigned int fd, unsigned int cmd, u32 arg)
+{
+
+ struct file * flip;
+ flip=fget(fd);
This causes a file descriptor leak, you never fput(flip), and in fact
you never even use the flip variable anywhere in this function so I
just removed these two lines. Next:
+ printk("solaris_setsockopt: optname (%#x) optval (%#x)\n", optname, val);
Please don't leave such debugging print statements enable in patches
you'd like me to apply to the real tree, I removed this.
Otherwise your patch was mostly fine, thanks for the work.
Later,
David S. Miller
davem@redhat.com
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/