Messages in this thread | | | Date | Mon, 05 Mar 2007 18:37:08 -0800 | From | "H. Peter Anvin" <> | Subject | Re: [patch v2] epoll use a single inode ... |
| |
Davide Libenzi wrote: > > Right now is using: > > this.len = sprintf(name, "[%u.%d]", current->pid, fd); > > That should be unique and not have the wraparound problem. Ack? >
NAK, very much NAK.
File descriptors aren't file structures, they're *pointers* to file structures.
It's perfectly possible -- downright common -- for a file descriptor to be inherited by another process, and then the pid is recycled -- collision.
-hpa - 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/
| |