On 6/4/2004 5:28 PM, David S. Miller wrote:

> Some bug fixes needed:
> 
> +#define NAME_OFFSET(de) ((int) ((de)->d_name - (char *) (de)))
> 
> Cast must be "(char __user *)" to avoid sparse warnings.

This updated patch takes care of the __user problems that Dave pointed out. Regarding __put_user vs __copy_to_user, __put_user is more optimal on ia64 like Andrew already noted. I realize it's not optimal on all platforms. So I suggest that we apply the patch for now and try to build something that's optimal for all platforms in the future.

	-Arun