Sukadev Bhattiprolu <sukadev at us.ibm.com> writes: > From: Sukadev Bhattiprolu <sukadev at us.ibm.com> > Subject: Use struct pid reference in autofs rather than pid_t > > Make autofs container-friendly by caching struct pid reference rather > than pid_t. Are autofs and autofs4 so tightly tied they must be converted together? If not 2 patches are more appropriate. Otherwise I'd say this looks fine at first glance. Just note that using struct pid protects from pid wraparound issues when dealing with user space so this should also be a small increase in correctness as well. At second glance this patch is very incorrect. It is missing get_pid and put_pid calls. The big difference between struct pid and pid_t values is that struct pid is reference counted. Your find_pid's should be find_get_pid's and you need the put_pids and unmount or whenever those values stop being valid. You also didn't take the required locks when using find_pid. Eric
