> <reads the code>>> Seems that it performs lookups only in the caller's PID namespace. > Maybe this is appropriate but it should be described and justified in> the changelog and in code comments, please. And in the forthcoming> manpage ;)

Well pids should always and only be looked up in the callers pidnamespace. Any other behavior is broken. It is probably wortha mention in a manpage but you should not need to justify usingabstractions as they were designed to be used.

All pointers are not encoded the same as void * pointers. Admittedlythe only case I can think of are function pointers on Itanium, butwhat is a little wrong today can easily become a lot wrong tomorrow.

Making the kcmp_ptr arguments void * seems the way to go though.

Now there is one interesting case we are not handling properly.If any of our pointers can be NULL which I think happens in thefile case we should return -EBADF instead of reporting two NULLpointers point to the same object.